Merge branch 'release/v0.8.1'

Author: Suszyński Krzysztof

Diff for: .github/CODEOWNERS

@@ -0,0 +1,3 @@
+# See: https://blog.github.com/2017-07-06-introducing-code-owners/
+# These owners will be the default owners for everything in the repo.
+*       @coi-gov-pl/puppet-team

Diff for: MANIFEST.in

@@ -18,3 +18,4 @@ recursive-include integration_tests */Dockerfile
 
 # Exclude environment files
 prune .circleci
+prune .github

Diff for: integration_tests/scripts/install-develop.sh

@@ -5,4 +5,4 @@ root=$(dirname $(dirname $(dirname ${here})))
 
 set -x
 
-pip install -e ${root}
+LC_ALL=C pip install -e ${root}

Diff for: integration_tests/test_ubuntu_1604.py

@@ -1,3 +1,4 @@
+import re
 import pytest
 from os.path import join
 
@@ -10,6 +11,38 @@ def sut():
     return join('debian', 'ubuntu-1604')
 
 
+@pytest.mark.ubuntu1604
+def test_puppeter_develop_install_on_ubuntu_1604_with_invalid_locale(phial, capsys, regex):
+    with capsys.disabled():
+        acceptance = PuppeterAcceptance(phial)
+        pattern = re.compile('^puppeter \d+\.\d+\.\d+(?:\.[a-z0-9]+)?$')
+
+        phial.exec('echo "export LC_ALL=mn-Mong-CN.UTF-8" >> /etc/profile.d/mandarin.sh')
+        acceptance.install_puppeter()
+        phial.exec('rm -vf /etc/profile.d/mandarin.sh')
+
+        exitcode, output, error = phial.exec('puppeter --version', capture=True)
+        assert error == ''
+        regex.pattern(pattern).matches(output)
+        assert exitcode == 0
+
+
+@pytest.mark.ubuntu1604
+def test_puppeter_production_install_on_ubuntu_1604_with_invalid_locale(phial, capsys, regex):
+    with capsys.disabled():
+        pattern = re.compile('^puppeter \d+\.\d+\.\d+(?:\.[a-z0-9]+)?$')
+
+        phial.exec('echo "export LC_ALL=mn-Mong-CN.UTF-8" >> /etc/profile.d/mandarin.sh')
+        exitcode, output, error = phial.exec('/puppeter-setup.sh')
+        assert exitcode == 0
+        phial.exec('rm -vf /etc/profile.d/mandarin.sh')
+
+        exitcode, output, error = phial.exec('puppeter --version', capture=True)
+        assert error == ''
+        regex.pattern(pattern).matches(output)
+        assert exitcode == 0
+
+
 @pytest.mark.ubuntu1604
 def test_agent_system_on_ubuntu_1604(phial, capsys, regex):
     with capsys.disabled():

Diff for: puppeter/__init__.py

@@ -4,7 +4,7 @@
 from typing import Type
 
 # The version of the app
-__version__ = '0.8.0'
+__version__ = '0.8.1'
 __program__ = 'puppeter'
 
 
@@ -16,8 +16,3 @@ def get_logger(cls):
 
 def __fullname(cls):
     return cls.__module__ + "." + cls.__name__
-
-
-if __name__ == '__main__':
-    from puppeter.main import main
-    main()

Diff for: puppeter/__main__.py

@@ -1,2 +1,4 @@
 from puppeter.main import main
-main()
+
+if __name__ == '__main__':
+    main()

Diff for: setup.sh

@@ -102,5 +102,5 @@ elif [ $osfamily == debian ]; then
   fi
 fi
 if [ $MODE == puppeter ]; then
-  run pip install puppeter
+  run LC_ALL=C pip install puppeter
 fi

Diff for: tests/test__main__.py

@@ -0,0 +1,21 @@
+import re
+
+import pytest
+
+from tests.helpers.commandline import captured_output
+
+
+def test_usage_printout():
+    from puppeter import __main__
+
+    with pytest.raises(SystemExit) as sysexit:
+        with captured_output() as (out, err):
+            __main__.main(['puppeter', '--help'])
+
+    stdout = out.getvalue().strip()
+    stderr = err.getvalue().strip()
+    assert sysexit.value.code == 0
+    assert stdout == ''
+    assert re.compile('Puppeter', re.MULTILINE).search(stderr)
+    assert re.compile('--answers FILE, -a FILE', re.MULTILINE).search(stderr)
+    assert re.compile('--execute, -e', re.MULTILINE).search(stderr)