Skip to content

Commit 48fe8f5

Browse files
committed
Fix Homebrew style: Use assert_path_exists instead of assert_predicate :exist?
Homebrew's style checker requires using assert_path_exists for file existence checks instead of assert_predicate with :exist?. Keep assert_predicate only for the :executable? checks.
1 parent e335fcb commit 48fe8f5

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

packaging/homebrew/mfc.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,15 +59,15 @@ def caveats
5959

6060
test do
6161
# Test that the binaries exist and are executable
62-
assert_predicate bin/"pre_process", :exist?
62+
assert_path_exists bin/"pre_process"
6363
assert_predicate bin/"pre_process", :executable?
64-
assert_predicate bin/"simulation", :exist?
64+
assert_path_exists bin/"simulation"
6565
assert_predicate bin/"simulation", :executable?
66-
assert_predicate bin/"post_process", :exist?
66+
assert_path_exists bin/"post_process"
6767
assert_predicate bin/"post_process", :executable?
6868

6969
# Verify examples were installed
70-
assert_predicate pkgshare/"examples", :exist?
71-
assert_predicate pkgshare/"examples/1D_sodshocktube/case.py", :exist?
70+
assert_path_exists pkgshare/"examples"
71+
assert_path_exists pkgshare/"examples/1D_sodshocktube/case.py"
7272
end
7373
end

0 commit comments

Comments
 (0)