fixed params naming

airtai · Oct 19, 2024 · a9a0156 · a9a0156
1 parent e5bdc42
commit a9a0156
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/tests/prometheus/basic.py b/tests/prometheus/basic.py
@@ -43,15 +43,15 @@ def settings_provider_factory(self):
             pytest.param(
                 AckStatus.acked,
                 RejectMessage,
-                id="acked_status_with_reject_message_exception",
+                id="acked status with reject message exception",
             ),
             pytest.param(
-                AckStatus.acked, Exception, id="acked_status_with_not_handler_exception"
+                AckStatus.acked, Exception, id="acked status with not handler exception"
             ),
-            pytest.param(AckStatus.acked, None, id="acked_status_without_exception"),
-            pytest.param(AckStatus.nacked, None, id="nacked_status_without_exception"),
+            pytest.param(AckStatus.acked, None, id="acked status without exception"),
+            pytest.param(AckStatus.nacked, None, id="nacked status without exception"),
             pytest.param(
-                AckStatus.rejected, None, id="rejected_status_without_exception"
+                AckStatus.rejected, None, id="rejected status without exception"
             ),
         ],
     )

diff --git a/tests/prometheus/test_metrics.py b/tests/prometheus/test_metrics.py
@@ -98,8 +98,8 @@ def test_add_received_message(
     @pytest.mark.parametrize(
         "is_default_buckets",
         [
-            pytest.param(True, id="with_default_buckets"),
-            pytest.param(False, id="with_custom_buckets"),
+            pytest.param(True, id="with default buckets"),
+            pytest.param(False, id="with custom buckets"),
         ],
     )
     def test_observe_received_messages_size(
@@ -260,11 +260,11 @@ def test_remove_received_message_in_process(
     @pytest.mark.parametrize(
         "status",
         [
-            pytest.param(ProcessingStatus.acked, id="acked_status"),
-            pytest.param(ProcessingStatus.nacked, id="nacked_status"),
-            pytest.param(ProcessingStatus.rejected, id="rejected_status"),
-            pytest.param(ProcessingStatus.skipped, id="skipped_status"),
-            pytest.param(ProcessingStatus.error, id="error_status"),
+            pytest.param(ProcessingStatus.acked, id="acked status"),
+            pytest.param(ProcessingStatus.nacked, id="nacked status"),
+            pytest.param(ProcessingStatus.rejected, id="rejected status"),
+            pytest.param(ProcessingStatus.skipped, id="skipped status"),
+            pytest.param(ProcessingStatus.error, id="error status"),
         ],
     )
     def test_add_received_processed_message(
@@ -457,8 +457,8 @@ def test_add_received_processed_message_exception(
     @pytest.mark.parametrize(
         "status",
         [
-            pytest.param(PublishingStatus.success, id="success_status"),
-            pytest.param(PublishingStatus.error, id="error_status"),
+            pytest.param(PublishingStatus.success, id="success status"),
+            pytest.param(PublishingStatus.error, id="error status"),
         ],
     )
     def test_add_published_message(