Change subscript

Author: 1amageek

Ballcap/DataRepresentable.swift

@@ -111,12 +111,15 @@ public extension DataRepresentable where Self: Object {
         }
     }
 
-    subscript<T: Any>(keyPath: WritableKeyPath<Model, T>) -> T? {
+    subscript<T: Any>(keyPath: WritableKeyPath<Model, T>) -> T {
         get {
-            return self.data?[keyPath: keyPath]
+            guard let data = self.data else {
+                fatalError("[Ballcap: DataRepresentable] This object has not data.")
+            }
+            return data[keyPath: keyPath]
         }
         set {
-            self.data?[keyPath: keyPath] = newValue!
+            self.data![keyPath: keyPath] = newValue
         }
     }
 

BallcapTests/CodableDocumentTests.swift

@@ -311,7 +311,7 @@ class CodableDocumentTests: XCTestCase {
             var ref: DocumentReference = Firestore.firestore().document("a/a")
         }
         let document: Document<Model> = Document()
-        XCTAssertEqual(document[\.ref]?.path, "a/a")
+        XCTAssertEqual(document[\.ref].path, "a/a")
     }
 
     func testDocumentSubScriptRefWrite() {

BallcapTests/ObjectTests.swift

@@ -89,6 +89,8 @@ class ObjectTests: XCTestCase {
         }
         let o: Obj = Obj(id: "a")
         XCTAssertEqual(o[\.path], "a")
+        o[\.path] = "b"
+        XCTAssertEqual(o[\.path], "b")
     }
 
     func testObjectSaveUpdateDelete() {