commit 482ea11c5645f052bb79d1e0fe3279eb6d3cc3fb
author Edwin Kempin <ekempin@google.com> Tue Nov 03 12:46:40 2020 +0100
committer Edwin Kempin <ekempin@google.com> Tue Nov 03 12:52:29 2020 +0100
tree 7841abf7aced58503a6e58ff50e67ed156a33c40
parent d0de104be7754493431a8db0e6e0fdb9c491d275

GetCodeOwnerProjectConfigIT: Fix tests

The implementations of 2 tests were swapped.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: If8b74f4b6bff1c71ecac5be8e4e8d7c5bad37021

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

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerProjectConfigIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerProjectConfigIT.java
index b820d98..e936dda 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerProjectConfigIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerProjectConfigIT.java
@@ -233,14 +233,6 @@
 
   @Test
   public void getConfigWithConfiguredOverrideApproval() throws Exception {
-    configureImplicitApprovals(project);
-    CodeOwnerProjectConfigInfo codeOwnerProjectConfigInfo =
-        projectCodeOwnersApiFactory.project(project).getConfig();
-    assertThat(codeOwnerProjectConfigInfo.general.implicitApprovals).isTrue();
-  }
-
-  @Test
-  public void getConfigWithEnabledImplicitApprovals() throws Exception {
     configureOverrideApproval(project, "Code-Review+2");
     CodeOwnerProjectConfigInfo codeOwnerProjectConfigInfo =
         projectCodeOwnersApiFactory.project(project).getConfig();
@@ -248,6 +240,14 @@
     assertThat(codeOwnerProjectConfigInfo.overrideApproval.value).isEqualTo(2);
   }
 
+  @Test
+  public void getConfigWithEnabledImplicitApprovals() throws Exception {
+    configureImplicitApprovals(project);
+    CodeOwnerProjectConfigInfo codeOwnerProjectConfigInfo =
+        projectCodeOwnersApiFactory.project(project).getConfig();
+    assertThat(codeOwnerProjectConfigInfo.general.implicitApprovals).isTrue();
+  }
+
   private void configureFileExtension(Project.NameKey project, String fileExtension)
       throws Exception {
     setConfig(project, null, GeneralConfig.KEY_FILE_EXTENSION, fileExtension);