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

Add make command to run a specific e2e test #1547

Merged
merged 9 commits into from
Jan 6, 2025

Conversation

swang392
Copy link
Contributor

@swang392 swang392 commented Dec 4, 2024

What does this PR do?

Adds a new make command to run a specific E2E test by setting the E2E_RUN_REGEX environment variable in your terminal to the name of the desired test.
Removes parentheses from E2E test names

Motivation

Sometimes we don't want to run the entire E2E test suite, especially when trying to check or develop one test.

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

  • Agent: vX.Y.Z
  • Cluster Agent: vX.Y.Z

Describe your test plan

  • set E2E_RUN_REGEX in your terminal window to the full name of an E2E test. ex: export E2E_RUN_REGEX=TestLocalKindSuite/TestGenericK8s/Minimal_DDA_config
  • run make e2e-tests
  • check that only the test you specified has run. You should see a line printing out the value of E2E_RUN_REGEX in the output.
  • remove the variable you set for E2E_RUN_REGEX -- export E2E_RUN_REGEX=
  • run make e2e-tests
  • check that all of the e2e tests run

Checklist

  • PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
  • PR has a milestone or the qa/skip-qa label

@swang392 swang392 added this to the v1.12.0 milestone Dec 4, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.94%. Comparing base (a408715) to head (f235750).

Additional details and impacted files

Impacted file tree graph

@@                   Coverage Diff                    @@
##           fanny/refactor-kind-e2e    #1547   +/-   ##
========================================================
  Coverage                    48.94%   48.94%           
========================================================
  Files                          227      227           
  Lines                        20636    20636           
========================================================
  Hits                         10101    10101           
  Misses                       10010    10010           
  Partials                       525      525           
Flag Coverage Δ
unittests 48.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a408715...f235750. Read the comment docs.

@fanny-jiang fanny-jiang changed the base branch from fanny/refactor-kind-e2e to fanny/refactor-e2e January 2, 2025 22:11
@fanny-jiang fanny-jiang changed the base branch from fanny/refactor-e2e to fanny/refactor-kind-e2e January 2, 2025 22:12
Makefile Outdated Show resolved Hide resolved
@swang392 swang392 marked this pull request as ready for review January 3, 2025 19:52
@swang392 swang392 requested a review from a team as a code owner January 3, 2025 19:52
@swang392 swang392 merged commit 8ad0c66 into fanny/refactor-kind-e2e Jan 6, 2025
20 checks passed
@swang392 swang392 deleted the swang392/e2e-refactor branch January 6, 2025 16:32
@khewonc khewonc removed this from the v1.12.0 milestone Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants