-
Notifications
You must be signed in to change notification settings - Fork 83
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
Fix test cases with Eventually semantics #183
Conversation
Signed-off-by: Megrez Lu <lujiajing1126@gmail.com>
test/cases/measure/data/data.go
Outdated
@@ -26,16 +26,16 @@ import ( | |||
"time" | |||
|
|||
"github.com/google/go-cmp/cmp" | |||
g "github.com/onsi/ginkgo/v2" | |||
gm "github.com/onsi/gomega" | |||
. "github.com/onsi/ginkgo/v2" |
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.
ci might fails
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.
ci might fails
Oh, I see the linter issues... Let me revert back this part
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.
But It seems we miss format for the test package?
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.
No. It doesn't ignore it. But we have to make the order deterministic.
Codecov Report
@@ Coverage Diff @@
## main #183 +/- ##
=======================================
Coverage 41.05% 41.05%
=======================================
Files 73 73
Lines 6835 6835
=======================================
Hits 2806 2806
Misses 3769 3769
Partials 260 260
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
* fix test cases with Eventually semantics
* fix test cases with Eventually semantics
* fix test cases with Eventually semantics
Signed-off-by: Megrez Lu lujiajing1126@gmail.com
This PR tends to fix test cases failures due to async write.
Before this PR, integration tests randomly failed.