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

Increased browser support for tests #723

Merged
merged 2 commits into from Jun 26, 2019
Merged

Conversation

DylanPiercey
Copy link
Member

@DylanPiercey DylanPiercey commented Jun 26, 2019

Description

This PR updates our tests to run in the following browsers:

  • Chrome 49
  • Firefox latest
  • Safari latest

Other browsers may not be possible with the current setup since some of them do not implement the features needed via the WebDriver spec. Still digging into to these to verify, however thought this was a good base to work off of.

References

Closes #497

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 2434

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.4%) to 90.24%

Totals Coverage Status
Change from base Build 2430: 1.4%
Covered Lines: 1159
Relevant Lines: 1246

💛 - Coveralls

Copy link
Contributor

@ianmcburnie ianmcburnie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before:

Screen Shot 2019-06-26 at 4 32 55 PM

After:

Screen Shot 2019-06-26 at 4 29 11 PM

Excellent stuff!

@ianmcburnie
Copy link
Contributor

p.s. I changed the scope of #497 to just firefox and safari. We can create a new ticket for IE and/or Edge if you think those are close too.

@ianmcburnie ianmcburnie added the squash: NO thanks For pull requests only label Jun 26, 2019
@ianmcburnie ianmcburnie merged commit e1d5c13 into 2.7.0 Jun 26, 2019
@ianmcburnie ianmcburnie deleted the increased-browser-support branch June 27, 2019 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squash: NO thanks For pull requests only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants