From d472c02274a1a99d0571c7d2f105120d13b0c94d Mon Sep 17 00:00:00 2001 From: Bryce Kwon Date: Sat, 7 Nov 2026 08:37:14 -1000 Subject: Check the Lua-less build through the helpers t0006 ran build/nolua/cgit by its path, so Valgrind never saw it, and its page checks passed a page that logged an error. The Makefile and README said the suite runs both binaries the same way, though only these four tests run the one without Lua. --- tests/t0006-nolua.sh | 29 +++++++++++++++-------------- 1 file changed, 15 insertions(+), 14 deletions(-) (limited to 'tests/t0006-nolua.sh') diff --git a/tests/t0006-nolua.sh b/tests/t0006-nolua.sh index f68e731..d0ec2fe 100755 --- a/tests/t0006-nolua.sh +++ b/tests/t0006-nolua.sh @@ -17,35 +17,36 @@ test_have_prereq CGIT_NOLUA || { test_done } -nolua_query() { CGIT_CONFIG="$PWD/$1" QUERY_STRING="$2" "$nolua"; } +# The helpers run the first cgit on PATH, which this binary now becomes, or +# under Valgrind the one the wrapper is pointed at. +if test -n "$CGIT_TEST_VALGRIND" +then + CGIT_VALGRIND_BINARY=$nolua + export CGIT_VALGRIND_BINARY +else + PATH="${nolua%/cgit}:$PATH" +fi test_expect_success 'the binary reports no Lua' ' - "$nolua" --version >tmp && + cgit --version >tmp && grep -F "[-] Lua scripting" tmp ' test_expect_success 'it serves a page' ' cgitrc_repo plainrc foo && - nolua_query plainrc "url=foo/log/" >tmp && - grep "^Status: 200" tmp && + cgit_ok plainrc "url=foo/log/" >tmp && grep ">commit 5" tmp ' test_expect_success 'an exec filter still runs' ' - { - cat plainrc && - echo "repo.commit-filter=exec:$FILTER_DIRECTORY/dump.sh" - } >execrc && - nolua_query execrc "url=foo/commit/" >tmp && + cgitrc_repo execrc foo "repo.commit-filter=exec:$FILTER_DIRECTORY/dump.sh" && + cgit_ok execrc "url=foo/commit/" >tmp && grep "COMMIT 5" tmp ' test_expect_success 'a lua filter is refused with one error page' ' - { - cat plainrc && - echo "repo.commit-filter=lua:$FILTER_DIRECTORY/dump.lua" - } >luarc && - nolua_query luarc "url=foo/commit/" >tmp 2>err && + cgitrc_repo luarc foo "repo.commit-filter=lua:$FILTER_DIRECTORY/dump.lua" && + cgit_rc luarc "url=foo/commit/" >tmp 2>err && grep "^Status: 500" tmp && test $(grep -c "^Status:" tmp) = 1 && grep "Built without Lua" err -- cgit 2.12.1