DataDog / system-tests

Test framework for libraries and agents.
Apache License 2.0
27 stars 9 forks source link

Add a test to verify that crashtracking properly times out #3339

Closed kevingosse closed 7 hours ago

kevingosse commented 5 days ago

Motivation

Add a test to verify that crashtracking properly times out when the telemetry endpoint takes too much time to reply.

Changes

Added a new test that sets an artificial delay on the telemetry endpoint, crashes the application, then verifies the container exits within 10 seconds.

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

kevingosse commented 4 days ago

Could we add the "test_crashtracking.py" to the manifest files instead of using the same decorator (missing_feature) for all method of this class?

Good call, done.