Skip to content

[Bug]: Message when limiting workers on a project is incorrect #35700

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
Open
alisterscott opened this issue Apr 23, 2025 · 2 comments · May be fixed by #35717
Open

[Bug]: Message when limiting workers on a project is incorrect #35700

alisterscott opened this issue Apr 23, 2025 · 2 comments · May be fixed by #35717
Assignees
@agg23
Labels
v1.53

Comments

@alisterscott
Copy link
Contributor

alisterscott commented Apr 23, 2025 •
edited
Loading

Version

1.52.0

Steps to reproduce

Steps

  1. Clone https://github.com/alisterscott/pw-workers-bug
  2. npm i
  3. npx playwright test --project=firefox shows two browsers which is correct and message Running 2 tests using 2 workers which is correct ✅
  4. npx playwright test --project=chromium shows one browser which is correct but message Running 2 tests using 2 workers ❌ should be Running 2 tests using 1 worker

Expected behavior

I expect to see Running x tests using y worker(s) where y is the project worker limit, not the overall limit

Actual behavior

Running 2 tests using 2 workers when only 1 is used

Additional context

No response

Environment

System:
    OS: macOS 15.3.1
    CPU: (8) arm64 Apple M1
    Memory: 165.31 MB / 16.00 GB
  Binaries:
    Node: 19.0.0 - /usr/local/bin/node
    npm: 8.19.2 - /usr/local/bin/npm
  IDEs:
    VSCode: 1.97.2 - /usr/local/bin/code
  Languages:
    Bash: 3.2.57 - /bin/bash
  npmPackages:
    @playwright/test: ^1.52.0 => 1.52.0
@agg23 agg23 linked a pull request Apr 23, 2025 that will close this issue
Open
@agg23 agg23 self-assigned this Apr 23, 2025
@agg23 agg23 added the v1.53 label Apr 23, 2025
@mxschmitt mxschmitt mentioned this issue Apr 25, 2025
Merged
@maschwenk
Copy link

@alisterscott just to be clear, you are saying the behavior is correct but the message is wrong? That's what I'm seeing

@alisterscott
Copy link
Contributor Author

@maschwenk yes behaviour correct but message is wrong

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agg23 agg23

Labels
v1.53
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(runner): incorporate set project worker count into status messages agg23/playwright
3 participants
@alisterscott @agg23 @maschwenk