commit e1a39d8f6a46d682ee83106165c32fa0d5c29f0b
author Changcheng Xiao <xchangcheng@google.com> Mon May 13 14:44:53 2019 +0200
committer Changcheng Xiao <xchangcheng@google.com> Mon May 13 17:57:08 2019 +0200
tree 209c8e52f09e023e9caf53b61c2a18a3dc4d4c06
parent 8e63759c810c41374b1a09948efc4115acf9dbb7

CheckSubmitRuleIT: enable ignored tests

Change-Id: I59872ed0afab00da506dbf063c5a38d7953eb6c4

javatests/com/google/gerrit/plugins/checks/acceptance/rules/ChecksSubmitRuleIT.java

diff --git a/javatests/com/google/gerrit/plugins/checks/acceptance/rules/ChecksSubmitRuleIT.java b/javatests/com/google/gerrit/plugins/checks/acceptance/rules/ChecksSubmitRuleIT.java
index d392baa..b0bc754 100644
--- a/javatests/com/google/gerrit/plugins/checks/acceptance/rules/ChecksSubmitRuleIT.java
+++ b/javatests/com/google/gerrit/plugins/checks/acceptance/rules/ChecksSubmitRuleIT.java
@@ -27,7 +27,6 @@
 import com.google.gerrit.plugins.checks.api.CheckState;
 import com.google.gerrit.reviewdb.client.PatchSet;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
 public class ChecksSubmitRuleIT extends AbstractCheckersTest {
@@ -128,7 +127,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void notStartedCheckOfNonApplicableCheckerDoesNotBlockSubmit() throws Exception {
     // Set up a checker which applies to a different repository than the considered change but still
     // post a check result to the change. -> Check is considered as optional.
@@ -143,7 +141,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void runningCheckOfNonApplicableCheckerDoesNotBlockSubmit() throws Exception {
     // Set up a checker which applies to a different repository than the considered change but still
     // post a check result to the change. -> Check is considered as optional.
@@ -172,7 +169,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void notStartedDisabledRequiredCheckDoesNotBlockSubmit() throws Exception {
     CheckerUuid checkerUuid = newRequiredChecker().disable().create();
     postCheckResult(checkerUuid, CheckState.NOT_STARTED);
@@ -184,7 +180,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void runningDisabledRequiredCheckDoesNotBlockSubmit() throws Exception {
     CheckerUuid checkerUuid = newRequiredChecker().disable().create();
     postCheckResult(checkerUuid, CheckState.RUNNING);
@@ -207,7 +202,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void notStartedOptionalCheckDoesNotBlockSubmit() throws Exception {
     CheckerUuid checkerUuid = newOptionalChecker().create();
     postCheckResult(checkerUuid, CheckState.NOT_STARTED);
@@ -219,7 +213,6 @@
   }
 
   @Test
-  @Ignore("TODO(aliceks): Fix behavior to make this test succeed.")
   public void runningOptionalCheckDoesNotBlockSubmit() throws Exception {
     CheckerUuid checkerUuid = newOptionalChecker().create();
     postCheckResult(checkerUuid, CheckState.RUNNING);