Merge pull request #30567 from nextcloud/bugfix/noid/npm-ci-for-karma

Use npm ci when running JS tests
nextcloud · Jan 10, 2022 · 8954e51 · 8954e51
2 parents 6572f80 + 6aba396
commit 8954e51
Show file tree

Hide file tree

Showing 25 changed files with 5,513 additions and 286 deletions.
diff --git a/apps/files/src/files-app-settings.js b/apps/files/src/files-app-settings.js
@@ -37,6 +37,9 @@ Object.assign(window.OCA.Files, { Settings: new Settings() })
 Object.assign(window.OCA.Files.Settings, { Setting })
 
 window.addEventListener('DOMContentLoaded', function() {
+	if (window.TESTING) {
+		return
+	}
 	// Init Vue app
 	// eslint-disable-next-line
 	new Vue({

diff --git a/apps/files/src/main-personal-settings.js b/apps/files/src/main-personal-settings.js
@@ -32,5 +32,7 @@ __webpack_nonce__ = btoa(getRequestToken())
 
 Vue.prototype.t = t
 
-const View = Vue.extend(PersonalSettings)
-new View().$mount('#files-personal-settings')
+if (!window.TESTING) {
+	const View = Vue.extend(PersonalSettings)
+	new View().$mount('#files-personal-settings')
+}
diff --git a/apps/systemtags/js/admin.js b/apps/systemtags/js/admin.js
@@ -179,6 +179,8 @@
 })();
 
 window.addEventListener('DOMContentLoaded', function() {
-	OCA.SystemTags.Admin.init();
+	if (!window.TESTING) {
+		OCA.SystemTags.Admin.init();
+	}
 });
 
diff --git a/autotest-js.sh b/autotest-js.sh
@@ -20,7 +20,7 @@ then
 fi
 
 # update/install test packages
-mkdir -p "$PREFIX" && $NPM install --link --prefix "$PREFIX" || exit 3
+mkdir -p "$PREFIX" && $NPM ci --link --prefix "$PREFIX" || exit 3
 
 # create scss test
 mkdir -p tests/css