2020-02-17 22:31:47 +08:00
|
|
|
#RUN: %fish -C 'set -l fish %fish' %s
|
|
|
|
# Test ALL THE FISH FILES
|
|
|
|
# in share/, that is - the tests are exempt because they contain syntax errors, on purpose
|
|
|
|
|
Revert "Revert "Speed up check-all-fish-files when executed locally""
This reverts commit e240d81ff841a86294e9e0e6b26bf42a1b4c18ed and
introduces a more compatible method of finding newly added fish scripts
to syntax check.
`find -newer` is the original and is supported by everything under the
sun (including FreeBSD, NetBSD, Solaris, OpenIndiana, macOS 10.10, WSL,
and more), and if not, the tests will succeed anyway. `find -mnewer` was
added later around the time `find -cnewer` and co (which checks the
creation date rather than the modification date) was introduced, but
apparently the GNU version of coreutils never introduced the `-mnewer`
alias for `-newer`.
Yes, this is hacky and yes it would be ideal if the build system is the
one that picked which tests to run rather than the test itself picking.
But let's not pretend that our tests are idealogically ideal or pure
right now and until we fix the mess that is our CMake test integration
(e.g. use ctest and configure each test to be run separately with
configurable payloads, etc) eight seconds is still eight seconds, and
again, the CI isn't affected.
2021-03-07 07:07:24 +08:00
|
|
|
set timestamp_file ./last_check_all_files
|
|
|
|
set -l find_args
|
|
|
|
if test -f $timestamp_file
|
|
|
|
set find_args -newer $timestamp_file
|
2021-03-06 12:40:40 +08:00
|
|
|
end
|
Revert "Revert "Speed up check-all-fish-files when executed locally""
This reverts commit e240d81ff841a86294e9e0e6b26bf42a1b4c18ed and
introduces a more compatible method of finding newly added fish scripts
to syntax check.
`find -newer` is the original and is supported by everything under the
sun (including FreeBSD, NetBSD, Solaris, OpenIndiana, macOS 10.10, WSL,
and more), and if not, the tests will succeed anyway. `find -mnewer` was
added later around the time `find -cnewer` and co (which checks the
creation date rather than the modification date) was introduced, but
apparently the GNU version of coreutils never introduced the `-mnewer`
alias for `-newer`.
Yes, this is hacky and yes it would be ideal if the build system is the
one that picked which tests to run rather than the test itself picking.
But let's not pretend that our tests are idealogically ideal or pure
right now and until we fix the mess that is our CMake test integration
(e.g. use ctest and configure each test to be run separately with
configurable payloads, etc) eight seconds is still eight seconds, and
again, the CI isn't affected.
2021-03-07 07:07:24 +08:00
|
|
|
set -l fail_count 0
|
|
|
|
for file in (find $__fish_data_dir/ -name "*.fish" $find_args 2>/dev/null; or find $__fish_data_dir/ -name "*.fish")
|
|
|
|
$fish -n $file; or set fail_count (math $fail_count + 1)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Prevent setting timestamp if any errors were encountered
|
|
|
|
if test "$fail_count" -eq 0
|
|
|
|
touch $timestamp_file
|
|
|
|
end
|
|
|
|
|
2020-02-17 22:31:47 +08:00
|
|
|
# No output is good output
|