-
Notifications
You must be signed in to change notification settings - Fork 83
feat(integration-tests): Validate the running package component-wise with validate_package_running.
#1659
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
base: main
Are you sure you want to change the base?
feat(integration-tests): Validate the running package component-wise with validate_package_running.
#1659
Conversation
…ig file; address comments.
…er-level clp-config.yml
…E_CONFIGS other than clp-text and clp-json
…equired components.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…nning of fixture names.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
validate_package_running (and supporting code) to test the package.validate_package_running and supporting code to test the package.
validate_package_running and supporting code to test the package.validate_package_running.
Description
This PR introduces the
validate_package_runningfunction to check if the containerized components of the given package are all running after spin-up.validate_package_runninguseslist_running_containers_with_prefixto match each of the components in the package'scomponent_listwith at least one running docker component ("at least one" because it searches by name prefix, to account for components that may sometimes have multiples, like workers).PR dependencies
This PR depends on the following being merged:
clp-textandclp-jsonpackages. #1437component_listdata member withinPackageConfig. #1658Checklist
breaking change.
Validation performed
Ran the following command from
clp/integration-tests:uv run pytest -m 'package'Both the
clp-jsonandclp-texttests passed.