Commit673294d

authored

chore: add e2e test for updating theme (#16897)

1 parent564b387 commit673294dCopy full SHA for 673294d

File tree

+31

-7

lines changed

+31

-7

lines changed

Lines changed: 28 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,28 @@`
	`1`	`+import{expect,test}from"@playwright/test";`
	`2`	`+import{users}from"../../constants";`
	`3`	`+import{login}from"../../helpers";`
	`4`	`+import{beforeCoderTest}from"../../hooks";`
	`5`	`+`
	`6`	`+test.beforeEach(({ page})=>{`
	`7`	`+beforeCoderTest(page);`
	`8`	`+});`
	`9`	`+`
	`10`	`+test("adjust user theme preference",async({ page})=>{`
	`11`	`+awaitlogin(page,users.member);`
	`12`	`+`
	`13`	`+awaitpage.goto("/settings/appearance",{waitUntil:"domcontentloaded"});`
	`14`	`+`
	`15`	`+awaitpage.getByText("Light",{exact:true}).click();`
	`16`	`+awaitexpect(page.getByLabel("Light")).toBeChecked();`
	`17`	`+`
	`18`	`+// Make sure the page is actually updated to use the light theme`
	`19`	`+const[root]=awaitpage.$$("html");`
	`20`	`+expect(awaitroot.evaluate((it)=>it.className)).toContain("light");`
	`21`	`+`
	`22`	`+awaitpage.goto("/",{waitUntil:"domcontentloaded"});`
	`23`	`+`
	`24`	`+// Make sure the page is still using the light theme after reloading and`
	`25`	`+// navigating away from the settings page.`
	`26`	`+const[homeRoot]=awaitpage.$$("html");`
	`27`	`+expect(awaithomeRoot.evaluate((it)=>it.className)).toContain("light");`
	`28`	`+});`

Lines changed: 3 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -5,11 +5,11 @@ import {`
`5`	`5`	`createTemplate,`
`6`	`6`	`createWorkspace,`
`7`	`7`	`echoResponsesWithParameters,`
	`8`	`+login,`
`8`	`9`	`openTerminalWindow,`
`9`	`10`	`requireTerraformProvisioner,`
`10`	`11`	`verifyParameters,`
`11`	`12`	`}from"../../helpers";`
`12`		`-import{login}from"../../helpers";`
`13`	`13`	`import{beforeCoderTest}from"../../hooks";`
`14`	`14`	`import{`
`15`	`15`	`fifthParameter,`
`@@ -150,9 +150,7 @@ test("create workspace with disable_param search params", async ({ page }) => {`
`150`	`150`	`awaitlogin(page,users.member);`
`151`	`151`	`awaitpage.goto(`
`152`	`152`	`/templates/${templateName}/workspace?disable_params=first_parameter,second_parameter`,
`153`		`-{`
`154`		`-waitUntil:"domcontentloaded",`
`155`		`-},`
	`153`	`+{waitUntil:"domcontentloaded"},`
`156`	`154`	`);`
`157`	`155`
`158`	`156`	`awaitexpect(page.getByLabel(/Firstparameter/i)).toBeDisabled();`
`@@ -173,9 +171,7 @@ test.skip("create docker workspace", async ({ context, page }) => {`
`173`	`171`	`// The workspace agents must be ready before we try to interact with the workspace.`
`174`	`172`	`awaitpage.waitForSelector(`
`175`	`173`	`//div[@role="status"][@data-testid="agent-status-ready"]`,
`176`		`-{`
`177`		`-state:"visible",`
`178`		`-},`
	`174`	`+{state:"visible"},`
`179`	`175`	`);`
`180`	`176`
`181`	`177`	`// Wait for the terminal button to be visible, and click it.`

Comments

(0)