Skip to content
Permalink
Browse files

respond to comments

  • Loading branch information
wyegelwel authored and thejunglejane committed Jul 31, 2018
1 parent 5cb4929 commit 12d820447e231c9130deaf694cc08345668ada5a
Showing with 21 additions and 4 deletions.
  1. +2 −2 README.rst
  2. +16 −0 docs/quickstart.rst
  3. +2 −1 marbles/core/marbles/setuptools.py
  4. +1 −1 marbles/core/setup.py
@@ -66,14 +66,14 @@ You can also run your existing `unittest` tests with `marbles`
python -m marbles test_module.py
Marbles also creates a setuptools command so if you are used to running
`python setup.py test`, you can now run:
``python setup.py test``, you can now run:

.. code-block:: bash
python setup.py marbles
You can go one step further and alias the command test to run marbles
by adding the following to setup.cfg:
by adding the following to :file:`setup.cfg`:

.. code-block:: bash
@@ -132,6 +132,22 @@ As we saw :ref:`above <Quickstart>`, you can also run vanilla unit tests with ma
python -m marbles /path/to/unittest_tests.py
Marbles also creates a setuptools command so if you are used to running
``python setup.py test``, you can now run:

.. code-block:: bash
python setup.py marbles
You can go one step further and alias the command test to run marbles
by adding the following to :file:`setup.cfg`:

.. code-block:: bash
[aliases]
test = marbles
Declaring marbles as a dependency
---------------------------------

@@ -10,7 +10,7 @@
# and overrides functions that reference unittest to instead
# use marbles.
# Hopefully you can find a better way to achieve the same goal.
class MarblesCommand(setuptools.command.test.test):
class MarblesTestCommand(setuptools.command.test.test):
description = 'Run tests using Marbles'

def run_tests(self):
@@ -22,6 +22,7 @@ def run_tests(self):
testRunner=self._resolve_as_ep(self.test_runner),
exit=False,
)

if not test.result.wasSuccessful():
msg = 'Test failed: %s' % test.result
self.announce(msg, log.ERROR)
@@ -77,7 +77,7 @@
},
entry_points={
"distutils.commands": [
"marbles = marbles.setuptools:MarblesCommand",
"marbles = marbles.setuptools:MarblesTestCommand",
],
},
classifiers=classifiers

0 comments on commit 12d8204

Please sign in to comment.
You can’t perform that action at this time.