diff options
author | Phaedrus Leeds <mwleeds@protonmail.com> | 2021-11-16 11:00:34 -0800 |
---|---|---|
committer | Phaedrus Leeds <mwl458@gmail.com> | 2022-01-04 10:44:37 -0800 |
commit | e5bec49b83964d73ce3d3ba9766f904413930ca9 (patch) | |
tree | 942c0a6b8f519bec8dc0c2cbcfafae5dd965b8c5 | |
parent | 63606694186bd775499ec32efdcb703873d6cf87 (diff) | |
download | flatpak-e5bec49b83964d73ce3d3ba9766f904413930ca9.tar.gz |
Make test suite logs prettier
This fixes a few issues with the unit test logs that make them ugly:
1. Currently some lines of output from a command will appear after the
line from xtrace which has the next command, since the command was
printing to stdout and xtrace uses stderr. E.g. "Installation complete."
will appear after "+ flatpak --user install -y ..." but it is from the
previous install command.
2. Lines of output have many spaces after them to pad them to the table
width but this is not needed for non-fancy output.
3. Lines of output are mixed with output from httpd since they don't end
with a newline character, e.g. "Installing… ▊
4%127.0.0.1 - - [16/Nov/2021 00:18:24] "GET /..."
(cherry picked from commit 388c23cfc51b6b9f03dcdf87b11c0078f11227ea)
-rw-r--r-- | app/flatpak-cli-transaction.c | 14 | ||||
-rw-r--r-- | tests/libtest.sh | 4 |
2 files changed, 11 insertions, 7 deletions
diff --git a/app/flatpak-cli-transaction.c b/app/flatpak-cli-transaction.c index 788be097..99bc8c6e 100644 --- a/app/flatpak-cli-transaction.c +++ b/app/flatpak-cli-transaction.c @@ -124,7 +124,7 @@ add_new_remote (FlatpakTransaction *transaction, if (self->disable_interaction) { - g_print (_("Configuring %s as new remote '%s'"), url, remote_name); + g_print (_("Configuring %s as new remote '%s'\n"), url, remote_name); return TRUE; } @@ -397,7 +397,7 @@ progress_changed_cb (FlatpakTransactionProgress *progress, g_print ("\r%s", str->str); /* redraw failed, just update the progress */ } else - g_print ("\n%s", str->str); + g_print ("%s\n", str->str); } static void @@ -455,7 +455,7 @@ new_operation (FlatpakTransaction *transaction, redraw (self); } else - g_print ("\n%-*s", self->table_width, text); + g_print ("%s\n", text); g_free (self->progress_msg); self->progress_msg = g_steal_pointer (&text); @@ -509,7 +509,7 @@ operation_error (FlatpakTransaction *transaction, redraw (self); } else - g_print ("\n%-*s\n", self->table_width, msg); /* override progress, and go to next line */ + g_print ("%s\n", msg); return TRUE; } @@ -548,7 +548,7 @@ operation_error (FlatpakTransaction *transaction, redraw (self); } else - g_printerr ("\n%-*s\n", self->table_width, text); + g_printerr ("%s\n", text); if (!non_fatal && self->stop_on_first_error) return FALSE; @@ -1052,7 +1052,7 @@ message_handler (const gchar *log_domain, redraw (self); } else - g_print ("\n%-*s\n", self->table_width, text); + g_print ("%s\n", text); } static gboolean @@ -1414,7 +1414,7 @@ flatpak_cli_transaction_run (FlatpakTransaction *transaction, redraw (self); } else - g_print ("\n%-*s", self->table_width, text); + g_print ("%s", text); g_print ("\n"); } diff --git a/tests/libtest.sh b/tests/libtest.sh index e12907f8..28aa5160 100644 --- a/tests/libtest.sh +++ b/tests/libtest.sh @@ -18,6 +18,10 @@ # Free Software Foundation, Inc., 59 Temple Place - Suite 330, # Boston, MA 02111-1307, USA. +# redirect stdout to stderr, otherwise the log will have command output out of +# order with xtrace output +exec 2>&1 + if [ -n "${G_TEST_SRCDIR:-}" ]; then test_srcdir="${G_TEST_SRCDIR}" else |