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

test: fix a couple failures on certain systems #16303

Merged
merged 1 commit into from Dec 7, 2023

Conversation

Bo98
Copy link
Member

@Bo98 Bo98 commented Dec 7, 2023

  • test/caveats_spec could fail if neither launchctl or systemd were available (e.g. Ubuntu images without systemd installed).
    • We should probably adjust generic OS tests to assume neither are available.
  • test/unpack_strategy_spec could fail if run as root as writable? is always true as root.

@MikeMcQuaid MikeMcQuaid merged commit cd6f3fe into Homebrew:master Dec 7, 2023
26 checks passed
@MikeMcQuaid
Copy link
Member

Thanks again @Bo98!

@Bo98 Bo98 deleted the test-fixes branch December 7, 2023 20:45
@github-actions github-actions bot added the outdated PR was locked due to age label Jan 7, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants