PR by Bits for Dev Agent Session.

You can ask for changes by mentioning @DataDog in a comment.

Feedback (especially what can be better) welcome in #code-gen-feedback!

What does this PR do?

• Deflakes TestMultipleSpanIntegrationTags by replacing a timing-sensitive tg.Wait call with assert.Eventually that waits for both spans_started and spans_finished to reach 10 within 1s (polling every 10ms).
• Keeps the final equality assertions unchanged once the expected counts are observed.

Motivation

• CI showed intermittent failures where counts were off by one (expected 10, actual 9 and expected 5, actual 4), indicating the test asserted before the asynchronous metrics aggregation had fully caught up.
• The tracer’s metrics emission is asynchronous; flush alone wasn’t sufficient to guarantee the counts had been recorded when tg.Wait returned.
• Using assert.Eventually ensures the test only proceeds once both counters reach their expected values, eliminating the race and stabilizing the test.
• Related failure: https://app.datadoghq.com/ci/test/AwAAAZmlrr_OlXRWBAAAABhBWm1scnJfT0FBQnBudTZZMmJpODRyWmYAAAAkZjE5OWE1YWYtMTk1Zi00MDE1LTg0OWItODRmNDE3MWNmM2RiAAcvXA

Reviewer's Checklist

• Changed code has unit tests for its functionality at or near 100% coverage.
• System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
• There is a benchmark for any new code, or changes to existing code.
• If this interacts with the agent in a new way, a system test has been added.
• New code is free of linting errors. You can check this by running ./scripts/lint.sh locally.
• Add an appropriate team label so this PR gets put in the right place for the release notes.
• Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!