Executor: Remove duplicate entry with dry-run argument

python-poetry · Oct 20, 2021 · 73b6da5 · 73b6da5
1 parent 82459bd
commit 73b6da5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/poetry/installation/executor.py b/poetry/installation/executor.py
@@ -310,12 +310,6 @@ def _do_execute_operation(self, operation: "OperationTypes") -> int:
             return 0
 
         if not self._enabled or self._dry_run:
-            self._io.write_line(
-                "  <fg=blue;options=bold>•</> {message}".format(
-                    message=operation_message,
-                )
-            )
-
             return 0
 
         result = getattr(self, f"_execute_{method}")(operation)
@@ -737,7 +731,9 @@ def _download_archive(self, operation: Union[Install, Update], link: Link) -> Pa
         return archive
 
     def _should_write_operation(self, operation: Operation) -> bool:
-        return not operation.skipped or self._dry_run or self._verbose
+        return (
+            not operation.skipped or self._dry_run or self._verbose or not self._enabled
+        )
 
     def _save_url_reference(self, operation: "OperationTypes") -> None:
         """

diff --git a/tests/console/commands/plugin/test_remove.py b/tests/console/commands/plugin/test_remove.py
@@ -108,7 +108,6 @@ def test_remove_installed_package_dry_run(app, tester, env):
 Package operations: 0 installs, 0 updates, 1 removal
 
   • Removing poetry-plugin (1.2.3)
-  • Removing poetry-plugin (1.2.3)
 """
 
     assert tester.io.fetch_output() == expected