-
Notifications
You must be signed in to change notification settings - Fork 144
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.
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
Implements deadline tests and chain epoch update to i64 #533
Conversation
* error in bitfield bounded iter found
vm/actor/tests/deadline_test.rs
Outdated
dls.due[3] = bf_seq(3 * PART_SIZE + 1, 1); | ||
dls.due[5] = bf_seq(3 * PART_SIZE + 2, 2 * PART_SIZE); | ||
|
||
compute_partitions_sector(&mut dls, PART_SIZE as u64, 1, &[4]).unwrap(); |
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.
wouldnt this just panic on the first one and end the test?
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.
yeah just assert each are an error or none (I missed the should panic)
.add_to(5, 3) | ||
.verify(&deadlines); | ||
|
||
// fill less full deadlines first |
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.
in spec-actors, they have this as a seperate test case
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.
Does that make a difference or is it just preference?
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.
I mean its not a BIG deal, but they two different test cases.
Looks like you're also missing this test https://github.com/filecoin-project/specs-actors/blob/master/actors/builtin/miner/deadlines_test.go#L229 |
Summary of changes
Changes introduced in this pull request:
u64
toi64
OptionalEpoch
typeReference issue to close (if applicable)
Closes #495
Other information and links