-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add smoke tests mode #656
Add smoke tests mode #656
Conversation
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.
Partial Review: Looking great! Awesome PR 🎉
@@ -261,32 +218,6 @@ func createDefaultClientBridgeConfig(t *test.SystemTest) ([]string, error) { | |||
) | |||
} | |||
|
|||
func runCreateBridgeClientTestConfig( |
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.
Why are these being removed?
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.
they were unused functions
@@ -22,6 +24,7 @@ func TestCreateBucket(testSetup *testing.T) { | |||
require.Equal(t, 200, resp.StatusCode()) | |||
}) | |||
|
|||
// FIXME - this should be 400 not 500 |
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.
@stewartie4 reminder to raise an issue for this
|
||
t.Parallel() | ||
// FIXME: we should never return a 500 to the end user |
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.
@stewartie4 reminder to raise an issue for this
Actions PR: 0chain/actions#155
Smoke tests run: https://github.com/0chain/system_test/actions/runs/4838450707/jobs/8623002789
(this run also includes flaky tests so the run is longer than it will be in other repos)