This is an automated email from the ASF dual-hosted git repository.
kszucs pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new 4fefff3 ARROW-4850: [CI] Ensure integration_test.py returns non-zero
on failures
4fefff3 is described below
commit 4fefff3aeeb22e1c4e082122b3ee7c49c6071def
Author: François Saint-Jacques <[email protected]>
AuthorDate: Wed Mar 13 13:55:56 2019 +0100
ARROW-4850: [CI] Ensure integration_test.py returns non-zero on failures
Author: François Saint-Jacques <[email protected]>
Closes #3884 from fsaintjacques/ARROW-4850-integration-failures and
squashes the following commits:
f9ce12b5 <François Saint-Jacques> ARROW-4850: Ensure integration_test.py
returns non-zero on failures
---
integration/integration_test.py | 19 ++++++++-----------
1 file changed, 8 insertions(+), 11 deletions(-)
diff --git a/integration/integration_test.py b/integration/integration_test.py
index e7e8edd..17e7afb 100644
--- a/integration/integration_test.py
+++ b/integration/integration_test.py
@@ -1307,17 +1307,14 @@ def run_all_tests(run_flight=False, debug=False,
tempdir=None):
if run_flight:
failures.extend(runner.run_flight())
- print()
- print('##########################################################')
- if not failures:
- print('-- All tests passed!')
- else:
- print('-- Tests completed, failures:')
- for producer, consumer, exc_info in failures:
- print("FAILED TEST:", producer.name, "producing, ",
- consumer.name, "consuming")
- traceback.print_exception(*exc_info)
- print()
+ if failures:
+ print("################# FAILURES #################")
+ for producer, consumer, exc_info in failures:
+ print("FAILED TEST:", producer.name, "producing, ",
+ consumer.name, "consuming")
+ traceback.print_exception(*exc_info)
+ print()
+ sys.exit(1)
def write_js_test_json(directory):
