Solves sonar: Only one method invocation is expected when testing run…

…time exceptions
damianszczepanik · Jan 1, 2024 · 15cad69 · 15cad69
1 parent 5603807
commit 15cad69
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml
@@ -21,7 +21,7 @@ jobs:
           distribution: adopt
 
       - name: Build with Maven
-        run: mvn --batch-mode package sonar:sonar -Dsonar.projectKey=damianszczepanik_silencio -Dsonar.organization=damianszczepanik -Dsonar.host.url=https://sonarcloud.io -Dsonar.login=$SONAR_TOKEN
+        run: mvn --batch-mode package sonar:sonar -Dsonar.projectKey=damianszczepanik_silencio -Dsonar.organization=damianszczepanik -Dsonar.host.url=https://sonarcloud.io -Dsonar.token=$SONAR_TOKEN
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
diff --git a/src/test/java/pl/szczepanik/silencio/utils/IOUtilityTest.java b/src/test/java/pl/szczepanik/silencio/utils/IOUtilityTest.java
@@ -56,12 +56,15 @@ void shouldFailOnIOException() throws Exception {
 
         String errorMessage = "Something is wrong!";
 
+        // given
+        URL url = new URL(URL_ADDRESS);
+
         // when
-        when(IOUtils.toString(new URL(URL_ADDRESS), StandardCharsets.UTF_8))
+        when(IOUtils.toString(url, StandardCharsets.UTF_8))
                 .thenThrow(new IOException(errorMessage));
 
         // then
-        assertThatThrownBy(() -> IOUtility.urlToString(new URL(URL_ADDRESS)))
+        assertThatThrownBy(() -> IOUtility.urlToString(url))
                 .isInstanceOf(IntegrityException.class)
                 .hasMessage(errorMessage);
     }