From 748bca0a4bb701055347fdaa0ca648a3cfa7375c Mon Sep 17 00:00:00 2001
From: Daniel Muehlbachler-Pietrzykowski <daniel@muehlbachler.io>
Date: Fri, 4 Mar 2022 14:22:23 +0100
Subject: [PATCH] Issue #403: upgrade to checkstyle 10.0; drop java 1.8 support

---
 .github/workflows/sonar-checkstyle-workflows.yml      | 11 -----------
 .gitignore                                            |  3 +++
 README.md                                             |  1 +
 pom.xml                                               |  6 +++---
 .../plugins/checkstyle/CheckstyleConstantsTest.java   |  3 ++-
 .../checkstyle/CheckstyleSeverityUtilsTest.java       |  3 ++-
 6 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/sonar-checkstyle-workflows.yml b/.github/workflows/sonar-checkstyle-workflows.yml
index 66082e69..83dea960 100644
--- a/.github/workflows/sonar-checkstyle-workflows.yml
+++ b/.github/workflows/sonar-checkstyle-workflows.yml
@@ -7,17 +7,6 @@ on:
   pull_request:
 
 jobs:
-  mvn-install-java-8:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - name: Set up JDK 8
-        uses: actions/setup-java@v1
-        with:
-          java-version: 8
-      - name: install
-        run: "./.ci/ci.sh install"
-
   mvn-install-java-11:
     runs-on: ubuntu-latest
     steps:
diff --git a/.gitignore b/.gitignore
index d6ef3594..ed17e064 100644
--- a/.gitignore
+++ b/.gitignore
@@ -15,6 +15,9 @@ target/
 .project
 .settings
 
+# ---- VSCode
+.vscode/
+
 # ---- Mac OS X
 .DS_Store?
 Icon?
diff --git a/README.md b/README.md
index b623e4c8..d88101ad 100644
--- a/README.md
+++ b/README.md
@@ -21,6 +21,7 @@ Compatibility matrix from checkstyle team:
 
 | Checkstyle Plugin | Sonar min | Sonar max | Checkstyle | Jdk |
 |-------------------|-----------|-----------|------------|-----|
+| 10.0              | 8.9       | 8.9+      | 10.0       | 11  |
 | 9.3               | 8.9       | 8.9+      | 9.3        | 1.8 |
 | 9.2.1             | 8.9       | 8.9+      | 9.2.1      | 1.8 |
 | 9.2               | 8.9       | 8.9+      | 9.2        | 1.8 |
diff --git a/pom.xml b/pom.xml
index 54aaa896..b1c9185a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -4,7 +4,7 @@
 
   <groupId>com.github.checkstyle</groupId>
   <artifactId>checkstyle-sonar-plugin</artifactId>
-  <version>9.4-SNAPSHOT</version>
+  <version>10.0-SNAPSHOT</version>
   <packaging>sonar-plugin</packaging>
 
   <name>Checkstyle SonarQube Plugin</name>
@@ -93,7 +93,7 @@
   </ciManagement>
 
   <properties>
-    <checkstyle.version>9.3</checkstyle.version>
+    <checkstyle.version>10.0</checkstyle.version>
     <sonar.version>8.9.0.43852</sonar.version>
     <sonar-java.version>7.2.0.26923</sonar-java.version>
     <maven.checkstyle.plugin.version>3.1.2</maven.checkstyle.plugin.version>
@@ -103,7 +103,7 @@
       8.35
     </maven.sevntu.checkstyle.plugin.checkstyle.version>
     <maven.jacoco.plugin.version>0.8.5</maven.jacoco.plugin.version>
-    <java.version>1.8</java.version>
+    <java.version>11</java.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <!-- we disable ITs by default as it need download of 200MB sonar application jar -->
     <skipITs>true</skipITs>
diff --git a/src/test/java/org/sonar/plugins/checkstyle/CheckstyleConstantsTest.java b/src/test/java/org/sonar/plugins/checkstyle/CheckstyleConstantsTest.java
index a4666c67..6ed347ee 100644
--- a/src/test/java/org/sonar/plugins/checkstyle/CheckstyleConstantsTest.java
+++ b/src/test/java/org/sonar/plugins/checkstyle/CheckstyleConstantsTest.java
@@ -23,6 +23,7 @@
 
 import java.lang.reflect.Constructor;
 
+import org.eclipse.jdt.core.dom.Modifier;
 import org.junit.Test;
 
 public class CheckstyleConstantsTest {
@@ -31,7 +32,7 @@ public class CheckstyleConstantsTest {
     public void privateConstructor() throws ReflectiveOperationException {
         final Constructor<CheckstyleConstants> constructor = CheckstyleConstants.class
                 .getDeclaredConstructor();
-        assertThat(constructor.isAccessible()).isFalse();
+        assertThat(Modifier.isPrivate(constructor.getModifiers()));
         constructor.setAccessible(true);
         constructor.newInstance();
     }
diff --git a/src/test/java/org/sonar/plugins/checkstyle/CheckstyleSeverityUtilsTest.java b/src/test/java/org/sonar/plugins/checkstyle/CheckstyleSeverityUtilsTest.java
index 5591a89b..425bd289 100644
--- a/src/test/java/org/sonar/plugins/checkstyle/CheckstyleSeverityUtilsTest.java
+++ b/src/test/java/org/sonar/plugins/checkstyle/CheckstyleSeverityUtilsTest.java
@@ -23,6 +23,7 @@
 
 import java.lang.reflect.Constructor;
 
+import org.eclipse.jdt.core.dom.Modifier;
 import org.junit.Assert;
 import org.junit.Test;
 import org.sonar.api.rules.RulePriority;
@@ -72,7 +73,7 @@ public void privateConstructor() throws ReflectiveOperationException {
         final Constructor<CheckstyleSeverityUtils> constructor = CheckstyleSeverityUtils.class
                 .getDeclaredConstructor();
 
-        assertThat(constructor.isAccessible()).isFalse();
+        assertThat(Modifier.isPrivate(constructor.getModifiers()));
         constructor.setAccessible(true);
         constructor.newInstance();
     }