From 89ff34f0bb5c0953d1ad513b2787d7a4c2dfece4 Mon Sep 17 00:00:00 2001
From: Laurent Clouet <laurent@nopnop.net>
Date: Fri, 7 Jun 2024 16:05:16 +0200
Subject: [PATCH] Ref: add new static analyser rule:
 category/java/bestpractices.xml/LooseCoupling

---
 rulesets/java-sheepit.xml                           | 2 +-
 src/main/java/com/sheepit/client/Configuration.java | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/rulesets/java-sheepit.xml b/rulesets/java-sheepit.xml
index f8a7470..97be775 100644
--- a/rulesets/java-sheepit.xml
+++ b/rulesets/java-sheepit.xml
@@ -21,7 +21,7 @@
 	<rule ref="category/java/bestpractices.xml/UnusedPrivateField" />
 	<rule ref="category/java/bestpractices.xml/UnusedPrivateMethod" />
 	<rule ref="category/java/bestpractices.xml/UseCollectionIsEmpty" />
-
+	<rule ref="category/java/bestpractices.xml/LooseCoupling" />
 	<rule ref="category/java/codestyle.xml/ExtendsObject" />
 	<rule ref="category/java/codestyle.xml/ForLoopShouldBeWhileLoop" />
 	<rule ref="category/java/codestyle.xml/TooManyStaticImports" />
diff --git a/src/main/java/com/sheepit/client/Configuration.java b/src/main/java/com/sheepit/client/Configuration.java
index e0072b1..737bc67 100644
--- a/src/main/java/com/sheepit/client/Configuration.java
+++ b/src/main/java/com/sheepit/client/Configuration.java
@@ -366,7 +366,7 @@ import lombok.Data;
 	 * Only logs instances where something was detected, otherwise is it will only print "FilesystemHealthCheck started"
 	 * @return an ArrayList of Strings containing all logs of the FSHealth check
 	 */
-	public ArrayList<String> filesystemHealthCheck() {
+	public List<String> filesystemHealthCheck() {
 		ArrayList<String> logs = new ArrayList<>();
 		String f = "FSHealth: ";
 		logs.add(f + "FilesystemHealthCheck started");