From 21ab1d0e195a38034f5ccadf3b0980701a8fbac7 Mon Sep 17 00:00:00 2001
From: Laurent Clouet <4409640-laurent.clouet@users.noreply.gitlab.com>
Date: Thu, 6 Jun 2024 01:42:04 +0000
Subject: [PATCH] Ref: add new static analyser rule:
 category/java/errorprone.xml/UnnecessaryCaseChange

---
 rulesets/java-sheepit.xml                               | 2 ++
 src/main/java/com/sheepit/client/standalone/Worker.java | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/rulesets/java-sheepit.xml b/rulesets/java-sheepit.xml
index b2b2ac6..631f488 100644
--- a/rulesets/java-sheepit.xml
+++ b/rulesets/java-sheepit.xml
@@ -49,6 +49,8 @@
 	<rule ref="category/java/errorprone.xml/UnnecessaryConversionTemporary" />
 	<rule ref="category/java/errorprone.xml/UnusedNullCheckInEquals" />
 	<rule ref="category/java/errorprone.xml/UselessOperationOnImmutable" />
+	<rule ref="category/java/errorprone.xml/UnnecessaryBooleanAssertion" />
+	<rule ref="category/java/errorprone.xml/UnnecessaryCaseChange" />
 	<rule ref="category/java/multithreading.xml/AvoidThreadGroup" />
 	<rule ref="category/java/multithreading.xml/DontCallThreadRun" />
 	<rule ref="category/java/multithreading.xml/DoubleCheckedLocking" />
diff --git a/src/main/java/com/sheepit/client/standalone/Worker.java b/src/main/java/com/sheepit/client/standalone/Worker.java
index 77e6c29..18ce973 100644
--- a/src/main/java/com/sheepit/client/standalone/Worker.java
+++ b/src/main/java/com/sheepit/client/standalone/Worker.java
@@ -406,7 +406,7 @@ public class Worker {
 			}
 			
 			if (shutdownMode != null) {
-				if (shutdownMode.toLowerCase().equals("wait") || shutdownMode.toLowerCase().equals("hard")) {
+				if (shutdownMode.equalsIgnoreCase("wait") || shutdownMode.equalsIgnoreCase("hard")) {
 					config.setShutdownMode(shutdownMode.toLowerCase());
 				}
 				else {