Chores (typos, unused imports, etc.) and revert Web5 User Agent test …

…endpoints Signed-off-by: Frank Hinek <[email protected]>
decentralized-identity · May 17, 2023 · 561f475 · 561f475
1 parent 1ceea37
commit 561f475
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 14 deletions.
diff --git a/packages/dids/tests/did-key.spec.ts b/packages/dids/tests/did-key.spec.ts
@@ -4,7 +4,7 @@ import { DidKeyApi } from '../src/did-key.js';
 const DidKey = new DidKeyApi();
 
 describe('DidKeyApi', () => {
-  it.only('works', async () => {
+  it('works', async () => {
     const didState = await DidKey.create();
 
     expect(didState.id).to.exist;

diff --git a/packages/web5-user-agent/tests/common/web5-user-agent.spec.ts b/packages/web5-user-agent/tests/common/web5-user-agent.spec.ts
@@ -2,8 +2,10 @@ import { expect } from 'chai';
 import { Encoder, RecordsWriteMessage, RecordsRead } from '@tbd54566975/dwn-sdk-js';
 import { TestAgent } from './utils/test-user-agent.js';
 
-let testAgent: TestAgent;
 let did: string;
+let dwnNodes: string[] = ['https://dwn.tbddev.org/dwn0'];
+// let dwnNodes: string[] = ['http://localhost:3000'];
+let testAgent: TestAgent;
 
 describe('Web5UserAgent', () => {
   before(async () => {
@@ -47,8 +49,7 @@ describe('Web5UserAgent', () => {
             type            : 'dwn',
             id              : 'dwn',
             serviceEndpoint : {
-              // nodes: ['https://dwn.tbddev.org/dwn0']
-              nodes: ['http://localhost:3000']
+              nodes: dwnNodes
             }
           }]
         }
@@ -80,8 +81,7 @@ describe('Web5UserAgent', () => {
             type            : 'dwn',
             id              : 'dwn',
             serviceEndpoint : {
-              // nodes: ['https://dwn.tbddev.org/dwn0']
-              nodes: ['http://localhost:3000']
+              nodes: dwnNodes
             }
           }]
         }
@@ -101,22 +101,20 @@ describe('Web5UserAgent', () => {
       expect(response.reply.status.code).to.equal(404);
     });
 
-    it('returns something when an jwark is smorked', async () => {
+    it('throws an error when DwnRequest fails validation', async () => {
       const { did: aliceDid } = await testAgent.createProfile({
         profileDidOptions: {
           services: [{
             type            : 'dwn',
             id              : 'dwn',
             serviceEndpoint : {
-              // nodes: ['https://dwn.tbddev.org/dwn0']
-              nodes: ['http://localhost:3000']
+              nodes: dwnNodes
             }
           }]
         }
       });
 
       try {
-        // TODO: (Moe -> Frank): what should sendDwnRequest return and how does it propogate back up to the caller?
         await testAgent.agent.sendDwnRequest({
           author         : aliceDid,
           target         : aliceDid,
@@ -138,8 +136,7 @@ describe('Web5UserAgent', () => {
             type            : 'dwn',
             id              : 'dwn',
             serviceEndpoint : {
-              // nodes: ['https://dwn.tbddev.org/dwn0']
-              nodes: ['http://localhost:3000']
+              nodes: dwnNodes
             }
           }]
         }

diff --git a/packages/web5/src/dwn-api.ts b/packages/web5/src/dwn-api.ts
@@ -17,7 +17,7 @@ import { DwnInterfaceName, DwnMethodName } from '@tbd54566975/dwn-sdk-js';
 
 import { Record } from './record.js';
 import { Protocol } from './protocol.js';
-import { dataToBlob, isDataSizeUnderCacheLimit, isEmptyObject } from './utils.js';
+import { dataToBlob, isEmptyObject } from './utils.js';
 
 // TODO: Export type ProtocolsConfigureDescriptor from dwn-sdk-js.
 export type ProtocolsConfigureDescriptor = {

diff --git a/packages/web5/tests/web5-dwn.spec.ts b/packages/web5/tests/web5-dwn.spec.ts
@@ -44,7 +44,7 @@ describe('web5.dwn', () => {
       });
 
       describe('from: did', () => {
-        xit('test neeed');
+        xit('test needed');
       });
     });
-Original file line number
+Diff line change
@@ Expand Up / @@ -44,7 +44,7 @@ describe('web5.dwn', () => { @@
           });
           describe('from: did', () => {
-            xit('test neeed');
+            xit('test needed');
           });
         });
@@ Expand Down @@