tree 82b01f5d1302477825d1df3d78b4c3d83501bf48
parent 02d36d4901fd58485c032d648e53831885e56d0e
author Edwin Kempin <ekempin@google.com> 1610101829 +0100
committer Edwin Kempin <ekempin@google.com> 1610374764 +0100

Fix getAllUsersAsCodeOwners_allVisible test

This test was supposed to test the code for handling
accounts.visibility = ALL, but since the admin user was doing the call
we instead landed in the shortcut for users that have the
VIEW_ALL_ACCOUNTS capability (which is already covered by
getAllUsersAsCodeOwners_withViewAllAccounts).

This was discovered by looking at the test coverage. Fixing this
increases the test coverage for AbstractGetCodeOwnersForPath slightly
from 95.4% to 96.2%.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I1ef01100677586daf10581c3876a639bb313b009