add some more tests

Author: Veikkosuhonen

e2e/chat.spec.ts

@@ -3,8 +3,7 @@ import { expect, test } from '@playwright/test'
 test.describe('Chat v2 Conversation tests', () => {
   test.beforeEach(async ({ page }) => {
     await page.goto('/v2')
-    // Press esc to close disclaimer modal if present
-    await page.keyboard.press('Escape')
+    await page.locator('#close-disclaimer').click()
   })
 
   test('Chat v2 mock response works', async ({ page }) => {
@@ -18,4 +17,22 @@ test.describe('Chat v2 Conversation tests', () => {
     await expect(page.getByTestId('user-message')).toContainText('testinen morjens')
     await expect(page.getByTestId('assistant-message')).toContainText('You are calling mock endpoint for streaming mock data')
   })
+
+  test('Can empty conversation', async ({ page }) => {
+    await page.locator('#model-selector').first().click()
+    await page.getByRole('option', { name: 'mock' }).click()
+
+    const chatInput = page.locator('#chat-input').first()
+    await chatInput.fill('tää tyhjennetään')
+    await chatInput.press('Shift+Enter')
+
+    await expect(page.getByTestId('user-message')).toContainText('tää tyhjennetään')
+    await expect(page.getByTestId('assistant-message')).toContainText('OVER', { timeout: 5000 })
+
+    page.on('dialog', (dialog) => dialog.accept())
+    await page.locator('#empty-conversation-button').click()
+
+    await expect(page.getByTestId('user-message')).not.toBeVisible()
+    await expect(page.getByTestId('assistant-message')).not.toBeVisible()
+  })
 })

e2e/chatUiFeatures.spec.ts

@@ -0,0 +1,26 @@
+import { expect, test } from '@playwright/test'
+
+test.describe('Chat v2 UI Features Tests', () => {
+  test.beforeEach(async ({ page }) => {
+    await page.goto('/v2')
+  })
+
+  test('Disclaimer is visible', async ({ page }) => {
+    await expect(page.locator('#close-disclaimer')).toBeVisible()
+  })
+
+  test('Disclaimer can be opened manually', async ({ page }) => {
+    await page.locator('#close-disclaimer').click()
+    await page.locator('#help-button').click()
+    await expect(page.locator('#close-disclaimer')).toBeVisible()
+  })
+
+  test('Settings can be opened and closed', async ({ page }) => {
+    await page.locator('#close-disclaimer').click()
+    await page.locator('#settings-button').click()
+    await expect(page.locator('#close-settings')).toBeVisible()
+
+    await page.keyboard.press('Escape')
+    await expect(page.locator('#close-settings')).not.toBeVisible()
+  })
+})

e2e/utils/logging.ts

@@ -0,0 +1,27 @@
+import type { Page } from '@playwright/test'
+
+export async function setupLogging(page: Page) {
+  page.on('requestfailed', (request) => {
+    console.log('Request failed: ' + request.url() + ' ' + request.failure()?.errorText)
+  })
+
+  page.on('request', (request) => {
+    console.log('Request: ' + request.url())
+  })
+
+  page.on('response', (response) => {
+    console.log('Request finished: ' + response.url() + ' ' + response.status())
+  })
+
+  page.on('console', (msg) => {
+    console.log(`Console message: ${msg.text()}`)
+  })
+
+  page.on('crash', () => {
+    console.log('Page crashed')
+  })
+
+  page.on('pageerror', (error) => {
+    console.log('Page error: ' + error.message)
+  })
+}

src/client/components/ChatV2/ChatV2.tsx

@@ -431,7 +431,7 @@ export const ChatV2 = () => {
           {course && <ChatInfo course={course} />}
 
           <Box sx={{ display: 'flex', flexDirection: 'column', gap: '0.6rem', mb: '2rem' }}>
-            <OutlineButtonBlack startIcon={<DeleteIcon />} onClick={handleReset}>
+            <OutlineButtonBlack startIcon={<DeleteIcon />} onClick={handleReset} id="empty-conversation-button">
               {t('chat:emptyConversation')}
             </OutlineButtonBlack>
             <Tooltip
@@ -447,10 +447,10 @@ export const ChatV2 = () => {
                 {t('email:save')}
               </OutlineButtonBlack>
             </Tooltip>
-            <OutlineButtonBlack startIcon={<SettingsIcon />} onClick={() => setSettingsModalOpen(true)}>
+            <OutlineButtonBlack startIcon={<SettingsIcon />} onClick={() => setSettingsModalOpen(true)} id="settings-button">
               {t('chat:settings')}
             </OutlineButtonBlack>
-            <OutlineButtonBlack startIcon={<HelpIcon />} onClick={() => setDisclaimerStatus({ open: true })}>
+            <OutlineButtonBlack startIcon={<HelpIcon />} onClick={() => setDisclaimerStatus({ open: true })} id="help-button">
               {t('info:title')}
             </OutlineButtonBlack>
           </Box>

src/client/components/ChatV2/SettingsModal.tsx

@@ -139,7 +139,7 @@ export const SettingsModal: React.FC<SettingsModalProps> = ({
           overflow: 'hidden',
         }}
       >
-        <IconButton onClick={() => setOpen(false)} sx={{ position: 'absolute', top: 10, right: 20, color: 'grey.500' }}>
+        <IconButton onClick={() => setOpen(false)} sx={{ position: 'absolute', top: 10, right: 20, color: 'grey.500' }} id="close-settings">
           <Close />
         </IconButton>
 

src/server/util/azure/mocks/mockFunctions.ts

@@ -46,6 +46,7 @@ export const getBasicStreamMock = (): MockResponseStreamEvent[] => {
 - To mock a file search fail response, write: **rag fail**
 - To mock a code block, write: **code block**
 - To mock a math block, write: **math block**
+OVER
 `
 
   const chunkedResponseText = chunkText(responseText)