Address some SonarCloud code smells on argument-exceptions2

Remove obsolete unused type parameter T from InitialParseExceptionParser Change test classes to default package visibility
JorelAli · Aug 12, 2023 · 5f6ca11 · 5f6ca11
1 parent a841cd3
commit 5f6ca11
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/...dapi-core/src/main/java/dev/jorel/commandapi/arguments/ExceptionHandlingArgumentType.java b/...dapi-core/src/main/java/dev/jorel/commandapi/arguments/ExceptionHandlingArgumentType.java
@@ -37,7 +37,7 @@ public record ExceptionHandlingArgumentType<T, ExceptionInformation>(
 		 * @param exceptionParser A function that parses the information in a {@link CommandSyntaxException} to create an
 		 * {@link ExceptionInformation} object.
 		 */
-		InitialParseExceptionParser<T, ExceptionInformation> exceptionParser
+		InitialParseExceptionParser<ExceptionInformation> exceptionParser
 ) implements ArgumentType<T> {
 
     @Override

diff --git a/...andapi-core/src/main/java/dev/jorel/commandapi/arguments/InitialParseExceptionParser.java b/...andapi-core/src/main/java/dev/jorel/commandapi/arguments/InitialParseExceptionParser.java
@@ -8,11 +8,10 @@
  * Extracts information about an exception that was thrown when an {@link ArgumentType} fails to parse.
  * See {@link InitialParseExceptionArgument#parseInitialParseException(CommandSyntaxException, StringReader)}.
  *
- * @param <T> The object returned when the wrapped {@link ArgumentType} is parsed.
  * @param <ExceptionInformation> The class that holds information about the exception.
  */
 @FunctionalInterface
-public interface InitialParseExceptionParser<T, ExceptionInformation> {
+public interface InitialParseExceptionParser<ExceptionInformation> {
     /**
      * Extracts information about an exception that was thrown when an {@link ArgumentType} fails to parse.
      * See {@link InitialParseExceptionArgument#parseInitialParseException(CommandSyntaxException, StringReader)}.

diff --git a/...est-tests/src/test/java/dev/jorel/commandapi/test/ArgumentParseExceptionArgumentTest.java b/...est-tests/src/test/java/dev/jorel/commandapi/test/ArgumentParseExceptionArgumentTest.java
@@ -12,7 +12,7 @@
 import java.util.Collection;
 import java.util.List;
 
-public class ArgumentParseExceptionArgumentTest extends TestBase {
+class ArgumentParseExceptionArgumentTest extends TestBase {
 
     /*********
      * Setup *

diff --git a/...test-tests/src/test/java/dev/jorel/commandapi/test/InitialParseExceptionArgumentTest.java b/...test-tests/src/test/java/dev/jorel/commandapi/test/InitialParseExceptionArgumentTest.java
@@ -10,7 +10,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class InitialParseExceptionArgumentTest extends TestBase {
+class InitialParseExceptionArgumentTest extends TestBase {
 
 	/*********
 	 * Setup *