review: routing changed to include testUser

guardian · Feb 4, 2025 · 2353b7c · 2353b7c
1 parent 870fd43
commit 2353b7c
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 22 deletions.
diff --git a/support-frontend/app/controllers/SubscriptionsController.scala b/support-frontend/app/controllers/SubscriptionsController.scala
@@ -14,7 +14,7 @@ import config.StringsConfig
 import lib.RedirectWithEncodedQueryString
 import play.api.mvc._
 import play.twirl.api.Html
-import services.CachedProductCatalogServiceProvider
+import services.{CachedProductCatalogServiceProvider, TestUserService}
 import services.pricing.{PriceSummary, PriceSummaryServiceProvider}
 import views.EmptyDiv
 import views.ViewHelpers.outputJson
@@ -30,6 +30,7 @@ class SubscriptionsController(
     settingsProvider: AllSettingsProvider,
     val supportUrl: String,
     stage: Stage,
+    testUserService: TestUserService,
     cachedProductCatalogServiceProvider: CachedProductCatalogServiceProvider,
 )(implicit val ec: ExecutionContext)
     extends AbstractController(components)
@@ -93,29 +94,31 @@ class SubscriptionsController(
     ) ++ paperMap
   }
 
-  def landing(countryCode: String): Action[AnyContent] = CachedAction() { implicit request =>
-    implicit val settings: AllSettings = settingsProvider.getAllSettings()
-    val title = "Support the Guardian | Get a Subscription"
-    val mainElement = EmptyDiv("subscriptions-landing-page")
-    val js = "subscriptionsLandingPage.js"
-    val pricingCopy = CountryGroup.byId(countryCode).map(getLandingPrices)
-    val productCatalog = cachedProductCatalogServiceProvider.fromStage(stage, false).get()
-
-    Ok(
-      views.html.main(
-        title,
-        mainElement,
-        RefPath(js),
-        Some(RefPath("subscriptionsLandingPage.css")),
-        description = stringsConfig.subscriptionsLandingDescription,
-        noindex = stage != PROD,
-      ) {
-        Html(s"""<script type="text/javascript">
+  def landing(countryCode: String): Action[AnyContent] = {
+    MaybeAuthenticatedAction { implicit request =>
+      implicit val settings: AllSettings = settingsProvider.getAllSettings()
+      val title = "Support the Guardian | Get a Subscription"
+      val mainElement = EmptyDiv("subscriptions-landing-page")
+      val js = "subscriptionsLandingPage.js"
+      val pricingCopy = CountryGroup.byId(countryCode).map(getLandingPrices)
+      val isTestUser = testUserService.isTestUser(request)
+      val productCatalog = cachedProductCatalogServiceProvider.fromStage(stage, isTestUser).get()
+
+      Ok(
+        views.html.main(
+          title,
+          mainElement,
+          RefPath(js),
+          Some(RefPath("subscriptionsLandingPage.css")),
+          description = stringsConfig.subscriptionsLandingDescription,
+          noindex = stage != PROD,
+        ) {
+          Html(s"""<script type="text/javascript">
               window.guardian.pricingCopy = ${outputJson(pricingCopy)};
               window.guardian.productCatalog = ${outputJson(productCatalog)}
             </script>""")
-      },
-    ).withSettingsSurrogateKey
+        },
+      ).withSettingsSurrogateKey
+    }
   }
-
 }
diff --git a/support-frontend/app/wiring/Controllers.scala b/support-frontend/app/wiring/Controllers.scala
@@ -62,6 +62,7 @@ trait Controllers {
     allSettingsProvider,
     appConfig.supportUrl,
     appConfig.stage,
+    testUsers,
     cachedProductCatalogServiceProvider,
   )