Skip to content

Conversation

nikita-tkachenko-datadog
Copy link
Contributor

What Does This Do

Adds Selenium module to the list of test instrumentations.

Motivation

Test instrumentations contain sample tests (that are not actual tests for the tracer, but rather fixtures that we use to verify that instrumentation works correctly).
These instrumentation modules need to be added to a special list: for the modules present additional configuration is done that excludes sample tests from being run.

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@nikita-tkachenko-datadog nikita-tkachenko-datadog requested a review from a team as a code owner September 12, 2025 13:39
@nikita-tkachenko-datadog nikita-tkachenko-datadog added comp: testing Testing tag: no release notes Changes to exclude from release notes labels Sep 12, 2025
@nikita-tkachenko-datadog nikita-tkachenko-datadog added the comp: ci visibility Continuous Integration Visibility label Sep 12, 2025
@nikita-tkachenko-datadog nikita-tkachenko-datadog changed the title Add Selenium module to list of test instrumentations to avoid running sample tests Add Selenium and Cucumber modules to list of test instrumentations to avoid running sample tests Sep 12, 2025
@datadog-datadog-prod-us1
Copy link
Contributor

datadog-datadog-prod-us1 bot commented Sep 12, 2025

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 60.15% (-0.02%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: b55b459 | Docs | Was this helpful? Give us feedback!

@nikita-tkachenko-datadog nikita-tkachenko-datadog merged commit 3b4a22b into master Sep 12, 2025
505 checks passed
@nikita-tkachenko-datadog nikita-tkachenko-datadog deleted the nikita-tkachenko/selenium-test-project branch September 12, 2025 14:40
@github-actions github-actions bot added this to the 1.54.0 milestone Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: ci visibility Continuous Integration Visibility comp: testing Testing tag: no release notes Changes to exclude from release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants