Skip to content

Commit

Permalink
fix failing test_cancel in 3.12 (#1322)
Browse files Browse the repository at this point in the history
* use dynamic schema in test_grant_access_to.py

* use dynamic schema in test_grant_access_to.py

* revert setup

* Specify env in test_cancel subprocess call

* remove chdir from test_cancel.py

* pip install dev-requirements.txt in github actions worker

* experimental updates to debug in github actions

* experimental updates to debug in github actions

* specify dbt exec path

* experiment

* remove editable install

* remove tmate

* remove maxfail

* remove unnecessary install
  • Loading branch information
colin-rogers-dbt authored Aug 21, 2024
1 parent 21aeca3 commit 01cb679
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion tests/functional/test_cancel.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def _get_info_schema_jobs_query(project_id, dataset_id, table_id):


def _run_dbt_in_subprocess(project, dbt_command):
os.chdir(project.project_root)

run_dbt_process = subprocess.Popen(
[
"dbt",
Expand All @@ -76,6 +76,7 @@ def _run_dbt_in_subprocess(project, dbt_command):
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
shell=False,
env=os.environ.copy(),
)
std_out_log = ""
while True:
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ commands =
bigquery: {envpython} -m pytest -n auto {posargs} -vv tests/functional -k "not TestPython" --profile service_account
deps =
-rdev-requirements.txt
-e.
.

[testenv:{python-tests,py38,py39,py310,py311,py312,py}]
description = python integration testing
Expand Down

0 comments on commit 01cb679

Please sign in to comment.