Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[test] Add e2e testing #125

Draft
wants to merge 60 commits into
base: yoann/inject-browser-sdk
Choose a base branch
from
Draft

[test] Add e2e testing #125

wants to merge 60 commits into from

Conversation

yoannmoinet
Copy link
Member

@yoannmoinet yoannmoinet commented Jan 24, 2025 •
edited
Loading

What and why?

It would feel safer to actually test in browser, IRL situations.

How?

Add e2e testing with Playwright.

  • new yarn cli dev-server command to run a local dev server.
  • add and setup playwright.
  • new yarn workspace @dd/tests e2e command to run playwright.
  • re-organise helpers around internal builds (bundlers and plugins).

@yoannmoinet
Fix and move plugin build script
d9251aa
@yoannmoinet
Add devserver cli
94ce3a8
@yoannmoinet
Add and setup playwright
462ad56
@yoannmoinet
Build plugins script update
7134f30
@yoannmoinet
Move the run of bundlers in its own file
f8a4872
@yoannmoinet
Move devserver into the CLI
98730bb
@yoannmoinet
Small fixes on bundling
5b50fff
@yoannmoinet
Save context params into cookie
5510859
@yoannmoinet
Update fixture project
8b9d7c9
@yoannmoinet
Fix buildPlugins not actually looping
0a127d7
@yoannmoinet
Update the globalSetup
3234608
@yoannmoinet
Add e2e test for Browser SDK
2ce1722
@yoannmoinet
Fix types
89374b0
@yoannmoinet
Use real webpack4 plugin
b9ee6ea
@yoannmoinet
Test on all browsers
bde975e
@yoannmoinet
Fix tests
a21aff2
@yoannmoinet
Fix injection testing and race condition
d35f1b6
@yoannmoinet
Integrity
f1e2e7e
@yoannmoinet
Update e2e config and cache
27c9713
@yoannmoinet
Update action
314ff62
@yoannmoinet
Fix cache action
b5b7117
@yoannmoinet
Add all the browsers
93aaacb
@yoannmoinet
Remove nb of workers in CI
3e14d94
@yoannmoinet
Add traces to e2e runs
eb1c86d
@yoannmoinet
Update reports
459a325
@yoannmoinet
Update CI runs
41fd54f
@yoannmoinet
Add requests failures in the test
7e7b111
@yoannmoinet
Only specific build locally
4b74db5
@yoannmoinet
Force colors in tests output
f38be14
@yoannmoinet
Tweak logs
204c3eb
@yoannmoinet
Update errors
e9e1694
@yoannmoinet
Change log command
e1d2a83
@yoannmoinet
Add more reporter in CI
aeb350e
@yoannmoinet
Add response status to logs
99b1a5d
@yoannmoinet
Do not clean dist folder
9703640
@yoannmoinet
Always save playwright cache
29728c0
@yoannmoinet
Actually call status()
3bc016d
@yoannmoinet
Move helpers
a9e1031
@yoannmoinet
More logs
7d1fbb7
@yoannmoinet
Reduce timeout to 10s
b67163d
@yoannmoinet
Fail on non 200
31e85e0
@yoannmoinet
Debug test esbuild not outputing files
5dc4576
@yoannmoinet
New line
c85d32d
@yoannmoinet
Add result details from builds
704d9e1
@yoannmoinet
Add more logs
7cc1b55
@yoannmoinet
Change bundle name
dd29362
@yoannmoinet
Remove plugin
d580910
@yoannmoinet
Clean debugs and logs
153f449
@yoannmoinet
Only run for esbuild
c7b3959
@yoannmoinet
Prevent vite from emptying the outDir
9697007
@yoannmoinet
Remove guardrails from debug
7ffa271
@yoannmoinet
Add docs
39fbeb9
@yoannmoinet
Extract project build
86e31be
@yoannmoinet
Re-organise the e2e tests to offer better flexibility and isolation
07930d6
@yoannmoinet
wip
878c3f7
@yoannmoinet
Revert diff on rum plugin
857a1d5
@yoannmoinet
Add missing plugin dependency
117746b
@yoannmoinet
Move unit tests in their own folder
da7a652
@yoannmoinet
Rename test commands
66c9326
@yoannmoinet
Remove useless things
60d4262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yoannmoinet