From 815d5b01bf9be1d476b110050b20ae980c76d4a8 Mon Sep 17 00:00:00 2001 From: Julian Andres Klode Date: Tue, 18 Feb 2025 15:46:04 +0100 Subject: test: Improve output cleaning for solver progress We don't really care about solver progress messages in testing; hide them when comparing output, and hide them better than before. Previously testsuccessequal ... --solver 3.0 behaved differently than tests run with globally forced on 3.0 solver. Stop cleaning up autoremovals, but do clean up "Calculating upgrade" as well. --- test/integration/framework | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/test/integration/framework b/test/integration/framework index 3784ece06..35c1a1032 100644 --- a/test/integration/framework +++ b/test/integration/framework @@ -1555,22 +1555,15 @@ downloadfile() { fi } -cleanup_solver3_pipe() { - if [ "$APT_SOLVER" != "3.0" ]; then - cat - else - # FIXME: We do not have support for listing autoremovals yet. - # FIXME: Progress output is different - sed -e '/Solving dependencies\.\.\./ d' \ - -e "/no longer required[.:]$/,/^Use '.* autoremove'/ d" - fi -} cleanup_output() { cat "$1" | sed \ -e '/gpgv: WARNING: This key is not suitable for signing in --compliance=gnupg mode/ d' \ -e '/^profiling:/ d' \ - | sed -e '/\.\.\.profiling:/ {N;s#\.\.\.profiling:.*\n#...#g}' \ - | cleanup_solver3_pipe \ + -e '/\.\.\.profiling:/ {N;s#\.\.\.profiling:.*\n#...#g}' \ + -e '/^Solving dependencies\.\.\.$/ d' \ + -e '/^Calculating upgrade\.\.\.$/ d' \ + -e 's/Solving dependencies\.\.\.//' \ + -e 's/Calculating upgrade\.\.\.//' \ >"$2" } -- cgit v1.2.3-70-g09d2