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

Write tests for 'brew list --version [--cask]'. #18999

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ascherer
Copy link
Contributor

@ascherer ascherer commented Dec 24, 2024

Following my recent PR (#18996), I would like to amend test/cmd/list_spec.rb with some test cases to check that the output is alphabetically sorted.

HELP WANTED! How can I create temporary test packages in the Caskroom/ instead of in the Cellar/?

I can only run 'brew tests' on Linux at this time, because my Mac Mini (macOS 14.7.2) fails due to missing file <mutex> (in mutex.cc).

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew typecheck with your changes locally?
  • Have you successfully run brew tests with your changes locally?

@ascherer ascherer force-pushed the test-list--cask--version branch 2 times, most recently from e9b772c to c1546e0 Compare December 24, 2024 20:47
HELP WANTED! How can I create test packages in the 'Caskroom'?

I can only run 'brew tests' on Linux, because my Mac Mini (macOS 14.7.2)
fails due to missing "mutex.h" (or some such).

At least the new test for 'brew list --version' succeeds (on Linux).

Squashed commits to (partial) success:
Try 'brew style list_spec.rb'.
Try 'rubocop -a'.
Generic OS fails.
Generic OS fails again.
The 'Caskroom' is not yet populated.
@ascherer ascherer force-pushed the test-list--cask--version branch from c1546e0 to 9de6a1c Compare December 25, 2024 09:37
'brew list --cask' seems to work, but both 'brew list --cask --version'
and 'brew list --version' fail because of empty cask list.
@ascherer ascherer force-pushed the test-list--cask--version branch from 9de6a1c to 9f09d39 Compare December 25, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant