-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
route logs to dbt-core/logs instead of each test folder #4711
Conversation
core/dbt/tests/fixtures/project.py
Outdated
@@ -12,6 +15,8 @@ | |||
|
|||
# These are the fixtures that are used in dbt core functional tests | |||
|
|||
INITIAL_ROOT = os.getcwd() |
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.
There's a builtin pytest fixture thing that you can use instead of this INITIAL_ROOT variable: request.config.rootdir. If you add the 'request' fixture to the signature of the 'logs_dir' fixture, that should work.
…into er/testing-rewrite-logging # Conflicts: # core/dbt/tests/fixtures/project.py
db991c9
to
507184f
Compare
Git seems confused because of my force push. I'm not sure what the best way to fix it is... I'd probably create a clean branch off of testing_rewrite and cherry-pick your commits onto that. |
@gshank I seem to have resolved the conflicts. |
Author: Emily Rockman <emily.rockman@dbtlabs.com> route logs to dbt-core/logs instead of each test folder (#4711) * Initial pass at switching integration tests to pytest * Reorganize dbt.tests.tables. Cleanup adapter handling * Move run_sql to TestProjInfo and TableComparison. Add comments, cleanup adapter schema setup * Tweak unique_schema name generation * Update CHANGELOG.md
resolves #4700
Description
Logs were routing to the folders the projects were crated in. I added the log-path config to the project so that logs are all written to
dbt-core/logs
now, same as where they were written with the current tests.Checklist
CHANGELOG.md
and added information about my change