-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Double Deal Flow #1481
Double Deal Flow #1481
Conversation
node/node_test.go
Outdated
t.Run("TestDealFlow", func(t *testing.T) { test.TestDealFlow(t, mockSbBuilder, 10*time.Millisecond) }) | ||
t.Run("TestDoubleDealFlow", func(t *testing.T) { test.TestDoubleDealFlow(t, mockSbBuilder, 10*time.Millisecond) }) |
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.
- TestDealFlow pass correctly since is the original test
- TestDoubleDealFlow doesn't pass.
api/test/deals.go
Outdated
|
||
r := bytes.NewReader(data) | ||
fcid, err := client.ClientImportLocal(ctx, r) | ||
makeDeal(t, ctx, 6, client, miner) |
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.
So most of the deal making is now on an extracted makeDeal
.
This makes TestDealFlow
only calling this once, and TestDoubleDealFlow
calling it twice... to be sure both are doing the exact same thing (with just randomness changing for the second deal).
The issue has been fixed in |
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Rebased now that last commit has |
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.
Thanks for the test! :)
…lowFAIL Double Deal Flow test
This PR has a new test for checking the issue #1455.
Test shouldn't pass since this PR adds the test to expose failing.The last rebase include the fix, tests should pass.