summaryrefslogtreecommitdiff
path: root/Meta/check-ak-test-files.sh
AgeCommit message (Collapse)Author
2022-09-13Meta: Fix check for AK test inclusionBen Wiederhake
basename from GNU coreutils 8.32 (the default on Debian Bullseye, as bash 5.2.0 does not include basename as a built-in command) does not accept multiple arguments. This will cause the command to fail with no output, and the error code not being propagated for some reason. This means that the loop never gets executed, and thus the check never actually does anything. This commit fixes that behavior by calling 'basename' multiple times.
2021-11-02Meta: Remove unnecessary -ithislooksfun
Using `xargs -i <cmd> {}` is just doing the default behavior of xargs, but with extra steps that also don't work on macOS.
2021-11-02Meta: Use grep -E/F, not grep -Pthislooksfun
grep -E and -F are POSIX standard, and meets all our matching needs.
2021-09-14Meta: Tweak the error in check-ak-test-files.sh a bitAli Mohammad Pur
This can also be tripped when the change was simply not being committed, avoid confusing the author by simply mentioning that.
2021-08-05Meta: Fix wrong path for AK Tests in check-ak-test-files.shMandar Kulkarni
2021-05-08Meta + Lagom: Fix paths after movement of tests to Tests/Brian Gianforcaro
Not sure how this didn't cause a build break when I originally moved them in #6891. Better fix them now.
2021-01-22Meta: Get building on NixOS (#5005)Jonathan Turner
2020-12-24Meta: Verify all AK test files are listed in CMakeLenny Maiorani
Problem: - It is possible for a new test file to be added to the `AK/Tests` directory without being added to the corresponding `CMakeLists.txt`. This results in the tests not being run. Solution: - As part of CI linting, verify that all the `AK/Tests/Test*.cpp` files are mentioned in the `CMakeLists.txt`.