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

To be compatible with connect middleware APIs

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() {},