Skip to content

[🚀 Feature]: Run tests on Mac in CI #12397

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

Closed
titusfortner opened this issue Jul 21, 2023 · 3 comments · Fixed by #12556
Closed

[🚀 Feature]: Run tests on Mac in CI #12397

titusfortner opened this issue Jul 21, 2023 · 3 comments · Fixed by #12556
Labels
B-build Includes scripting, bazel and CI integrations I-enhancement Something could be better

Comments

@titusfortner
Copy link
Member

Feature and motivation

Something (maybe Ruby) needs to run on Mac. We don't have any coverage for Safari or for Selenium Manager actually working properly on Mac.

I happened to hit a selenium manager bug trying to do something before the upcoming release; we need CI to catch this.

Usage example

n/a

@titusfortner titusfortner added I-enhancement Something could be better B-build Includes scripting, bazel and CI integrations labels Jul 21, 2023
@karenhernandeze
Copy link

Can i work in this?

@titusfortner
Copy link
Member Author

Sure.

titusfortner added a commit that referenced this issue Sep 13, 2023
* Add macos coverage to Ruby CI (#12397)

* guard is not present in before all hook so do not set driver

* after hook still executes even when guard disposition of the test is skip so explicitly skip it too

---------

Co-authored-by: Titus Fortner <[email protected]>
Co-authored-by: titusfortner <[email protected]>
Copy link

github-actions bot commented Dec 6, 2023

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B-build Includes scripting, bazel and CI integrations I-enhancement Something could be better
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants