-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
mobile: Mark AndroidEngineExplicitFlowTest as flaky #36426
Conversation
Signed-off-by: Fredy Wijaya <fredyw@google.com>
@@ -481,7 +481,7 @@ public void post_multipleRequests_randomBehavior() throws Exception { | |||
mockWebServer.enqueue(new MockResponse().setBody("hello, world")); | |||
RequestScenario requestScenario = | |||
new RequestScenario() | |||
.setHttpMethod(RequestMethod.GET) | |||
.setHttpMethod(RequestMethod.POST) |
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 changing to a POST? and should the url used on the line below be changed too?
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.
Because the test name is post_
and there's already the one that tests GET
in
public void get_multipleRequests_randomBehavior() throws Exception { |
I checked all the other tests and they all are correct, except this.
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.
Oh you mean the path. Done, I updated the path to post/
, but I don't think it matters.
Signed-off-by: Fredy Wijaya <fredyw@google.com>
The
AndroidEngineExplicitFlowTest.post_multipleRequests_randomBehavior
seems to be the one that's causing the flakiness. Addingflaky = True
and fixing the HTTP method toPOST
as what the test is intended seemed to reduce the flakiness.Risk Level: low (test only)
Testing:
bazel test --runs_per_test=100 //test/java/integration:android_engine_explicit_flow_test
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a