-
Notifications
You must be signed in to change notification settings - Fork 3
feat: implement storeIdentityPublicKeyHashes
method
#387
Conversation
* | ||
* @returns {Promise<void>} | ||
*/ | ||
async storeIdentityIdPublicKeys(identityId, publicKeyHashes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storeIdentityIdPublicKeyHashes
?
@@ -3,6 +3,7 @@ const getIdentityFixture = require('@dashevo/dpp/lib/test/fixtures/getIdentityFi | |||
const getDataContractFixture = require('@dashevo/dpp/lib/test/fixtures/getDataContractFixture'); | |||
|
|||
const CachedStateRepositoryDecorator = require('../../../lib/dpp/CachedStateRepositoryDecorator'); | |||
const { expect } = require('chai'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expect
supposed to be global, isn't it?
* | ||
* @returns {Promise<void>} | ||
*/ | ||
async storeIdentityIdPublicKeyHashes(identityId, publicKeyHashes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storeIdentityPublicKeyHashes
lib/dpp/DriveStateRepository.js
Outdated
* | ||
* @returns {Promise<void>} | ||
*/ | ||
async storeIdentityIdPublicKeyHashes(identityId, publicKeyHashes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storeIdentityPublicKeyHashes
@@ -76,6 +77,20 @@ describe('CachedStateRepositoryDecorator', () => { | |||
}); | |||
}); | |||
|
|||
describe('#storeIdentityIdPublicKeyHashes', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storeIdentityPublicKeyHashes
storeIdentityIdPublicKeys
methodstoreIdentityPublicKeyHashes
method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Issue being fixed or feature implemented
Implemented batch method
storeIdentityIdPublicKeys
What was done?
Implemented batch method
storeIdentityIdPublicKeys
How Has This Been Tested?
Unit tests.
Breaking Changes
None.
Checklist:
For repository code-owners and collaborators only