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

Use, and extend as necessary, happy path fixture for tests/functional/artifacts/test_artifacts.py #10286

Closed
1 task done
Tracked by #9649
QMalcolm opened this issue Jun 10, 2024 · 1 comment
Closed
1 task done
Tracked by #9649
Assignees

Comments

@QMalcolm
Copy link
Contributor

QMalcolm commented Jun 10, 2024

Housekeeping

  • I am a maintainer of dbt-core

Short description

This ticket should build on the work from #10256. Specifically we want to update the tests in tests/functional/artifacts/test_artifacts.py to use the the happy path project fixture where relevant. This may require adding to the happy path fixture.

Acceptance criteria

  • We use the happy path project fixture for tests in tests/functional/artifacts/test_artifacts.py
@QMalcolm QMalcolm added Refinement Maintainer input needed and removed Refinement Maintainer input needed labels Jun 10, 2024
@ChenyuLInx ChenyuLInx self-assigned this Jun 18, 2024
@ChenyuLInx
Copy link
Contributor

After looking into this test, it is fairly involved with the models being executed. We decided to not convert it to using happy path fixture.

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

No branches or pull requests

2 participants