Skip to content

Nextv5.0 checkpoint history removal #4170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
RenanSouza2 wants to merge 13 commits into from
Closed

Nextv5.0 checkpoint history removal #4170

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
62894b1
mid commit
RenanSouza2 Apr 8, 2023
3b8610f
proposal 1
RenanSouza2 Apr 8, 2023
578bc8b
Checkpoint tests some fixes
RenanSouza2 Apr 15, 2023
d53e29d
returns zero as past value
RenanSouza2 Apr 15, 2023
d89cf69
more tests
RenanSouza2 Apr 15, 2023
83f4f64
Votes
RenanSouza2 Apr 15, 2023
bf22828
ERC20Votes
RenanSouza2 Apr 15, 2023
9b13eb3
ERC20VotesComp
RenanSouza2 Apr 15, 2023
a91837c
Tested
RenanSouza2 Apr 15, 2023
73ce437
compiled
RenanSouza2 Apr 16, 2023
8b9029b
lint
RenanSouza2 Apr 16, 2023
16ae23b
some fixes
RenanSouza2 Apr 17, 2023
da3ac53
enable tests
RenanSouza2 Apr 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
ERC20VotesComp
  • Loading branch information
@RenanSouza2
RenanSouza2 committed Apr 15, 2023
commit 9b13eb313b5a8d6b74e995b1078cc3c8b334aee4
4 changes: 2 additions & 2 deletions test/token/ERC20/extensions/ERC20VotesComp.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ contract('ERC20VotesComp', function (accounts) {

describe('getPriorVotes', function () {
it('reverts if block number >= current block', async function () {
await expectRevert(this.token.getPriorVotes(other1, 5e10), 'Checkpoints: block not yet mined');
await expectRevert(this.token.getPriorVotes(other1, 5e10), 'Votes: block not yet mined');
});

it('returns 0 if there are no checkpoints', async function () {
Expand Down Expand Up @@ -470,7 +470,7 @@ contract('ERC20VotesComp', function (accounts) {
});

it('reverts if block number >= current block', async function () {
await expectRevert(this.token.getPastTotalSupply(5e10), 'Checkpoints: block not yet mined');
await expectRevert(this.token.getPastTotalSupply(5e10), 'Votes: block not yet mined');
});

it('returns 0 if there are no checkpoints', async function () {
Expand Down