v0.2.35: more terse output, hiding traceback from build.executables.run

setup.py

@@ -14,7 +14,7 @@
 ]
 
 setup(name='uranium',
-      version='0.2.34b',
+      version='0.2.35b',
       description='a build system for python',
       long_description=open('README.rst').read(),
       author='Yusuke Tsutsumi',

tests/test_executables.py

@@ -1,6 +1,18 @@
+from uranium.exceptions import NonZeroExitCodeException
+
+
 def test_hello_world(capfd, executables):
     desired_output = "hello, world"
     executables.run(["echo", "{0}".format(desired_output)],
                     subprocess_args={"stdin": None})
     out, err = capfd.readouterr()
     assert out.strip() == "hello, world"
+
+
+def test_exception_contains_executable_name(capfd, executables):
+    try:
+        executables.run(["grep"])
+    except NonZeroExitCodeException as e:
+        assert "grep" in str(e)
+    else:
+        assert False

tests/test_executables_full.py

@@ -2,7 +2,7 @@
 from .conftest import URANIUM_SOURCE_ROOT
 
 
-def test_pytest_avaliable_after_install(tmpdir):
+def test_pytest_available_after_install(tmpdir):
     """ pytest should be available after an installation. """
 
     UBUILD = """
@@ -19,3 +19,22 @@ def main(build):
     print("stdout:\n" + str(out))
     print("stderr:\n" + str(err))
     assert "2.7.0" in out.decode("utf-8")
+
+
+def test_no_traceback_on_bad_error_code(tmpdir):
+    """ pytest should be available after an installation. """
+
+    UBUILD = """
+def main(build):
+    # grep with no arguments is a non-zero exit code.
+    build.executables.run(["grep"])
+    """.strip()
+
+    tmpdir.join("ubuild.py").write(UBUILD)
+    code, out, err = execute_script(
+        "uranium_standalone", "--uranium-dir", URANIUM_SOURCE_ROOT,
+        cwd=tmpdir.strpath
+    )
+    print("stdout:\n" + str(out))
+    print("stderr:\n" + str(err))
+    assert "Traceback" not in out.decode("utf-8")

uranium/exceptions.py

@@ -15,23 +15,23 @@ class HooksException(UraniumException):
     pass
 
 
-class NonZeroExitCodeException(UraniumException):
+class PluginException(UraniumException):
+    """ an exception that occurred with the plugin """
     pass
 
 
-class PackageException(UraniumException):
-    """ exceptions with the package object """
+class ScriptException(UraniumException):
     pass
 
 
-class PluginException(UraniumException):
-    """ an exception that occurred with the plugin """
+class ConfigException(ScriptException):
     pass
 
 
-class ScriptException(UraniumException):
+class NonZeroExitCodeException(ScriptException):
     pass
 
 
-class ConfigException(UraniumException):
+class PackageException(UraniumException):
+    """ exceptions with the package object """
     pass

uranium/executables.py

@@ -70,7 +70,11 @@ def main(build):
         exit_code = popen.returncode
 
         if exit_code != 0 and fail_on_error:
-            raise NonZeroExitCodeException("received non-zero exit code: {0}".format(exit_code))
+            raise NonZeroExitCodeException(
+                "{0} returned non-zero exit code: {1}".format(
+                    args[0], exit_code
+                )
+            )
         return exit_code, out, err
 
     def install_script(self, name, body, execution_dir="base"):

uranium/lib/log_templates.py

@@ -1,11 +1,7 @@
 STARTING_URANIUM = """
-================
-STARTING URANIUM
-================
+=== START URANIUM ===
 """.strip()
 
 ENDING_URANIUM = """
-================
-URANIUM FINISHED
-================
+=== URANIUM FINISHED ===
 """.strip()