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.

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

Represent browsers as Browser objects internally #3218

Closed
flotwig opened this issue Jan 24, 2019 · 1 comment
Closed

Represent browsers as Browser objects internally #3218

flotwig opened this issue Jan 24, 2019 · 1 comment
Assignees
Labels
type: chore Work is required w/ no deliverable to end user
Milestone

Comments

@flotwig
Copy link
Contributor

flotwig commented Jan 24, 2019

Current behavior:

Inside the launcher package, browser is treated as an object with a name, a version, a path to the browser...

packages/launcher/lib/types.ts

Inside the server package, the browser is treated as an object (at first), but is quickly reduced to just the name (chrome, chromium, ...)

This means there's not much flexibility in how we can detect different browsers and then pass that information to the rest of Cypress.

Desired behavior:

Modify the internal representation of browsers from just the name to always be the full Browser object, to enable new functionality like selecting browser by version (#3183), using arbitrary paths to browsers (#1026), and using proper display names in the desktop-gui.

@flotwig flotwig mentioned this issue Jan 24, 2019
14 tasks
@jennifer-shehane jennifer-shehane added stage: ready for work The issue is reproducible and in scope type: chore Work is required w/ no deliverable to end user process: code labels Jan 25, 2019
@flotwig flotwig added stage: in progress and removed stage: ready for work The issue is reproducible and in scope labels Jan 25, 2019
@jennifer-shehane jennifer-shehane added stage: needs review The PR code is done & tested, needs review and removed stage: in progress labels Jan 29, 2019
@brian-mann brian-mann added this to the Sprint 22 milestone Feb 13, 2019
@jennifer-shehane jennifer-shehane added stage: pending release and removed stage: needs review The PR code is done & tested, needs review labels Feb 14, 2019
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Mar 15, 2019

Released in 3.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: chore Work is required w/ no deliverable to end user
Projects
None yet
Development

No branches or pull requests

3 participants