From 2e69417d67c63b7f3eb275c7cf1f1f215c1cdf41 Mon Sep 17 00:00:00 2001 From: Henning Makholm Date: Fri, 22 Aug 2025 15:27:37 +0200 Subject: [PATCH] rust integration test: use all output from codeql test run The integration test expectes to find a certain phrase from the extractor repeated in the _stderr_ of `codeql test run`. However, that subcommand is about to start reproducing the extractor output as-is, which means the phrase will instead appear in _stdout_. Change the integration test to capture all of the output, so it will keep passing across the change. --- rust/ql/integration-tests/qltest/test_qltest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust/ql/integration-tests/qltest/test_qltest.py b/rust/ql/integration-tests/qltest/test_qltest.py index edd62bf600ef..1fd525890cea 100644 --- a/rust/ql/integration-tests/qltest/test_qltest.py +++ b/rust/ql/integration-tests/qltest/test_qltest.py @@ -20,7 +20,7 @@ def test(codeql, rust, expected_files, dir): codeql.test.run(dir) def test_failing_cargo_check(codeql, rust): - out = codeql.test.run("failing_cargo_check", _assert_failure=True, _capture="stderr") + out = codeql.test.run("failing_cargo_check", _assert_failure=True, _capture="all") # TODO: QL test output redirection is currently broken on windows, leaving it up for follow-up work if not runs_on.windows: assert "requested cargo check failed" in out