|
 |
e41f67 |
From 9b42fce32bff25e0569581f76f532b9d57865aef Mon Sep 17 00:00:00 2001
|
|
|
e41f67 |
From: =?UTF-8?q?V=C3=ADt=20Ondruch?= <vondruch@redhat.com>
|
|
|
e41f67 |
Date: Mon, 27 Jul 2020 14:56:05 +0200
|
|
|
e41f67 |
Subject: [PATCH] Timeout the test_bug_reporter_add witout raising error.
|
|
|
e41f67 |
|
|
|
e41f67 |
While timeouting the threads might be still good idea, it does not seems
|
|
|
e41f67 |
the timeout impacts the TestBugReporter#test_bug_reporter_add result,
|
|
|
e41f67 |
because the output of the child process has been already collected
|
|
|
e41f67 |
earlier.
|
|
|
e41f67 |
|
|
|
e41f67 |
It seems that when the system is under heavy load, the thread might not
|
|
|
e41f67 |
be sheduled to finish its processing. Even finishing the child process
|
|
|
e41f67 |
might take tens of seconds and therefore the test case finish might take
|
|
|
e41f67 |
a while.
|
|
|
e41f67 |
---
|
|
|
e41f67 |
test/-ext-/bug_reporter/test_bug_reporter.rb | 2 +-
|
|
|
e41f67 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
e41f67 |
|
|
|
e41f67 |
diff --git a/test/-ext-/bug_reporter/test_bug_reporter.rb b/test/-ext-/bug_reporter/test_bug_reporter.rb
|
|
|
e41f67 |
index 628fcd0340..2c677cc8a7 100644
|
|
|
e41f67 |
--- a/test/-ext-/bug_reporter/test_bug_reporter.rb
|
|
|
e41f67 |
+++ b/test/-ext-/bug_reporter/test_bug_reporter.rb
|
|
|
e41f67 |
@@ -22,7 +22,7 @@ def test_bug_reporter_add
|
|
|
e41f67 |
args = ["--disable-gems", "-r-test-/bug_reporter",
|
|
|
e41f67 |
"-C", tmpdir]
|
|
|
e41f67 |
stdin = "register_sample_bug_reporter(12345); Process.kill :SEGV, $$"
|
|
|
e41f67 |
- assert_in_out_err(args, stdin, [], expected_stderr, encoding: "ASCII-8BIT")
|
|
|
e41f67 |
+ assert_in_out_err(args, stdin, [], expected_stderr, encoding: "ASCII-8BIT", timeout_error: nil)
|
|
|
e41f67 |
ensure
|
|
|
e41f67 |
FileUtils.rm_rf(tmpdir) if tmpdir
|
|
|
e41f67 |
end
|
|
|
e41f67 |
--
|
|
|
e41f67 |
2.27.0
|
|
|
e41f67 |
|