Skip to content

Commit 40268f3

Browse files
committed
Fix test failure that is caused by remote call moved to an async Task.
1 parent e9bd05f commit 40268f3

File tree

1 file changed

+15
-12
lines changed

1 file changed

+15
-12
lines changed

Yosemite/YosemiteTests/Stores/ProductStore+FilterProductsTests.swift

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -285,18 +285,21 @@ final class ProductStore_FilterProductsTests: XCTestCase {
285285
remote.whenLoadingAllProducts(siteID: sampleSiteID, thenReturn: .success([Product.fake()]))
286286

287287
// When
288-
let synchronizeAction = ProductAction.synchronizeProducts(siteID: sampleSiteID,
289-
pageNumber: defaultPageNumber,
290-
pageSize: defaultPageSize,
291-
stockStatus: filteredStockStatus,
292-
productStatus: filteredProductStatus,
293-
productType: filteredProductType,
294-
productCategory: filteredProductCategory,
295-
sortOrder: filteredProductSortOrder,
296-
productIDs: [1, 2],
297-
excludedProductIDs: [30, 45],
298-
onCompletion: { _ in })
299-
productStore.onAction(synchronizeAction)
288+
waitFor { promise in
289+
productStore.onAction(ProductAction.synchronizeProducts(siteID: self.sampleSiteID,
290+
pageNumber: self.defaultPageNumber,
291+
pageSize: self.defaultPageSize,
292+
stockStatus: filteredStockStatus,
293+
productStatus: filteredProductStatus,
294+
productType: filteredProductType,
295+
productCategory: filteredProductCategory,
296+
sortOrder: filteredProductSortOrder,
297+
productIDs: [1, 2],
298+
excludedProductIDs: [30, 45],
299+
onCompletion: { _ in
300+
promise(())
301+
}))
302+
}
300303

301304
// Then
302305
XCTAssertTrue(remote.synchronizeProductsTriggered)

0 commit comments

Comments
 (0)