Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit2377ab6

Browse files
authored
test(site): improve E2E framework (#9438)
1 parenta910e93 commit2377ab6

12 files changed

+132
-11
lines changed

‎.github/workflows/ci.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -554,7 +554,7 @@ jobs:
554554
-run:pnpm playwright:install
555555
working-directory:site
556556

557-
-run:pnpm playwright:test
557+
-run:pnpm playwright:test --workers 1
558558
env:
559559
DEBUG:pw:api
560560
working-directory:site

‎site/e2e/hooks.ts

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
import{Page}from"@playwright/test"
2+
3+
exportconstbeforeCoderTest=async(page:Page)=>{
4+
// eslint-disable-next-line no-console -- Show everything that was printed with console.log()
5+
page.on("console",(msg)=>console.log("[onConsole] "+msg.text()))
6+
7+
page.on("request",(request)=>{
8+
if(!isApiCall(request.url())){
9+
return
10+
}
11+
12+
// eslint-disable-next-line no-console -- Log HTTP requests for debugging purposes
13+
console.log(
14+
`[onRequest] method=${request.method()} url=${request.url()} postData=${
15+
request.postData() ?request.postData() :""
16+
}`,
17+
)
18+
})
19+
page.on("response",async(response)=>{
20+
if(!isApiCall(response.url())){
21+
return
22+
}
23+
24+
constshouldLogResponse=
25+
!response.url().endsWith("/api/v2/deployment/config")&&
26+
!response.url().endsWith("/api/v2/debug/health")
27+
28+
letresponseText=""
29+
try{
30+
if(shouldLogResponse){
31+
constbuffer=awaitresponse.body()
32+
responseText=buffer.toString("utf-8")
33+
responseText=responseText.replace(/\n$/g,"")
34+
}else{
35+
responseText="skipped..."
36+
}
37+
}catch(error){
38+
responseText="not_available"
39+
}
40+
41+
// eslint-disable-next-line no-console -- Log HTTP requests for debugging purposes
42+
console.log(
43+
`[onResponse] url=${response.url()} status=${response.status()} body=${responseText}`,
44+
)
45+
})
46+
}
47+
48+
constisApiCall=(urlString:string):boolean=>{
49+
consturl=newURL(urlString)
50+
constapiPath="/api/v2"
51+
52+
returnurl.pathname.startsWith(apiPath)
53+
}

‎site/e2e/playwright.config.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ export default defineConfig({
3030
timeout:60000,
3131
},
3232
],
33+
reporter:[["./reporter.ts"]],
3334
use:{
3435
baseURL:`http://localhost:${port}`,
3536
video:"retain-on-failure",

‎site/e2e/reporter.ts

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
importtype{
2+
FullConfig,
3+
Suite,
4+
TestCase,
5+
TestResult,
6+
FullResult,
7+
Reporter,
8+
}from"@playwright/test/reporter"
9+
10+
classCoderReporterimplementsReporter{
11+
onBegin(config:FullConfig,suite:Suite){
12+
// eslint-disable-next-line no-console -- Helpful for debugging
13+
console.log(`Starting the run with${suite.allTests().length} tests`)
14+
}
15+
16+
onTestBegin(test:TestCase){
17+
// eslint-disable-next-line no-console -- Helpful for debugging
18+
console.log(`Starting test${test.title}`)
19+
}
20+
21+
onStdOut(chunk:string,test:TestCase,_:TestResult):void{
22+
// eslint-disable-next-line no-console -- Helpful for debugging
23+
console.log(
24+
`[stdout] [${test ?test.title :"unknown"}]:${chunk.replace(
25+
/\n$/g,
26+
"",
27+
)}`,
28+
)
29+
}
30+
31+
onStdErr(chunk:string,test:TestCase,_:TestResult):void{
32+
// eslint-disable-next-line no-console -- Helpful for debugging
33+
console.log(
34+
`[stderr] [${test ?test.title :"unknown"}]:${chunk.replace(
35+
/\n$/g,
36+
"",
37+
)}`,
38+
)
39+
}
40+
41+
onTestEnd(test:TestCase,result:TestResult){
42+
// eslint-disable-next-line no-console -- Helpful for debugging
43+
console.log(`Finished test${test.title}:${result.status}`)
44+
}
45+
46+
onEnd(result:FullResult){
47+
// eslint-disable-next-line no-console -- Helpful for debugging
48+
console.log(`Finished the run:${result.status}`)
49+
}
50+
}
51+
52+
// eslint-disable-next-line no-unused-vars -- Playwright config uses it
53+
exportdefaultCoderReporter

‎site/e2e/tests/app.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import { test } from "@playwright/test"
22
import{randomUUID}from"crypto"
33
import*ashttpfrom"http"
44
import{createTemplate,createWorkspace,startAgent}from"../helpers"
5+
import{beforeCoderTest}from"../hooks"
6+
7+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
58

69
test("app",async({ context, page})=>{
710
constappContent="Hello World"

‎site/e2e/tests/createWorkspace.spec.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import{test,Page}from"@playwright/test"
1+
import{test}from"@playwright/test"
22
import{
33
createTemplate,
44
createWorkspace,
@@ -16,11 +16,9 @@ import {
1616
sixthParameter,
1717
}from"../parameters"
1818
import{RichParameter}from"../provisionerGenerated"
19+
import{beforeCoderTest}from"../hooks"
1920

20-
test.beforeEach(async({ page}:{page:Page})=>{
21-
// eslint-disable-next-line no-console -- For debugging purposes
22-
page.on("console",(msg)=>console.log("Console: "+msg.text()))
23-
})
21+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
2422

2523
test("create workspace",async({ page})=>{
2624
consttemplate=awaitcreateTemplate(page,{

‎site/e2e/tests/gitAuth.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ import { gitAuth } from "../constants"
33
import{Endpoints}from"@octokit/types"
44
import{GitAuthDevice}from"api/typesGenerated"
55
import{Awaiter,createServer}from"../helpers"
6+
import{beforeCoderTest}from"../hooks"
7+
8+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
69

710
// Ensures that a Git auth provider with the device flow functions and completes!
811
test("git auth device",async({ page})=>{

‎site/e2e/tests/outdatedAgent.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,12 @@ import {
77
sshIntoWorkspace,
88
startAgentWithCommand,
99
}from"../helpers"
10+
import{beforeCoderTest}from"../hooks"
1011

1112
constagentVersion="v0.14.0"
1213

14+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
15+
1316
test("ssh with agent "+agentVersion,async({ page})=>{
1417
consttoken=randomUUID()
1518
consttemplate=awaitcreateTemplate(page,{

‎site/e2e/tests/outdatedCLI.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,12 @@ import {
77
sshIntoWorkspace,
88
startAgent,
99
}from"../helpers"
10+
import{beforeCoderTest}from"../hooks"
1011

1112
constclientVersion="v0.14.0"
1213

14+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
15+
1316
test("ssh with client "+clientVersion,async({ page})=>{
1417
consttoken=randomUUID()
1518
consttemplate=awaitcreateTemplate(page,{

‎site/e2e/tests/restartWorkspace.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@ import {
99

1010
import{firstBuildOption,secondBuildOption}from"../parameters"
1111
import{RichParameter}from"../provisionerGenerated"
12+
import{beforeCoderTest}from"../hooks"
13+
14+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
1215

1316
test("restart workspace with ephemeral parameters",async({ page})=>{
1417
constrichParameters:RichParameter[]=[firstBuildOption,secondBuildOption]

‎site/e2e/tests/updateWorkspace.spec.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import{test,Page}from"@playwright/test"
1+
import{test}from"@playwright/test"
22

33
import{
44
createTemplate,
@@ -18,11 +18,9 @@ import {
1818
secondBuildOption,
1919
}from"../parameters"
2020
import{RichParameter}from"../provisionerGenerated"
21+
import{beforeCoderTest}from"../hooks"
2122

22-
test.beforeEach(async({ page}:{page:Page})=>{
23-
// eslint-disable-next-line no-console -- For debugging purposes
24-
page.on("console",(msg)=>console.log("Console: "+msg.text()))
25-
})
23+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
2624

2725
test("update workspace, new optional, immutable parameter added",async({
2826
page,

‎site/e2e/tests/webTerminal.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
import{test}from"@playwright/test"
22
import{createTemplate,createWorkspace,startAgent}from"../helpers"
33
import{randomUUID}from"crypto"
4+
import{beforeCoderTest}from"../hooks"
5+
6+
test.beforeEach(async({ page})=>awaitbeforeCoderTest(page))
47

58
test("web terminal",async({ context, page})=>{
69
consttoken=randomUUID()

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp