Skip to content

Commit e0692c4

Browse files
author
Sharma Elanthiriayan
committed
OrderTaxLine tests in OrdersUpsertUseCaseTests
- Remove `makeOrderTaxLine()` method and `.fake()` to generate tax lines.
1 parent a0d1c2c commit e0692c4

File tree

1 file changed

+3
-15
lines changed

1 file changed

+3
-15
lines changed

Yosemite/YosemiteTests/Stores/Order/OrdersUpsertUseCaseTests.swift

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ final class OrdersUpsertUseCaseTests: XCTestCase {
123123

124124
func test_it_persists_order_tax_line_in_storage() throws {
125125
// Given
126-
let taxLine = makeOrderTaxLine().copy(taxID: 1)
126+
let taxLine = OrderTaxLine.fake().copy(taxID: 1)
127127
let order = makeOrder().copy(siteID: 3, taxes: [taxLine])
128128
let useCase = OrdersUpsertUseCase(storage: viewStorage)
129129

@@ -137,13 +137,13 @@ final class OrdersUpsertUseCaseTests: XCTestCase {
137137

138138
func test_it_replaces_existing_order_tax_line_in_storage() throws {
139139
// Given
140-
let originalTaxLine = makeOrderTaxLine().copy(taxID: 1, ratePercent: 0.0)
140+
let originalTaxLine = OrderTaxLine.fake().copy(taxID: 1, ratePercent: 0.0)
141141
let order = makeOrder().copy(siteID: 3, taxes: [originalTaxLine])
142142
let useCase = OrdersUpsertUseCase(storage: viewStorage)
143143
useCase.upsert([order])
144144

145145
// When
146-
let taxLine = makeOrderTaxLine().copy(taxID: 1, ratePercent: 5.0)
146+
let taxLine = OrderTaxLine.fake().copy(taxID: 1, ratePercent: 5.0)
147147
useCase.upsert([order.copy(taxes: [taxLine])])
148148

149149
// Then
@@ -257,16 +257,4 @@ private extension OrdersUpsertUseCaseTests {
257257
totalTax: "0.00",
258258
attributes: attributes)
259259
}
260-
261-
func makeOrderTaxLine() -> Networking.OrderTaxLine {
262-
.init(taxID: 0,
263-
rateCode: "TAX",
264-
rateID: 0,
265-
label: "Tax label",
266-
isCompoundTaxRate: true,
267-
totalTax: "0.0",
268-
totalShippingTax: "0.0",
269-
ratePercent: 0.0,
270-
attributes: [])
271-
}
272260
}

0 commit comments

Comments
 (0)