SkipCodeOwnerConfigValidationPushOptionIT: Fix a test
codeOwnersSkipOptionIsOmittedIfUserCannotSkipTheCodeOwnersValidation()
should test that the skip option is omitted if the user cannot skip the
code owners validation, but the test created a change for a disabled
branch so that the skip option would have been omitted also if the user
had the necessary permission.
Bug: Google b/279897514
Change-Id: I476433b9a96873ac15321a2ee80645bd72bf9814
Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/SkipCodeOwnerConfigValidationPushOptionIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/SkipCodeOwnerConfigValidationPushOptionIT.java
index 3abee10..6c74375 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/SkipCodeOwnerConfigValidationPushOptionIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/SkipCodeOwnerConfigValidationPushOptionIT.java
@@ -79,7 +79,6 @@
SkipCodeOwnerConfigValidationPushOption.DESCRIPTION)));
}
- @GerritConfig(name = "plugin.code-owners.disabledBranch", value = "refs/heads/master")
@Test
public void codeOwnersSkipOptionIsOmittedIfUserCannotSkipTheCodeOwnersValidation()
throws Exception {
@@ -87,7 +86,7 @@
// capability.
requestScopeOperations.setApiUser(user.id());
- Change.Id changeId = changeOperations.newChange().project(project).branch("master").create();
+ Change.Id changeId = changeOperations.newChange().project(project).create();
ValidationOptionInfos validationOptionsInfos =
gApi.changes().id(project.get(), changeId.get()).getValidationOptions();
assertThat(validationOptionsInfos.validationOptions).isEmpty();
