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

test: more RDBMS integration tests in CI #2286

Merged
merged 35 commits into from
Mar 30, 2023
Merged
Show file tree
Hide file tree
Changes from 33 commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
a2611c6
-added docker compose
Jelenkee Mar 18, 2023
a60b835
-added dependencies
Jelenkee Mar 18, 2023
131f98b
-added real test cases
Jelenkee Mar 18, 2023
e183c69
-added mssql
Jelenkee Mar 18, 2023
2024146
refactoring
Jelenkee Mar 19, 2023
3063a48
-added go-task
Jelenkee Mar 21, 2023
454b51f
-more testcases
Jelenkee Mar 21, 2023
384e95c
-added github workflow
Jelenkee Mar 21, 2023
5ac360e
Merge branch 'main' into integration-tests
Jelenkee Mar 22, 2023
7a962a6
-fixed formatting
Jelenkee Mar 22, 2023
cf70d16
-fixed linting
Jelenkee Mar 22, 2023
2f221ee
-fixed workflow yaml
Jelenkee Mar 22, 2023
f347697
-fixed workflow yaml again
Jelenkee Mar 22, 2023
3fbb495
-added more testcases
Jelenkee Mar 24, 2023
fb585b9
Merge branch 'PRQL:main' into integration-tests
Jelenkee Mar 25, 2023
7264d69
Apply suggestions from code review
Jelenkee Mar 25, 2023
cc4a623
-refactored test workflow
Jelenkee Mar 25, 2023
b35f92e
Fix order of args to `assert_display_snapshot`
max-sixty Mar 25, 2023
f7ef223
Specify platform for mysql image
max-sixty Mar 25, 2023
1334ae2
Merge branch 'main' into integration-tests
Jelenkee Mar 26, 2023
ba803d7
-added csv import
Jelenkee Mar 26, 2023
2887814
-fixed or skipped all prql files
Jelenkee Mar 28, 2023
290d644
Merge branch 'main' into integration-tests
Jelenkee Mar 28, 2023
9d512f8
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Mar 28, 2023
02b42d1
-fixed sqlite
Jelenkee Mar 28, 2023
0c1b74f
added snaps§
Jelenkee Mar 28, 2023
ab7dd16
Merge branch 'main' into integration-tests
max-sixty Mar 28, 2023
58958d3
Update prql-compiler/prqlc/src/cli.rs
max-sixty Mar 28, 2023
eb56f7a
-removed test on macOS
Jelenkee Mar 29, 2023
5c5aa21
empty
Jelenkee Mar 29, 2023
b936ae5
-skip integration test on windows
Jelenkee Mar 29, 2023
6b5b099
-fixed taskfile
Jelenkee Mar 29, 2023
8a39f9f
-added double quotes
Jelenkee Mar 29, 2023
b9a5f46
Update .github/workflows/test-rust.yaml
max-sixty Mar 29, 2023
9a38890
Merge branch 'main' into integration-tests
max-sixty Mar 30, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/workflows/test-rust.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,19 @@ jobs:
with:
command: test
args: ${{ inputs.target_option }} --no-run --locked
- name: Run docker compose
run: docker compose up -d
working-directory: ./prql-compiler/tests/integration-rdbms
if: ${{ inputs.os == 'ubuntu-latest' }}
- name: Wait for database
uses: ifaxity/wait-on-action@v1
with:
resource: "tcp:5432 tcp:3306 tcp:1433"
timeout: 60000
if: ${{ inputs.os == 'ubuntu-latest' }}
- name: skip integration if not linux
run: echo "SKIP_INTEGRATION=true" >> $GITHUB_ENV
max-sixty marked this conversation as resolved.
Show resolved Hide resolved
if: ${{ inputs.os != 'ubuntu-latest' }}
# Only check unreferenced snapshots on the default target tests on ubuntu
#
# (Maybe there's a nicer approach where we can parameterize one step
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/test-taskfile.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-taskfile
cancel-in-progress: true

env:
SKIP_INTEGRATION: true

jobs:
test-taskfile:
runs-on: macos-latest
Expand Down
Loading