From e0aff91fa43829fd27b57fcb38439a02b537029d Mon Sep 17 00:00:00 2001 From: Bug Magnet Date: Thu, 26 Oct 2023 16:51:08 +0200 Subject: [PATCH] PR Feedback --- .../MigrationManagerTests.swift | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/ios/MullvadVPNTests/MigrationManagerTests.swift b/ios/MullvadVPNTests/MigrationManagerTests.swift index 4ef4e227e2af..9a3242d73747 100644 --- a/ios/MullvadVPNTests/MigrationManagerTests.swift +++ b/ios/MullvadVPNTests/MigrationManagerTests.swift @@ -45,13 +45,13 @@ final class MigrationManagerTests: XCTestCase { let settings = TunnelSettingsV3() try SettingsManager.writeSettings(settings) - let nothingToMigrationExpectation = expectation(description: "No migration") + let nothingToMigrateExpectation = expectation(description: "No migration") manager.migrateSettings(store: store, proxyFactory: proxyFactory) { result in if case .nothing = result { - nothingToMigrationExpectation.fulfill() + nothingToMigrateExpectation.fulfill() } } - wait(for: [nothingToMigrationExpectation], timeout: 1) + wait(for: [nothingToMigrateExpectation], timeout: 1) } func testNothingToMigrateIfRecordedSettingsVersionHigherThanLatestSettings() throws { @@ -59,26 +59,26 @@ final class MigrationManagerTests: XCTestCase { let settings = FutureVersionSettings() try write(settings: settings, version: Int.max - 1, in: store) - let nothingToMigrationExpectation = expectation(description: "No migration") + let nothingToMigrateExpectation = expectation(description: "No migration") manager.migrateSettings(store: store, proxyFactory: proxyFactory) { result in if case .nothing = result { - nothingToMigrationExpectation.fulfill() + nothingToMigrateExpectation.fulfill() } } - wait(for: [nothingToMigrationExpectation], timeout: 1) + wait(for: [nothingToMigrateExpectation], timeout: 1) } func testNothingToMigrateWhenSettingsAreNotFound() throws { let store = InMemorySettingsStore() SettingsManager.unitTestStore = store - let nothingToMigrationExpectation = expectation(description: "No migration") + let nothingToMigrateExpectation = expectation(description: "No migration") manager.migrateSettings(store: store, proxyFactory: proxyFactory) { result in if case .nothing = result { - nothingToMigrationExpectation.fulfill() + nothingToMigrateExpectation.fulfill() } } - wait(for: [nothingToMigrationExpectation], timeout: 1) + wait(for: [nothingToMigrateExpectation], timeout: 1) // Reset the `SettingsManager` unit test store to avoid affecting other tests // since it's a globally shared instance