DataDog / dd-trace-py

Datadog Python APM Client
https://ddtrace.readthedocs.io/
Other
504 stars 396 forks source link

fix(ci_visibility): properly resolve decorated functions for test source file information [backport 2.8] #9643

Closed romainkomorndatadog closed 4 days ago

romainkomorndatadog commented 5 days ago

Backport 203e89489f1e698a127317dc57b01e9d6467e859 from #9586 to 2.8.

Fixes an issue where decorated test functions could resolve to the wrong location when certain decorators (eg: @mock.patch) were used.

This also fixes the fact that source file info paths might not always be relative to the current repo root.

The test added in this PR verifies the above by using a variety of decorators as well as executing from the nested_dir directory instead of the git repo root, with the source file info properly showing the path as nested_dir/test_mydecorators.py.

Checklist

Reviewer Checklist

pr-commenter[bot] commented 5 days ago

Benchmarks

Benchmark execution time: 2024-06-26 16:55:40

Comparing candidate commit 80f1bc19f1de11664cfc0443fff05d05266f4800 in PR branch backport-9586-to-2.8 with baseline commit bad58cc1d422464ac73a0e19dad6c667b9ffdf42 in branch 2.8.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 201 metrics, 9 unstable metrics.