Fix Java ITs

SonarSource · Aug 9, 2024 · 8d3a71e · 8d3a71e
1 parent ce8ae45
commit 8d3a71e
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 9 deletions.
diff --git a/its/projects/WebConfig.CSharp/WebConfig.CSharp/Dummy.cs b/its/projects/WebConfig.CSharp/WebConfig.CSharp/Dummy.cs
@@ -1,5 +1,5 @@
 public class Dummy
 {
     // This is present just to have a file to analyze.
-    public int Prop => 42;
+    public static int Prop => 42;
 }
diff --git a/its/src/test/java/com/sonar/it/csharp/MultipleProjectsTest.java b/its/src/test/java/com/sonar/it/csharp/MultipleProjectsTest.java
@@ -98,11 +98,12 @@ void firstProjectFirstClassIssues() {
       .collect(Collectors.toList());
 
     assertThat(barIssues)
-      .hasSize(2)
+      .hasSize(3)
       .extracting(Issues.Issue::getRule, Issues.Issue::getLine)
       .containsExactlyInAnyOrder(
         tuple("csharpsquid:S1134", 3),
-        tuple("csharpsquid:S1135", 4));
+        tuple("csharpsquid:S1135", 4),
+        tuple("csharpsquid:S2325", 16));
   }
 
   @Test
@@ -114,7 +115,9 @@ void secondProjectFirstClassIssues() {
 
     assertThat(barIssues)
       .extracting(Issues.Issue::getRule, Issues.Issue::getLine)
-      .containsExactlyInAnyOrder(tuple("csharpsquid:S1135", 1));
+      .containsExactlyInAnyOrder(
+        tuple("csharpsquid:S1135", 1),
+        tuple("csharpsquid:S2325", 10));
   }
 
   /* Files */

diff --git a/its/src/test/java/com/sonar/it/csharp/NoSonarTest.java b/its/src/test/java/com/sonar/it/csharp/NoSonarTest.java
@@ -31,6 +31,7 @@
 
 import static com.sonar.it.csharp.Tests.ORCHESTRATOR;
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.tuple;
 
 @ExtendWith(Tests.class)
 public class NoSonarTest {
@@ -49,11 +50,13 @@ public static void init() throws Exception {
   @Test
   void excludeNoSonarComment() {
     List<Issues.Issue> issues = TestUtils.getIssues(ORCHESTRATOR, PROJECT);
-    assertThat(issues).hasSize(2);
-    assertThat(issues.get(0).getLine()).isEqualTo(3);
-    assertThat(issues.get(0).getRule()).isEqualTo("csharpsquid:S101");
-    assertThat(issues.get(1).getLine()).isEqualTo(5);
-    assertThat(issues.get(1).getRule()).isEqualTo("external_roslyn:CA1822");
+    assertThat(issues)
+      .hasSize(3)
+      .extracting(Issues.Issue::getRule, Issues.Issue::getLine)
+      .containsExactlyInAnyOrder(
+        tuple("csharpsquid:S101", 3),
+        tuple("external_roslyn:CA1822", 5),
+        tuple("csharpsquid:S2325", 5));
   }
 
   @Test