WIP - hugs.localdomain Wed, Jun 05 04:56 pm

[no-changelog-required]

Author: airhorns

spec/class-model-snapshotted-views.spec.ts

@@ -24,10 +24,13 @@ class Outer extends ClassModel({ name: types.string, examples: types.array(ViewE
 }
 
 describe("class model snapshotted views", () => {
-  describe.each([
-    ["read-only", true],
-    ["observable", false],
-  ])("%s", (_name, readOnly) => {
+  // describe.each([
+  //   ["read-only", true],
+  //   ["observable", false],
+  // ])("%s", (_name, readOnly) => {
+  describe("whatever", () => {
+    const readOnly = false;
+
     test("instances don't require the snapshot to include the cache", () => {
       const instance = create(ViewExample, { key: "1", name: "Test" }, readOnly);
       expect(instance.slug).toEqual("test");
@@ -114,6 +117,7 @@ describe("class model snapshotted views", () => {
     //   expect(AllOptionalB.is(instanceB)).toBe(false);
     // });
   });
+
   test("an observable instance saves the view value in a snapshot when changed", () => {
     const instance = ViewExample.create({ key: "1", name: "Test" });
     expect(instance.slug).toEqual("test");

src/class-model.ts

@@ -517,7 +517,10 @@ export const isClassModel = (type: IAnyType): type is IClassModelType<any, any,
 const buildSnapshottedViewProcessor = (snapshottedViews: SnapshottedViewMetadata[], mstType: MSTIModelType<any, any>) => {
   return mstTypes.snapshotProcessor(mstType, {
     postProcessor(snapshot, node) {
-      if (!node) return snapshot; // FIXME -- undesirable to ever run without a node because we are thusly not putting a snapshot of the view in the snapshot
+      if (!node) {
+        debugger;
+        return snapshot; // FIXME -- undesirable to ever run without a node because we are thusly not putting a snapshot of the view in the snapshot
+      }
       for (const snapshottedView of snapshottedViews) {
         storeViewOnSnapshot(node, snapshottedView, snapshot);
       }