Improve error reporting for object validate command

Author: dgarros

infrahub_sdk/ctl/object.py

@@ -7,6 +7,7 @@
 from ..async_typer import AsyncTyper
 from ..ctl.client import initialize_client
 from ..ctl.utils import catch_exception, init_logging
+from ..exceptions import ValidationError
 from ..spec.object import ObjectFile
 from .parameters import CONFIG_PARAM
 from .utils import load_yamlfile_from_disk_and_exit
@@ -39,8 +40,21 @@ async def load(
     files = load_yamlfile_from_disk_and_exit(paths=paths, file_type=ObjectFile, console=console)
     client = initialize_client()
 
+    has_errors = False
+
     for file in files:
-        await file.validate_format(client=client, branch=branch)
+        try:
+            await file.validate_format(client=client, branch=branch)
+        except ValidationError as e:
+            has_errors = True
+            if file.multiple_documents:
+                console.print(f"[red] File '{file.location}' [{file.document_position}] is not valid!")
+            else:
+                console.print(f"[red] File '{file.location}' is not valid!")
+            console.print(f"[red] {e.message}")
+
+    if has_errors:
+        raise typer.Exit(1)
 
     for file in files:
         await file.process(client=client, branch=branch)
@@ -63,5 +77,21 @@ async def validate(
     files = load_yamlfile_from_disk_and_exit(paths=paths, file_type=ObjectFile, console=console)
     client = initialize_client()
 
+    has_errors = False
     for file in files:
-        await file.validate_format(client=client, branch=branch)
+        try:
+            await file.validate_format(client=client, branch=branch)
+            if file.multiple_documents:
+                console.print(f"[green] File '{file.location}' [{file.document_position}] is Valid!")
+            else:
+                console.print(f"[green] File '{file.location}' is Valid!")
+        except ValidationError as e:
+            has_errors = True
+            if file.multiple_documents:
+                console.print(f"[red] File '{file.location}' [{file.document_position}] is not valid!")
+            else:
+                console.print(f"[red] File '{file.location}' is not valid!")
+            console.print(f"[red] {e.message}")
+
+    if has_errors:
+        raise typer.Exit(1)