From 7522a5f0e8ec84ed362c6ef3c14886870b83d510 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADtor=20De=20Ara=C3=BAjo?= Date: Thu, 6 Feb 2025 10:19:23 +0000 Subject: [PATCH] tag renaming --- ddtrace/internal/ci_visibility/recorder.py | 8 -------- tests/ci_visibility/test_quarantine.py | 2 +- tests/contrib/pytest/test_pytest_quarantine.py | 4 ++-- 3 files changed, 3 insertions(+), 11 deletions(-) diff --git a/ddtrace/internal/ci_visibility/recorder.py b/ddtrace/internal/ci_visibility/recorder.py index 55981b058b5..1d6d51d0eea 100644 --- a/ddtrace/internal/ci_visibility/recorder.py +++ b/ddtrace/internal/ci_visibility/recorder.py @@ -463,14 +463,6 @@ def is_test_management_enabled(cls): os.getenv("DD_TEST_MANAGEMENT_ENABLED", default=True) ) - @classmethod - def should_skip_quarantined_tests(cls): - if cls._instance is None: - return False - return cls._instance._api_settings.quarantine.skip_quarantined_tests and asbool( - os.getenv("DD_TEST_MANAGEMENT_ENABLED", default=True) - ) - @classmethod def should_collect_coverage(cls): return cls._instance._api_settings.coverage_enabled or asbool( diff --git a/tests/ci_visibility/test_quarantine.py b/tests/ci_visibility/test_quarantine.py index ad15a9ce698..f22a01f938c 100644 --- a/tests/ci_visibility/test_quarantine.py +++ b/tests/ci_visibility/test_quarantine.py @@ -53,4 +53,4 @@ def test_quarantine_tags_set(self): session.finish() assert test._span.get_tag("test.management.is_quarantined") == "true" - assert session._span.get_tag("test_session.quarantine.enabled") == "true" + assert session._span.get_tag("test.test_management.enabled") == "true" diff --git a/tests/contrib/pytest/test_pytest_quarantine.py b/tests/contrib/pytest/test_pytest_quarantine.py index 44fb7360e66..6927b358374 100644 --- a/tests/contrib/pytest/test_pytest_quarantine.py +++ b/tests/contrib/pytest/test_pytest_quarantine.py @@ -263,7 +263,7 @@ def test_quarantine_spans_without_atr(self): spans = self.pop_spans() [session_span] = _get_spans_from_list(spans, "session") - assert session_span.get_tag("test_session.quarantine.enabled") == "true" + assert session_span.get_tag("test.test_management.enabled") == "true" [module_span] = _get_spans_from_list(spans, "module") [suite_span_fail_quarantined] = _get_spans_from_list(spans, "suite", "test_fail_quarantined.py") @@ -296,7 +296,7 @@ def test_quarantine_spans_with_atr(self): spans = self.pop_spans() [session_span] = _get_spans_from_list(spans, "session") - assert session_span.get_tag("test_session.quarantine.enabled") == "true" + assert session_span.get_tag("test.test_management.enabled") == "true" [module_span] = _get_spans_from_list(spans, "module") [suite_span_fail_quarantined] = _get_spans_from_list(spans, "suite", "test_fail_quarantined.py")