summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Holesovsky <kendy@collabora.com>2017-08-17 14:04:22 +0200
committerJan Holesovsky <kendy@collabora.com>2017-08-17 14:04:22 +0200
commitd78c0b164b3ae8278d51d453a16a69aefe4fec2a (patch)
tree7405acf73e46b0e6526d33cf2def1810e748e7a7
parentaccess_header: Pass the access_header around + unit test. (diff)
downloadonline-d78c0b164b3ae8278d51d453a16a69aefe4fec2a.tar.gz
online-d78c0b164b3ae8278d51d453a16a69aefe4fec2a.zip
unit tests: Really bail out on error; and no need for a tempfile.
Change-Id: I53c1ab62bf9293217a5cada54c358292364ed60a
-rw-r--r--test/Makefile.am2
-rw-r--r--test/test.cpp4
2 files changed, 3 insertions, 3 deletions
diff --git a/test/Makefile.am b/test/Makefile.am
index 5f7b769d0c..005618de93 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -98,4 +98,4 @@ all-local: unittest
@echo "Running build-time unit tests. For more thorough testing, please run 'make check'."
@echo
@fc-cache "@LO_PATH@"/share/fonts/truetype
- @${top_builddir}/test/unittest 2> unittest.log || { cat unittest.log ; exit 1 ; }
+ @${top_builddir}/test/unittest
diff --git a/test/test.cpp b/test/test.cpp
index ed2f15af01..e989e3fa79 100644
--- a/test/test.cpp
+++ b/test/test.cpp
@@ -65,7 +65,7 @@ int main(int argc, char** argv)
Log::initialize("tst", loglevel, true, false, {});
- runClientTests(true, verbose);
+ return runClientTests(true, verbose)? 0: 1;
}
static bool IsStandalone = false;
@@ -134,7 +134,7 @@ bool runClientTests(bool standalone, bool verbose)
outputter.setNoWrap();
outputter.write();
- return result.wasSuccessful() ? 0 : 1;
+ return result.wasSuccessful();
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */