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
test/cask: Fix "System Settings" caveats for (non-)Ventura OSes #15001
Conversation
- "System Preferences" was updated to be "System Settings" in macOS Ventura. This naming was updated in our caveats generator in PR 14997. - This test was failing for me locally (on Ventura) because it was hardcoded to expect `System Preferences => Security & Privacy`, but getting `System Settings => Privacy & Security`. - This is not the most important thing in the world, we can probably live with the failing test, but I thought I'd try to fix it anyway.
Review period will end on 2023-03-21 at 00:00:00 UTC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to rename this since it will apply to all future macOS versions as well (I assume) and not just ventura
.
But otherwise 👍
Review period skipped due to |
- Let's actually show in the fixtures JSON filename what's changed between the two.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Good catch! I wrote this test on a Mac that doesn't run on Ventura so that's why I didn't catch it originally. |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?System Preferences => Security & Privacy
, but gettingSystem Settings => Privacy & Security
.