Use factory for autofill prefs to fix tests

duckduckgo · Sep 19, 2024 · 64e0d24 · 64e0d24
1 parent d0b7ece
commit 64e0d24
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/DuckDuckGo/DataImport/Model/DataImportViewModel.swift b/DuckDuckGo/DataImport/Model/DataImportViewModel.swift
@@ -133,9 +133,11 @@ struct DataImportViewModel {
 
 #endif
 
- lazy var isPasswordManagerAutolockEnabled: Bool = {
- AutofillPreferences().isAutoLockEnabled
- }()
+ var isPasswordManagerAutolockEnabled: Bool {
+ autofillPreferencesFactory().isAutoLockEnabled
+ }
+
+ private let autofillPreferencesFactory: () -> AutofillPreferencesPersistor
 
  init(importSource: Source? = nil,
  screen: Screen? = nil,
@@ -147,6 +149,7 @@ struct DataImportViewModel {
  requestPrimaryPasswordCallback: @escaping @MainActor (Source) -> String? = Self.requestPrimaryPasswordCallback,
  openPanelCallback: @escaping @MainActor (DataType) -> URL? = Self.openPanelCallback,
  reportSenderFactory: @escaping ReportSenderFactory = { FeedbackSender().sendDataImportReport },
+ autofillPreferencesFactory: @escaping () -> AutofillPreferencesPersistor = { AutofillPreferences() },
  onFinished: @escaping () -> Void = {},
  onCancelled: @escaping () -> Void = {}) {
 
@@ -161,6 +164,7 @@ struct DataImportViewModel {
 
  self.browserProfiles = ThirdPartyBrowser.browser(for: importSource).map(loadProfiles)
  self.selectedProfile = browserProfiles?.defaultProfile
+ self.autofillPreferencesFactory = autofillPreferencesFactory
 
  self.selectedDataTypes = importSource.supportedDataTypes