From 653c9ad82dd3680fe01431f9efa1eb913abdf3b3 Mon Sep 17 00:00:00 2001 From: Sviatoslav Sydorenko Date: Fri, 17 Feb 2023 04:16:57 +0100 Subject: [PATCH] =?UTF-8?q?=E2=9C=A8=20Add=20module=20check=20mode=20for?= =?UTF-8?q?=20`started`=20state=20(#18)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Prior to this patch, it was attempting to do actual mutation unconditionally. --- plugins/modules/parallels_desktop.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/plugins/modules/parallels_desktop.py b/plugins/modules/parallels_desktop.py index fa10b44..ee8791d 100644 --- a/plugins/modules/parallels_desktop.py +++ b/plugins/modules/parallels_desktop.py @@ -594,13 +594,15 @@ def ensure_app_started( # noqa: WPS231 try: parallels_pid = self.get_parallels_pid() except LookupError: + parallels_pid = -1 spawn_parallels_cmd = 'open', '-a', 'Parallels Desktop', '--hide' # NOTE: This may error out with rc=1 and the following stderr: # NOTE: "LSOpenURLsWithRole() failed for the application # NOTE: /Applications/Parallels Desktop.app with error -610.\n" - self.run_with_raise(spawn_parallels_cmd) - parallels_pid = self.get_parallels_pid() + if not self.check_mode: + self.run_with_raise(spawn_parallels_cmd) + parallels_pid = self.get_parallels_pid() return { 'msg': 'The {app!s} app (process: `{proc!s}`; '