[INTERNAL] Update test setup

Alignment with current best practices (e.g. in Testing Tutorial) - Remove usage of <base> tag - Usage of ui5:// protocol instead of relative page path - Fix coverage exclusion for test modules
SAP · Oct 22, 2024 · b53d30e · b53d30e
1 parent 01334af
commit b53d30e
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 16 deletions.
diff --git a/webapp/test/Test.qunit.html b/webapp/test/Test.qunit.html
@@ -2,11 +2,10 @@
 <html>
 <head>
 	<meta charset="utf-8">
-	<base href="../">
 	<script
-		src="resources/sap/ui/test/starter/runTest.js"
+		src="../resources/sap/ui/test/starter/runTest.js"
 		data-sap-ui-resource-roots='{
-			"test-resources.sap.ui.demo.todo": "./test"
+			"test-resources.sap.ui.demo.todo": "./"
 		}'
 	></script>
 </head>

diff --git a/webapp/test/integration/FilterJourney.js b/webapp/test/integration/FilterJourney.js
@@ -2,7 +2,7 @@
 
 sap.ui.define([
 	"sap/ui/test/opaQunit",
-	"sap/ui/demo/todo/test/integration/pages/App"
+	"./pages/App"
 ], (opaTest) => {
 	"use strict";
 

diff --git a/webapp/test/integration/SearchJourney.js b/webapp/test/integration/SearchJourney.js
@@ -3,7 +3,7 @@
 sap.ui.define([
 	"sap/ui/Device",
 	"sap/ui/test/opaQunit",
-	"sap/ui/demo/todo/test/integration/pages/App"
+	"./pages/App"
 ], (Device, opaTest) => {
 	"use strict";
 

diff --git a/webapp/test/integration/TodoListJourney.js b/webapp/test/integration/TodoListJourney.js
@@ -2,7 +2,7 @@
 
 sap.ui.define([
 	"sap/ui/test/opaQunit",
-	"sap/ui/demo/todo/test/integration/pages/App"
+	"./pages/App"
 ], (opaTest) => {
 	"use strict";
 

diff --git a/webapp/test/integration/opaTests.qunit.js b/webapp/test/integration/opaTests.qunit.js
@@ -1,9 +1,9 @@
 sap.ui.define([
 	"sap/ui/test/Opa5",
-	"sap/ui/demo/todo/test/integration/arrangements/Startup",
-	"sap/ui/demo/todo/test/integration/TodoListJourney",
-	"sap/ui/demo/todo/test/integration/SearchJourney",
-	"sap/ui/demo/todo/test/integration/FilterJourney"
+	"./arrangements/Startup",
+	"./TodoListJourney",
+	"./SearchJourney",
+	"./FilterJourney"
 ], (Opa5, Startup) => {
 	"use strict";
 

diff --git a/webapp/test/testsuite.qunit.html b/webapp/test/testsuite.qunit.html
@@ -2,13 +2,12 @@
 <html>
 <head>
 	<meta charset="utf-8">
-	<base href="../">
 	<title>QUnit test suite for Todo App</title>
 	<script
-		src="resources/sap/ui/test/starter/createSuite.js"
+		src="../resources/sap/ui/test/starter/createSuite.js"
 		data-sap-ui-testsuite="test-resources/sap/ui/demo/todo/testsuite.qunit"
 		data-sap-ui-resource-roots='{
-			"test-resources.sap.ui.demo.todo": "./test"
+			"test-resources.sap.ui.demo.todo": "./"
 		}'
 	></script>
 </head>

diff --git a/webapp/test/testsuite.qunit.js b/webapp/test/testsuite.qunit.js
@@ -3,7 +3,7 @@ sap.ui.define(function () {
 	return {
 		name: "QUnit test suite for Todo App",
 		defaults: {
-			page: "test/Test.qunit.html?testsuite={suite}&test={name}",
+			page: "ui5://test-resources/sap/ui/demo/todo/Test.qunit.html?testsuite={suite}&test={name}",
 			qunit: {
 				version: 2
 			},
@@ -16,11 +16,11 @@ sap.ui.define(function () {
 			},
 			coverage: {
 				only: "sap/ui/demo/todo/",
-				never: "sap/ui/demo/todo/test/"
+				never: "test-resources/sap/ui/demo/todo/"
 			},
 			loader: {
 				paths: {
-					"sap/ui/demo/todo": "./"
+					"sap/ui/demo/todo": "../"
 				}
 			}
 		},