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

Add browserstack support #30

Merged
merged 2 commits into from
Aug 1, 2016
Merged

Conversation

Cruikshanks
Copy link
Member

This PR will add support for running the tests using Browserstack to Quke.

Found when a test failed the output actually showed this exception was being thrown. The root was the way the after hook was using the `return` statement.

This change modifies the structure of the hook to fix that error.
@Cruikshanks Cruikshanks added the enhancement New feature or request label Jul 27, 2016
@Cruikshanks Cruikshanks self-assigned this Jul 27, 2016
@Cruikshanks Cruikshanks force-pushed the feature/add-browserstack-support branch 8 times, most recently from 7b51183 to b0bb2fd Compare August 1, 2016 11:26
[Browserstack](https://www.browserstack.com/) is an online testing solution which allows you to run your web app in a multitude of platform, OS and browser combinations.

Our QA representatives have expressed an interest in being able to run tests written in Quke through Browserstack hence this change adds that ability.

The changes cover

- Updating `lib/quke/config.rb` to handle the new configuration details needed for browserstack
- Updating `.config.example.yml` with details of the new options
- Adding support for setting the browserstack credentials in environment variables as well as in a `.config.yml` file
- Adding a code to cover registration of the browserstack driver
- Moving all driver registration to its own class
- New rake task to quickly run Quke using the browserstack driver
- Update to the default profile to cover allow separate running of @Quke tests and @browserstack tests
- New feature and steps to confirm browserstack is running (cannot test against the local server at this time using browserstack.local)
- Updated documentation on how to integrate with Browserstack
- Updated travis.yml build config to include browserstack integration test
@Cruikshanks Cruikshanks force-pushed the feature/add-browserstack-support branch from b0bb2fd to 55cb453 Compare August 1, 2016 11:30
@Cruikshanks Cruikshanks merged commit c843329 into master Aug 1, 2016
@Cruikshanks Cruikshanks deleted the feature/add-browserstack-support branch August 1, 2016 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant