Change migrate version

Author: intitni

…erviceUpdateMigration/MigrateTo230.swift …erviceUpdateMigration/MigrateTo240.swiftCore/Sources/ServiceUpdateMigration/MigrateTo230.swift renamed to Core/Sources/ServiceUpdateMigration/MigrateTo240.swift Core/Sources/ServiceUpdateMigration/MigrateTo230.swift renamed to Core/Sources/ServiceUpdateMigration/MigrateTo240.swift

@@ -3,9 +3,9 @@ import Foundation
 import Keychain
 import Preferences
 
-func migrateTo230(
+func migrateTo240(
     defaults: UserDefaults = .shared,
-    keychain: KeychainType = Keychain(scope: "apikey")
+    keychain: KeychainType = Keychain.apiKey
 ) throws {
     let key = UserDefaultPreferenceKeys().embeddingModels.key
     if defaults.value(forKey: key) != nil { return }

Core/Sources/ServiceUpdateMigration/ServiceUpdateMigrator.swift

@@ -26,8 +26,8 @@ public struct ServiceUpdateMigrator {
         if old <= 135 {
             try migrateFromLowerThanOrEqualToVersion135()
         }
-        if old <= 229 {
-            try migrateTo230()
+        if old < 240 {
+            try migrateTo240()
         }
     }
 }

…teMigrationTests/MigrateTo230Tests.swift …teMigrationTests/MigrateTo240Tests.swiftCore/Tests/ServiceUpdateMigrationTests/MigrateTo230Tests.swift renamed to Core/Tests/ServiceUpdateMigrationTests/MigrateTo240Tests.swift Core/Tests/ServiceUpdateMigrationTests/MigrateTo230Tests.swift renamed to Core/Tests/ServiceUpdateMigrationTests/MigrateTo240Tests.swift

@@ -4,17 +4,17 @@ import XCTest
 
 @testable import ServiceUpdateMigration
 
-final class MigrateTo230Tests: XCTestCase {
-    let userDefaults = UserDefaults(suiteName: "MigrateTo230Tests")!
+final class MigrateTo240Tests: XCTestCase {
+    let userDefaults = UserDefaults(suiteName: "MigrateTo240Tests")!
 
     override func tearDown() async throws {
-        userDefaults.removePersistentDomain(forName: "MigrateTo230Tests")
+        userDefaults.removePersistentDomain(forName: "MigrateTo240Tests")
     }
 
-    func test_migrateTo230_no_data_to_migrate() async throws {
+    func test_migrateTo240_no_data_to_migrate() async throws {
         let keychain = FakeKeyChain()
 
-        try migrateTo230(defaults: userDefaults, keychain: keychain)
+        try migrateTo240(defaults: userDefaults, keychain: keychain)
 
         XCTAssertTrue(try keychain.getAll().isEmpty, "No api key to migrate")
 
@@ -70,7 +70,7 @@ final class MigrateTo230Tests: XCTestCase {
         }
     }
 
-    func test_migrateTo230_migrate_data_use_openAI() async throws {
+    func test_migrateTo240_migrate_data_use_openAI() async throws {
         let keychain = FakeKeyChain()
 
         userDefaults.set("Key1", forKey: "OpenAIAPIKey")
@@ -85,7 +85,7 @@ final class MigrateTo230Tests: XCTestCase {
         userDefaults.set("openAI", forKey: "ChatFeatureProvider")
         userDefaults.set("openAI", forKey: "EmbeddingFeatureProvider")
 
-        try migrateTo230(defaults: userDefaults, keychain: keychain)
+        try migrateTo240(defaults: userDefaults, keychain: keychain)
 
         XCTAssertEqual(try keychain.getAll(), [
             "OpenAI": "Key1",
@@ -156,7 +156,7 @@ final class MigrateTo230Tests: XCTestCase {
         }
     }
 
-    func test_migrateTo230_migrate_data_use_azureOpenAI() async throws {
+    func test_migrateTo240_migrate_data_use_azureOpenAI() async throws {
         let keychain = FakeKeyChain()
 
         userDefaults.set("Key1", forKey: "OpenAIAPIKey")
@@ -171,7 +171,7 @@ final class MigrateTo230Tests: XCTestCase {
         userDefaults.set("azureOpenAI", forKey: "ChatFeatureProvider")
         userDefaults.set("azureOpenAI", forKey: "EmbeddingFeatureProvider")
 
-        try migrateTo230(defaults: userDefaults, keychain: keychain)
+        try migrateTo240(defaults: userDefaults, keychain: keychain)
 
         let chatModels = userDefaults.value(for: \.chatModels)
         let embeddingModels = userDefaults.value(for: \.embeddingModels)