From 6b19fb50305d9fd6fd403592043f98770eb0dfc5 Mon Sep 17 00:00:00 2001
From: Stefan de Reuver <9864602+Horofic@users.noreply.github.com>
Date: Thu, 18 Jul 2024 09:36:40 +0200
Subject: [PATCH] Add review suggestions

---
 acquire/utils.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/acquire/utils.py b/acquire/utils.py
index 41cc9de..c394867 100644
--- a/acquire/utils.py
+++ b/acquire/utils.py
@@ -329,9 +329,13 @@ def check_and_set_acquire_args(
         raise ValueError("--skip-parent can only be set with --children")
 
     if args.output_type == "zip" and args.compress_method not in ZIP_COMPRESSION_METHODS:
-        raise ValueError(f"Invalid compression method for zip: {ZIP_COMPRESSION_METHODS.keys()}")
+        raise ValueError(
+            f"Invalid compression method for zip, allowed are: {', '.join(ZIP_COMPRESSION_METHODS.keys())}"
+        )
     if args.output_type == "tar" and args.compress_method not in TAR_COMPRESSION_METHODS:
-        raise ValueError(f"Invalid compression method for tar: {TAR_COMPRESSION_METHODS.keys()}")
+        raise ValueError(
+            f"Invalid compression method for tar, allowed are: {', '.join(TAR_COMPRESSION_METHODS.keys())}"
+        )
 
 
 def get_user_name() -> str: