[PATCH] regression-tests: remove assert_logs from test 28

[Ondrej Lichtner]

From: Ondrej Lichtner <olichtne(a)redhat.com&gt;

These checked if stdout from an exec_cmd was printed out in the result
summary. This behaviour changed in 14370a671f614cc728922a13f68963618695c656,
breaking the 28th regression test.

However, these assert_log commands are not needed for the purposes of
this regression test - the required functionality is tested from within
the python task. Because of that it's safe to remove them, in order to
fix the regression test.

Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
---
 regression-tests/tests/28/run.sh | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/regression-tests/tests/28/run.sh b/regression-tests/tests/28/run.sh
index 9901282..a066c0f 100755
--- a/regression-tests/tests/28/run.sh
+++ b/regression-tests/tests/28/run.sh
@@ -25,9 +25,6 @@ assert_status "pass" "$rv1"
 assert_status "pass" "$rv2"
 assert_status "pass" "$rv3"
 assert_status "pass" "$rv4"
-assert_log "INFO" "stdout:.*test" "$log1"
-assert_log "INFO" "stdout:.*test" "$log2"
-assert_log "INFO" "stdout:.*test" "$log4"
 
 rm -f test.log
 
-- 
2.10.0

[Jan Tluka]

Fri, Oct 07, 2016 at 02:44:22PM CEST, olichtne(a)redhat.com wrote:
> From: Ondrej Lichtner <olichtne(a)redhat.com&gt;
>
> These checked if stdout from an exec_cmd was printed out in the result
> summary. This behaviour changed in 14370a671f614cc728922a13f68963618695c656,
> breaking the 28th regression test.
>
> However, these assert_log commands are not needed for the purposes of
> this regression test - the required functionality is tested from within
> the python task. Because of that it's safe to remove them, in order to
> fix the regression test.
>
> Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com&gt; 
Acked-by: Jan Tluka <jtluka(a)redhat.com&gt;