Skip to content

Commit

Permalink
Fix broken tests due to revert statement change
Browse files Browse the repository at this point in the history
  • Loading branch information
wojciech-turek committed Sep 18, 2023
1 parent 7ea97bf commit 39c335f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/asset/test/AuthSuperValidator.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ describe('AuthSuperValidator, (/packages/asset/contracts/AuthSuperValidator.sol)
DEFAULT_ADMIN_ROLE,
authValidatorAdmin.address
)
).to.be.revertedWith("AuthSuperValidator: can't renounce admin role");
).to.be.revertedWith('AuthSuperValidator: cant renounce admin role');
});
it('should allow admin to set signer for a given contract address', async function () {
const {MockContract, AuthValidatorContractAsAdmin, backendSigner} =
Expand Down
4 changes: 2 additions & 2 deletions packages/asset/test/Catalyst.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ describe('Catalyst (/packages/asset/contracts/Catalyst.sol)', function () {
initializer: 'initialize',
}
)
).to.revertedWith("Catalyst: CID can't be empty");
).to.revertedWith('Catalyst: CID cant be empty');
});
});
describe('Admin Role', function () {
Expand Down Expand Up @@ -395,7 +395,7 @@ describe('Catalyst (/packages/asset/contracts/Catalyst.sol)', function () {
it('cant add invalid token uri', async function () {
const {catalystAsAdmin} = await runCatalystSetup();
await expect(catalystAsAdmin.addNewCatalystType('')).to.be.revertedWith(
"Catalyst: CID can't be empty"
'Catalyst: CID cant be empty'
);
});
it('cant set invalid trusted forwarder', async function () {
Expand Down

1 comment on commit 39c335f

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage for this commit

96.15%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
packages/asset/contracts
   Asset.sol94.53%89.58%96.43%98.08%116, 207, 309, 309–310, 76
   AssetCreate.sol94.51%83.33%100%100%138, 140, 173, 293, 72
   AssetReveal.sol94.35%86.21%96.55%98.89%143, 147, 181, 376, 416, 424, 441, 75, 98
   AuthSuperValidator.sol90%83.33%100%88.89%51–52
   Catalyst.sol95%91.94%95.45%98.21%125, 127, 140, 152, 224, 80
packages/asset/contracts/interfaces
   IAsset.sol100%100%100%100%
   IAssetCreate.sol100%100%100%100%
   IAssetReveal.sol100%100%100%100%
   ICatalyst.sol100%100%100%100%
   ITokenUtils.sol100%100%100%100%
packages/asset/contracts/libraries
   TokenIdUtils.sol100%100%100%100%
packages/dependency-metatx/contracts
   ERC2771Handler.sol100%100%100%100%
   ERC2771HandlerAbstract.sol100%100%100%100%
   ERC2771HandlerUpgradeable.sol95.45%83.33%100%100%43
packages/dependency-metatx/contracts/test
   ERC2771HandlerTest.sol100%100%100%100%
   ERC2771HandlerUpgradeableTest.sol100%100%100%100%
   MockTrustedForwarder.sol0%0%0%0%15, 18–19, 19, 19–20
packages/dependency-operator-filter/contracts
   OperatorFiltererUpgradeable.sol82.35%85%71.43%83.33%17, 51–52, 61–62, 71, 87
   OperatorFilterSubscription.sol60%50%100%50%19–20
packages/dependency-operator-filter/contracts/interfaces
   IOperatorFilterRegistry.sol100%100%100%100%
packages/dependency-royalty-management/contracts
   MultiRoyaltyDistributor.sol84.06%65%90%92.31%104, 144, 25, 41, 41, 41, 41, 60–61, 97
   RoyaltyDistributor.sol78.95%50%80%90%20, 50, 56
   RoyaltyManager.sol96.39%87.50%100%100%104, 47, 98
   RoyaltySplitter.sol92.08%75%92.31%97.06%117, 157, 185, 218, 65, 72, 83
packages/dependency-royalty-management/contracts/interfaces
   IERC20Approve.sol100%100%100%100%
   IMultiRoyaltyDistributor.sol100%100%100%100%
   IMultiRoyaltyRecipients.sol100%100%100%100%
   IRoyaltyManager.sol100%100%100%100%
   IRoyaltyUGC.sol100%100%100%100%

Please sign in to comment.