Skip to content

Conversation

@ykim-akamai
Copy link
Contributor

@ykim-akamai ykim-akamai commented Jan 20, 2025

📝 Description

In efforts to unify test commands across all repository, the commands are refactored in Makefile. Refer to "How to Test" Section

Also related GHA workflow yamls have been updated with the latest commands

✔️ How to Test

  • Integration test: make test-int
  • Unit test: make test-unit
  • Smoke test: make test-smoke

Integration test arguments and usage:

TEST_SUITE: Optional, specify a test suite (e.g. domains), Default to run everything if not set
TEST_CASE: Optional, specify a test case (e.g. 'test_create_a_domain')
TEST_ARGS: Optional, additional arguments for pytest (e.g. '-v' for verbose mode)

e.g.
make TEST_SUITE=domains test-int
make TEST_CASE=test_create_a_domain test-int
make TEST_CASE=test_create_a_domain TEST_ARGS="-v" test-int

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

@ykim-akamai ykim-akamai added testing for updates to the testing suite in the changelog. repo-ci-improvement for improvements in the repository or CI workflow in the changelog. labels Jan 20, 2025
@ykim-akamai ykim-akamai requested a review from a team as a code owner January 20, 2025 18:33
@ykim-akamai ykim-akamai requested review from ezilber-akamai and zliang-akamai and removed request for a team January 20, 2025 18:33
Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New test commands working locally!

@ykim-akamai ykim-akamai merged commit 45db566 into linode:dev Jan 27, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

repo-ci-improvement for improvements in the repository or CI workflow in the changelog. testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants