Skip to content

Commit

Permalink
attempt to fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
dwreeves committed Oct 29, 2023
1 parent 8d42ab9 commit 2cda884
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 7 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,20 @@ on:
branches:
- main
jobs:
test-duckdb:
pre-commit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v4
- uses: pre-commit/[email protected]
integration-tests:
runs-on: ubuntu-latest
services:
postgres:
image: postgres
env:
POSTGRES_PASSWORD: postgres
POSTGRES_DB: postgres
options: >-
--health-cmd pg_isready
--health-interval 10s
Expand All @@ -24,11 +31,10 @@ jobs:
dbt_core: [1.4.*, 1.6.*]
db_target: [dbt-duckdb, dbt-postgres]
steps:
- uses: actions/checkout@v1
- uses: actions/setup-python@v1
- uses: actions/checkout@v4
- uses: actions/setup-python@v4
with:
python-version: "3.10"
architecture: x64
- name: Install Poetry
uses: snok/install-poetry@v1
with:
Expand All @@ -45,9 +51,9 @@ jobs:
env:
DBT_CORE_VERSION: ${{ matrix.dbt_core }}
DBT_PROVIDER_PACKAGE: ${{ matrix.db_target }}
- name: Lint
run: ./run lint
- name: Test
run: ./run test "${DBT_TARGET}"
env:
DBT_TARGET: ${{ matrix.db_target }}
POSTGRES_USER: postgres
POSTGRES_PASSWORD: postgres
2 changes: 1 addition & 1 deletion integration_tests/profiles/profiles.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ dbt_linreg_profile:
type: postgres
user: '{{ env_var("POSTGRES_USER", "postgres") }}'
password: '{{ env_var("POSTGRES_PASSWORD", "postgres") }}'
host: '{{ env_var("POSTGRES_HOST", "postgres") }}'
host: '{{ env_var("POSTGRES_HOST", "localhost") }}'
port: '{{ env_var("POSTGRES_PORT", "5432") | as_number }}'
dbname: '{{ env_var("POSTGRES_DB", "postgres") }}'
schema: '{{ env_var("POSTGRES_SCHEMA", "public") }}'

0 comments on commit 2cda884

Please sign in to comment.