Fix condition check for Pay at location status

Author: AdamGrzybkowski

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

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

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

@@ -228,7 +228,7 @@ class BookingMapperTest : BaseUnitTest() {
     }
 
     @Test
-    fun `given completed 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 PayAtLocation`() {
         // GIVEN
         val booking = sampleBooking().let { original ->
             val paymentInfo = BookingPaymentInfo(
@@ -239,7 +239,7 @@ class BookingMapperTest : BaseUnitTest() {
                 total = BigDecimal("55.00"),
                 totalTax = BigDecimal("0.00")
             )
-            val orderWithPayment = original.order.copy(paymentInfo = paymentInfo, status = "completed")
+            val orderWithPayment = original.order.copy(paymentInfo = paymentInfo, status = "processing")
             original.copy(order = orderWithPayment)
         }