dev-cmd/livecheck: avoid watchlist in test

The existing watchlist test in `dev-cmd/livecheck_spec.rb` will only
pass if the testing environment doesn't contain a livecheck watchlist
file. When a watchlist file is present, it ends up being treated as
empty (formulae and casks aren't available in tests) and produces an
`Invalid usage: No formulae or casks to check` error instead. We don't
have to worry about a watchlist file on CI but it's a potential issue
when running `brew test` locally.

This provides a bogus `HOMEBREW_LIVECHECK_WATCHLIST` value to the
`#brew` call, to ensure that any watchlist file in the testing
environment is not used for this test.

Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
This commit is contained in:
Sam Ford 2023-08-16 12:53:50 -04:00
parent e475de2a4a
commit 1d404e437b
No known key found for this signature in database
GPG Key ID: 7AF5CBEE1DD6F76D

View File

@ -20,7 +20,7 @@ describe "brew livecheck" do
end
it "gives an error when no arguments are given and there's no watchlist", :integration_test do
expect { brew "livecheck" }
expect { brew "livecheck", "HOMEBREW_LIVECHECK_WATCHLIST" => ".this_should_not_exist" }
.to output(/Invalid usage: A watchlist file is required when no arguments are given\./).to_stderr
.and not_to_output.to_stdout
.and be_a_failure