test: fix vitests by running them in real browser to workaround Node.JS limitations

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Author: susnux

.github/workflows/node-test.yml

@@ -51,6 +51,8 @@ jobs:
           CYPRESS_INSTALL_BINARY: 0
         run: |
           npm ci
+          # Install browsers for tests
+          npm npx playwright install chromium
           npm run build --if-present
 
       - name: Test

REUSE.toml

@@ -6,7 +6,7 @@ SPDX-PackageSupplier = "Nextcloud GmbH <https://nextcloud.com/impressum/>"
 SPDX-PackageDownloadLocation = "https://github.com/nextcloud-libraries/nextcloud-auth"
 
 [[annotations]]
-path = ["package.json", "package-lock.json", "tsconfig.json"]
+path = ["package.json", "package-lock.json", "tsconfig.json", "test/tsconfig.json"]
 precedence = "aggregate"
 SPDX-FileCopyrightText = "2019 Nextcloud GmbH and Nextcloud contributors"
 SPDX-License-Identifier = "GPL-3.0-or-later"

lib/guest.ts

@@ -74,6 +74,17 @@ export function setGuestNickname(nickname: string): void {
 	getGuestUser().displayName = nickname
 }
 
+/**
+ * Reset the guest user state.
+ *
+ * @internal
+ */
+export function resetGuestUser(): void {
+	currentUser = undefined
+	browserStorage.removeItem('guestUid')
+	browserStorage.removeItem('guestNickname')
+}
+
 /**
  * Generate a random UUID (version 4) if the crypto API is not available.
  * If the crypto API is available, it uses the less secure `randomUUID` method.

package-lock.json

@@ -43,9 +43,9 @@
     "@nextcloud/eslint-config": "^9.0.0-rc.9",
     "@nextcloud/typings": "^1.10.0",
     "@nextcloud/vite-config": "^2.5.2",
+    "@vitest/browser-playwright": "^4.1.4",
     "@vitest/coverage-v8": "^4.1.4",
     "eslint": "^10.2.0",
-    "happy-dom": "^20.8.9",
     "msw": "^2.13.2",
     "typedoc": "^0.28.18",
     "typescript": "^6.0.2",

package.json

@@ -2,14 +2,15 @@
  * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: GPL-3.0-or-later
  */
-import { randomBytes } from 'crypto'
+
 import { beforeEach, describe, expect, test, vi } from 'vitest'
 
 /**
  * Mock `<meta>` element with nonce
  */
 function mockNonce() {
-	const nonce = randomBytes(16).toString('base64')
+	const bytes = globalThis.crypto.getRandomValues(new Uint8Array(16))
+	const nonce = bytes.toBase64()
 	const el = document.createElement('meta')
 	el.name = 'csp-nonce'
 	el.nonce = nonce