Skip to content

Commit

Permalink
Added testcases for updateCollection (#356)
Browse files Browse the repository at this point in the history
  • Loading branch information
adi44 authored Aug 23, 2021
1 parent 3074bb7 commit 4a289ad
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions test/AssetManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,16 @@ describe('AssetManager', function () {
await assertRevert(tx, 'Job exists in this collection');
});

it('updateCollection should only work for collections which exists', async function () {
const tx = assetManager.updateCollection(10, 2, 5);
assertRevert(tx, 'Collection ID not present');
});

it('updateCollection should only work for collections which are currently active', async function () {
await assetManager.setAssetStatus(false, 3);
const tx = assetManager.updateCollection(3, 2, 5);
assertRevert(tx, 'Collection is inactive');
});
// it('should be able to get result using proxy', async function () {
// await delegator.upgradeDelegate(assetManager.address);
// assert(await delegator.delegate() === assetManager.address);
Expand Down

0 comments on commit 4a289ad

Please sign in to comment.