refactor: migrate subscription tests to Playwright (16-18/19)

Author: fricklerhandwerk

src/webview/templates/subscriptions/package_subscription.html

@@ -17,7 +17,7 @@ <h1 class="page-title">Package Subscription</h1>
     </div>
 
   {% else %}
-    <div class="rounded-box column gap">
+    <div id="{{ package_name }}" class="rounded-box column gap">
       <h2 class="heading">{{ package_name }}</h2>
       <p>Subscribe to receive notifications about security alerts suggestions that may affect this package.</p>
       <div class="toggler {% if is_subscribed %}toggler-on{% else %}toggler-off{% endif %}">

src/webview/tests/test_subscriptions.py

@@ -186,6 +186,24 @@ def test_user_does_not_receive_notification_when_auto_subscribe_disabled(
     expect(badge).to_have_text("1")
 
 
+def test_package_subscription(
+    live_server: LiveServer,
+    as_staff: Page,
+    drv: NixDerivation,
+) -> None:
+    """Test that the package subscription page displays correctly for valid packages"""
+    url = reverse(
+        "webview:subscriptions:package", kwargs={"package_name": drv.attribute}
+    )
+    as_staff.goto(live_server.url + url)
+    package = as_staff.locator(f"#{drv.attribute}")
+    subscribe = package.get_by_role("button", name="Subscribe")
+    subscribe.click()
+    unsubscribe = package.get_by_role("button", name="Unsubscribe")
+    unsubscribe.click()
+    subscribe.click()
+
+
 class SubscriptionTests(TestCase):
     def setUp(self) -> None:
         # Create test user with social account
@@ -292,22 +310,6 @@ def setUp(self) -> None:
             parent_evaluation=self.evaluation,
         )
 
-    def test_package_subscription_page_shows_valid_package(self) -> None:
-        """Test that the package subscription page displays correctly for valid packages"""
-        url = reverse(
-            "webview:subscriptions:package", kwargs={"package_name": "firefox"}
-        )
-        response = self.client.get(url)
-
-        self.assertEqual(response.status_code, 200)
-        self.assertTemplateUsed(response, "subscriptions/package_subscription.html")
-
-        # Check context
-        self.assertEqual(response.context["package_name"], "firefox")
-        self.assertTrue(response.context["package_exists"])
-        self.assertFalse(response.context["is_subscribed"])
-        self.assertIsNone(response.context["error_message"])
-
     def test_package_subscription_page_shows_invalid_package(self) -> None:
         """Test that the package subscription page shows error for invalid packages"""
         url = reverse(
@@ -325,48 +327,6 @@ def test_package_subscription_page_shows_invalid_package(self) -> None:
         self.assertIsNotNone(response.context["error_message"])
         self.assertIn("does not exist", response.context["error_message"])
 
-    def test_package_subscription_page_subscribe_action(self) -> None:
-        """Test subscribing to a package via the package subscription page"""
-        url = reverse(
-            "webview:subscriptions:package", kwargs={"package_name": "firefox"}
-        )
-        response = self.client.post(url, {"action": "subscribe"})
-
-        # Should redirect back to the same page
-        self.assertEqual(response.status_code, 302)
-        self.assertIn("firefox", response.url)
-
-        # Follow redirect and check subscription status
-        response = self.client.get(response.url)
-        self.assertEqual(response.status_code, 200)
-        self.assertTrue(response.context["is_subscribed"])
-
-    def test_package_subscription_page_unsubscribe_action(self) -> None:
-        """Test unsubscribing from a package via the package subscription page"""
-        # First subscribe to the package
-        self.user.profile.package_subscriptions.append("firefox")
-        self.user.profile.save(update_fields=["package_subscriptions"])
-
-        url = reverse(
-            "webview:subscriptions:package", kwargs={"package_name": "firefox"}
-        )
-
-        # Verify initially subscribed
-        response = self.client.get(url)
-        self.assertTrue(response.context["is_subscribed"])
-
-        # Unsubscribe
-        response = self.client.post(url, {"action": "unsubscribe"})
-
-        # Should redirect back to the same page
-        self.assertEqual(response.status_code, 302)
-        self.assertIn("firefox", response.url)
-
-        # Follow redirect and check subscription status
-        response = self.client.get(response.url)
-        self.assertEqual(response.status_code, 200)
-        self.assertFalse(response.context["is_subscribed"])
-
 
 def test_maintainer_notification_many_packages_in_suggestion(
     make_suggestion: Callable[..., CVEDerivationClusterProposal],