Change confusing variable name

jenkinsci · May 29, 2023 · 82a1838 · 82a1838
1 parent 2e50c13
commit 82a1838
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/src/test/java/com/dabsquared/gitlabjenkins/webhook/build/MergeRequestBuildActionTest.java b/src/test/java/com/dabsquared/gitlabjenkins/webhook/build/MergeRequestBuildActionTest.java
@@ -104,22 +104,23 @@ public void build() throws IOException {
             testProject.addTrigger(mockTrigger);
             executeMergeRequestAction(testProject, getJson("MergeRequestEvent.json"));
         } finally {
-            ArgumentCaptor<MergeRequestHook> pushHookArgumentCaptor = ArgumentCaptor.forClass(MergeRequestHook.class);
-            verify(mockTrigger).onPost(pushHookArgumentCaptor.capture());
-            assertThat(pushHookArgumentCaptor.getValue().getProject(), is(notNullValue()));
-            assertThat(pushHookArgumentCaptor.getValue().getProject().getWebUrl(), is(notNullValue()));
-            assertThat(pushHookArgumentCaptor.getValue().getUser(), is(notNullValue()));
-            assertThat(pushHookArgumentCaptor.getValue().getUser().getName(), containsString("Administrator"));
-            assertThat(pushHookArgumentCaptor.getValue().getUser().getUsername(), containsString("root"));
+            ArgumentCaptor<MergeRequestHook> mergeRequestHookArgumentCaptor =
+                    ArgumentCaptor.forClass(MergeRequestHook.class);
+            verify(mockTrigger).onPost(mergeRequestHookArgumentCaptor.capture());
+            assertThat(mergeRequestHookArgumentCaptor.getValue().getProject(), is(notNullValue()));
+            assertThat(mergeRequestHookArgumentCaptor.getValue().getProject().getWebUrl(), is(notNullValue()));
+            assertThat(mergeRequestHookArgumentCaptor.getValue().getUser(), is(notNullValue()));
+            assertThat(mergeRequestHookArgumentCaptor.getValue().getUser().getName(), containsString("Administrator"));
+            assertThat(mergeRequestHookArgumentCaptor.getValue().getUser().getUsername(), containsString("root"));
             assertThat(
-                    pushHookArgumentCaptor
+                    mergeRequestHookArgumentCaptor
                             .getValue()
                             .getObjectAttributes()
                             .getLastCommit()
                             .getAuthor(),
                     is(notNullValue()));
             assertThat(
-                    pushHookArgumentCaptor
+                    mergeRequestHookArgumentCaptor
                             .getValue()
                             .getObjectAttributes()
                             .getLastCommit()