feat: set --no-interaction when stdin is not a tty (#245)

cleo/application.py

@@ -501,7 +501,11 @@ def _configure_io(self, io: IO) -> None:
         elif io.input.has_parameter_option("--no-ansi", True):
             io.decorated(False)
 
-        if io.input.has_parameter_option(["--no-interaction", "-n"], True):
+        if io.input.has_parameter_option(["--no-interaction", "-n"], True) or (
+            io.input._interactive is None
+            and io.input.stream
+            and not io.input.stream.isatty()
+        ):
             io.interactive(False)
 
         shell_verbosity = int(os.getenv("SHELL_VERBOSITY", 0))

cleo/io/inputs/input.py

@@ -21,7 +21,7 @@ def __init__(self, definition: Definition | None = None) -> None:
         self._stream: TextIO = None  # type: ignore[assignment]
         self._options: dict[str, Any] = {}
         self._arguments: dict[str, Any] = {}
-        self._interactive = True
+        self._interactive: bool | None = None
 
         if definition is None:
             self._definition = Definition()
@@ -56,7 +56,7 @@ def read(self, length: int, default: str = "") -> str:
         """
         Reads the given amount of characters from the input stream.
         """
-        if not self._interactive:
+        if not self.is_interactive():
             return default
 
         return self._stream.read(length)
@@ -65,7 +65,7 @@ def read_line(self, length: int = -1, default: str = "") -> str:
         """
         Reads a line from the input stream.
         """
-        if not self._interactive:
+        if not self.is_interactive():
             return default
 
         return self._stream.readline(length)
@@ -83,7 +83,7 @@ def is_closed(self) -> bool:
         return self._stream.closed
 
     def is_interactive(self) -> bool:
-        return self._interactive
+        return True if self._interactive is None else self._interactive
 
     def interactive(self, interactive: bool = True) -> None:
         self._interactive = interactive

cleo/testers/application_tester.py

@@ -40,7 +40,7 @@ def execute(
         self,
         args: str = "",
         inputs: str | None = None,
-        interactive: bool | None = None,
+        interactive: bool = True,
         verbosity: Verbosity | None = None,
         decorated: bool = False,
         supports_utf8: bool = True,