From 424f300ad27d3d1f2fa9640ecaa66f8a0bb05583 Mon Sep 17 00:00:00 2001
From: Norman Maurer <norman_maurer@apple.com>
Date: Thu, 2 Mar 2023 09:35:22 +0100
Subject: [PATCH] Skip revapi checks for testsuite modules

Motivation:

We should skip revapi checks for testsuite modules as these are not expected to be public API

Modifications:

Add configuration for skipping

Result:

No API check for testsuites
---
 testsuite-autobahn/pom.xml                         | 1 +
 testsuite-http2/pom.xml                            | 1 +
 testsuite-native-image-client-runtime-init/pom.xml | 1 +
 testsuite-native-image-client/pom.xml              | 1 +
 testsuite-native-image/pom.xml                     | 1 +
 testsuite-native/pom.xml                           | 1 +
 testsuite-osgi/pom.xml                             | 1 +
 testsuite-shading/pom.xml                          | 1 +
 testsuite/pom.xml                                  | 1 +
 9 files changed, 9 insertions(+)

diff --git a/testsuite-autobahn/pom.xml b/testsuite-autobahn/pom.xml
index 551d90daeb3..7787e0c9c36 100644
--- a/testsuite-autobahn/pom.xml
+++ b/testsuite-autobahn/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_autobahn</javaModuleName>
diff --git a/testsuite-http2/pom.xml b/testsuite-http2/pom.xml
index 16a09d5e415..2c64acca6e3 100644
--- a/testsuite-http2/pom.xml
+++ b/testsuite-http2/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_http2</javaModuleName>
diff --git a/testsuite-native-image-client-runtime-init/pom.xml b/testsuite-native-image-client-runtime-init/pom.xml
index d64604aeafc..e86902ce5b8 100644
--- a/testsuite-native-image-client-runtime-init/pom.xml
+++ b/testsuite-native-image-client-runtime-init/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_native_image_client_runtime_init</javaModuleName>
diff --git a/testsuite-native-image-client/pom.xml b/testsuite-native-image-client/pom.xml
index 957cb086296..9074f9a5caf 100644
--- a/testsuite-native-image-client/pom.xml
+++ b/testsuite-native-image-client/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_native_image_client</javaModuleName>
diff --git a/testsuite-native-image/pom.xml b/testsuite-native-image/pom.xml
index ab079072938..ca1215eae0d 100644
--- a/testsuite-native-image/pom.xml
+++ b/testsuite-native-image/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_native_image</javaModuleName>
diff --git a/testsuite-native/pom.xml b/testsuite-native/pom.xml
index a4182de00af..03fa1a4b4af 100644
--- a/testsuite-native/pom.xml
+++ b/testsuite-native/pom.xml
@@ -30,6 +30,7 @@
 
   <properties>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <skipNativeTestsuite>false</skipNativeTestsuite>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
diff --git a/testsuite-osgi/pom.xml b/testsuite-osgi/pom.xml
index 8a518530af9..843d4eb4de7 100644
--- a/testsuite-osgi/pom.xml
+++ b/testsuite-osgi/pom.xml
@@ -32,6 +32,7 @@
     <exam.version>4.13.0</exam.version>
     <argLine.java9.extras>--add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/jdk.internal.loader=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.security=ALL-UNNAMED</argLine.java9.extras>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_osgi</javaModuleName>
diff --git a/testsuite-shading/pom.xml b/testsuite-shading/pom.xml
index 82a172e7dcc..c9efa584850 100644
--- a/testsuite-shading/pom.xml
+++ b/testsuite-shading/pom.xml
@@ -40,6 +40,7 @@
     <skipShadingTestsuite>true</skipShadingTestsuite>
     <shadedPackagePrefix>io.netty.</shadedPackagePrefix>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <!-- Do not deploy this module -->
     <skipDeploy>true</skipDeploy>
     <javaModuleName>io.netty.testsuite_shading</javaModuleName>
diff --git a/testsuite/pom.xml b/testsuite/pom.xml
index 6d3fe3a4692..df73b969267 100644
--- a/testsuite/pom.xml
+++ b/testsuite/pom.xml
@@ -120,6 +120,7 @@
     <!-- Needed for SSL tests as these use the SelfSignedCertificate --> 
     <argLine.java9.extras>--add-exports java.base/sun.security.x509=ALL-UNNAMED</argLine.java9.extras>
     <japicmp.skip>true</japicmp.skip>
+    <revapi.skip>true</revapi.skip>
     <javaModuleName>io.netty.testsuite</javaModuleName>
   </properties>