diff options
| author | Jonatan Rhodin <jonatan.rhodin@mullvad.net> | 2024-10-08 16:47:36 +0200 |
|---|---|---|
| committer | Jonatan Rhodin <jonatan.rhodin@mullvad.net> | 2024-10-09 16:21:01 +0200 |
| commit | cfe99ea983818b9462a9240c67ba7d2c28ba6c94 (patch) | |
| tree | 06b46a345d96b0aa3283faeffc9f3fc5f58e550f | |
| parent | 892da35cbc36db7b733fb4416b5c16189571896e (diff) | |
| download | mullvadvpn-cfe99ea983818b9462a9240c67ba7d2c28ba6c94.tar.xz mullvadvpn-cfe99ea983818b9462a9240c67ba7d2c28ba6c94.zip | |
Fix unit tests
| -rw-r--r-- | android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/ConnectViewModelTest.kt | 1 | ||||
| -rw-r--r-- | android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModelTest.kt | 16 |
2 files changed, 13 insertions, 4 deletions
diff --git a/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/ConnectViewModelTest.kt b/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/ConnectViewModelTest.kt index 267813d62a..9696a30539 100644 --- a/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/ConnectViewModelTest.kt +++ b/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/ConnectViewModelTest.kt @@ -192,6 +192,7 @@ class ConnectViewModelTest { country = "Sweden", city = "Gothenburg", hostname = "Host", + entryHostname = "EntryHost", latitude = 57.7065, longitude = 11.967, ) diff --git a/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModelTest.kt b/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModelTest.kt index d04cdc5d26..340809fbb3 100644 --- a/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModelTest.kt +++ b/android/app/src/test/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModelTest.kt @@ -33,6 +33,7 @@ import net.mullvad.mullvadvpn.lib.model.WireguardTunnelOptions import net.mullvad.mullvadvpn.repository.AutoStartAndConnectOnBootRepository import net.mullvad.mullvadvpn.repository.RelayListRepository import net.mullvad.mullvadvpn.repository.SettingsRepository +import net.mullvad.mullvadvpn.repository.WireguardConstraintsRepository import net.mullvad.mullvadvpn.usecase.SystemVpnSettingsAvailableUseCase import org.junit.jupiter.api.AfterEach import org.junit.jupiter.api.BeforeEach @@ -49,6 +50,7 @@ class VpnSettingsViewModelTest { private val mockRelayListRepository: RelayListRepository = mockk() private val mockAutoStartAndConnectOnBootRepository: AutoStartAndConnectOnBootRepository = mockk() + private val mockWireguardConstraintsRepository: WireguardConstraintsRepository = mockk() private val mockSettingsUpdate = MutableStateFlow<Settings?>(null) private val portRangeFlow = MutableStateFlow(emptyList<PortRange>()) @@ -70,6 +72,7 @@ class VpnSettingsViewModelTest { relayListRepository = mockRelayListRepository, dispatcher = UnconfinedTestDispatcher(), autoStartAndConnectOnBootRepository = mockAutoStartAndConnectOnBootRepository, + wireguardConstraintsRepository = mockWireguardConstraintsRepository, ) } @@ -179,12 +182,17 @@ class VpnSettingsViewModelTest { } @Test - fun `onWireguardPortSelected should invoke updateSelectedWireguardConstraint with Constraint Only with same port`() = + fun `onWireguardPortSelected should invoke setWireguardPort with Constraint Only with same port`() = runTest { // Arrange val wireguardPort: Constraint<Port> = Constraint.Only(Port(99)) - val wireguardConstraints = WireguardConstraints(port = wireguardPort) - coEvery { mockRelayListRepository.updateSelectedWireguardConstraints(any()) } returns + val wireguardConstraints = + WireguardConstraints( + port = wireguardPort, + useMultihop = false, + entryLocation = Constraint.Any, + ) + coEvery { mockWireguardConstraintsRepository.setWireguardPort(any()) } returns Unit.right() // Act @@ -192,7 +200,7 @@ class VpnSettingsViewModelTest { // Assert coVerify(exactly = 1) { - mockRelayListRepository.updateSelectedWireguardConstraints(wireguardConstraints) + mockWireguardConstraintsRepository.setWireguardPort(wireguardConstraints.port) } } |
