Merge pull request #18 from sentrysoftware/feature/Fix-issue-17-immut…

…able-lists [#17] Fix Unsupported Operation Exception when using CiscoUCSRest request using CLI
sentrysoftware · Jan 11, 2024 · d1ba024 · d1ba024
2 parents 2061abe + dfd42b1
commit d1ba024
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/main/java/org/sentrysoftware/ipmi/client/IpmiClient.java b/src/main/java/org/sentrysoftware/ipmi/client/IpmiClient.java
@@ -24,7 +24,7 @@
 
 import static org.sentrysoftware.ipmi.client.Utils.execute;
 
-import java.util.Collections;
+import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeoutException;
@@ -80,7 +80,7 @@ public static List<Sensor> getSensors(final IpmiClientConfiguration ipmiConfigur
 			try (GetSensorsRunner runner = new GetSensorsRunner(ipmiConfiguration)) {
 				runner.doRun();
 				List<Sensor> result = runner.getResult();
-				return result != null ? result : Collections.emptyList();
+				return result != null ? result : new ArrayList<>();
 			}
 		}, ipmiConfiguration.getTimeout() * 1000);
 	}
@@ -101,7 +101,7 @@ public static List<Fru> getFrus(final IpmiClientConfiguration ipmiConfiguration)
 			try (GetFrusRunner runner = new GetFrusRunner(ipmiConfiguration)) {
 				runner.doRun();
 				List<Fru> result = runner.getResult();
-				return result != null ? result : Collections.emptyList();
+				return result != null ? result : new ArrayList<>();
 			}
 		}, ipmiConfiguration.getTimeout() * 1000);
 	}

diff --git a/src/main/java/org/sentrysoftware/ipmi/client/runner/GetFrusRunner.java b/src/main/java/org/sentrysoftware/ipmi/client/runner/GetFrusRunner.java
@@ -23,7 +23,6 @@
  */
 
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import java.util.stream.Collectors;
 
@@ -238,6 +237,6 @@ private List<FruRecord> getFruRecords(int fruId) throws Exception {
 			// Nothing can be done
 		}
 
-		return Collections.emptyList();
+		return new ArrayList<>();
 	}
 }