-
Notifications
You must be signed in to change notification settings - Fork 21
feat(PTP-490): object parameters in all methods / JSDoc / standardized parameter name #41
Conversation
…tion of parameter name
const expectedContractMetadata = { name: 'testContract', symbol: 'TST', tokenType: 'ERC721' }; | ||
expect(contractMetadata).toStrictEqual(expectedContractMetadata); | ||
}); | ||
}); | ||
|
||
describe('As an account I should be able to get the list of NFTs by address', () => { | ||
it('should return list of NFTs by address', async () => { | ||
const nfts = await sdk.getNFTs(process.env.WALLET_PUBLIC_ADDRESS); | ||
const nfts = await sdk.getNFTs({ publicAddress: process.env.WALLET_PUBLIC_ADDRESS }); |
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.
We should adapt the environment variable as well :)
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.
not mandatory as it doesn't affects user experience
.gitignore
Outdated
storage | ||
.DS_STORE |
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.
Can you ignore .vscode
folder too ?
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.
.idea too , i use webstorm :p
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.
ok let me add that
Kudos, SonarCloud Quality Gate passed! |
No description provided.