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

Fix runner for JsTestDriver based tests on OS X #5045

Merged
merged 2 commits into from
Nov 2, 2016
Merged

Fix runner for JsTestDriver based tests on OS X #5045

merged 2 commits into from
Nov 2, 2016

Conversation

Vinai
Copy link
Contributor

@Vinai Vinai commented Jun 15, 2016

This PR introduces two changes:

  1. Do not use Xvfb when running the tests on OS X
  2. Use the correct path to the firefox executable under OS X

Background

Only applications compiled for X11 can use Xvfb to run in headless mode, since actually it's not the browsers running in headless mode but the X11 server.
Firefox (or Chrome or Safari) on OS X do not use X11 but Aqua. This means they can not run in headless mode.
It is possible to install Xvfb on OS X using the XQuartz project, but even then the browsers will not use the headless X11 server.
It is possible to compile Firefox on OS X to use X11, but that is quite involved and should not be required to run the JS tests.
Under OS X the JsTestRunner should simply start an instance of Firefox in the background and run the tests.

@MomotenkoNatalia
Copy link
Contributor

MomotenkoNatalia commented Jul 11, 2016

@Vinai Thanks for contributing! Our internal ticket - MAGETWO-57857

@MomotenkoNatalia MomotenkoNatalia added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Oct 19, 2016
@mmansoor-magento mmansoor-magento merged commit d3fb125 into magento:develop Nov 2, 2016
mmansoor-magento pushed a commit that referenced this pull request Nov 2, 2016
Bugs
- MAGETWO-60004 [GITHUB] Fix error with the WYSIWYG and Greek characters #5548
- MAGETWO-57857 [GITHUB] Fix runner for JsTestDriver based tests on OS X #5045
- MAGETWO-56789 [GITHUB][PR] block newsletter title #4958
magento-engcom-team pushed a commit that referenced this pull request Nov 21, 2019
[arcticfoxes] MC-22813: Changing Quantity and Adding to cart throws exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Frontend bugfix Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants