refactor: 테스트 개선

Author: six-standard

insight/tasks/weekly_newsletter_batch.py

@@ -145,8 +145,8 @@ def _get_weekly_trend_html(self) -> str:
 
             # 템플릿 렌더링이 제대로 되지 않은 경우 배치 종료
             if (
-                "이 주의 트렌딩 글" not in weekly_trend_html
-                or "트렌드 분석" not in weekly_trend_html
+                "이번 주의 트렌딩 글" not in weekly_trend_html
+                or "주간 트렌드 분석" not in weekly_trend_html
             ):
                 logger.error(
                     f"Failed to build weekly trend HTML for newsletter #{weekly_trend['id']}"

insight/tests/conftest.py

@@ -160,9 +160,9 @@ def sample_newsletter(user):
         email_message=EmailMessage(
             to=[user.email],
             from_email=settings.DEFAULT_FROM_EMAIL,
-            subject="Test Newsletter",
-            text_body="Test content",
-            html_body="<div>Test content</div>",
+            subject="벨로그 대시보드 주간 뉴스레터 #1",
+            text_body="Weekly Report Test content",
+            html_body="<div>Weekly Report<br/>Velog Dashboard<br/>활동 리포트<br/>대시보드 보러가기</div>",
         ),
     )
 

insight/tests/tasks/test_weekly_newsletter_batch.py

@@ -196,6 +196,8 @@ def test_build_newsletters_success(
             assert len(newsletters) == 1
             assert newsletters[0].user_id == user.id
             assert newsletters[0].email_message.to[0] == user.email
+            # 제목 포맷 검증
+            assert "벨로그 대시보드 주간 뉴스레터" in newsletters[0].email_message.subject
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     def test_send_newsletters_success(

insight/tests/tasks/test_weekly_newsletter_template.py

@@ -61,8 +61,9 @@ def test_get_weekly_trend_html_success(
 
             assert trending_summary[0]["title"] in weekly_trend_html
             assert trend_analysis["insights"] in weekly_trend_html
-            assert "이 주의 트렌딩 글" in weekly_trend_html
-            assert "트렌드 분석" in weekly_trend_html
+            assert "벨로그 주간 트렌드" in weekly_trend_html
+            assert "이번 주의 트렌딩 글" in weekly_trend_html
+            assert "주간 트렌드 분석" in weekly_trend_html
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     @pytest.mark.django_db
@@ -124,12 +125,11 @@ def test_get_user_weekly_trend_html_success(
 
         assert trending_summary[0]["title"] in user_weekly_trend_html
         assert trend_analysis["insights"] in user_weekly_trend_html
-        assert (
-            f'<b>{user_weekly_stats["new_posts"]}개</b>의 글을 작성'
-            in user_weekly_trend_html
-        )
+        assert f'{user_weekly_stats["new_posts"]}개의 글' in user_weekly_trend_html
         assert "마지막으로 글을 작성하신지" not in user_weekly_trend_html
         assert user.username in user_weekly_trend_html
+        assert "이번주에 작성한 글" in user_weekly_trend_html
+        assert "주간 내 활동 분석" in user_weekly_trend_html
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     def test_get_user_weekly_trend_html_inactive_user(
@@ -147,13 +147,16 @@ def test_get_user_weekly_trend_html_inactive_user(
         insight_data = inactive_user_weekly_trend.insight
         user_weekly_reminder = insight_data.get("user_weekly_reminder")
 
-        assert "이번주에 쓴 글" not in user_weekly_trend_html
-        assert "내 글을 분석해보면?" not in user_weekly_trend_html
-        assert "글을 작성하지 않으셨네요" in user_weekly_trend_html
-        assert (
-            f'마지막으로 글을 작성하신지 {user_weekly_reminder["days_ago"]}일이 지났어요'
-            in user_weekly_trend_html
-        )
+        assert "이번주에 작성한 글" not in user_weekly_trend_html
+        assert "주간 내 활동 분석" not in user_weekly_trend_html
+        # days_ago가 있는 경우와 없는 경우 모두 처리
+        if user_weekly_reminder.get("days_ago"):
+            assert (
+                f'😭 마지막으로 글을 작성하신지 {user_weekly_reminder["days_ago"]}일이 지났어요!'
+                in user_weekly_trend_html
+            )
+        else:
+            assert "😭 글을 작성하지 않으셨네요!" in user_weekly_trend_html
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     def test_get_user_weekly_trend_html_exception(
@@ -193,6 +196,8 @@ def test_get_newsletter_html_success(self, mock_logger, newsletter_batch):
         assert weekly_trend_html in newsletter_html
         assert user_weekly_trend_html in newsletter_html
         assert "대시보드 보러가기" in newsletter_html
+        assert "Weekly Report" in newsletter_html
+        assert "Velog Dashboard" in newsletter_html
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     def test_get_newsletter_html_expired_token_user(
@@ -210,10 +215,12 @@ def test_get_newsletter_html_expired_token_user(
         )
 
         # 템플릿 렌더링 검증
-        assert "토큰이 만료" in newsletter_html
+        assert "🚨 잠시만요, 토큰이 만료된 것 같아요!" in newsletter_html
+        assert "토큰이 만료되어 정상적으로 통계를 수집할 수 없었어요" in newsletter_html
         assert weekly_trend_html in newsletter_html
         assert user_weekly_trend_html not in newsletter_html
         assert "대시보드 보러가기" in newsletter_html
+        assert "활동 리포트" in newsletter_html
 
     @patch("insight.tasks.weekly_newsletter_batch.logger")
     def test_get_newsletter_html_exception(