Codecov Report

Merging #1311 (e78a8d6) into master (649385c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1311   +/-   ##
=======================================
  Coverage   80.67%   80.67%           
=======================================
  Files          68       68           
  Lines        3607     3607           
=======================================
  Hits         2910     2910           
  Misses        697      697           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 649385c...e78a8d6. Read the comment docs.

Since there are two groups we can't depend on the one we changed to
always be the first one returned.

Instead fetch the group we want and then test our assertion against
that group.

Thanks, this will fix a lot of the flakiness.

There's probably some more weirdness and unnecessary dependency on execution order in functional tests, because they were rewritten from a single module with plain asserts. I'll try to get to that at some point.