fix: use testid instead of placeholder for e2e test element lookup

Author: ErikBjare

e2e/conversation.spec.ts

@@ -122,7 +122,7 @@ test.describe('Conversation Flow', () => {
     const message = 'Hello. We are testing, just say exactly "Hello world" without anything else.';
 
     // Type a message
-    await page.getByPlaceholder('Send a message...').fill(message);
+    await page.getByTestId('chat-input').fill(message);
     await page.keyboard.press('Enter');
 
     // Should show the message in the conversation

src/components/BrowserPreview.tsx

@@ -31,7 +31,11 @@ export const BrowserPreview: FC<Props> = ({ defaultUrl = 'http://localhost:8080'
 
   const handleRefresh = () => {
     // Force refresh by appending a dummy parameter if URL is unchanged
-    setCurrentUrl(inputValue === currentUrl ? `${inputValue}${inputValue.includes('?') ? '&' : '?'}_refresh=${Date.now()}` : inputValue);
+    setCurrentUrl(
+      inputValue === currentUrl
+        ? `${inputValue}${inputValue.includes('?') ? '&' : '?'}_refresh=${Date.now()}`
+        : inputValue
+    );
     setLogs([]); // Clear logs on refresh
   };
 

src/components/ChatInput.tsx

@@ -129,6 +129,7 @@ export const ChatInput: FC<Props> = ({
                     <Textarea
                       ref={textareaRef}
                       value={message}
+                      data-testid="chat-input"
                       onChange={(e) => {
                         setMessage(e.target.value);
                         // Auto-adjust height