Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

datadog-ci synthetics run-tests --failOnMissingTests does not fail on missing tests #1103

Closed
PSanetra opened this issue Nov 7, 2023 · 2 comments
Labels
enhancement New feature or request synthetics Related to [synthetics]

Comments

@PSanetra
Copy link

PSanetra commented Nov 7, 2023

Bug description

Specifying --failOnMissingTests on datadog-ci synthetics run-tests should result in a failed CI pipeline, but even if there are no tests found, the pipeline succeeds. 😕

Describe what you expected

datadog-ci synthetics run-tests should return a non 0 exit code if --failOnMissingTests and no tests were executed.

Steps to reproduce the issue

The following command was executed in a GitLab CI Job:

$ datadog-ci synthetics run-tests --failOnMissingTests --datadogSite datadoghq.eu --apiKey "$DATADOG_SYNTHETICS_API_KEY" --appKey "$DATADOG_SYNTHETICS_APP_KEY" -s "non_existing_tag:non_existing_value env:dev"
No test to run.
00:01
Job succeeded

Additional context

Docker Image: datadog/ci:v2.23.0

Command

synthetics

@PSanetra PSanetra added the bug Something isn't working label Nov 7, 2023
@github-actions github-actions bot added the synthetics Related to [synthetics] label Nov 7, 2023
@Drarig29
Copy link
Contributor

Drarig29 commented Nov 8, 2023

Hi @PSanetra!

Thanks for reporting! When we added this flag, we didn't want to make side effects on Datadog (e.g. a deleted test) turn a lot of CI pipelines red, because of tests that are not explicitly listed in your repository.

But we can hear that referencing tests via explicit public IDs or via a -s query should work the same.

We created a card to tackle this 🙇

@Drarig29 Drarig29 added enhancement New feature or request and removed bug Something isn't working labels Nov 8, 2023
@Drarig29
Copy link
Contributor

We forgot to close this, this was released in v2.25.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request synthetics Related to [synthetics]
Projects
None yet
Development

No branches or pull requests

2 participants