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

write tests for the base_images package #30661

Conversation

alafanechere
Copy link
Contributor

@alafanechere alafanechere commented Sep 21, 2023

What

Add tests on top of the base_images packages.
Reaching 57% coverage.
Auto run it in the CI

@vercel
Copy link

vercel bot commented Sep 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Sep 21, 2023 8:10pm

@alafanechere alafanechere force-pushed the augustin/declare-first-base-image branch from 1de4960 to 1533e75 Compare September 21, 2023 20:09
@alafanechere alafanechere force-pushed the augustin/09-21-write_tests_for_the_base_images_package branch from 95405c7 to ef228a7 Compare September 21, 2023 20:09
@@ -32,3 +32,15 @@ jobs:
sentry_dsn: ${{ secrets.SENTRY_AIRBYTE_CI_DSN }}
github_token: ${{ secrets.GH_PAT_MAINTENANCE_OCTAVIA }}
subcommand: "test airbyte-ci/connectors/pipelines"
- name: Run airbyte-ci/connectors/base_images tests
Copy link
Contributor

Choose a reason for hiding this comment

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

WOOO!!

Copy link
Contributor

@bnchrch bnchrch left a comment

Choose a reason for hiding this comment

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

These are great. Thank you. Approved

@alafanechere alafanechere merged commit 1557e0e into augustin/declare-first-base-image Sep 22, 2023
25 of 26 checks passed
@alafanechere alafanechere deleted the augustin/09-21-write_tests_for_the_base_images_package branch September 22, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants