-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Metadata: Check that normalized files exist #26133
Conversation
efc679c
to
b938ede
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice additional check! Just minor pytest related suggestions.
airbyte-ci/connectors/metadata_service/lib/tests/fixtures/__init__.py
Outdated
Show resolved
Hide resolved
airbyte-ci/connectors/metadata_service/lib/tests/fixtures/__init__.py
Outdated
Show resolved
Hide resolved
return "exists" in image_name and "exists" in version | ||
|
||
|
||
def setup_upload_mocks(mocker, version_blob_md5_hash, latest_blob_md5_hash, local_file_md5_hash): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: in pytest the setup
teardown
pattern is usually declared through the use of fixtures.
Co-authored-by: Augustin <augustin@airbyte.io>
1173ed6
to
5627a63
Compare
* Check that normalized files exist * refactor fixtures * Mocks * Update tests * Format * Apply suggestions from code review Co-authored-by: Augustin <augustin@airbyte.io> * Revert path --------- Co-authored-by: Augustin <augustin@airbyte.io>
* Check that normalized files exist * refactor fixtures * Mocks * Update tests * Format * Apply suggestions from code review Co-authored-by: Augustin <augustin@airbyte.io> * Revert path --------- Co-authored-by: Augustin <augustin@airbyte.io>
What
Checks that the normalization images exist on metadata file upload
and adds tests to ensure were doing this check
closes #25538