commit 404d7efe2ea583eaa1419e9fec7e97a904994228
author Youssef Elghareeb <ghareeb@google.com> Wed Dec 08 16:57:09 2021 +0100
committer Youssef Elghareeb <ghareeb@google.com> Wed Dec 08 16:58:33 2021 +0100
tree 8e6e39a2beb469b64bdcf860d88c616a3394ebd7
parent 8713b072a7e27a39d4c9c58ec27336e1c31998f3

Remove the "enable_submit_requirements" experiment config in tests

This experiment was ramped up to 100% and is not used in Gerrit core
anymore. Removing from tests.

Change-Id: I6d8ff54e79fa80987ce0f1c46cc0a23ce3a7009b

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

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnerHasOperandsIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnerHasOperandsIT.java
index 070cbb5..1610647 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnerHasOperandsIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnerHasOperandsIT.java
@@ -44,7 +44,6 @@
 import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT;
 import com.google.gerrit.plugins.codeowners.backend.CodeOwnerApprovalHasOperand;
 import com.google.gerrit.plugins.codeowners.backend.CodeOwnersInternalServerErrorException;
-import com.google.gerrit.server.experiments.ExperimentFeaturesConstants;
 import com.google.gerrit.server.query.change.ChangeData;
 import com.google.gerrit.server.query.change.ChangeQueryBuilder;
 import com.google.inject.Inject;
@@ -97,9 +96,6 @@
   }
 
   @Test
-  @GerritConfig(
-      name = "experiments.enabled",
-      value = ExperimentFeaturesConstants.GERRIT_BACKEND_REQUEST_FEATURE_ENABLE_SUBMIT_REQUIREMENTS)
   public void hasApproval_satisfied() throws Exception {
     codeOwnerConfigOperations
         .newCodeOwnerConfig()
@@ -123,9 +119,6 @@
   }
 
   @Test
-  @GerritConfig(
-      name = "experiments.enabled",
-      value = ExperimentFeaturesConstants.GERRIT_BACKEND_REQUEST_FEATURE_ENABLE_SUBMIT_REQUIREMENTS)
   public void hasApproval_unsatisfiedIfChangeIsClosed() throws Exception {
     codeOwnerConfigOperations
         .newCodeOwnerConfig()
@@ -157,9 +150,6 @@
   }
 
   @Test
-  @GerritConfig(
-      name = "experiments.enabled",
-      value = ExperimentFeaturesConstants.GERRIT_BACKEND_REQUEST_FEATURE_ENABLE_SUBMIT_REQUIREMENTS)
   public void hasApproval_internalServerError() throws Exception {
     ChangeData changeData = createChange().getChange();
 
@@ -186,9 +176,6 @@
   }
 
   @Test
-  @GerritConfig(
-      name = "experiments.enabled",
-      value = ExperimentFeaturesConstants.GERRIT_BACKEND_REQUEST_FEATURE_ENABLE_SUBMIT_REQUIREMENTS)
   public void hasApproval_ruleErrorForNonParsableCodeOwnerConfig() throws Exception {
     String nameOfInvalidCodeOwnerConfigFile = getCodeOwnerConfigFileName();
     createNonParseableCodeOwnerConfig(nameOfInvalidCodeOwnerConfigFile);
@@ -205,9 +192,6 @@
 
   @Test
   @GerritConfig(name = "plugin.code-owners.disabled", value = "true")
-  @GerritConfig(
-      name = "experiments.enabled",
-      value = ExperimentFeaturesConstants.GERRIT_BACKEND_REQUEST_FEATURE_ENABLE_SUBMIT_REQUIREMENTS)
   public void hasEnabled_notMatchingWhenCodeOwnersIsDisabledForTheChange() throws Exception {
     Change change =
         createChange("Change Adding A File", "foo/bar.baz", "file content").getChange().change();