Fix unit tests

Author: hichamboushaba

plugins/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/di/WCDatabaseModule.kt

@@ -5,11 +5,13 @@ import dagger.Binds
 import dagger.Module
 import dagger.Provides
 import org.wordpress.android.fluxc.module.DatabaseModule
+import org.wordpress.android.fluxc.persistence.OrderSqlUtils
 import org.wordpress.android.fluxc.persistence.TransactionExecutor
 import org.wordpress.android.fluxc.persistence.WCAndroidDatabase
 import org.wordpress.android.fluxc.persistence.dao.AddonsDao
 import org.wordpress.android.fluxc.persistence.dao.CouponsDao
 import org.wordpress.android.fluxc.persistence.dao.OrdersDao
+import javax.inject.Inject
 import javax.inject.Singleton
 
 @Module(
@@ -49,6 +51,11 @@ interface WCDatabaseModule {
 
         @Provides fun provideWooPaymentsDepositsOverviewDao(database: WCAndroidDatabase) =
             database.wooPaymentsDepositsOverviewDao
+
+        /**
+         * OrderSqlUtils is a Kotlin object, we can't use [Inject] to inject it.
+         */
+        @Provides fun provideOrderSqlUtils() = OrderSqlUtils
     }
     @Binds fun bindTransactionExecutor(database: WCAndroidDatabase): TransactionExecutor
 }

plugins/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/store/OrderUpdateStore.kt

@@ -38,7 +38,8 @@ class OrderUpdateStore @Inject internal constructor(
     private val coroutineEngine: CoroutineEngine,
     private val wcOrderRestClient: OrderRestClient,
     private val ordersDaoDecorator: OrdersDaoDecorator,
-    private val siteSqlUtils: SiteSqlUtils
+    private val siteSqlUtils: SiteSqlUtils,
+    private val orderSqlUtils: OrderSqlUtils
 ) {
     suspend fun updateCustomerOrderNote(
         orderId: Long,
@@ -277,7 +278,7 @@ class OrderUpdateStore @Inject internal constructor(
                 WooResult(result.error)
             } else {
                 ordersDaoDecorator.deleteOrder(site.localId(), orderId)
-                OrderSqlUtils.deleteOrderSummaryById(site, orderId)
+                orderSqlUtils.deleteOrderSummaryById(site, orderId)
                 WooResult(Unit)
             }
         }

plugins/woocommerce/src/test/java/org/wordpress/android/fluxc/store/OrderUpdateStoreTest.kt

@@ -25,6 +25,7 @@ import org.wordpress.android.fluxc.network.rest.wpcom.wc.order.CoreOrderStatus
 import org.wordpress.android.fluxc.network.rest.wpcom.wc.order.OrderDto.Billing
 import org.wordpress.android.fluxc.network.rest.wpcom.wc.order.OrderDto.Shipping
 import org.wordpress.android.fluxc.network.rest.wpcom.wc.order.OrderRestClient
+import org.wordpress.android.fluxc.persistence.OrderSqlUtils
 import org.wordpress.android.fluxc.persistence.SiteSqlUtils
 import org.wordpress.android.fluxc.persistence.dao.OrdersDaoDecorator
 import org.wordpress.android.fluxc.store.WCOrderStore.OnOrderChanged
@@ -45,6 +46,7 @@ class OrderUpdateStoreTest {
     private val siteSqlUtils: SiteSqlUtils = mock {
         on { getSiteWithLocalId(any()) } doReturn site
     }
+    private val orderSqlUtils: OrderSqlUtils = mock()
     private val ordersDaoDecorator: OrdersDaoDecorator = mock {
         onBlocking { getOrder(TEST_REMOTE_ORDER_ID, TEST_LOCAL_SITE_ID) } doReturn initialOrder
     }
@@ -56,9 +58,10 @@ class OrderUpdateStoreTest {
                         TestCoroutineScope().coroutineContext,
                         mock()
                 ),
-                orderRestClient,
-                ordersDaoDecorator,
-                siteSqlUtils
+                wcOrderRestClient = orderRestClient,
+                ordersDaoDecorator = ordersDaoDecorator,
+                siteSqlUtils = siteSqlUtils,
+                orderSqlUtils = orderSqlUtils
         )
     }
 
@@ -518,6 +521,7 @@ class OrderUpdateStoreTest {
         )
 
         verify(ordersDaoDecorator).deleteOrder(site.localId(), TEST_REMOTE_ORDER_ID)
+        verify(orderSqlUtils).deleteOrderSummaryById(site, TEST_REMOTE_ORDER_ID)
     }
 
     private companion object {