summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPetr Rockai <prockai@redhat.com>2014-10-01 11:28:15 +0200
committerPetr Rockai <prockai@redhat.com>2014-11-18 19:02:11 +0100
commitce42323bd1510bfeaf635f37769920332d2035ea (patch)
tree6e43acd674fe9a3edb2f2c3a2c0fc2b556dad33b
parentaa05be29a0b9c43418b09c16460597ba846dc96a (diff)
downloadlvm2-ce42323bd1510bfeaf635f37769920332d2035ea.tar.gz
test: Properly detect warnings from testcases.
-rw-r--r--test/lib/brick-shelltest.h21
1 files changed, 19 insertions, 2 deletions
diff --git a/test/lib/brick-shelltest.h b/test/lib/brick-shelltest.h
index 71194cfc3..7a2b8f0db 100644
--- a/test/lib/brick-shelltest.h
+++ b/test/lib/brick-shelltest.h
@@ -543,8 +543,22 @@ struct KMsg : Source {
};
struct Observer : Sink {
- Observer() {}
- void push( std::string ) {}
+ TimedBuffer stream;
+
+ bool warnings;
+ Observer() : warnings( false ) {}
+
+ void push( std::string s ) {
+ stream.push( s );
+ }
+
+ void sync( bool force ) {
+ while ( !stream.empty( force ) ) {
+ TimedBuffer::Line line = stream.shift( force );
+ if ( line.second.find( "TEST WARNING" ) != std::string::npos )
+ warnings = true;
+ }
+ }
};
struct IO : Sink {
@@ -850,6 +864,9 @@ struct TestCase {
else
r = Journal::FAILED;
+ if ( r == Journal::PASSED && io.observer().warnings )
+ r = Journal::WARNED;
+
io.close();
if ( iobuf && ( r == Journal::FAILED || r == Journal::TIMEOUT ) )