betterddb 0.5.2 → 0.5.4

package/lib/builders/batch-get-builder.js

@@ -16,9 +16,21 @@ class BatchGetBuilder {
      * Returns an array of parsed items of type T.
      */
     async execute() {
+        if (this.keys.length === 0) {
+            return [];
+        }
+        const seen = new Set();
+        const deduplicatedKeys = this.keys.filter((key) => {
+            const keyString = JSON.stringify(key);
+            if (seen.has(keyString)) {
+                return false;
+            }
+            seen.add(keyString);
+            return true;
+        });
         const tableName = this.parent.getTableName();
         // Build an array of keys using the parent's key builder.
-        const keysArray = this.keys.map(key => this.parent.buildKey(key));
+        const keysArray = deduplicatedKeys.map(key => this.parent.buildKey(key));
         // Construct the BatchGet parameters.
         const params = {
             RequestItems: {

package/lib/builders/query-builder.js

@@ -144,7 +144,6 @@ class QueryBuilder {
         this.expressionAttributeNames['#entity'] = 'entityType';
         this.expressionAttributeValues[':entity_value'] = this.parent.getEntityType();
         params.FilterExpression = this.filterConditions.join(' AND ');
-        console.log(params);
         const result = await this.parent.getClient().send(new lib_dynamodb_1.QueryCommand(params));
         return { items: this.parent.getSchema().array().parse(result.Items), lastKey: (_c = result.LastEvaluatedKey) !== null && _c !== void 0 ? _c : undefined };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "betterddb",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "description": "A definition-based DynamoDB wrapper library that provides a schema-driven and fully typesafe DAL.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

package/src/builders/batch-get-builder.ts

@@ -14,9 +14,22 @@ export class BatchGetBuilder<T> {
    * Returns an array of parsed items of type T.
    */
   public async execute(): Promise<T[]> {
+    if (this.keys.length === 0) {
+      return [];
+    }
+
+    const seen = new Set();
+    const deduplicatedKeys = this.keys.filter((key) => {
+      const keyString = JSON.stringify(key);
+      if (seen.has(keyString)) {
+        return false;
+      }
+      seen.add(keyString);
+      return true;
+    });
     const tableName = this.parent.getTableName();
     // Build an array of keys using the parent's key builder.
-    const keysArray = this.keys.map(key => this.parent.buildKey(key));
+    const keysArray = deduplicatedKeys.map(key => this.parent.buildKey(key));
 
     // Construct the BatchGet parameters.
     const params: BatchGetItemInput = {

package/src/builders/query-builder.ts

@@ -168,7 +168,6 @@ export class QueryBuilder<T> {
     this.expressionAttributeValues[':entity_value'] = this.parent.getEntityType();
     params.FilterExpression = this.filterConditions.join(' AND ');
 
-    console.log(params);
     const result = await this.parent.getClient().send(new QueryCommand(params));
     return {items: this.parent.getSchema().array().parse(result.Items) as T[], lastKey: result.LastEvaluatedKey ?? undefined};
   }

package/test/batch-get.test.ts

@@ -78,4 +78,20 @@ describe('BetterDDB - Get Operation', () => {
     expect(users.some(user => user.id === 'user-123')).toBe(true);
     expect(users.some(user => user.id === 'user-124')).toBe(true);
   });
+
+  it('should retrieve an item using GetBuilder that does not exist', async () => {
+    const users = await userDdb.batchGet([{ id: 'user-123', email: 'jane@example.com' }]).execute();
+    expect(users.length).toEqual(0);
+  });
+
+  it('should return an empty array if no keys are provided', async () => {
+    const users = await userDdb.batchGet([]).execute();
+    expect(users.length).toEqual(0);
+  });
+
+  it('should deduplicate keys', async () => {
+    const users = await userDdb.batchGet([{ id: 'user-123', email: 'john@example.com' }, { id: 'user-123', email: 'john@example.com' }]).execute();
+    expect(users.length).toEqual(1);
+    expect(users[0].id).toEqual('user-123');
+  });
 });

package/test/get.test.ts

@@ -77,4 +77,9 @@ describe('BetterDDB - Get Operation', () => {
     expect(user).not.toBeNull();
     expect(user?.id).toBe('user-123');
   });
+
+  it('should retrieve an item using GetBuilder that does not exist', async () => {
+    const user = await userDdb.get({ id: 'user-123', email: 'jane@example.com' }).execute();
+    expect(user).toBeNull();
+  });
 });

package/test/query.test.ts

@@ -179,4 +179,14 @@ describe('BetterDDB - Query Operation', () => {
       expect(result.name).toContain('B');
     });
   });
+
+  it('should query items using QueryBuilder with where clause on an index', async () => {
+    // Query the GSI for email "alice@example.com". Two items match.
+    // Then apply two filter conditions: name begins_with "Alice" and name contains "B" should only match one.
+    const results = await userDdb.query({ email: 'alice@example.com' })
+      .usingIndex('EmailIndex')
+      .where('begins_with', { email: 'alice' })
+      .execute();
+    expect(results.items.length).toEqual(2);
+  });
 });