From f09ce7ffcfedc77369f3b28ed7290d4aa34b9df5 Mon Sep 17 00:00:00 2001 From: Leendert de Borst Date: Thu, 6 Nov 2025 13:49:19 +0100 Subject: [PATCH] Update swift CredentialMatcher tests (#1335) --- .../ios/VaultUI/Selection/CredentialProviderView.swift | 2 +- ...ntialFilterTests.swift => CredentialMatcherTests.swift} | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) rename apps/mobile-app/ios/VaultUITests/{CredentialFilterTests.swift => CredentialMatcherTests.swift} (98%) diff --git a/apps/mobile-app/ios/VaultUI/Selection/CredentialProviderView.swift b/apps/mobile-app/ios/VaultUI/Selection/CredentialProviderView.swift index 6b46dff70..b2e72a19d 100644 --- a/apps/mobile-app/ios/VaultUI/Selection/CredentialProviderView.swift +++ b/apps/mobile-app/ios/VaultUI/Selection/CredentialProviderView.swift @@ -266,7 +266,7 @@ public class CredentialProviderViewModel: ObservableObject { } func filterCredentials() { - filteredCredentials = CredentialFilter.filterCredentials(credentials, searchText: searchText) + filteredCredentials = CredentialMatcher.filterCredentials(credentials, searchText: searchText) } func handleSelection(username: String, password: String) { diff --git a/apps/mobile-app/ios/VaultUITests/CredentialFilterTests.swift b/apps/mobile-app/ios/VaultUITests/CredentialMatcherTests.swift similarity index 98% rename from apps/mobile-app/ios/VaultUITests/CredentialFilterTests.swift rename to apps/mobile-app/ios/VaultUITests/CredentialMatcherTests.swift index 32c026fb1..a0da12533 100644 --- a/apps/mobile-app/ios/VaultUITests/CredentialFilterTests.swift +++ b/apps/mobile-app/ios/VaultUITests/CredentialMatcherTests.swift @@ -2,7 +2,7 @@ import XCTest @testable import VaultUI @testable import VaultModels -final class CredentialFilterTests: XCTestCase { +final class CredentialMatcherTests: XCTestCase { private var testCredentials: [Credential] = [] override func setUp() { @@ -220,8 +220,8 @@ final class CredentialFilterTests: XCTestCase { // Test that dumpert.nl credential doesn't match nl.marktplaats.android package // They both contain "nl" in the name but shouldn't match since "nl" is just a TLD let reversedDomainCredentials = [ - createTestCredential(serviceName: "Dumpert.nl", serviceUrl: "", username: "user@dumpert.nl"), - createTestCredential(serviceName: "Marktplaats.nl", serviceUrl: "", username: "user@marktplaats.nl"), + createTestCredential(serviceName: "Dumpert.nl", serviceUrl: "dumpert.nl", username: "user@dumpert.nl"), + createTestCredential(serviceName: "Marktplaats.nl", serviceUrl: "nl.marktplaats.android", username: "user@marktplaats.nl"), ] let matches = CredentialMatcher.filterCredentials(reversedDomainCredentials, searchText: "nl.marktplaats.android") @@ -277,6 +277,7 @@ final class CredentialFilterTests: XCTestCase { createTestCredential(serviceName: "Bank Account", serviceUrl: "https://secure-bank.com", username: "user@bank.com"), createTestCredential(serviceName: "AliExpress", serviceUrl: "https://aliexpress.com", username: "user@aliexpress.com"), createTestCredential(serviceName: "Reddit", serviceUrl: "", username: "user@reddit.com"), + createTestCredential(serviceName: "Marktplaats", serviceUrl: "nl.marktplaats.android", username: "user@marktplaats.nl"), ] }