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

[BUG] npm init playwright fails on path spaces #9261

Closed
sahmeepee opened this issue Oct 1, 2021 · 0 comments · Fixed by #9275
Closed

[BUG] npm init playwright fails on path spaces #9261

sahmeepee opened this issue Oct 1, 2021 · 0 comments · Fixed by #9275
Assignees
@mxschmitt

Comments

@sahmeepee
Copy link

Context:

  • Playwright Version: 1.15.1
  • Operating System: Windows 10
  • Node.js version: 12.19.0
  • Browser: n/a
  • Extra: [any specific details about your environment]

Code Snippet

mkdir "c:\temp\project with spaces"
cd "c:\temp\project with spaces"
npm init playwright myprojectname

Describe the bug

This command fails if there is a space anywhere in the path with an error "failed to install browsers" because it looks for a directory (e.g. "c:\temp\project") which doesn't exist.
@mxschmitt mxschmitt self-assigned this Oct 1, 2021
@mxschmitt mxschmitt mentioned this issue Oct 2, 2021
Merged
@mxschmitt mxschmitt added the v1.16 label Oct 2, 2021
bors bot pushed a commit to ttbud/ttbud that referenced this issue Oct 6, 2021
@renovate-bot
Update dependency @playwright/test to v1.15.2 (#820)
64d6b86 
[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@playwright/test](https://playwright.dev) ([source](https://togithub.com/Microsoft/playwright)) | [`1.15.1` -> `1.15.2`](https://renovatebot.com/diffs/npm/@playwright%2ftest/1.15.1/1.15.2) | [![age](https://badges.renovateapi.com/packages/npm/@playwright%2ftest/1.15.2/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@playwright%2ftest/1.15.2/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@playwright%2ftest/1.15.2/compatibility-slim/1.15.1)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@playwright%2ftest/1.15.2/confidence-slim/1.15.1)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>Microsoft/playwright</summary>

### [`v1.15.2`](https://togithub.com/Microsoft/playwright/releases/v1.15.2)

[Compare Source](https://togithub.com/Microsoft/playwright/compare/v1.15.1...v1.15.2)

#### Highlights

This patch includes bug fixes for the following issues:

[microsoft/playwright#9261 - \[BUG] npm init playwright fails on path spaces[microsoft/playwright#9298 - \[Question]: Should new Headers methods work in RouteAsync ?

#### Browser Versions

-   Chromium 96.0.4641.0
-   Mozilla Firefox 92.0
-   WebKit 15.0

This version of Playwright was also tested against the following stable channels:

-   Google Chrome 93
-   Microsoft Edge 93

***

1.15.2-1633455481000

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/ttbud/ttbud).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mxschmitt mxschmitt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(install): installation with folders which contain spaces mxschmitt/playwright
2 participants
@sahmeepee @mxschmitt