commit c34e17a5a96d6865941a60560ffc9cbd551c82ce
author Edwin Kempin <ekempin@google.com> Mon Nov 30 09:52:19 2020 +0100
committer Edwin Kempin <ekempin@google.com> Mon Nov 30 09:54:24 2020 +0100
tree b143b20ae4c861d247e1546d1530da2898dc339d
parent 049c9224212059d5b31812f23dd79098af7bf063

Fix names of comments that do not match the parameter names

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

javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnersTest.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnersTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnersTest.java
index cb065a0..fafc620 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnersTest.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnersTest.java
@@ -64,7 +64,7 @@
             NullPointerException.class,
             () ->
                 codeOwners.get(
-                    CodeOwnerConfig.Key.create(project, "master", "/"), /* folderPath= */ null));
+                    CodeOwnerConfig.Key.create(project, "master", "/"), /* revision= */ null));
     assertThat(npe).hasMessageThat().isEqualTo("revision");
   }
 

javatests/com/google/gerrit/plugins/codeowners/backend/findowners/FindOwnersCodeOwnerConfigParserTest.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/findowners/FindOwnersCodeOwnerConfigParserTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/findowners/FindOwnersCodeOwnerConfigParserTest.java
index 9fbc92c..f0ec10b 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/backend/findowners/FindOwnersCodeOwnerConfigParserTest.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/backend/findowners/FindOwnersCodeOwnerConfigParserTest.java
@@ -614,7 +614,7 @@
             NullPointerException.class,
             () ->
                 FindOwnersCodeOwnerConfigParser.replaceEmail(
-                    /* content= */ null, admin.email(), user.email()));
+                    /* codeOwnerConfigFileContent= */ null, admin.email(), user.email()));
     assertThat(npe).hasMessageThat().isEqualTo("codeOwnerConfigFileContent");
   }