From f8755ae02ed58f5b3ce55e33d90589202aadd15e Mon Sep 17 00:00:00 2001
From: Julien HENRY <julien.henry@sonarsource.com>
Date: Thu, 2 May 2024 15:04:30 +0200
Subject: Fix some quality flaws

---
 .../sonar/scanner/bootstrap/GlobalTempFolderProviderTest.java    | 9 ++++-----
 .../test/java/org/sonar/scanner/bootstrap/PluginFilesTest.java   | 2 +-
 .../java/org/sonar/scanner/http/ScannerWsClientProviderTest.java | 8 +++-----
 3 files changed, 8 insertions(+), 11 deletions(-)

(limited to 'sonar-scanner-engine/src/test/java')

diff --git a/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/GlobalTempFolderProviderTest.java b/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/GlobalTempFolderProviderTest.java
index 9d8522a897f..2a27b7c5653 100644
--- a/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/GlobalTempFolderProviderTest.java
+++ b/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/GlobalTempFolderProviderTest.java
@@ -19,7 +19,6 @@
  */
 package org.sonar.scanner.bootstrap;
 
-import com.google.common.collect.ImmutableMap;
 import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
@@ -49,7 +48,7 @@ class GlobalTempFolderProviderTest {
     Files.delete(workingDir);
 
     var tempFolder = underTest.provide(
-      new ScannerProperties(ImmutableMap.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, workingDir.toAbsolutePath().toString())), sonarUserHome);
+      new ScannerProperties(Map.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, workingDir.toAbsolutePath().toString())), sonarUserHome);
     tempFolder.newDir();
     tempFolder.newFile();
 
@@ -71,7 +70,7 @@ class GlobalTempFolderProviderTest {
     }
 
     underTest.provide(
-      new ScannerProperties(ImmutableMap.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, workingDir.toAbsolutePath().toString())), sonarUserHome);
+      new ScannerProperties(Map.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, workingDir.toAbsolutePath().toString())), sonarUserHome);
 
     // this also checks that all other temps were deleted
     assertThat(workingDir.toFile().list()).hasSize(1);
@@ -88,7 +87,7 @@ class GlobalTempFolderProviderTest {
   }
 
   @Test
-  void createTempFolderFromSonarHome(@TempDir Path sonarUserHomePath) throws Exception {
+  void createTempFolderFromSonarHome(@TempDir Path sonarUserHomePath) {
     // with sonar home, it will be in {sonar.home}/.sonartmp
     when(sonarUserHome.getPath()).thenReturn(sonarUserHomePath);
     
@@ -109,7 +108,7 @@ class GlobalTempFolderProviderTest {
     when(sonarUserHome.getPath()).thenReturn(sonarUserHomePath);
     String globalWorkDir = ".";
     ScannerProperties globalProperties = new ScannerProperties(
-      ImmutableMap.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, globalWorkDir));
+      Map.of(CoreProperties.GLOBAL_WORKING_DIRECTORY, globalWorkDir));
 
     var tempFolder = underTest.provide(globalProperties, sonarUserHome);
     File newFile = tempFolder.newFile();
diff --git a/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/PluginFilesTest.java b/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/PluginFilesTest.java
index 907e13f92bc..97d33f200fa 100644
--- a/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/PluginFilesTest.java
+++ b/sonar-scanner-engine/src/test/java/org/sonar/scanner/bootstrap/PluginFilesTest.java
@@ -78,7 +78,7 @@ class PluginFilesTest {
   private PluginFiles underTest;
 
   @BeforeEach
-  void setUp(@TempDir Path sonarUserHomeDir) throws Exception {
+  void setUp(@TempDir Path sonarUserHomeDir) {
     when(sonarUserHome.getPath()).thenReturn(sonarUserHomeDir);
 
     HttpConnector connector = HttpConnector.newBuilder().acceptGzip(true).url(sonarqube.url("/")).build();
diff --git a/sonar-scanner-engine/src/test/java/org/sonar/scanner/http/ScannerWsClientProviderTest.java b/sonar-scanner-engine/src/test/java/org/sonar/scanner/http/ScannerWsClientProviderTest.java
index e0ee3121acc..e605999ddd3 100644
--- a/sonar-scanner-engine/src/test/java/org/sonar/scanner/http/ScannerWsClientProviderTest.java
+++ b/sonar-scanner-engine/src/test/java/org/sonar/scanner/http/ScannerWsClientProviderTest.java
@@ -66,9 +66,7 @@ class ScannerWsClientProviderTest {
   private static final GlobalAnalysisMode GLOBAL_ANALYSIS_MODE = new GlobalAnalysisMode(new ScannerProperties(Collections.emptyMap()));
   private static final AnalysisWarnings ANALYSIS_WARNINGS = warning -> {
   };
-  @TempDir
-  private Path sonarUserHomeDir;
-  private final SonarUserHome sonarUserHome = mock(SonarUserHome.class);
+  private SonarUserHome sonarUserHome = mock(SonarUserHome.class);
   private final Map<String, String> scannerProps = new HashMap<>();
 
   private final ScannerWsClientProvider underTest = new ScannerWsClientProvider();
@@ -78,9 +76,9 @@ class ScannerWsClientProviderTest {
   private final Properties systemProps = new Properties();
 
   @BeforeEach
-  void configureMocks() {
+  void configureMocks(@TempDir Path sonarUserHomeDir) {
     when(system2.properties()).thenReturn(systemProps);
-    when(sonarUserHome.getPath()).thenReturn(sonarUserHomeDir);
+    sonarUserHome = new SonarUserHome(sonarUserHomeDir);
   }
 
   @Test
-- 
cgit v1.2.3