[test visibility] Add test source file to test suites

[juan-fernandez]

What does this PR do?

Add test.source.file, test.source.start and test.codeowners to test suites, in addition to tests, which already had those tags.

Motivation

Test suites are test files and it's useful for users to be able to look at test files in Datadog UI. This allows customer to do this.

Plugin Checklist

• Unit tests.

[github-actions]

Overall package size

Self size: 7.1 MB
Deduped: 62.47 MB
No deduping: 62.75 MB

Dependency sizes

| name | version | self size | total size | |------|---------|-----------|------------| | @datadog/native-appsec | 8.1.1 | 18.67 MB | 18.68 MB | | @datadog/native-iast-taint-tracking | 3.1.0 | 12.27 MB | 12.28 MB | | @datadog/pprof | 5.3.0 | 9.85 MB | 10.22 MB | | protobufjs | 7.2.5 | 2.77 MB | 5.16 MB | | @datadog/native-iast-rewriter | 2.4.1 | 2.14 MB | 2.23 MB | | @opentelemetry/core | 1.14.0 | 872.87 kB | 1.47 MB | | @datadog/native-metrics | 2.0.0 | 898.77 kB | 1.3 MB | | @opentelemetry/api | 1.8.0 | 1.21 MB | 1.21 MB | | jsonpath-plus | 9.0.0 | 580.4 kB | 1.03 MB | | import-in-the-middle | 1.8.1 | 71.67 kB | 785.15 kB | | msgpack-lite | 0.1.26 | 201.16 kB | 281.59 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | pprof-format | 2.1.0 | 111.69 kB | 111.69 kB | | @datadog/sketches-js | 2.1.0 | 109.9 kB | 109.9 kB | | semver | 7.6.3 | 95.82 kB | 95.82 kB | | lodash.sortby | 4.7.0 | 75.76 kB | 75.76 kB | | lru-cache | 7.14.0 | 74.95 kB | 74.95 kB | | ignore | 5.3.1 | 51.46 kB | 51.46 kB | | int64-buffer | 0.1.10 | 49.18 kB | 49.18 kB | | shell-quote | 1.8.1 | 44.96 kB | 44.96 kB | | istanbul-lib-coverage | 3.2.0 | 29.34 kB | 29.34 kB | | rfdc | 1.3.1 | 25.21 kB | 25.21 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | limiter | 1.1.5 | 23.17 kB | 23.17 kB | | dc-polyfill | 0.1.4 | 23.1 kB | 23.1 kB | | retry | 0.13.1 | 18.85 kB | 18.85 kB | | jest-docblock | 29.7.0 | 8.99 kB | 12.76 kB | | crypto-randomuuid | 1.0.0 | 11.18 kB | 11.18 kB | | koalas | 1.0.2 | 6.47 kB | 6.47 kB | | path-to-regexp | 0.1.10 | 6.38 kB | 6.38 kB | | module-details-from-path | 1.0.3 | 4.47 kB | 4.47 kB |

_{🤖 This report was automatically generated by heaviest-objects-in-the-universe}

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-09-13 10:55:05

Comparing candidate commit 896b0fb in PR branch juan-fernandez/add-test-source-to-suites with baseline commit 8b5b271 in branch master.

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.19%. Comparing base (bd2f8b2) to head (043a1f9).
Report is 10 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4677      +/-   ##
==========================================
+ Coverage   62.59%   69.19%   +6.59%     
==========================================
  Files          58        1      -57     
  Lines        2267      198    -2069     
  Branches       33       33              
==========================================
- Hits         1419      137    -1282     
+ Misses        848       61     -787     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[romainkomorndatadog]

LGTM but I feel like we have an implicit "suite maps to whole file" assumption and I wonder if we should have something more explicit.

Maybe specifying the file without listing a start line? 🤔

[juan-fernandez]

LGTM but I feel like we have an implicit "suite maps to whole file" assumption and I wonder if we should have something more explicit.

Maybe specifying the file without listing a start line? 🤔

if the suite maps to part of a file, wouldn't it work to pass the start line? If a suite maps to multiple files, this won't work, but we mentioned that source files in test suites would be best effort and the libraries would take the decision whether to include the tag or not, right?