Skip to content

Commit 2482e41

Browse files
committed
Fix unit tests
1 parent de0f2bf commit 2482e41

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

Networking/NetworkingTests/Mapper/ReportOrderMapperTests.swift

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ import XCTest
66
///
77
class ReportOrderMapperTests: XCTestCase {
88

9+
/// Sample SiteID
10+
///
11+
let siteID = 1234
12+
913
/// Verifies that the broken response causes the mapper to return an unknown status
1014
///
1115
func testBrokenResponseReturnsUnknownStatus() {
@@ -43,8 +47,8 @@ class ReportOrderMapperTests: XCTestCase {
4347
XCTAssertNotNil(orderStatuses)
4448
XCTAssertEqual(orderStatuses.count, 9)
4549

46-
let ciaOrderStatus = OrderStatus(name: "CIA Investigation", slug: "cia-investigation", total: 10)
47-
let preorderedOrderStatus = OrderStatus(name: "Pre ordered", slug: "pre-ordered", total: 1)
50+
let ciaOrderStatus = OrderStatus(name: "CIA Investigation", siteID: 1234, slug: "cia-investigation", total: 10)
51+
let preorderedOrderStatus = OrderStatus(name: "Pre ordered", siteID: 1234, slug: "pre-ordered", total: 1)
4852

4953
XCTAssertEqual(orderStatuses[0].slug, "pending")
5054
XCTAssertEqual(orderStatuses[0].name, "Pending payment")
@@ -93,7 +97,7 @@ private extension ReportOrderMapperTests {
9397
///
9498
func mapOrderStatusResult(from filename: String) throws -> [OrderStatus] {
9599
let response = Loader.contentsOf(filename)!
96-
let mapper = ReportOrderTotalsMapper()
100+
let mapper = ReportOrderTotalsMapper(siteID: 1234)
97101
return try mapper.map(response: response)
98102
}
99103

0 commit comments

Comments
 (0)