chore(CI): Update for ColdBox 8, Lucee 7, and full BoxLang Prime support

elpete · elpete · commit c1b5bbd7e484 · 2025-10-11T07:20:11.000-06:00
diff --git a/.github/workflows/cron.yml b/.github/workflows/cron.yml
@@ -11,15 +11,34 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        cfengine: ["lucee@5", "lucee@6", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang@1", "boxlang-cfml@1"]
+        cfengine: ["lucee@5", "lucee@6", "lucee@7", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang-cfml@1"]
+        coldbox: ["^6.0.0", "^7.0.0", "^8.0.0"]
         experimental: [ false ]
         include:
           - cfengine: "lucee@be"
+            coldbox: "^6.0.0"
+            experimental: true
+          - cfengine: "lucee@be"
+            coldbox: "^7.0.0"
+            experimental: true
+          - cfengine: "lucee@be"
+            coldbox: "^8.0.0"
+            experimental: true
+          - cfengine: "adobe@be"
+            coldbox: "^6.0.0"
+            experimental: true
+          - cfengine: "adobe@be"
+            coldbox: "^7.0.0"
             experimental: true
           - cfengine: "adobe@be"
+            coldbox: "^8.0.0"
             experimental: true
           - cfengine: "boxlang@be"
+            coldbox: "^8.0.0"
             experimental: true
+          - cfengine: "boxlang@1"
+            coldbox: "^8.0.0"
+            experimental: false
     steps:
       - name: Checkout Repository
         uses: actions/checkout@v3.2.0
@@ -38,6 +57,7 @@ jobs:
       - name: Install dependencies
         run: |
           box install
+          box install coldbox@{{ matrix.coldbox }} --noSave
 
       - name: Start server
         run: |
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -19,15 +19,34 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        cfengine: ["lucee@5", "lucee@6", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang@1", "boxlang-cfml@1"]
+        cfengine: ["lucee@5", "lucee@6", "lucee@7", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang-cfml@1"]
+        coldbox: ["^6.0.0", "^7.0.0", "^8.0.0"]
         experimental: [ false ]
         include:
           - cfengine: "lucee@be"
+            coldbox: "^6.0.0"
+            experimental: true
+          - cfengine: "lucee@be"
+            coldbox: "^7.0.0"
+            experimental: true
+          - cfengine: "lucee@be"
+            coldbox: "^8.0.0"
+            experimental: true
+          - cfengine: "adobe@be"
+            coldbox: "^6.0.0"
+            experimental: true
+          - cfengine: "adobe@be"
+            coldbox: "^7.0.0"
             experimental: true
           - cfengine: "adobe@be"
+            coldbox: "^8.0.0"
             experimental: true
           - cfengine: "boxlang@be"
+            coldbox: "^8.0.0"
             experimental: true
+          - cfengine: "boxlang@1"
+            coldbox: "^8.0.0"
+            experimental: false
     steps:
       - name: Checkout Repository
         uses: actions/checkout@v3.2.0
@@ -46,6 +65,7 @@ jobs:
       - name: Install dependencies
         run: |
           box install
+          box install coldbox@{{ matrix.coldbox }} --noSave
 
       - name: Start server
         run: |
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -14,8 +14,13 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        cfengine: ["lucee@5", "lucee@6", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang@1", "boxlang-cfml@1"]
+        cfengine: ["lucee@5", "lucee@6", "lucee@7", "adobe@2021", "adobe@2023", "adobe@2025", "boxlang-cfml@1"]
+        coldbox: ["^6.0.0", "^7.0.0", "^8.0.0"]
         experimental: [ false ]
+        include:
+          - cfengine: "boxlang@1"
+            coldbox: "^8.0.0"
+            experimental: false
     steps:
       - name: Checkout Repository
         uses: actions/checkout@v3.2.0
@@ -34,6 +39,7 @@ jobs:
       - name: Install dependencies
         run: |
           box install
+          box install coldbox@{{ matrix.coldbox }} --noSave
 
       - name: Start server
         run: |
diff --git a/models/CfhttpHttpClient.cfc b/models/CfhttpHttpClient.cfc
@@ -254,9 +254,10 @@ component implements="HyperHttpClientInterface" {
 			var queryParams = req.getQueryParams();
 			for ( var param in queryParams ) {
 				cfhttpparam(
-					type  = "url",
-					name  = param.name,
-					value = param.value
+					type    = "url",
+					name    = param.name,
+					value   = param.value,
+					encoded = true // this will be ignored by ACF, but used by Lucee and BoxLang (currently)
 				);
 			}
 
diff --git a/models/HyperRequest.cfc b/models/HyperRequest.cfc
@@ -216,7 +216,7 @@ component accessors="true" {
 
 		// This is overwritten by the HyperBuilder if WireBox exists.
 		variables.interceptorService = {
-			"processState" : function() {
+			"announce" : function() {
 			}
 		};
 
@@ -1096,15 +1096,15 @@ component accessors="true" {
 		for ( var callback in variables.requestCallbacks ) {
 			callback( this );
 		}
-		variables.interceptorService.processState( "onHyperRequest", { "request" : this } );
+		variables.interceptorService.announce( "onHyperRequest", { "request" : this } );
 
 		try {
 			var res = shouldFake() ? generateFakeRequest() : variables.httpClient.send( this );
 
 			for ( var callback in variables.responseCallbacks ) {
 				callback( res );
 			}
-			variables.interceptorService.processState(
+			variables.interceptorService.announce(
 				"onHyperResponse",
 				{
 					"response" : res,
@@ -1166,7 +1166,7 @@ component accessors="true" {
 		for ( var callback in variables.requestCallbacks ) {
 			callback( this );
 		}
-		variables.interceptorService.processState( "onHyperRequest", { "request" : this } );
+		variables.interceptorService.announce( "onHyperRequest", { "request" : this } );
 
 		return httpClient.debug( this );
 	}
diff --git a/server-boxlang-cfml@1.json b/server-boxlang-cfml@1.json
@@ -8,18 +8,18 @@
         "http":{
             "port":"8500"
         },
-        "rewrites": {
-            "enable": true
+        "rewrites":{
+            "enable":true
         }
     },
-    "JVM": {
-        "heapSize": "1024",
-        "javaVersion": "openjdk21_jre",
+    "JVM":{
+        "heapSize":"1024",
+        "javaVersion":"openjdk21_jre",
         "args":"-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=8888"
     },
-    "openBrowser": false,
-    "env": {},
-    "scripts": {
-        "onServerInitialInstall": "install bx-esapi,bx-compat-cfml"
+    "openBrowser":false,
+    "env":{},
+    "scripts":{
+        "onServerInitialInstall":"install bx-esapi,bx-compat-cfml"
     }
 }
diff --git a/server-boxlang@1.json b/server-boxlang@1.json
@@ -8,18 +8,18 @@
         "http":{
             "port":"8500"
         },
-        "rewrites": {
-            "enable": true
+        "rewrites":{
+            "enable":true
         }
     },
-    "JVM": {
-        "heapSize": "1024",
-        "javaVersion": "openjdk21_jre",
+    "JVM":{
+        "heapSize":"1024",
+        "javaVersion":"openjdk21_jre",
         "args":"-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=8888"
     },
-    "openBrowser": false,
-    "env": {},
-    "scripts": {
-        "onServerInitialInstall": "install bx-esapi"
+    "openBrowser":false,
+    "env":{},
+    "scripts":{
+        "onServerInitialInstall":"install bx-esapi"
     }
 }
diff --git a/server-lucee@7.json b/server-lucee@7.json
@@ -0,0 +1,16 @@
+{
+    "name":"hyper-lucee@7",
+    "app":{
+        "serverHomeDirectory":".engine/lucee7",
+        "cfengine":"lucee@7"
+    },
+    "web":{
+        "http":{
+            "port":"8500"
+        },
+        "rewrites":{
+            "enable":"true"
+        }
+    },
+    "openBrowser":"false"
+}
diff --git a/tests/Application.cfc b/tests/Application.cfc
@@ -12,7 +12,11 @@ component {
     this.mappings[ "/root" ] = rootPath;
     this.mappings[ "/hyper" ] = rootPath;
     this.mappings[ "/globber" ] = rootPath & "modules/globber";
-    this.mappings[ "/testingModuleRoot" ] = listDeleteAt( rootPath, listLen( rootPath, '\/' ), "\/" );
+    testingModuleRootMapping = listDeleteAt( rootPath, listLen( rootPath, '\/' ), "\/" );
+    if ( left( testingModuleRootMapping, 1 ) != "/" ) {
+        testingModuleRootMapping = "/" & testingModuleRootMapping;
+    }
+    this.mappings[ "/testingModuleRoot" ] = testingModuleRootMapping;
     this.mappings[ "/app" ] = testsPath & "resources/app";
     this.mappings[ "/coldbox" ] = testsPath & "resources/app/coldbox";
     this.mappings[ "/testbox" ] = rootPath & "/testbox";
diff --git a/tests/resources/app/config/Router.cfc b/tests/resources/app/config/Router.cfc
@@ -1,10 +1,6 @@
 component {
 
 	function configure() {
-		// Set Full Rewrites
-		setFullRewrites( false );
-
-		// Conventions based routing
 		route( ":handler/:action?" ).end();
 	}
 
diff --git a/tests/resources/app/handlers/api.cfc b/tests/resources/app/handlers/api.cfc
@@ -81,7 +81,11 @@ component {
 	function sleep( event, rc, prc ) {
 		param rc.delay = 0;
 		sleep( rc.delay * 1000 );
-		return event.renderData( type = "plain", statusCode = 200, data = "Slept for #rc.delay# seconds." );
+		return event.renderData(
+			type       = "plain",
+			statusCode = 200,
+			data       = "Slept for #rc.delay# seconds."
+		);
 	}
 
 }
diff --git a/tests/specs/integration/AsyncSpec.cfc b/tests/specs/integration/AsyncSpec.cfc
@@ -9,10 +9,6 @@ component extends="tests.resources.ModuleIntegrationSpec" appMapping="/app" {
 	function run() {
 		describe( "async requests", function() {
 			it( "can send a request asynchronously", function() {
-				if ( isBoxLang() ) {
-					return skip( "Variadic parameters are not yet supported in BoxLang." );
-				}
-
 				var future = hyper.setUrl( "https://jsonplaceholder.typicode.com/posts/1" ).sendAsync();
 				expect( future ).toBeInstanceOf( "Future", "A Future object should have been returned." );
 				var res = future.get();
diff --git a/tests/specs/integration/GetSpec.cfc b/tests/specs/integration/GetSpec.cfc
@@ -48,10 +48,10 @@ component extends="tests.resources.ModuleIntegrationSpec" appMapping="/app" {
 			} );
 
 			it( "deserializes query string parameters in the url (to reserialize later)", function() {
-				var res = hyper
+				var req = hyper
 					.setBaseUrl( "https://jsonplaceholder.typicode.com" )
-					.setUrl( "/posts?param=with+spaces" )
-					.get();
+					.setUrl( "/posts?param=with+spaces" );
+				var res = req.get();
 				expect( res.getRequest().getFullUrl( withQueryString = true ) ).toBeWithCase(
 					"https://jsonplaceholder.typicode.com/posts?param=with+spaces"
 				);
diff --git a/tests/specs/unit/HyperResponseSpec.cfc b/tests/specs/unit/HyperResponseSpec.cfc
@@ -228,7 +228,7 @@ component extends="testbox.system.BaseSpec" {
 					headers         = {
 						"Set-Cookie" : [
 							"foo=bar;path=/;secure;samesite=none;httponly",
-							"baz=qux;path=/;expires=Mon, 31 Dec 2038 23:59:59 GMT",
+							"baz=qux;path=/;expires=Fri, 31 Dec 2038 23:59:59 GMT",
 							"one=two;max-age=2592000;domain=example.com"
 						]
 					}
@@ -244,7 +244,7 @@ component extends="testbox.system.BaseSpec" {
 					"baz" : {
 						"value"   : "qux",
 						"path"    : "/",
-						"expires" : "Mon, 31 Dec 2038 23:59:59 GMT"
+						"expires" : "Fri, 31 Dec 2038 23:59:59 GMT"
 					},
 					"one" : {
 						"value"   : "two",
@@ -262,7 +262,7 @@ component extends="testbox.system.BaseSpec" {
 					headers         = {
 						"Set-Cookie" : [
 							"foo=bar;path=/;secure;samesite=none;httponly",
-							"baz=qux;path=/;expires=Mon, 31 Dec 2038 23:59:59 GMT",
+							"baz=qux;path=/;expires=Fri, 31 Dec 2038 23:59:59 GMT",
 							"one=two;max-age=2592000;domain=example.com"
 						]
 					}

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,7 @@ component accessors="true" {`
`216`	`216`
`217`	`217`	`// This is overwritten by the HyperBuilder if WireBox exists.`
`218`	`218`	`variables.interceptorService = {`
`219`		`- "processState" : function() {`
	`219`	`+ "announce" : function() {`
`220`	`220`	`}`
`221`	`221`	`};`
`222`	`222`
`@@ -1096,15 +1096,15 @@ component accessors="true" {`
`1096`	`1096`	`for ( var callback in variables.requestCallbacks ) {`
`1097`	`1097`	`callback( this );`
`1098`	`1098`	`}`
`1099`		`- variables.interceptorService.processState( "onHyperRequest", { "request" : this } );`
	`1099`	`+ variables.interceptorService.announce( "onHyperRequest", { "request" : this } );`
`1100`	`1100`
`1101`	`1101`	`try {`
`1102`	`1102`	`var res = shouldFake() ? generateFakeRequest() : variables.httpClient.send( this );`
`1103`	`1103`
`1104`	`1104`	`for ( var callback in variables.responseCallbacks ) {`
`1105`	`1105`	`callback( res );`
`1106`	`1106`	`}`
`1107`		`- variables.interceptorService.processState(`
	`1107`	`+ variables.interceptorService.announce(`
`1108`	`1108`	`"onHyperResponse",`
`1109`	`1109`	`{`
`1110`	`1110`	`"response" : res,`
`@@ -1166,7 +1166,7 @@ component accessors="true" {`
`1166`	`1166`	`for ( var callback in variables.requestCallbacks ) {`
`1167`	`1167`	`callback( this );`
`1168`	`1168`	`}`
`1169`		`- variables.interceptorService.processState( "onHyperRequest", { "request" : this } );`
	`1169`	`+ variables.interceptorService.announce( "onHyperRequest", { "request" : this } );`
`1170`	`1170`
`1171`	`1171`	`return httpClient.debug( this );`
`1172`	`1172`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,6 @@`
`1`	`1`	`component {`
`2`	`2`
`3`	`3`	`function configure() {`
`4`		`- // Set Full Rewrites`
`5`		`- setFullRewrites( false );`
`6`		`-`
`7`		`- // Conventions based routing`
`8`	`4`	`route( ":handler/:action?" ).end();`
`9`	`5`	`}`
`10`	`6`