Merge pull request #13 from textury/fix-tags

fix: fix length of undefined error
textury · Feb 16, 2022 · 818b90c · 818b90c
2 parents 46649b8 + de2b23a
commit 818b90c
Show file tree

Hide file tree

Showing 2 changed files with 51 additions and 1 deletion.
diff --git a/src/__tests__/ardb.ts b/src/__tests__/ardb.ts
@@ -140,6 +140,31 @@ describe('USING ARWEAVE', () => {
 
     expect(txs[0].id).not.toBe('5AYV-RdPCoyfjeeabHVnDGvXrFNM5azcTlkNpp7RQhE');
   });
+
+  test('tags', async () => {
+    let txs = await ardb
+      .search('transactions')
+      .appName('SmartWeaveAction')
+      .tag('Type', 'ArweaveActivity')
+      .only('id')
+      .sort('HEIGHT_ASC')
+      .find();
+
+    let tagsTx = await ardb
+      .search('transactions')
+      .tag('Type', 'ArweaveActivity')
+      .tags([
+        {
+          name: 'App-Name',
+          values: 'SmartWeaveAction'
+        }
+      ])
+      .only('id')
+      .sort('HEIGHT_ASC')
+      .find();
+
+      expect(txs.length).toEqual(tagsTx.length)
+  })
 });
 
 describe('USING BLOCKWEAVE', () => {
@@ -275,4 +300,29 @@ describe('USING BLOCKWEAVE', () => {
 
     expect(txs[0].id).not.toBe('5AYV-RdPCoyfjeeabHVnDGvXrFNM5azcTlkNpp7RQhE');
   });
+
+  test('tags', async () => {
+    let txs = await ardb
+      .search('transactions')
+      .appName('SmartWeaveAction')
+      .tag('Type', 'ArweaveActivity')
+      .only('id')
+      .sort('HEIGHT_ASC')
+      .find();
+
+    let tagsTx = await ardb
+      .search('transactions')
+      .tag('Type', 'ArweaveActivity')
+      .tags([
+        {
+          name: 'App-Name',
+          values: 'SmartWeaveAction'
+        }
+      ])
+      .only('id')
+      .sort('HEIGHT_ASC')
+      .find();
+
+      expect(txs.length).toEqual(tagsTx.length)
+  })
 });
diff --git a/src/ardb.ts b/src/ardb.ts
@@ -154,7 +154,7 @@ export default class ArDB {
       });
     }
 
-    const $oTags = this.options.tags.length ? this.options.tags : [];
+    const $oTags = this.options.tags && this.options.tags.length ? this.options.tags : [];
     this.options.tags = [...ts, ...$oTags];
     return this;
   }