Add GH Action test matrix for OS/netty/transport

.github/workflows/check_transport.yml

@@ -0,0 +1,28 @@
+name: Check Matrix
+
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macOS-latest, windows-latest]
+        transport: [native, nio]
+        exclude:
+          # excludes native on Windows (there's none)
+          - os: windows-latest
+            transport: native
+
+    steps:
+    - uses: actions/checkout@v1
+    - name: Set up JDK 1.8
+      uses: actions/setup-java@v1
+      with:
+        java-version: 1.8
+    - name: Build with Gradle
+# IF WE NEED TO TEST DIFFERENT VERSIONS OF NETTY, WE CAN ADD forceNettyVersion TO THE MATRIX
+#      run: ./gradlew check -PforceTransport=${{ matrix.transport }} -PforceNettyVersion=${{ matrix.netty }}
+      run: ./gradlew clean check -PforceTransport=${{ matrix.transport }}

build.gradle

@@ -60,8 +60,17 @@ ext {
   testAddonVersion = reactorCoreVersion
   assertJVersion = '3.6.1'
 
+  // Netty
+  nettyDefaultVersion = '4.1.43.Final'
+  if (!project.hasProperty("forceNettyVersion")) {
+	nettyVersion = nettyDefaultVersion
+  }
+  else {
+	nettyVersion = forceNettyVersion
+	println "Netty version defined from command line: ${forceNettyVersion}"
+  }
+
   // Libraries
-  nettyVersion = '4.1.43.Final'
   jacksonDatabindVersion = '2.5.1'
 
   // Testing
@@ -203,6 +212,7 @@ configure(rootProject) { project ->
 		println 'Bamboo CI detected, avoiding use of mavenLocal()'
 	  }
 	  maven { url 'https://repo.spring.io/libs-snapshot' }
+	  maven { url 'https://oss.sonatype.org/content/repositories/snapshots' }
 	}
 	maven { url 'https://repo.spring.io/libs-milestone' }
 	maven { url 'https://repo.spring.io/libs-release' }
@@ -242,8 +252,32 @@ configure(rootProject) { project ->
 
 	compile "io.netty:netty-handler:${nettyVersion}"
 	compile "io.netty:netty-codec-http:${nettyVersion}"
-	compile "io.netty:netty-transport-native-epoll:${nettyVersion}:linux-x86_64"
-	optional "io.netty:netty-transport-native-kqueue:${nettyVersion}"
+
+	//transport resolution: typical build forces epoll but not kqueue transitively
+	//on the other hand, if we want to make transport-specific tests, we'll make all
+	// native optional at compile time and add correct native/nio to testRuntime
+	if (project.hasProperty("forceTransport")) {
+	  //so that the main code compiles
+	  optional "io.netty:netty-transport-native-epoll:${nettyVersion}"
+	  optional "io.netty:netty-transport-native-kqueue:${nettyVersion}"
+	  //now we explicitly add correctly qualified native, or do nothing if we want to test NIO
+	  if (forceTransport == "native") {
+		if (osdetector.os == "osx") {
+		  testRuntime "io.netty:netty-transport-native-kqueue:${nettyVersion}${os_suffix}"
+		}
+		else if (osdetector.os == "linux") {
+		  testRuntime "io.netty:netty-transport-native-epoll:${nettyVersion}${os_suffix}"
+		}
+	  }
+	  else if (forceTransport != "nio") {
+		throw new InvalidUserDataException("invalid -PforceTranport option " + forceTransport + ", should be native|nio")
+	  }
+	}
+	else {
+	  //classic build to be distributed
+	  compile "io.netty:netty-transport-native-epoll:${nettyVersion}:linux-x86_64"
+	  optional "io.netty:netty-transport-native-kqueue:${nettyVersion}"
+	}
 
 	// Testing
 	testCompile "com.fasterxml.jackson.core:jackson-databind:$jacksonDatabindVersion"