test: do not use admin user as test user

Author: Veikkosuhonen

e2e/README.md

@@ -1,9 +1,21 @@
 # Notes about e2e tests
 
 When writing new spec file, make sure to import the test function from the fixtures file, like this:
+```ts
+import { teacherTest as test } from './fixtures'
 ```
-import { test } from './fixtures'
+
+So that the global foreach function runs for your tests. For different user roles (`studentTest`, `teacherTest`, `adminTest`), import the corresponding test function:
+```ts
+import { studentTest as test } from './fixtures'
 ```
-.
 
-So that the global foreach function runs for your tests.
+## Test setup and data
+
+Test user headers are defined in `src/shared/testData.ts`. TEST_COURSES are also defined there.
+
+Before each test, the test data of a test user is reset by calling the `/test/reset-test-data` endpoint. It is found at `src/server/routes/testUtils.ts`.
+
+When running the tests, the headers `x-test-user-idx` and `x-test-user-role` are set, defining the test user's id and iam-based role. They are read at `src/server/middleware/user.ts`.
+
+The tests are isolated so that each (should at least) modifies their own data discriminated by the user's id. This allows parallel execution.

e2e/chat.spec.ts

@@ -1,6 +1,6 @@
 import { expect } from '@playwright/test'
 import { acceptDisclaimer } from './utils/test-helpers'
-import { test } from './fixtures'
+import { teacherTest as test } from './fixtures'
 
 test.describe('Chat v2 Conversation tests', () => {
   test.beforeEach(async ({ page }) => {

e2e/chatFeatures.spec.ts

@@ -1,6 +1,6 @@
 import { expect } from '@playwright/test'
 import { acceptDisclaimer } from './utils/test-helpers'
-import { test } from './fixtures'
+import { teacherTest as test } from './fixtures'
 
 test.describe('Chat v2 UI Features Tests', () => {
   test.beforeEach(async ({ page }) => {

e2e/courseChatRag.spec.ts

@@ -1,6 +1,6 @@
 import { expect } from '@playwright/test'
 import { acceptDisclaimer } from './utils/test-helpers'
-import { test } from './fixtures'
+import { studentTest as test } from './fixtures'
 
 test.describe('Course Chat v2', () => {
   test.beforeEach(async ({ page }) => {

e2e/fixtures.ts

@@ -1,7 +1,7 @@
 import { test as base } from '@playwright/test'
 import { setupLogging } from './utils/logging'
 
-export const test = base.extend<{ forEachTest: void }>({
+export const adminTest = base.extend<{ forEachTest: void }>({
   forEachTest: [
     async ({ page, request }, use) => {
       // This code runs before every test.
@@ -32,6 +32,37 @@ export const test = base.extend<{ forEachTest: void }>({
   ], // automatically starts for every test.
 })
 
+export const teacherTest = base.extend<{ forEachTest: void }>({
+  forEachTest: [
+    async ({ page, request }, use) => {
+      // This code runs before every test.
+
+      // setupLogging(page)
+
+      /*
+       * Parallel worker isolation: each worker has its own test user identified by the worker index.
+       */
+
+      const testUserIdx = studentTest.info().workerIndex
+      const testUserRole = 'teacher'
+
+      page.context().setExtraHTTPHeaders({
+        'x-test-user-index': String(testUserIdx),
+        'x-test-user-role': testUserRole,
+      })
+
+      await request.post('/api/test/reset-test-data', { data: { testUserIdx, testUserRole } })
+
+      // Run the test
+      await use()
+
+      // This code runs after every test.
+      // console.log('Last URL:', page.url());
+    },
+    { auto: true },
+  ], // automatically starts for every test.
+})
+
 export const studentTest = base.extend<{ forEachTest: void }>({
   forEachTest: [
     async ({ page, request }, use) => {

e2e/healthcheck.spec.ts

@@ -1,4 +1,5 @@
-import { test, expect } from '@playwright/test'
+import { expect } from '@playwright/test'
+import { studentTest as test } from './fixtures'
 
 test.describe('Health Check Tests', () => {
   test.beforeEach(async ({ page }) => {})

e2e/prompts.spec.ts

@@ -1,5 +1,5 @@
-import { test } from './fixtures'
 import { expect } from '@playwright/test'
+import { teacherTest as test } from './fixtures'
 import { acceptDisclaimer } from './utils/test-helpers'
 
 test.describe('Prompts', () => {