@sprucelabs/data-stores 28.3.270 → 28.3.272

package/build/esm/tests/databaseAssertUtil.js

@@ -603,7 +603,7 @@ const databaseAssertUtil = {
             assert.isEqual(updated.name, 'updated', 'updateOne() did not update the fields. I tried to update the name field to "updated".');
             assert.isEqual(updated.isPublic, true, 'Make sure you return existing fields from updateOne()! So, the actual record.');
             const match = yield db.findOne(this.collectionName, { id: updated.id });
-            assert.isEqualDeep(match, updated, 'The record in the database did not actually update!');
+            assert.isEqualDeep(match, Object.assign(Object.assign({}, inserted), { name: 'updated' }), 'The record in the database did not actually updated! Check that fields are not lost as well as updated.');
             yield this.shutdown(db);
         });
     },
@@ -1408,9 +1408,9 @@ const databaseAssertUtil = {
                 undefinedField: undefined,
                 name: generateId(),
             });
-            assert.isTruthy(createdUndefined);
+            assert.isTruthy(createdUndefined, 'upsertOne() should have returned a record.');
             assert.isTrue(createdUndefined.undefinedField === null ||
-                createdUndefined.undefinedfield === null);
+                createdUndefined.undefinedfield === null, 'upsertOne() should make undefined values as null.');
             const createdNull = yield db.upsertOne(this.collectionName, {
                 nullField: null,
                 name: '234',
@@ -1418,21 +1418,21 @@ const databaseAssertUtil = {
                 nullField: null,
                 name: generateId(),
             });
-            assert.isTruthy(createdNull);
+            assert.isTruthy(createdNull, 'upsertOne() should have returned a record.');
             let all = yield db.find(this.collectionName);
             assert.isLength(all, 2, 'I expected to find 2 records after 2 upserts.');
             const updatedUndefined = yield db.upsertOne(this.collectionName, {
                 undefinedField: undefined,
             }, { undefinedField: 'now defined' });
-            assert.isEqual(updatedUndefined.id, createdUndefined.id);
-            assert.isEqual((_a = updatedUndefined.undefinedField) !== null && _a !== void 0 ? _a : updatedUndefined.undefinedfield, 'now defined');
+            assert.isEqual(updatedUndefined.id, createdUndefined.id, 'upsertOne() should have returned the same record.');
+            assert.isEqual((_a = updatedUndefined.undefinedField) !== null && _a !== void 0 ? _a : updatedUndefined.undefinedfield, 'now defined', 'upsertOne() should have made the updates i asked for.');
             const updatedNull = yield db.upsertOne(this.collectionName, {
                 nullField: null,
             }, { nullField: 'now defined' });
-            assert.isEqual(updatedNull.id, createdNull.id);
-            assert.isEqual((_b = updatedNull.nullField) !== null && _b !== void 0 ? _b : updatedNull.nullfield, 'now defined', 'nullField should have upserted to "now defined"');
+            assert.isEqual(updatedNull.id, createdNull.id, "upsertOne() should have returned the same record. I'm querying based on a null value.");
+            assert.isEqual((_b = updatedNull.nullField) !== null && _b !== void 0 ? _b : updatedNull.nullfield, 'now defined', 'upsertOne() should have made the updates i asked for.');
             all = yield db.find(this.collectionName);
-            assert.isLength(all, 2);
+            assert.isLength(all, 2, 'I expected to find 2 records after 2 upserts. Make sure queries are honoring null values.');
             yield this.shutdown(db);
         });
     },

package/build/tests/databaseAssertUtil.js

@@ -548,7 +548,7 @@ const databaseAssertUtil = {
         test_utils_1.assert.isEqual(updated.name, 'updated', 'updateOne() did not update the fields. I tried to update the name field to "updated".');
         test_utils_1.assert.isEqual(updated.isPublic, true, 'Make sure you return existing fields from updateOne()! So, the actual record.');
         const match = await db.findOne(this.collectionName, { id: updated.id });
-        test_utils_1.assert.isEqualDeep(match, updated, 'The record in the database did not actually update!');
+        test_utils_1.assert.isEqualDeep(match, Object.assign(Object.assign({}, inserted), { name: 'updated' }), 'The record in the database did not actually updated! Check that fields are not lost as well as updated.');
         await this.shutdown(db);
     },
     async assertCanSortById(connect) {
@@ -1289,9 +1289,9 @@ const databaseAssertUtil = {
             undefinedField: undefined,
             name: (0, generateId_1.default)(),
         });
-        test_utils_1.assert.isTruthy(createdUndefined);
+        test_utils_1.assert.isTruthy(createdUndefined, 'upsertOne() should have returned a record.');
         test_utils_1.assert.isTrue(createdUndefined.undefinedField === null ||
-            createdUndefined.undefinedfield === null);
+            createdUndefined.undefinedfield === null, 'upsertOne() should make undefined values as null.');
         const createdNull = await db.upsertOne(this.collectionName, {
             nullField: null,
             name: '234',
@@ -1299,21 +1299,21 @@ const databaseAssertUtil = {
             nullField: null,
             name: (0, generateId_1.default)(),
         });
-        test_utils_1.assert.isTruthy(createdNull);
+        test_utils_1.assert.isTruthy(createdNull, 'upsertOne() should have returned a record.');
         let all = await db.find(this.collectionName);
         test_utils_1.assert.isLength(all, 2, 'I expected to find 2 records after 2 upserts.');
         const updatedUndefined = await db.upsertOne(this.collectionName, {
             undefinedField: undefined,
         }, { undefinedField: 'now defined' });
-        test_utils_1.assert.isEqual(updatedUndefined.id, createdUndefined.id);
-        test_utils_1.assert.isEqual((_a = updatedUndefined.undefinedField) !== null && _a !== void 0 ? _a : updatedUndefined.undefinedfield, 'now defined');
+        test_utils_1.assert.isEqual(updatedUndefined.id, createdUndefined.id, 'upsertOne() should have returned the same record.');
+        test_utils_1.assert.isEqual((_a = updatedUndefined.undefinedField) !== null && _a !== void 0 ? _a : updatedUndefined.undefinedfield, 'now defined', 'upsertOne() should have made the updates i asked for.');
         const updatedNull = await db.upsertOne(this.collectionName, {
             nullField: null,
         }, { nullField: 'now defined' });
-        test_utils_1.assert.isEqual(updatedNull.id, createdNull.id);
-        test_utils_1.assert.isEqual((_b = updatedNull.nullField) !== null && _b !== void 0 ? _b : updatedNull.nullfield, 'now defined', 'nullField should have upserted to "now defined"');
+        test_utils_1.assert.isEqual(updatedNull.id, createdNull.id, "upsertOne() should have returned the same record. I'm querying based on a null value.");
+        test_utils_1.assert.isEqual((_b = updatedNull.nullField) !== null && _b !== void 0 ? _b : updatedNull.nullfield, 'now defined', 'upsertOne() should have made the updates i asked for.');
         all = await db.find(this.collectionName);
-        test_utils_1.assert.isLength(all, 2);
+        test_utils_1.assert.isLength(all, 2, 'I expected to find 2 records after 2 upserts. Make sure queries are honoring null values.');
         await this.shutdown(db);
     },
     async assertCanSaveAndGetNullAndUndefined(connect) {

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "28.3.270",
+  "version": "28.3.272",
   "files": [
     "build/**/*",
     "!build/__tests__",