Permalink
Browse files

dist_cmd_test: drop support for making the command non-fatal

  • Loading branch information...
1 parent 2048018 commit fa7252d495d23232e956f57556f561e12165f10d @strohel committed Jul 26, 2013
Showing with 3 additions and 15 deletions.
  1. +1 −2 README.rst
  2. +2 −13 support/dist_cmd_test.py
View
@@ -147,8 +147,7 @@ your PyBayes build works as expected.
Since version 0.4, testing is integrated into the `setup.py` script and can be run without
installing PyBayes. In order to run PyBayes test-suite, simply issue ``./setup.py test`` from within
the source directory. To run tests during installation procedure, simply install like this:
-``./setup.py build test install``. Failing tests cause installation to fail, but this can be
-overriden using `--non-fatal` option of the `test` command.
+``./setup.py build test install``. With this command, failing tests prevent installation.
If you want to test your already installed PyBayes instance, simply issue
``python -m pybayes.tests`` anytime, anywhere. :-)
View
@@ -18,18 +18,10 @@ class test(Command):
"""Test PyBayes in the build directory"""
description = 'run unit test-suite of PyBayes within build directory'
- user_options = [
- ('fatal', None,
- 'make test failures fatal [default]'),
- ('non-fatal', None,
- 'make test failures fatal (opposite of `--fatal`)')
- ]
- boolean_options = ['fatal']
- negative_opt = {'non-fatal':'fatal'}
+ user_options = []
def initialize_options(self):
self.build_lib = None
- self.fatal = 1
def finalize_options(self):
self.set_undefined_options('build', ('build_lib', 'build_lib'))
@@ -50,9 +42,6 @@ def run(self):
if not result.wasSuccessful():
raise Exception("There were test failures")
except Exception as e:
- if self.fatal:
- raise DistutilsExecError(e)
- else:
- log.warn("ignoring exception: {0}".format(e))
+ raise DistutilsExecError(e)
finally:
sys.path = orig_path

0 comments on commit fa7252d

Please sign in to comment.