[INTERNAL] TestRunner: Use parseurl module instead of req.path

To be compatible with connect middleware APIs

Author: RandomByte

lib/middleware/testRunner.js

@@ -3,6 +3,7 @@ const fs = require("graceful-fs");
 const readFile = promisify(fs.readFile);
 const path = require("path");
 const mime = require("mime-types");
+const parseurl = require("parseurl");
 const log = require("@ui5/logger").getLogger("server:middleware:testRunner");
 
 const testRunnerResourceRegEx = /\/test-resources\/sap\/ui\/qunit\/(testrunner\.(html|css)|TestRunner.js)$/;
@@ -31,7 +32,7 @@ function serveResource(res, resourcePath, resourceContent) {
 function createMiddleware({resources}) {
 	return async function(req, res, next) {
 		try {
-			const pathname = req.path; // TODO: Other middlewares use the parseurl module here but I don't get why
+			const pathname = parseurl(req).pathname;
 			const parts = testRunnerResourceRegEx.exec(pathname);
 			const resourceName = parts && parts[1];
 

test/lib/server/middleware/testRunner.js

@@ -23,7 +23,7 @@ function callMiddleware(reqPath) {
 	const middleware = testRunnerMiddleware({resources: {}});
 	return new Promise((resolve, reject) => {
 		const req = {
-			path: reqPath
+			url: `http://localhost${reqPath}`
 		};
 		const res = {
 			setHeader: function() {},