- Issue created by @nicxvan
- π¬π§United Kingdom catch
Quick fix is fine but also we should use π Add PerformanceTestTrait::assertMetrics() so it is easier to write performance tests Active in this test - it was the one performance test that didn't get converted in that issue.
- πΊπΈUnited States dww
Thanks for that pointer, @catch. Letβs see what we can do to make this stable again.
- πΊπΈUnited States dww
Per Slack, catch is okay to commit this now and do the conversion in a follow up. I or nicxvan will open that follow-up momentarily, but catch only has 29 minutes to connnit this m, so hereβs a quick RTBC. Thereβs a pipeline in the linked issue that shows this change allows that to pass. This is a trivial change. Ready!
Thanks,
-Derek - π¬π§United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 11.1.x, thanks!
- πΊπΈUnited States dww
Thanks!
π Use PerformanceTestTrait::assertMetrics() in the navigation performance test Active