frequency lowercase for user communication

guardian · Sep 9, 2024 · dc5ebc7 · dc5ebc7
1 parent 5b01d20
commit dc5ebc7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lambda/src/main/scala/pricemigrationengine/model/BillingPeriod.scala b/lambda/src/main/scala/pricemigrationengine/model/BillingPeriod.scala
@@ -13,11 +13,11 @@ object BillingPeriod {
   val notificationPaymentFrequencyMapping = Map(
     // This map is used to convert a CohortItem's billingPeriod in to the user friendly representation in letters
     // and emails.
-    "Month" -> "Monthly",
-    "Quarter" -> "Quarterly",
-    "Quarterly" -> "Quarterly",
-    "Semi_Annual" -> "Semiannually",
-    "Annual" -> "Annually"
+    "Month" -> "monthly",
+    "Quarter" -> "quarterly",
+    "Quarterly" -> "quarterly",
+    "Semi_Annual" -> "semiannually",
+    "Annual" -> "annually"
   )
 
   def toString(period: BillingPeriod): String = {

diff --git a/lambda/src/test/scala/pricemigrationengine/handlers/NotificationHandlerTest.scala b/lambda/src/test/scala/pricemigrationengine/handlers/NotificationHandlerTest.scala
@@ -23,7 +23,7 @@ class NotificationHandlerTest extends munit.FunSuite {
   private val startDateUserFriendlyFormat = "1 January 2020"
   private val currency = "GBP"
   private val billingPeriod = "Month"
-  private val billingPeriodInNotification = "Monthly"
+  private val billingPeriodInNotification = "monthly" // lowercase variation for the user communication
   private val oldPrice = BigDecimal(10.00)
 
   // The estimated new price is the price without cap