Andras/BoundlessFlowCampus2K

Fork 0

Added Playwright info to README #1049

Open

e24aleli wants to merge 3 commits from write-in-wiki-about-the-playwrite-#1043 into team_1_week_7

e24aleli commented

2026-05-20 12:56:27 +00:00

Collaborator

No description provided.

e24aleli added 1 commit

2026-05-20 12:56:28 +00:00

Added Playwright help to README

Playwright Tests / test (pull_request) Waiting to run

Details

5b60d33033

a22erigr changed target branch from main to team_1_week_7

2026-05-20 12:57:26 +00:00

a22erigr added the

needs review

label

2026-05-20 12:58:09 +00:00

e24aleli referenced this pull request

2026-05-20 12:58:59 +00:00

write in wiki about the playwrite #1043

a22erigr

2026-05-20 13:00:02 +00:00

added the
Under-review
label
requested review from f21marfo

f21marfo reviewed

2026-05-20 13:26:49 +00:00

f21marfo left a comment

Collaborator

Either something is wrong on my end, or there is required information missing in the readme. Tried running the commands and had the following results.

Running npm run test:responsive did not work and generated the following terminal output.

npx playwright test --ui
Terminates with the following error.

npx playwright show-report
Seems to work, but just brings up old test results in the repository.

Either something is wrong on my end, or there is required information missing in the readme. Tried running the commands and had the following results. Running `npm run test:responsive` did not work and generated the following terminal output. ![image](/attachments/7a78f80b-d1d9-4cb1-a24e-6c4c07252034) npx playwright test --ui Terminates with the following error. ![image](/attachments/111ba16a-88e7-4fb0-bc90-da28c6fbb566) npx playwright show-report Seems to work, but just brings up old test results in the repository. ![image](/attachments/193002a4-0a76-459e-a5dd-522394c0bbfc)

image.png

14 KiB

image.png

57 KiB

image.png

471 KiB

e24aleli commented

2026-05-20 13:34:05 +00:00

Author

Collaborator

# 1. Clear out your existing Playwright cache
npx playwright uninstall --all

# 2. Re-install fresh, validated browser binaries
npx playwright install firefox chromium webkit

# 3. Force dependency validation
npx playwright install-deps

Right now the directoríes that playwright creates after install is in the main branch i think we need to remove all regarding playwright except for the test dir and include it in the install script. This way the directories is correctly installed instead of being "pre-installed" when fetching a branch currently.

I believe the problem is currently that we need playwright to be in the installation script so that it is installed on the computer because i had to run the following to make it work when i reinstalled after merge. This is sort of speculation as of now the test itself might need to be looked in to aswell its very simple.. ``` # 1. Clear out your existing Playwright cache npx playwright uninstall --all # 2. Re-install fresh, validated browser binaries npx playwright install firefox chromium webkit # 3. Force dependency validation npx playwright install-deps ``` Right now the directoríes that playwright creates after install is in the main branch i think we need to remove all regarding playwright except for the test dir and include it in the install script. This way the directories is correctly installed instead of being "pre-installed" when fetching a branch currently.

f21marfo commented

2026-05-20 13:50:27 +00:00

Collaborator

I just tried clearing out and reinstalling playwright, still can't seem to get the commands to work.

e24aleli commented

2026-05-20 14:17:55 +00:00

Author

Collaborator

@f21marfo found the problem!!

Inside of package.json
it should say:

for some reason on this branch and probably other ones its not there

@f21marfo found the problem!! Inside of `package.json` it should say:![image](/attachments/6f7fe93a-8e36-4a02-900d-e5ec89c7fbf0) for some reason on this branch and probably other ones its not there

image.png

9.5 KiB

a24julot added the

Team 1

label

2026-05-20 14:30:07 +00:00

f21marfo commented

2026-05-21 07:26:08 +00:00

Collaborator

Still can't make it work, but now the first command gives a different error message.

Which is honestly puzzling, since "npx playwright show-report" does work, so obviously "playwright" isn't truly an unknown command.

npx playwright test --ui remains unchanged.

Still can't make it work, but now the first command gives a different error message. ![image](/attachments/503640f7-0843-4fc8-a0bf-d0610087ebb2) Which is honestly puzzling, since "npx playwright show-report" does work, so obviously "playwright" isn't truly an unknown command. npx playwright test --ui remains unchanged.

image.png

18 KiB

e24aleli commented

2026-05-21 08:55:03 +00:00

Author

Collaborator

This is truly weird and you have tried to refetch the branch?

f21marfo commented

2026-05-21 09:02:26 +00:00

Collaborator

Yes, even wiped and refetched everything and run the scripts, clean.

a24sakma commented

2026-05-21 09:25:41 +00:00

Collaborator

@e24aleli && @f21marfo
I think we have some issues that needs to be addressed.
so the libraries that gets installed. shall be in the package.json
the command that runs the script shall also be in the package
there shall be an example on the test that can be runned.

Right now those packages gets installed globally on your laptop and it is outside the project

@e24aleli && @f21marfo I think we have some issues that needs to be addressed. so the libraries that gets installed. shall be in the `package.json` the command that runs the script shall also be in the `package` there shall be an example on the test that can be runned. Right now those packages gets installed globally on your laptop and it is outside the project

a22erigr requested reviews from a24sakma, f21marfo

2026-05-21 09:29:46 +00:00

a24sakma added the

further-changes-needed

label

2026-05-21 09:35:45 +00:00

e24aleli added 1 commit

2026-05-21 09:55:14 +00:00

Changed package.json to make playwwright work 13b44ed660

e24aleli added 1 commit

2026-05-21 10:01:33 +00:00

Added playwright to deps 137d3b5616

Otherwise manual install is necessary

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin write-in-wiki-about-the-playwrite-#1043:write-in-wiki-about-the-playwrite-#1043

git switch write-in-wiki-about-the-playwrite-#1043

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch team_1_week_7

git merge --no-ff write-in-wiki-about-the-playwrite-#1043

git switch write-in-wiki-about-the-playwrite-#1043

git rebase team_1_week_7

git switch team_1_week_7

git merge --ff-only write-in-wiki-about-the-playwrite-#1043

git switch write-in-wiki-about-the-playwrite-#1043

git rebase team_1_week_7

git switch team_1_week_7

git merge --no-ff write-in-wiki-about-the-playwrite-#1043

git switch team_1_week_7

git merge --squash write-in-wiki-about-the-playwrite-#1043