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

Add tox-based Integration Testing support for this package #318

Open
emmyoop opened this issue Sep 9, 2024 · 2 comments · May be fixed by #319
Open

Add tox-based Integration Testing support for this package #318

emmyoop opened this issue Sep 9, 2024 · 2 comments · May be fixed by #319

Comments

@emmyoop
Copy link

emmyoop commented Sep 9, 2024

Description

dbt Labs is running integration tests for popular open source packages as part of continuous testing and release of dbt.

Add support for testing this package using tox, as described in Standardizing integration tests for popular dbt packages.

Acceptance Criteria

  1. Tests reside in the /integration_tests directory.
  2. Supported adapters are defined in supported_adapters.env at the project root. The variable SUPPORTED_ADAPTERS lists the supported adapters, separated by commas. (Note: Only adapters running in dbt Cloud are currently supported.)
  3. tox is used for running tests.
  4. Each adapter test is defined in the tox.ini file using the naming pattern testenv:dbt_integration_<ADAPTER>.
  5. The profiles.yml for integration tests uses specific naming conventions for the target and environment variables. Examples can be found in dbt-package-testing.

Who will this benefit?

This change will enable:

  • dbt Labs to test this package, ensuring that dbt Cloud customers can confidently receive continuous dbt updates ("Versionless")
  • dbt Labs to proactively identify any incompatibilities or functional regressions before they are released in final versions of dbt Core OSS.
  • Package maintainers to consistently test changes in the same manner as dbt Labs will test new releases.

Final Note

dbt Labs will commence running tests only after the new tests have been released and are available in the latest package version on the hub.

@clausherther
Copy link
Contributor

Thanks for the head up @emmyoop. Will you also be pushing a PR for these changes on both packages?

@jtcohen6 jtcohen6 linked a pull request Sep 16, 2024 that will close this issue
@jtcohen6
Copy link

@clausherther I've just opened:

Let me know what you think - happy to incorporate suggested changes.

(Thanks @emmyoop for helping me get set up!)

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 a pull request may close this issue.

3 participants