use logger, update tests

TBD54566975 · Oct 11, 2024 · 0bdb0e1 · 0bdb0e1
1 parent 96e25b2
commit 0bdb0e1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/agent/src/agent-did-resolver-cache.ts b/packages/agent/src/agent-did-resolver-cache.ts
@@ -72,7 +72,7 @@ export class AgentDidResolverCache extends DidResolverCacheLevel implements DidR
               } catch(error: any) {
                 // if the error is not due to no changes detected, log the error
                 if (error.message && !error.message.includes('No changes detected, update aborted')) {
-                  console.error(`Error updating DID: ${error.message}`);
+                  logger.error(`Error updating DID: ${error.message}`);
                 }
               }
             }

diff --git a/packages/agent/src/local-key-manager.ts b/packages/agent/src/local-key-manager.ts
@@ -423,6 +423,7 @@ export class LocalKeyManager implements AgentKeyManager {
     // Compute the key URI for the key.
     const keyUri = await this.getKeyUri({ key: privateKey });
 
+    // Store the key in the key store.
     await this._keyStore.set({
       id                : keyUri,
       data              : privateKey,

diff --git a/packages/agent/tests/agent-did-resolver-cach.spec.ts b/packages/agent/tests/agent-did-resolver-cach.spec.ts
@@ -5,7 +5,6 @@ import { TestAgent } from './utils/test-agent.js';
 import sinon from 'sinon';
 import { expect } from 'chai';
 import { BearerDid, DidJwk } from '@web5/dids';
-import { BearerIdentity } from '../src/bearer-identity.js';
 
 describe('AgentDidResolverCache',  () => {
   let resolverCache: AgentDidResolverCache;
@@ -81,7 +80,7 @@ describe('AgentDidResolverCache',  () => {
 
     const getStub = sinon.stub(resolverCache['cache'], 'get').resolves(JSON.stringify({ ttlMillis: Date.now() - 1000, value: { didDocument: { id: did.uri } } }));
     const resolveSpy = sinon.spy(testHarness.agent.did, 'resolve').withArgs(did.uri);
-    const nextTickSpy = sinon.stub(resolverCache['cache'], 'nextTick').resolves();
+    sinon.stub(resolverCache['cache'], 'nextTick').resolves();
     const didApiStub = sinon.stub(testHarness.agent.did, 'get');
     const updateSpy = sinon.stub(testHarness.agent.did, 'update').resolves();
     didApiStub.withArgs({ didUri: did.uri, tenant: testHarness.agent.agentDid.uri }).resolves(new BearerDid({
@@ -104,7 +103,7 @@ describe('AgentDidResolverCache',  () => {
 
     const getStub = sinon.stub(resolverCache['cache'], 'get').resolves(JSON.stringify({ ttlMillis: Date.now() - 1000, value: { didDocument: { id: did.uri } } }));
     const resolveSpy = sinon.spy(testHarness.agent.did, 'resolve').withArgs(did.uri);
-    const nextTickSpy = sinon.stub(resolverCache['cache'], 'nextTick').resolves();
+    sinon.stub(resolverCache['cache'], 'nextTick').resolves();
     const didApiStub = sinon.stub(testHarness.agent.did, 'get');
     const updateSpy = sinon.stub(testHarness.agent.did, 'update').rejects(new Error('Some Error'));
     const consoleErrorSpy = sinon.stub(console, 'error');
@@ -137,7 +136,7 @@ describe('AgentDidResolverCache',  () => {
 
   it('throws if the error is anything other than a notFound error', async () => {
     const did = testHarness.agent.agentDid.uri;
-    const getStub = sinon.stub(resolverCache['cache'], 'get').rejects(new Error('Some Error'));
+    sinon.stub(resolverCache['cache'], 'get').rejects(new Error('Some Error'));
 
     try {
       await resolverCache.get(did);