Fix exit code in case of undefined variable. Closes #48.

CHANGES.md

@@ -6,6 +6,7 @@ Major release, unreleased
 - The option `--keep-trailing-newline` was removed in favor of making
   it default. The old behaviour can be achieved with the new option
   `--remove-trailing-newline`.
+- Fixed an exit code in case of undefined variable from 0 to 1.
 
 Version 4.3
 -----------

tests/test_cli.py

@@ -231,10 +231,12 @@ def test_mode_is_none():
 
 
 def test_mode_is_pedantic():
-    """gh-42"""
-    cmd = r'echo -n "{{ foo }}" | yasha --mode=pedantic -'
-    out = check_output(cmd, shell=True, stderr=subprocess.STDOUT)
-    assert out == b"UndefinedError: 'foo' is undefined\n"
+    """gh-42, and gh-48"""
+    with pytest.raises(subprocess.CalledProcessError) as err:
+        cmd = r'echo -n "{{ foo }}" | yasha --mode=pedantic -'
+        out = check_output(cmd, shell=True, stderr=subprocess.STDOUT)
+    out = err.value.output
+    assert out == b"Error: Variable 'foo' is undefined\n"
 
 
 def test_mode_is_debug():

yasha/cli.py

@@ -243,4 +243,4 @@ def cli(
         t_stream.enable_buffering(size=5)
         t_stream.dump(output, encoding=yasha.ENCODING)
     except JinjaUndefinedError as e:
-        click.echo("UndefinedError: {}".format(e), err=True)
+        raise ClickException("Variable {}".format(e))