Adding upstream version 0.13.1.upstream/0.13.1

Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>

1 files changed, 56 insertions, 0 deletions

diff --git a/qa/test_stdin.py b/qa/test_stdin.py
new file mode 100644
index 0000000..fff636f
--- /dev/null
+++ b/qa/test_stdin.py
@@ -0,0 +1,56 @@
+# -*- coding: utf-8 -*-
+# pylint: disable=too-many-function-args,unexpected-keyword-arg
+import io
+import subprocess
+from qa.shell import echo, gitlint
+from qa.base import BaseTestCase
+from qa.utils import ustr, DEFAULT_ENCODING
+
+
+class StdInTests(BaseTestCase):
+    """ Integration tests for various STDIN scenarios for gitlint """
+
+    def test_stdin_pipe(self):
+        """ Test piping input into gitlint.
+            This is the equivalent of doing:
+            $ echo "foo" | gitlint
+        """
+        # NOTE: There is no use in testing this with _tty_in=True, because if you pipe something into a command
+        # there never is a TTY connected to stdin (per definition). We're setting _tty_in=False here to be explicit
+        # but note that this is always true when piping something into a command.
+        output = gitlint(echo(u"WIP: Pïpe test."),
+                         _cwd=self.tmp_git_repo, _tty_in=False, _err_to_out=True, _ok_code=[3])
+        self.assertEqualStdout(output, self.get_expected("test_stdin/test_stdin_pipe_1"))
+
+    def test_stdin_pipe_empty(self):
+        """ Test the scenario where no TTY is attached an nothing is piped into gitlint. This occurs in
+            CI runners like Jenkins and Gitlab, see https://github.com/jorisroovers/gitlint/issues/42 for details.
+            This is the equivalent of doing:
+            $ echo -n "" | gitlint
+        """
+        commit_msg = u"WIP: This ïs a title.\nContent on the sëcond line"
+        self.create_simple_commit(commit_msg)
+
+        # We need to set _err_to_out explicitly for sh to merge stdout and stderr output in case there's
+        # no TTY attached to STDIN
+        # http://amoffat.github.io/sh/sections/special_arguments.html?highlight=_tty_in#err-to-out
+        output = gitlint(echo("-n", ""), _cwd=self.tmp_git_repo, _tty_in=False, _err_to_out=True, _ok_code=[3])
+
+        self.assertEqual(ustr(output), self.get_expected("test_stdin/test_stdin_pipe_empty_1"))
+
+    def test_stdin_file(self):
+        """ Test the scenario where STDIN is a regular file (stat.S_ISREG = True)
+            This is the equivalent of doing:
+            $ gitlint < myfile
+        """
+        tmp_commit_msg_file = self.create_tmpfile(u"WIP: STDIN ïs a file test.")
+
+        with io.open(tmp_commit_msg_file, encoding=DEFAULT_ENCODING) as file_handle:
+
+            # We need to use subprocess.Popen() here instead of sh because when passing a file_handle to sh, it will
+            # deal with reading the file itself instead of passing it on to gitlint as a STDIN. Since we're trying to
+            # test for the condition where stat.S_ISREG == True that won't work for us here.
+            p = subprocess.Popen(u"gitlint", stdin=file_handle, cwd=self.tmp_git_repo,
+                                 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+            output, _ = p.communicate()
+            self.assertEqual(ustr(output), self.get_expected("test_stdin/test_stdin_file_1"))