Skip to content

Commit ea05a97

Browse files
author
Sharma Elanthiriayan
committed
SimplePaymentsSummaryViewModelTests fix test_noteAdded_event_is_tracked_after_editing_note test
- Mock `getSimplePaymentsTaxesToggleState` to ensure that toggle state is fixed inside the test. - Expect `simplePaymentsFlowTaxesToggled` event in received events as view model loads the toggle state during initialization.
1 parent 2da8a4e commit ea05a97

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

WooCommerce/WooCommerceTests/ViewRelated/Orders/Simple Payments/SimplePaymentsSummaryViewModelTests.swift

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,18 +231,34 @@ final class SimplePaymentsSummaryViewModelTests: XCTestCase {
231231

232232
func test_noteAdded_event_is_tracked_after_editing_note() {
233233
// Given
234+
let mockStores = MockStoresManager(sessionManager: .testingInstance)
235+
mockStores.whenReceivingAction(ofType: AppSettingsAction.self) { action in
236+
switch action {
237+
case let .getSimplePaymentsTaxesToggleState(_, onCompletion):
238+
onCompletion(.success(true))
239+
case .setSimplePaymentsTaxesToggleState:
240+
break // No op
241+
default:
242+
XCTFail("Unexpected action: \(action)")
243+
}
244+
}
245+
234246
let mockAnalytics = MockAnalyticsProvider()
235247
let viewModel = SimplePaymentsSummaryViewModel(providedAmount: "1.0",
236248
totalWithTaxes: "1.0",
237249
taxAmount: "0.0",
250+
stores: mockStores,
238251
analytics: WooAnalytics(analyticsProvider: mockAnalytics))
239252

240253
// When
241254
viewModel.noteViewModel.newNote = "content"
242255
viewModel.noteViewModel.updateNote(onFinish: { _ in })
243256

244257
// Then
245-
assertEqual(mockAnalytics.receivedEvents, [WooAnalyticsStat.simplePaymentsFlowNoteAdded.rawValue])
258+
assertEqual(mockAnalytics.receivedEvents, [
259+
WooAnalyticsStat.simplePaymentsFlowTaxesToggled.rawValue, // Event triggered when view model loads the toggle state during initialization
260+
WooAnalyticsStat.simplePaymentsFlowNoteAdded.rawValue
261+
])
246262
}
247263

248264
func test_taxesToggled_event_is_tracked_after_switching_taxes_toggle() {

0 commit comments

Comments
 (0)