tree 05d1900dc486955495ee57677d333999f7a7094e
parent 6dfc66e6ab24370d575af5e2e919e620a9182859
author Han-Wen Nienhuys <hanwen@google.com> 1542236799 -0800
committer Dave Borowitz <dborowitz@google.com> 1542238217 -0800

GroupsIT: separate order check from element check

The caller of assertIncludes only cares that the set of includes is the
same. Additionally, assertIncludes itself cares that the output of this
particular extension API is sorted. But we can check this within the
body of assertIncludes without requiring callers to manually sort their
arguments.

The previous approach had the pitfall that "group-9" sorts after
"group-10", meaning that addMultipleIncludes might be making an
incorrect assumption that g1's name sorts before g2's name, depending on
the exact subset of test methods that get run.

Change-Id: Ic4378ba919c03061e0d46b9e8715dec2920478d5