Skip to content

Commit bc9659f

Browse files
davidjsonnAmxx
andauthored
fix(test): remove redundant nullish coalescing in ERC721 behavior tests (#6199)
Co-authored-by: Hadrien Croubois <[email protected]>
1 parent fc2d30a commit bc9659f

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

test/token/ERC721/ERC721.behavior.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -480,15 +480,13 @@ function shouldBehaveLikeERC721() {
480480

481481
const itApproves = function () {
482482
it('sets the approval for the target address', async function () {
483-
expect(await this.token.getApproved(tokenId)).to.equal(this.approved ?? this.approved);
483+
expect(await this.token.getApproved(tokenId)).to.equal(this.approved);
484484
});
485485
};
486486

487487
const itEmitsApprovalEvent = function () {
488488
it('emits an approval event', async function () {
489-
await expect(this.tx)
490-
.to.emit(this.token, 'Approval')
491-
.withArgs(this.owner, this.approved ?? this.approved, tokenId);
489+
await expect(this.tx).to.emit(this.token, 'Approval').withArgs(this.owner, this.approved, tokenId);
492490
});
493491
};
494492

0 commit comments

Comments
 (0)