Merge pull request #14809 from woocommerce/issue/WOOMOB-1569_pay_on_site

[WOOMOB-1569] - Rename Pay at location to Pay on site

Author: AdamGrzybkowski

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/BookingMapper.kt

@@ -113,7 +113,7 @@ class BookingMapper @Inject constructor(
         paymentMethod: String?,
     ): BookingStatus {
         return if (orderStatus != "completed" && paymentMethod == "cod") {
-            BookingStatus.PayAtLocation
+            BookingStatus.PayOnSite
         } else {
             when (this) {
                 BookingEntity.Status.Paid -> BookingStatus.Paid

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/compose/BookingStatusTag.kt

@@ -26,7 +26,7 @@ fun BookingStatusTag(
 }
 
 sealed interface BookingStatus {
-    data object PayAtLocation : BookingStatus
+    data object PayOnSite : BookingStatus
     data object Unpaid : BookingStatus
     data object PendingConfirmation : BookingStatus
     data object Confirmed : BookingStatus
@@ -45,15 +45,15 @@ private fun BookingStatus.text(): String {
         BookingStatus.Paid -> stringResource(R.string.booking_payment_status_paid)
         BookingStatus.Cancelled -> stringResource(R.string.booking_payment_status_cancelled)
         BookingStatus.Complete -> stringResource(R.string.booking_payment_status_complete)
-        BookingStatus.PayAtLocation -> stringResource(R.string.booking_payment_status_pay_at_location)
+        BookingStatus.PayOnSite -> stringResource(R.string.booking_payment_status_pay_on_site)
         is BookingStatus.Unknown -> key
     }
 }
 
 @Composable
 fun BookingStatus.backgroundColor(): Color {
     return when (this) {
-        BookingStatus.PayAtLocation -> R.color.tag_bg_booking_yellow
+        BookingStatus.PayOnSite -> R.color.tag_bg_booking_yellow
         else -> R.color.tagView_bg
     }.let { colorResource(it) }
 }
@@ -70,10 +70,10 @@ private fun PaymentStatusTagPreview() {
 
 @LightDarkThemePreviews
 @Composable
-private fun PaymentStatusTagPayAtLocationPreview() {
+private fun PaymentStatusTagPayOnSitePreview() {
     WooThemeWithBackground {
         BookingStatusTag(
-            state = BookingStatus.PayAtLocation
+            state = BookingStatus.PayOnSite
         )
     }
 }

WooCommerce/src/main/res/values/strings.xml

@@ -4254,7 +4254,7 @@
     <string name="booking_payment_status_cancelled">Cancelled</string>
     <string name="booking_payment_status_confirmed">Confirmed</string>
     <string name="booking_payment_status_complete">Complete</string>
-    <string name="booking_payment_status_pay_at_location">Pay at location</string>
+    <string name="booking_payment_status_pay_on_site">Pay on site</string>
     <string name="booking_attendance_status_booked">Booked</string>
     <string name="booking_attendance_status_checked_in">Checked-in</string>
     <string name="booking_attendance_status_cancelled">Cancelled</string>

WooCommerce/src/test/kotlin/com/woocommerce/android/ui/bookings/BookingMapperTest.kt

@@ -255,7 +255,7 @@ class BookingMapperTest : BaseUnitTest() {
     }
 
     @Test
-    fun `given processing order with COD payment method, when mapped to summary model, then status is PayAtLocation`() {
+    fun `given processing order with COD payment method, when mapped to summary model, then status is PayOnSite`() {
         // GIVEN
         val booking = sampleBooking().let { original ->
             val paymentInfo = BookingPaymentInfo(
@@ -274,7 +274,7 @@ class BookingMapperTest : BaseUnitTest() {
         val model = mapper.run { booking.toBookingSummaryModel(AttendanceUpdateStatus.Idle) }
 
         // THEN
-        assertThat(model.status).isEqualTo(BookingStatus.PayAtLocation)
+        assertThat(model.status).isEqualTo(BookingStatus.PayOnSite)
     }
 
     private fun sampleBooking(