Merge pull request #3 from JimRoepcke/feature/do-not-use-set-in-index…

…esWithKeys Do not use Set(keys) in indexesWithKeys
JimRoepcke · Feb 13, 2018 · 86eb549 · 86eb549
2 parents 4d56966 + 2a2aa21
commit 86eb549
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Sources/YapDatabaseExtensions.swift b/Sources/YapDatabaseExtensions.swift
@@ -233,7 +233,7 @@ extension Persistable {
         Keys>(_ keys: Keys) -> [YapDB.Index] where
         Keys: Sequence,
         Keys.Iterator.Element == String {
-            return Set(keys).map { YapDB.Index(collection: collection, key: $0) }
+            return keys.map { YapDB.Index(collection: collection, key: $0) }
     }
 
     /**

diff --git a/Tests/YapDatabaseExtensionsTests.swift b/Tests/YapDatabaseExtensionsTests.swift
@@ -48,7 +48,7 @@ class PersistableTests: XCTestCase {
     func test__indexes_from_keys() {
         let keys = [ "beatle-1", "beatle-2", "beatle-3", "beatle-4", "beatle-2" ]
         let indexes = Person.indexesWithKeys(keys)
-        XCTAssertEqual(indexes.count, 4)
+        XCTAssertEqual(indexes.count, 5)
     }
 }