-
-
Notifications
You must be signed in to change notification settings - Fork 173
Test for location input on /settings page. #1204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Changes from 5 commits
67c5c4f
e6883a5
8724657
ed62c79
4699c2b
1fad2e0
2df741c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,4 +1,4 @@ | ||||||||||||||||||||||||||||||||||||||||
| import test from "@playwright/test"; | ||||||||||||||||||||||||||||||||||||||||
| import { test, expect } from "@playwright/test"; | ||||||||||||||||||||||||||||||||||||||||
| import { loggedInAsUserOne } from "./utils"; | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| test.describe("Unauthenticated setttings Page", () => { | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -10,6 +10,68 @@ test.describe("Authenticated settings Page", () => { | |||||||||||||||||||||||||||||||||||||||
| test.beforeEach(async ({ page }) => { | ||||||||||||||||||||||||||||||||||||||||
| await loggedInAsUserOne(page); | ||||||||||||||||||||||||||||||||||||||||
| }); | ||||||||||||||||||||||||||||||||||||||||
| // | ||||||||||||||||||||||||||||||||||||||||
| // Replace with tests for authenticated users | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Test for changing username | ||||||||||||||||||||||||||||||||||||||||
| test("Username input field", async ({ page }) => { | ||||||||||||||||||||||||||||||||||||||||
| await page.goto("http://localhost:3000/settings"); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Wait for the username input field to be visible | ||||||||||||||||||||||||||||||||||||||||
| await page.locator('input[id="username"]').waitFor(); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Test that the input field is visible and has the correct attributes | ||||||||||||||||||||||||||||||||||||||||
| const inputField = page.locator('input[id="username"]'); | ||||||||||||||||||||||||||||||||||||||||
| await expect(inputField).toBeVisible(); | ||||||||||||||||||||||||||||||||||||||||
| await expect(inputField).toHaveAttribute("type", "text"); | ||||||||||||||||||||||||||||||||||||||||
| await expect(inputField).toHaveAttribute("autocomplete", "username"); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Test that the error message appears when the input field is invalid | ||||||||||||||||||||||||||||||||||||||||
| await inputField.fill("45&p^x#@!96%*()"); | ||||||||||||||||||||||||||||||||||||||||
| await page.locator('button[type="submit"]').click(); | ||||||||||||||||||||||||||||||||||||||||
| const errorMessage = page.locator( | ||||||||||||||||||||||||||||||||||||||||
| 'p:text-is("Username can only contain alphanumerics and dashes.")', | ||||||||||||||||||||||||||||||||||||||||
| ); | ||||||||||||||||||||||||||||||||||||||||
| await expect(errorMessage).toBeVisible(); | ||||||||||||||||||||||||||||||||||||||||
| await expect(errorMessage).toHaveText( | ||||||||||||||||||||||||||||||||||||||||
| "Username can only contain alphanumerics and dashes.", | ||||||||||||||||||||||||||||||||||||||||
| ); | ||||||||||||||||||||||||||||||||||||||||
| // Reset the form | ||||||||||||||||||||||||||||||||||||||||
| await page.locator('button:has-text("Reset")').click(); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Test that the input field can be filled with a valid value and saves it | ||||||||||||||||||||||||||||||||||||||||
| await inputField.fill("codu-rules"); | ||||||||||||||||||||||||||||||||||||||||
| await page.locator('button[type="submit"]').click(); | ||||||||||||||||||||||||||||||||||||||||
| await expect(inputField).toHaveValue("codu-rules"); | ||||||||||||||||||||||||||||||||||||||||
| }); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Tests location input, autocomplete, and saved values | ||||||||||||||||||||||||||||||||||||||||
| test("location input is visible", async ({ page }) => { | ||||||||||||||||||||||||||||||||||||||||
| await page.goto("http://localhost:3000/settings"); | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| // Test to see if input is visible | ||||||||||||||||||||||||||||||||||||||||
| await page.locator('input[id="location"]').isVisible(); | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
| await page.locator('input[id="location"]').isVisible(); | |
| await expect(page.locator('input[id="location"]')).toBeVisible(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add missing page reload in persistence test.
The test comment mentions verifying persistence after page reload, but the actual reload step is missing.
Add the reload step:
await page.fill('input[id="location"]', "A fun place to visit.");
await page.locator('button[type="submit"]').click();
await expect(page.locator('input[id="location"]')).toHaveValue(
"A fun place to visit.",
);
+ // Verify persistence after reload
+ await page.reload();
+ await expect(page.locator('input[id="location"]')).toHaveValue(
+ "A fun place to visit.",
+ );📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| // Test to see if change in location persists after submit and page reload | |
| await page.fill('input[id="location"]', "A fun place to visit."); | |
| await page.locator('button[type="submit"]').click(); | |
| await expect(page.locator('input[id="location"]')).toHaveValue( | |
| "A fun place to visit.", | |
| ); | |
| }); | |
| // Test to see if change in location persists after submit and page reload | |
| await page.fill('input[id="location"]', "A fun place to visit."); | |
| await page.locator('button[type="submit"]').click(); | |
| await expect(page.locator('input[id="location"]')).toHaveValue( | |
| "A fun place to visit.", | |
| ); | |
| // Verify persistence after reload | |
| await page.reload(); | |
| await expect(page.locator('input[id="location"]')).toHaveValue( | |
| "A fun place to visit.", | |
| ); | |
| }); |
Uh oh!
There was an error while loading. Please reload this page.