Flaky test: WebHookAction_NoUUID_IsBadRequest

[ryanbrandenburg]

This test fails occasionally with the following error:

Microsoft.AspNetCore.Routing.Matchers.AmbiguousMatchException : The request matched multiple endpoints. Matches: 
BitbucketStronglyTypedCoreReceiver.Controllers.BitbucketController.BitbucketForIt (BitbucketStronglyTypedCoreReceiver)
BitbucketStronglyTypedCoreReceiver.Controllers.BitbucketController.BitbucketForPush (BitbucketStronglyTypedCoreReceiver)
BitbucketStronglyTypedCoreReceiver.Controllers.BitbucketController.Bitbucket (BitbucketStronglyTypedCoreReceiver)
   at Microsoft.AspNetCore.Routing.EndpointConstraints.EndpointConstraintEndpointSelector.SelectAsync(HttpContext httpContext, IEndpointFeature feature, CandidateSet candidates)
   at Microsoft.AspNetCore.Routing.Matchers.DfaMatcher.MatchAsync(HttpContext httpContext, IEndpointFeature feature)
   at Microsoft.AspNetCore.Routing.GlobalRoutingMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.TestHost.HttpContextBuilder.<>c__DisplayClass10_0.<<SendAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.TestHost.ClientHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Mvc.Testing.Handlers.CookieContainerHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Mvc.Testing.Handlers.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.AspNetCore.WebHooks.FunctionalTest.BitbucketStronglyTypedCoreReceiverTest.WebHookAction_NoUUID_IsBadRequest() in /_/test/Microsoft.AspNetCore.WebHooks.FunctionalTest/BitbucketStronglyTypedCoreReceiverTest.cs:line 170
--- End of stack trace from previous location where exception was thrown ---

[ryanbrandenburg]

WebHookAction_NoCode_IsBadRequest failed again.

[ryanbrandenburg]

WebHookAction_NoCode_IsBadRequest failed again.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

[ryanbrandenburg]

WebHookAction_WithBody_Succeeds failed again.

[Eilon]

@dougbu - Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

• Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

  • [x] File a bug against the product (#318, now aspnet/Routing#630)
  • Is it possible to change the test to avoid the flakiness?
  • Yes? Go to the "Change the test!" section.
  • No?
    • [x] Disable the test (PR #320, commit bedc118aef -- really a workaround)
    • [ ] Wait for other bug to be resolved
    • [ ] Wait for us to get build that has the fix
    • [ ] Re-enable our test (TODO: Link to PR/commit)
    • [ ] Close this bug

• Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

  • Is there is a way to change our test to avoid this flakiness?
  • Yes? Change the test!
    • [ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
    • [ ] Run the test 100 times locally as a sanity check.
    • [ ] Close this bug
  • No?
    • [ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

[dougbu]

See #318 for product bug

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NoEventHeader_IsNotFound failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NoEventHeader_IsNotFound failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NonPost_IsNotAllowed failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NoUUID_IsBadRequest failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_NoCode_IsBadRequest failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_WithBody_Succeeds failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_WithBody_Succeeds failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[ryanbrandenburg]

WebHookAction_WithBody_Succeeds failed again.

This comment was made automatically. If there is a problem contact @ryanbrandenburg.

[dougbu]

@rynowak much appreciated.

Is this done? And, should the workflow above be updated?

[Eilon]

As long as the underlying issue was resolved, just closing is fine.