@fireproof/core 0.20.0-dev-preview-40 → 0.20.0-dev-preview-50

package/tests/fireproof/database.test.ts

@@ -59,19 +59,19 @@ describe("basic Ledger with record", function () {
     await db.close();
     await db.destroy();
   });
-  beforeEach(async function () {
+  beforeEach(async () => {
     await sthis.start();
     db = fireproof("factory-name");
     const ok = await db.put<Doc>({ _id: "hello", value: "world" });
     expect(ok.id).toBe("hello");
   });
-  it("should get", async function () {
+  it("should get", async () => {
     const doc = await db.get<Doc>("hello");
     expect(doc).toBeTruthy();
     expect(doc._id).toBe("hello");
     expect(doc.value).toBe("world");
   });
-  it("should update", async function () {
+  it("should update", async () => {
     const ok = await db.put({ _id: "hello", value: "universe" });
     expect(ok.id).toBe("hello");
     const doc = await db.get<Doc>("hello");
@@ -79,20 +79,20 @@ describe("basic Ledger with record", function () {
     expect(doc._id).toBe("hello");
     expect(doc.value).toBe("universe");
   });
-  it("should del last record", async function () {
+  it("should del last record", async () => {
     const ok = await db.del("hello");
     expect(ok.id).toBe("hello");
 
     const e = await db.get("hello").catch((e) => e);
     expect(e.message).toMatch(/Not found/);
   });
-  it("has changes", async function () {
+  it("has changes", async () => {
     const { rows } = await db.changes([]);
     expect(rows.length).toBe(1);
     expect(rows[0].key).toBe("hello");
     expect(rows[0].value._id).toBe("hello");
   });
-  it("is not persisted", async function () {
+  it("is not persisted", async () => {
     const db2 = fireproof("factory-name");
     const { rows } = await db2.changes([]);
     expect(rows.length).toBe(1);
@@ -114,7 +114,7 @@ describe("named Ledger with record", function () {
     await db.close();
     await db.destroy();
   });
-  beforeEach(async function () {
+  beforeEach(async () => {
     await sthis.start();
     db = fireproof("test-db-name");
     /** @type {Doc} */
@@ -122,13 +122,13 @@ describe("named Ledger with record", function () {
     const ok = await db.put(doc);
     expect(ok.id).toBe("hello");
   });
-  it("should get", async function () {
+  it("should get", async () => {
     const doc = await db.get<Doc>("hello");
     expect(doc).toBeTruthy();
     expect(doc._id).toBe("hello");
     expect(doc.value).toBe("world");
   });
-  it("should update", async function () {
+  it("should update", async () => {
     const ok = await db.put({ _id: "hello", value: "universe" });
     expect(ok.id).toBe("hello");
     const doc = await db.get<Doc>("hello");
@@ -136,20 +136,61 @@ describe("named Ledger with record", function () {
     expect(doc._id).toBe("hello");
     expect(doc.value).toBe("universe");
   });
-  it("should del last record", async function () {
+  it("should update with null value", async function () {
+    const ok = await db.put({ _id: "hello", value: null });
+    expect(ok.id).toBe("hello");
+    const doc = await db.get<Doc>("hello");
+    expect(doc).toBeTruthy();
+    expect(doc._id).toBe("hello");
+    expect(doc.value).toBeNull();
+    expect(Object.keys(doc).includes("value")).toBeTruthy();
+  });
+  it("should update with undefined value", async function () {
+    const ok = await db.put({ _id: "hello", value: undefined });
+    expect(ok.id).toBe("hello");
+    const doc = await db.get<Doc>("hello");
+    expect(doc).toBeTruthy();
+    expect(doc._id).toBe("hello");
+
+    // expect 'value' not to be in keys
+    expect(Object.keys(doc).includes("value")).toBeFalsy();
+  });
+  it("should update with NaN value", async function () {
+    const ok = await db.put({ _id: "hello", value: NaN });
+    expect(ok.id).toBe("hello");
+    const doc = await db.get<Doc>("hello");
+    expect(doc).toBeTruthy();
+    expect(doc._id).toBe("hello");
+
+    // expect 'value' not to be in keys
+    expect(Object.keys(doc).includes("value")).toBeFalsy();
+  });
+  it("should not update with Infinity value", async function () {
+    const ok = await db.put({ _id: "hello", value: Infinity }).catch((e) => e);
+    expect(ok.message).toMatch(/IPLD/);
+  });
+  it("should not update with undefined array value", async function () {
+    const ok = await db.put({ _id: "hello", value: [undefined] }).catch((e) => e);
+    expect(ok.message).toMatch(/IPLD/);
+  });
+  it("should not update with NaN array value", async function () {
+    const ok = await db.put({ _id: "hello", value: [NaN] }).catch((e) => e);
+    expect(ok.message).toMatch(/IPLD/);
+  });
+  it("should del last record", async () => {
     const ok = await db.del("hello");
     expect(ok.id).toBe("hello");
 
     const e = await db.get("hello").catch((e) => e);
     expect(e.message).toMatch(/Not found/);
   });
-  it("has changes", async function () {
+  it("has changes", async () => {
     const { rows } = await db.changes([]);
     expect(rows.length).toBe(1);
     expect(rows[0].key).toBe("hello");
     expect(rows[0].value._id).toBe("hello");
   });
-  it("should have a key", async function () {
+  it("should have a key", async () => {
     const { rows } = await db.changes([]);
     expect(rows.length).toBe(1);
     const blocks = db.ledger.crdt.blockstore as bs.EncryptedBlockstore;
@@ -161,7 +202,7 @@ describe("named Ledger with record", function () {
     // expect(loader.keyId?.length).toBe(64);
     // expect(loader.key).not.toBe(loader.keyId);
   });
-  it("should work right with a sequence of changes", async function () {
+  it("should work right with a sequence of changes", async () => {
     const numDocs = 10;
     for (let i = 0; i < numDocs; i++) {
       const doc = { _id: `id-${i}`, hello: "world" };
@@ -198,7 +239,7 @@ describe("named Ledger with record", function () {
     expect(rows4.length).toBe(5);
   });
 
-  it("should work right after compaction", async function () {
+  it("should work right after compaction", async () => {
     const numDocs = 10;
     for (let i = 0; i < numDocs; i++) {
       const doc = { _id: `id-${i}`, hello: "world" };
@@ -222,7 +263,7 @@ describe("named Ledger with record", function () {
 //   /** @type {Ledger} */
 //   let db
 //   const writes = []
-//   beforeEach(async function () {
+//   beforeEach(async () =>{
 //     await resetDirectory(dataDir, 'test-parallel-writes')
 //     db = new Ledger('test-parallel-writes', { public: true })
 //     /** @type {Doc} */
@@ -256,7 +297,7 @@ describe("basic Ledger parallel writes / public ordered", () => {
     expect(crdt.clock.head.length).toBe(1);
   });
 
-  it("has changes ordered", async function () {
+  it("has changes ordered", async () => {
     const { rows, clock } = await db.changes([]);
     expect(clock[0]).toBe(db.ledger.crdt.clock.head[0]);
     expect(rows.length).toBe(10);
@@ -299,7 +340,7 @@ describe("basic Ledger parallel writes / public", () => {
       expect(doc.hello).toBe("world");
     }
   });
-  it("should del all", async function () {
+  it("should del all", async () => {
     for (let i = 0; i < 10; i++) {
       const id = `id-${i}`;
       const ok = await db.del(id);
@@ -309,7 +350,7 @@ describe("basic Ledger parallel writes / public", () => {
       expect(e.message).toMatch(/Not found/);
     }
   });
-  it("should delete all in parallel", async function () {
+  it("should delete all in parallel", async () => {
     const deletes: Promise<DocResponse>[] = [];
     for (let i = 0; i < 10; i++) {
       const id = `id-${i}`;
@@ -322,7 +363,7 @@ describe("basic Ledger parallel writes / public", () => {
       expect(e.message).toMatch(/Not found/);
     }
   });
-  it("has changes not ordered", async function () {
+  it("has changes not ordered", async () => {
     const { rows, clock } = await db.changes([]);
     expect(clock[0]).toBe(db.ledger.crdt.clock.head[0]);
     expect(rows.length).toBe(10);
@@ -333,7 +374,7 @@ describe("basic Ledger parallel writes / public", () => {
       expect(rows[i].clock).toBeTruthy();
     }
   });
-  it("should not have a key", async function () {
+  it("should not have a key", async () => {
     const { rows } = await db.changes([]);
     expect(rows.length).toBe(10);
     // expect(db.opts.public).toBeTruthy();
@@ -358,7 +399,7 @@ describe("basic Ledger with subscription", function () {
     await db.close();
     await db.destroy();
   });
-  beforeEach(async function () {
+  beforeEach(async () => {
     await sthis.start();
     db = fireproof("factory-name");
     didRun = 0;
@@ -371,7 +412,7 @@ describe("basic Ledger with subscription", function () {
       }, true);
     });
   });
-  it("should run on put", async function () {
+  it("should run on put", async () => {
     const all = await db.allDocs();
     expect(all.rows.length).toBe(0);
     const doc = { _id: "hello", message: "world" };
@@ -384,7 +425,7 @@ describe("basic Ledger with subscription", function () {
     expect(ok.id).toBe("hello");
     expect(didRun).toBe(1);
   });
-  it("should unsubscribe", async function () {
+  it("should unsubscribe", async () => {
     unsubscribe();
     const doc = { _id: "hello", message: "again" };
     const ok = await db.put(doc);
@@ -402,14 +443,14 @@ describe("basic Ledger with no update subscription", function () {
     await db.close();
     await db.destroy();
   });
-  beforeEach(async function () {
+  beforeEach(async () => {
     db = fireproof("factory-name");
     didRun = 0;
     unsubscribe = db.subscribe(() => {
       didRun++;
     });
   });
-  it("should run on put", async function () {
+  it("xshould run on put", async function () {
     const all = await db.allDocs();
     expect(all.rows.length).toBe(0);
     /** @type {Doc} */
@@ -418,8 +459,8 @@ describe("basic Ledger with no update subscription", function () {
     const ok = await db.put(doc);
     expect(ok.id).toBe("hello");
     expect(didRun).toBe(1);
-  });
-  it("should unsubscribe", async function () {
+  }, 10000);
+  it("should unsubscribe", async () => {
     unsubscribe();
     const doc = { _id: "hello", message: "again" };
     const ok = await db.put(doc);
@@ -438,7 +479,7 @@ describe("ledger with files input", () => {
     await db.close();
     await db.destroy();
   });
-  beforeEach(async function () {
+  beforeEach(async () => {
     await sthis.start();
     imagefiles = await buildBlobFiles();
     db = fireproof("fireproof-with-images");
@@ -453,11 +494,11 @@ describe("ledger with files input", () => {
     result = await db.put(doc);
   });
 
-  it("Should upload images", async function () {
+  it("Should upload images", async () => {
     expect(result.id).toBe("images-main");
   });
 
-  it("Should fetch the images", async function () {
+  it("Should fetch the images", async () => {
     const doc = await db.get(result.id);
     const files = doc._files as DocFiles;
     expect(files).toBeTruthy();
@@ -488,7 +529,7 @@ describe("ledger with files input", () => {
     // expect(file.name).toBe('fireproof.png') // see https://github.com/fireproof-storage/fireproof/issues/70
   });
 
-  it("should update the file document data without changing the files", async function () {
+  it("should update the file document data without changing the files", async () => {
     interface Doc {
       type: string;
     }
@@ -521,7 +562,7 @@ describe("ledger with files input", () => {
 
     expect(file.type).toBe(imagefiles[0].file.type);
     expect(file.size).toBe(imagefiles[0].file.size);
-  });
+  }, 1000000);
 });
 
 describe("StoreURIRuntime", () => {
@@ -551,14 +592,14 @@ describe("StoreURIRuntime", () => {
   it("default toStoreURIRuntime", () => {
     expect(JSON.parse(JSON.stringify(toStoreURIRuntime(sthis, "test")))).toEqual({
       data: {
-        data: "my://bla/storage?name=test&store=data&storekey=%40test-data%40&suffix=.car&urlGen=fromEnv",
-        file: "my://bla/storage?name=test&store=data&storekey=%40test-data%40&urlGen=fromEnv",
+        car: "my://bla/storage?name=test&store=car&storekey=%40test-data%40&suffix=.car&urlGen=fromEnv",
+        file: "my://bla/storage?name=test&store=file&storekey=%40test-data%40&urlGen=fromEnv",
         meta: "my://bla/storage?name=test&store=meta&storekey=%40test-meta%40&urlGen=fromEnv",
         wal: "my://bla/storage?name=test&store=wal&storekey=%40test-wal%40&urlGen=fromEnv",
       },
       idx: {
-        data: "my://bla/storage?index=idx&name=test&store=data&storekey=%40test-data-idx%40&suffix=.car&urlGen=fromEnv",
-        file: "my://bla/storage?index=idx&name=test&store=data&storekey=%40test-data-idx%40&urlGen=fromEnv",
+        car: "my://bla/storage?index=idx&name=test&store=car&storekey=%40test-data-idx%40&suffix=.car&urlGen=fromEnv",
+        file: "my://bla/storage?index=idx&name=test&store=file&storekey=%40test-data-idx%40&urlGen=fromEnv",
         meta: "my://bla/storage?index=idx&name=test&store=meta&storekey=%40test-meta-idx%40&urlGen=fromEnv",
         wal: "my://bla/storage?index=idx&name=test&store=wal&storekey=%40test-wal-idx%40&urlGen=fromEnv",
       },
@@ -568,14 +609,14 @@ describe("StoreURIRuntime", () => {
   it("no name toStoreURIRuntime(not more)", () => {
     expect(JSON.parse(JSON.stringify(toStoreURIRuntime(sthis, "gogo")))).toEqual({
       data: {
-        data: "my://bla/storage?name=gogo&store=data&storekey=%40gogo-data%40&suffix=.car&urlGen=fromEnv",
-        file: "my://bla/storage?name=gogo&store=data&storekey=%40gogo-data%40&urlGen=fromEnv",
+        car: "my://bla/storage?name=gogo&store=car&storekey=%40gogo-data%40&suffix=.car&urlGen=fromEnv",
+        file: "my://bla/storage?name=gogo&store=file&storekey=%40gogo-data%40&urlGen=fromEnv",
         meta: "my://bla/storage?name=gogo&store=meta&storekey=%40gogo-meta%40&urlGen=fromEnv",
         wal: "my://bla/storage?name=gogo&store=wal&storekey=%40gogo-wal%40&urlGen=fromEnv",
       },
       idx: {
-        data: "my://bla/storage?index=idx&name=gogo&store=data&storekey=%40gogo-data-idx%40&suffix=.car&urlGen=fromEnv",
-        file: "my://bla/storage?index=idx&name=gogo&store=data&storekey=%40gogo-data-idx%40&urlGen=fromEnv",
+        car: "my://bla/storage?index=idx&name=gogo&store=car&storekey=%40gogo-data-idx%40&suffix=.car&urlGen=fromEnv",
+        file: "my://bla/storage?index=idx&name=gogo&store=file&storekey=%40gogo-data-idx%40&urlGen=fromEnv",
         meta: "my://bla/storage?index=idx&name=gogo&store=meta&storekey=%40gogo-meta-idx%40&urlGen=fromEnv",
         wal: "my://bla/storage?index=idx&name=gogo&store=wal&storekey=%40gogo-wal-idx%40&urlGen=fromEnv",
       },
@@ -589,11 +630,11 @@ describe("StoreURIRuntime", () => {
           toStoreURIRuntime(sthis, "xxx", {
             base: "my://storage-base",
             data: {
-              data: "my://storage-data?name=yyy",
+              car: "my://storage-data?name=yyy",
               meta: "my://storage-meta",
             },
             idx: {
-              data: "my://storage-idx-data?name=yyy&index=bla",
+              car: "my://storage-idx-data?name=yyy&index=bla",
               meta: "my://storage-idx-meta",
             },
           }),
@@ -601,14 +642,14 @@ describe("StoreURIRuntime", () => {
       ),
     ).toEqual({
       data: {
-        data: "my://storage-data?name=yyy&store=data&storekey=%40yyy-data%40&suffix=.car",
-        file: "my://storage-data?name=yyy&store=data&storekey=%40yyy-data%40",
+        car: "my://storage-data?name=yyy&store=car&storekey=%40yyy-data%40&suffix=.car",
+        file: "my://storage-data?name=yyy&store=file&storekey=%40yyy-data%40",
         meta: "my://storage-meta?name=xxx&store=meta&storekey=%40xxx-meta%40",
         wal: "my://storage-base?name=xxx&store=wal&storekey=%40xxx-wal%40",
       },
       idx: {
-        data: "my://storage-idx-data?index=bla&name=yyy&store=data&storekey=%40yyy-data-idx%40&suffix=.car",
-        file: "my://storage-idx-data?index=bla&name=yyy&store=data&storekey=%40yyy-data-idx%40",
+        car: "my://storage-idx-data?index=bla&name=yyy&store=car&storekey=%40yyy-data-idx%40&suffix=.car",
+        file: "my://storage-idx-data?index=bla&name=yyy&store=file&storekey=%40yyy-data-idx%40",
         meta: "my://storage-idx-meta?index=idx&name=xxx&store=meta&storekey=%40xxx-meta-idx%40",
         wal: "my://storage-base?index=idx&name=xxx&store=wal&storekey=%40xxx-wal-idx%40",
       },
@@ -619,14 +660,14 @@ describe("StoreURIRuntime", () => {
     sthis.env.delete("FP_STORAGE_URL");
     expect(JSON.parse(JSON.stringify(toStoreURIRuntime(sthis, "maxi")))).toEqual({
       data: {
-        data: "murks://fp?name=maxi&store=data&storekey=%40maxi-data%40&suffix=.car&urlGen=default",
-        file: "murks://fp?name=maxi&store=data&storekey=%40maxi-data%40&urlGen=default",
+        car: "murks://fp?name=maxi&store=car&storekey=%40maxi-data%40&suffix=.car&urlGen=default",
+        file: "murks://fp?name=maxi&store=file&storekey=%40maxi-data%40&urlGen=default",
         meta: "murks://fp?name=maxi&store=meta&storekey=%40maxi-meta%40&urlGen=default",
         wal: "murks://fp?name=maxi&store=wal&storekey=%40maxi-wal%40&urlGen=default",
       },
       idx: {
-        data: "murks://fp?index=idx&name=maxi&store=data&storekey=%40maxi-data-idx%40&suffix=.car&urlGen=default",
-        file: "murks://fp?index=idx&name=maxi&store=data&storekey=%40maxi-data-idx%40&urlGen=default",
+        car: "murks://fp?index=idx&name=maxi&store=car&storekey=%40maxi-data-idx%40&suffix=.car&urlGen=default",
+        file: "murks://fp?index=idx&name=maxi&store=file&storekey=%40maxi-data-idx%40&urlGen=default",
         meta: "murks://fp?index=idx&name=maxi&store=meta&storekey=%40maxi-meta-idx%40&urlGen=default",
         wal: "murks://fp?index=idx&name=maxi&store=wal&storekey=%40maxi-wal-idx%40&urlGen=default",
       },
@@ -642,16 +683,16 @@ describe("StoreURIRuntime", () => {
         stores: [
           {
             data: {
-              data: "my://bla/storage?name=test&store=data&storekey=%40test-data%40&suffix=.car&urlGen=fromEnv",
-              file: "my://bla/storage?name=test&store=data&storekey=%40test-data%40&urlGen=fromEnv",
+              car: "my://bla/storage?name=test&store=car&storekey=%40test-data%40&suffix=.car&urlGen=fromEnv",
+              file: "my://bla/storage?name=test&store=file&storekey=%40test-data%40&urlGen=fromEnv",
               meta: "my://bla/storage?name=test&store=meta&storekey=%40test-meta%40&urlGen=fromEnv",
               wal: "my://bla/storage?name=test&store=wal&storekey=%40test-wal%40&urlGen=fromEnv",
             },
           },
           {
             idx: {
-              data: "my://bla/storage?index=idx&name=test&store=data&storekey=%40test-data-idx%40&suffix=.car&urlGen=fromEnv",
-              file: "my://bla/storage?index=idx&name=test&store=data&storekey=%40test-data-idx%40&urlGen=fromEnv",
+              car: "my://bla/storage?index=idx&name=test&store=car&storekey=%40test-data-idx%40&suffix=.car&urlGen=fromEnv",
+              file: "my://bla/storage?index=idx&name=test&store=file&storekey=%40test-data-idx%40&urlGen=fromEnv",
               meta: "my://bla/storage?index=idx&name=test&store=meta&storekey=%40test-meta-idx%40&urlGen=fromEnv",
               wal: "my://bla/storage?index=idx&name=test&store=wal&storekey=%40test-wal-idx%40&urlGen=fromEnv",
             },