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

Move end-to-end tests to dedicated test directory #3271

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jaegeral
Copy link
Collaborator

WARNING: This PR will likely have broken end2end tests for Github.

This pull request moves the end-to-end tests from ./end_to_end_tests to ./tests/end_to_end_tests. This change streamlines the project's folder structure by consolidating all test-related files within the tests directory. This improves organization and makes it clearer where different types of tests reside.

Specifically:

  • Moved the ./end_to_end_tests directory and its contents to ./tests/end_to_end_tests.
  • Updated any references to the old end-to-end test path within the codebase to reflect the new location. (If applicable, list specific files or modules that were updated. E.g., "Updated references in run_e2e_tests.sh and configuration files.")

This change aims to improve the overall project structure and maintainability. It does not introduce any new functionality or change existing behavior. Keeping all tests within the tests directory allows for better organization and discoverability of both unit and end-to-end tests.

Checks

  • All tests succeed.
  • Unit tests added.
  • e2e tests added.
  • Documentation updated.

Closing issues

Related to #3184

@jaegeral jaegeral added the Code Health Code health improvements label Jan 30, 2025
@jaegeral jaegeral requested a review from jkppr January 30, 2025 10:04
@jaegeral jaegeral marked this pull request as ready for review January 30, 2025 10:04
@jaegeral jaegeral marked this pull request as draft January 30, 2025 10:11
@jaegeral
Copy link
Collaborator Author

This might be a bit more involved then expected. I will keep the PR as draft for now.

In case it is open for one year, it shall be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Health Code health improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant