Add two new decorators to replace the force skip option

[nccatoni]

Motivation

Changes

Workflow

1. ⚠️ Create your PR as draft ⚠️
2. Work on you PR until the CI passes
3. Mark it as ready for review
   • Test logic is modified? -> Get a review from RFC owner.
   • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

• Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
• A docker base image is modified?
  • the relevant build-XXX-image label is present
• A scenario is added, removed or renamed?
  • Get a review from R&P team

[github-actions]

CODEOWNERS have been resolved as:

tests/test_the_test/test_slow_and_crash_decorators.py                   @DataDog/system-tests-core
conftest.py                                                             @DataDog/system-tests-core
pyproject.toml                                                          @DataDog/system-tests-core
utils/__init__.py                                                       @DataDog/system-tests-core
utils/_decorators.py                                                    @DataDog/system-tests-core

[datadog-official]

⚠️ Tests

✨ Fix all issues with Cursor

⚠️ Warnings

❄️ 2 New flaky tests detected

tests.debugger.test_debugger_probe_snapshot.Test_Debugger_Method_Probe_Snaphots_With_SCM.test_mix_snapshot[poc] from system_tests_suite (Datadog) (Fix with Cursor)

ValueError: Snapshot log6c01d-cba3-49d8-805c-792971e93e75 was not received.

self = <tests.debugger.test_debugger_probe_snapshot.Test_Debugger_Method_Probe_Snaphots_With_SCM object at 0x7fc2794c2c30>

    @missing_feature(context.library == "nodejs", reason="Not yet implemented", force_skip=True)
    @missing_feature(context.library == "golang", reason="Not yet implemented", force_skip=True)
    def test_mix_snapshot(self):
        self._assert()
>       self._validate_snapshots()

...

tests.debugger.test_debugger_probe_snapshot.Test_Debugger_Method_Probe_Snaphots_With_SCM.test_mix_snapshot[spring-boot-openliberty] from system_tests_suite (Datadog) (Fix with Cursor)

ValueError: Snapshot logec865-d3ad-455f-9c2b-590e96d64adf was not received.

self = <tests.debugger.test_debugger_probe_snapshot.Test_Debugger_Method_Probe_Snaphots_With_SCM object at 0x7f27684556a0>

    @missing_feature(context.library == "nodejs", reason="Not yet implemented", force_skip=True)
    @missing_feature(context.library == "golang", reason="Not yet implemented", force_skip=True)
    def test_mix_snapshot(self):
        self._assert()
>       self._validate_snapshots()

...

ℹ️ Info

🧪 All tests passed

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ca3e14e | Docs | Datadog PR Page | Was this helpful? Give us feedback!}

[cbeauchesne]

Could you write some test?

By reading your code, I think it does not do what we expect (skip if it's annotated with flaky/bug/missing_feature), but inconditionally skip the test.

[cbeauchesne]

I like this !

[Copilot]

Pull request overview

This PR introduces two new decorators (@slow and @scenario_crash) that replace the force_skip option in existing decorators like @bug and @missing_feature. These decorators leverage pytest's skip_if_xfail marker to conditionally skip tests only when combined with a declaration marker (bug, missing_feature, etc.).

Changes:

• Added @slow and @scenario_crash decorators as aliases to pytest.mark.skip_if_xfail
• Updated conftest.py to skip tests that have both skip_if_xfail and declaration markers
• Added comprehensive test coverage for the new decorators

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
utils/_decorators.py	Defines the new slow and scenario_crash decorators
utils/init.py	Exports the new decorators in the public API
tests/test_the_test/test_slow_and_crash_decorators.py	Comprehensive test suite validating decorator behavior
pyproject.toml	Registers the new skip_if_xfail pytest marker
conftest.py	Implements skip logic for tests with both markers

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.