more fixy

Author: adamrtalbot

seqerakit/cli.py

@@ -174,7 +174,12 @@ def handle_block(self, block, args, destroy=False, dryrun=False):
         }
 
         # Get the on_exists option from args for backward compatibility
-        on_exists = args.get("on_exists", OnExists.FAIL)
+        on_exists = args.get("on_exists", "fail")
+        if isinstance(on_exists, str):
+            try:
+                on_exists = OnExists[on_exists.upper()]
+            except KeyError:
+                on_exists = OnExists.FAIL
 
         # Global settings take precedence over block-level settings
         # First check the global --on-exists parameter
@@ -188,9 +193,11 @@ def handle_block(self, block, args, destroy=False, dryrun=False):
             on_exists = OnExists.OVERWRITE
 
         if not dryrun:
-            logging.debug(
-                f" on_exists is set to '{on_exists.name.lower()}' for {block}\n"
+            # Use on_exists.name.lower() only if it's an enum, otherwise use the string
+            on_exists_str = (
+                on_exists.name.lower() if hasattr(on_exists, "name") else on_exists
             )
+            logging.debug(f" on_exists is set to '{on_exists_str}' for {block}\n")
             should_continue = self.overwrite_method.handle_overwrite(
                 block, args["cmd_args"], on_exists=on_exists
             )

seqerakit/overwrite.py

@@ -178,10 +178,10 @@ def handle_overwrite(
                     self.delete_resource(block, operation, sp_args)
                 else:  # fail
                     raise ResourceExistsError(
-                        f" The {block} resource already exists and"
-                        " will not be created. Please set 'on_exists: overwrite' "
-                        " to replace the resource or set 'on_exists: ignore' to "
-                        " ignore this error.\n"
+                        f"The {block} resource already exists and "
+                        "will not be created. Please set 'on_exists: overwrite' "
+                        "to replace the resource or set 'on_exists: ignore' to "
+                        "ignore this error.\n"
                     )
             return True
         return True