commit 4d01dad9264976aef91fcff6ba5bdb22b404510d
author Edwin Kempin <ekempin@google.com> Fri Mar 04 17:57:08 2022 +0100
committer Edwin Kempin <ekempin@google.com> Fri Mar 04 17:57:43 2022 +0100
tree 964f0d9aacc8f7374016742afbeb2aa942b2fe6a
parent 99737da8e4246b68fd084a20a14393d10b2374ca

GeneralConfigTest: Fix comments for null parameters

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

javatests/com/google/gerrit/plugins/codeowners/backend/config/GeneralConfigTest.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/config/GeneralConfigTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/config/GeneralConfigTest.java
index 43d8bfe..86801cc 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/backend/config/GeneralConfigTest.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/backend/config/GeneralConfigTest.java
@@ -585,7 +585,7 @@
           throws Exception {
     Config cfg = new Config();
     cfg.setString(
-        SECTION_CODE_OWNERS, /* scubsection= */ null, KEY_REJECT_NON_RESOLVABLE_IMPORTS, "true");
+        SECTION_CODE_OWNERS, /* subsection= */ null, KEY_REJECT_NON_RESOLVABLE_IMPORTS, "true");
     assertThat(generalConfig.getRejectNonResolvableImports(project, cfg)).isTrue();
   }
 
@@ -1525,7 +1525,7 @@
     NullPointerException npe =
         assertThrows(
             NullPointerException.class,
-            () -> generalConfig.validateProjectLevelConfig(/*project= */ null, new Config()));
+            () -> generalConfig.validateProjectLevelConfig(/* fileName= */ null, new Config()));
     assertThat(npe).hasMessageThat().isEqualTo("fileName");
   }
 
@@ -1536,7 +1536,7 @@
             NullPointerException.class,
             () ->
                 generalConfig.validateProjectLevelConfig(
-                    "code-owners.config", /* pluginConfig= */ null));
+                    "code-owners.config", /* projectLevelConfig= */ null));
     assertThat(npe).hasMessageThat().isEqualTo("projectLevelConfig");
   }