Fix SQLite Cache Key Bug

Sources/ApolloSQLite/SQLiteNormalizedCache.swift

@@ -33,11 +33,19 @@ public final class SQLiteNormalizedCache {
   }
 
   private func recordCacheKey(forFieldCacheKey fieldCacheKey: CacheKey) -> CacheKey {
-    var components = fieldCacheKey.components(separatedBy: ".")
-    if components.count > 1 {
-      components.removeLast()
+    let components = fieldCacheKey.components(separatedBy: ".")
+    var updatedComponents = [String]()
+    for component in components {
+      if updatedComponents.last?.last?.isNumber ?? false && component.first?.isNumber ?? false {
+        updatedComponents[updatedComponents.count - 1].append(".\(component)")
+      } else {
+        updatedComponents.append(component)
+      }
+    }
+    if updatedComponents.count > 1 {
+      updatedComponents.removeLast()
     }
-    return components.joined(separator: ".")
+    return updatedComponents.joined(separator: ".")
   }
 
   private func createTableIfNeeded() throws {

Tests/ApolloCacheDependentTests/LoadQueryFromStoreTests.swift

@@ -319,7 +319,44 @@ class LoadQueryFromStoreTests: XCTestCase {
       }
     }
   }
-
+
+
+  func testLoadingQueryWithFloats() throws {
+    let starshipLength = 1234.5
+
+    let initialRecords: RecordSet = [
+      "QUERY_ROOT": ["starshipLength(length:1234.5)": Reference(key: "starshipLength(length:1234.5)")],
+      "starshipLength(length:1234.5)": ["__typename": "Starship",
+                                        "name": "Millennium Falcon",
+                                        "length": starshipLength,
+                                        "coordinates": [[0.0,1.0], [2.0,3.0]]]
+    ]
+
+
+    withCache(initialRecords: initialRecords) { (cache) in
+      store = ApolloStore(cache: cache)
+
+      let query = StarshipLengthQuery(length: starshipLength)
+
+      load(query: query) { result in
+        switch result {
+        case .success(let graphQLResult):
+          XCTAssertNil(graphQLResult.errors)
+
+          guard let data = graphQLResult.data else {
+            XCTFail("No data returned with result")
+            return
+          }
+
+          XCTAssertEqual(data.starshipLength?.name, "Millennium Falcon")
+          XCTAssertEqual(data.starshipLength?.length, starshipLength)
+        case .failure(let error):
+          XCTFail("Unexpected error: \(error)")
+        }
+      }
+    }
+  }
+
   // MARK: - Helpers
 
   private func load<Query: GraphQLQuery>(query: Query, resultHandler: @escaping GraphQLResultHandler<Query.Data>) {

Tests/StarWarsAPI/API.swift

@@ -5548,6 +5548,121 @@ public final class StarshipQuery: GraphQLQuery {
   }
 }
 
+public final class StarshipLengthQuery: GraphQLQuery {
+  /// The raw GraphQL definition of this operation.
+  public let operationDefinition =
+    """
+    query StarshipLength($length: Float!) {
+      starshipLength(length: $length) {
+        __typename
+        name
+        coordinates
+        length
+      }
+    }
+    """
+
+  public let operationName = "StarshipLength"
+
+  public let operationIdentifier: String? = "f890b11aaa5d41e6d53ced4549beb27b9157c8c301d47e076909262ed9edbfb1"
+
+  public var length: Double
+
+  public init(length: Double) {
+    self.length = length
+  }
+
+  public var variables: GraphQLMap? {
+    return ["length": length]
+  }
+
+  public struct Data: GraphQLSelectionSet {
+    public static let possibleTypes = ["Query"]
+
+    public static let selections: [GraphQLSelection] = [
+      GraphQLField("starshipLength", arguments: ["length": GraphQLVariable("length")], type: .object(StarshipLength.selections)),
+    ]
+
+    public private(set) var resultMap: ResultMap
+
+    public init(unsafeResultMap: ResultMap) {
+      self.resultMap = unsafeResultMap
+    }
+
+    public init(starshipLength: StarshipLength? = nil) {
+      self.init(unsafeResultMap: ["__typename": "Query", "starshipLength": starshipLength.flatMap { (value: StarshipLength) -> ResultMap in value.resultMap }])
+    }
+
+    public var starshipLength: StarshipLength? {
+      get {
+        return (resultMap["starshipLength"] as? ResultMap).flatMap { StarshipLength(unsafeResultMap: $0) }
+      }
+      set {
+        resultMap.updateValue(newValue?.resultMap, forKey: "starshipLength")
+      }
+    }
+
+    public struct StarshipLength: GraphQLSelectionSet {
+      public static let possibleTypes = ["Starship"]
+
+      public static let selections: [GraphQLSelection] = [
+        GraphQLField("__typename", type: .nonNull(.scalar(String.self))),
+        GraphQLField("name", type: .nonNull(.scalar(String.self))),
+        GraphQLField("coordinates", type: .list(.nonNull(.list(.nonNull(.scalar(Double.self)))))),
+        GraphQLField("length", type: .scalar(Double.self)),
+      ]
+
+      public private(set) var resultMap: ResultMap
+
+      public init(unsafeResultMap: ResultMap) {
+        self.resultMap = unsafeResultMap
+      }
+
+      public init(name: String, coordinates: [[Double]]? = nil, length: Double? = nil) {
+        self.init(unsafeResultMap: ["__typename": "Starship", "name": name, "coordinates": coordinates, "length": length])
+      }
+
+      public var __typename: String {
+        get {
+          return resultMap["__typename"]! as! String
+        }
+        set {
+          resultMap.updateValue(newValue, forKey: "__typename")
+        }
+      }
+
+      /// The name of the starship
+      public var name: String {
+        get {
+          return resultMap["name"]! as! String
+        }
+        set {
+          resultMap.updateValue(newValue, forKey: "name")
+        }
+      }
+
+      public var coordinates: [[Double]]? {
+        get {
+          return resultMap["coordinates"] as? [[Double]]
+        }
+        set {
+          resultMap.updateValue(newValue, forKey: "coordinates")
+        }
+      }
+
+      /// Length of the starship, along the longest axis
+      public var length: Double? {
+        get {
+          return resultMap["length"] as? Double
+        }
+        set {
+          resultMap.updateValue(newValue, forKey: "length")
+        }
+      }
+    }
+  }
+}
+
 public final class ReviewAddedSubscription: GraphQLSubscription {
   /// The raw GraphQL definition of this operation.
   public let operationDefinition =

Tests/StarWarsAPI/Starship.graphql

@@ -4,3 +4,11 @@ query Starship {
     coordinates
   }
 }
+
+query StarshipLength($length: Float!) {
+  starshipLength(length: $length) {
+    name
+    coordinates
+    length
+  }
+}

Tests/StarWarsAPI/operationIdsPath.json

@@ -127,6 +127,10 @@
     "name": "Starship",
     "source": "query Starship {\n  starship(id: 3000) {\n    __typename\n    name\n    coordinates\n  }\n}"
   },
+  "f890b11aaa5d41e6d53ced4549beb27b9157c8c301d47e076909262ed9edbfb1": {
+    "name": "StarshipLength",
+    "source": "query StarshipLength($length: Float!) {\n  starshipLength(length: $length) {\n    __typename\n    name\n    coordinates\n    length\n  }\n}"
+  },
   "38644c5e7cf4fd506b91d2e7010cabf84e63dfcd33cf1deb443b4b32b55e2cbe": {
     "name": "ReviewAdded",
     "source": "subscription ReviewAdded($episode: Episode) {\n  reviewAdded(episode: $episode) {\n    __typename\n    episode\n    stars\n    commentary\n  }\n}"

Tests/StarWarsAPI/schema.json

@@ -204,6 +204,33 @@
               },
               "isDeprecated": false,
               "deprecationReason": null
+            },
+            {
+              "name": "starshipLength",
+              "description": "",
+              "args": [
+                {
+                  "name": "length",
+                  "description": "",
+                  "type": {
+                    "kind": "NON_NULL",
+                    "name": null,
+                    "ofType": {
+                      "kind": "SCALAR",
+                      "name": "Float",
+                      "ofType": null
+                    }
+                  },
+                  "defaultValue": null
+                }
+              ],
+              "type": {
+                "kind": "OBJECT",
+                "name": "Starship",
+                "ofType": null
+              },
+              "isDeprecated": false,
+              "deprecationReason": null
             }
           ],
           "inputFields": null,
@@ -2184,4 +2211,4 @@
       ]
     }
   }
-}
+}