Run CI/test on Java 8/11/17/21 versions

Makes minor visibility changes to ensure they can be run on Java 17+ as well without using reflection hacks.

Author: chadlwilson

.github/workflows/maven.yml

@@ -16,15 +16,20 @@ on:
 
 jobs:
   build:
-
+    name: Java ${{ matrix.java_version }}
     runs-on: ubuntu-latest
 
+    strategy:
+      matrix:
+        java_version: [ '8', '11', '17', '21' ]
+      fail-fast: false
+
     steps:
     - uses: actions/checkout@v4
-    - name: Set up JDK 8 
+    - name: Set up JDK
       uses: actions/setup-java@v4
       with:
-        java-version: '8'
+        java-version: ${{ matrix.java_version }}
         distribution: 'temurin'
         cache: maven
     - name: Build with Maven

pom.xml

@@ -145,6 +145,12 @@
       <version>3.0.3</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>jakarta.activation</groupId>
+      <artifactId>jakarta.activation-api</artifactId>
+      <version>1.2.2</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
   <build>

src/main/java/org/jruby/rack/DefaultRackApplicationFactory.java

@@ -352,11 +352,11 @@ protected void loadJRubyRack(final Ruby runtime) {
     /**
      * Initializes the runtime (exports the context, boots the Rack handler).
      *
-     * NOTE: (package) visible due specs
+     * NOTE: visible due to specs
      *
      * @param runtime
      */
-    void initRuntime(final Ruby runtime) {
+    public void initRuntime(final Ruby runtime) {
         loadJRubyRack(runtime);
         // set $servlet_context :
         runtime.getGlobalVariables().set(
@@ -406,7 +406,7 @@ void initRuntime(final Ruby runtime) {
      * @param runtime
      * @return the rack version requirement
      */
-    String checkAndSetRackVersion(final Ruby runtime) {
+    public String checkAndSetRackVersion(final Ruby runtime) {
         String rackVersion = null;
         try {
             rackVersion = IOHelpers.rubyMagicCommentValue(rackupScript, "rack.version:");

src/main/java/org/jruby/rack/embed/Config.java

@@ -81,7 +81,7 @@ public String getProperty(String key, String defaultValue) {
     } */
 
     @SuppressWarnings("unchecked")
-    void doInitialize(final Ruby runtime) {
+    public void doInitialize(final Ruby runtime) {
         setOut( runtime.getOut() );
         setErr( runtime.getErr() );
         rubyENV = runtime.getENV();