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

javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java
index 4dbac76..c811514 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java
@@ -861,6 +861,7 @@
         .containsExactly(user.id(), user2.id(), admin.id());
 
     // Query code owners with a limit.
+    requestScopeOperations.setApiUser(user.id());
     codeOwnerInfos = queryCodeOwners(getCodeOwnersApi().query().withLimit(2), "/foo/bar/baz.md");
     assertThat(codeOwnerInfos).hasSize(2);
     assertThatList(codeOwnerInfos)