Skip to content

Commit

Permalink
Merge pull request #247 from duckdb/jwills_whats_up_mother_duck
Browse files Browse the repository at this point in the history
See if I can isolate whatever issue is springing up with the MD integration tests
  • Loading branch information
jwills authored Sep 11, 2023
2 parents b14b66e + cf2420d commit 5ae8df0
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ['3.9', '3.10', '3.11']
python-version: ['3.9']

env:
TOXENV: "buenavista"
Expand Down
4 changes: 4 additions & 0 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,10 @@ def dbt_profile_target(profile_type, bv_server_process, tmp_path_factory):
profile["path"] = str(tmp_path_factory.getbasetemp() / "tmp.db")
elif profile_type == "md":
# Test against MotherDuck
if "MOTHERDUCK_TOKEN" not in os.environ:
raise ValueError(
"Please set the MOTHERDUCK_TOKEN environment variable to run tests against MotherDuck"
)
profile["disable_transactions"] = True
profile["path"] = "md:test"
elif profile_type == "memory":
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ deps =
-rdev-requirements.txt
-e.

[testenv:{buenavista,py39,py310,py311,py}]
[testenv:{buenavista,py39}]
description = adapter functional testing using a Buena Vista server
skip_install = True
passenv = *
Expand Down

0 comments on commit 5ae8df0

Please sign in to comment.