Test getting code owners when all users are global code owners This slightly increases the test coverage for AbstractGetCodeOwnersForPath from 96.2% to 96.9%. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: I2feeb31f0453a1a41a44bd0c7fd6ed691d03d023
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 c811514..eaaddb3 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/AbstractGetCodeOwnersForPathIT.java
@@ -642,6 +642,30 @@ } @Test + @GerritConfig(name = "plugin.code-owners.globalCodeOwner", value = "*") + public void getAllUsersAsGlobalCodeOwners() throws Exception { + TestAccount user2 = accountCreator.user2(); + + List<CodeOwnerInfo> codeOwnerInfos = queryCodeOwners("/foo/bar/baz.md"); + assertThat(codeOwnerInfos) + .comparingElementsUsing(hasAccountId()) + .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) + .element(0) + .hasAccountIdThat() + .isAnyOf(user.id(), user2.id(), admin.id()); + assertThatList(codeOwnerInfos) + .element(1) + .hasAccountIdThat() + .isAnyOf(user.id(), user2.id(), admin.id()); + } + + @Test @GerritConfig( name = "plugin.code-owners.globalCodeOwner", values = {"global.owner1@example.com", "global.owner2@example.com"})