Skip to content

Commit

Permalink
test: don't run spec tests with stack
Browse files Browse the repository at this point in the history
We cannot handle postgis and other postgresql dependencies with
stack.

Also delete test/with_tmp_db since it's no longer used.
  • Loading branch information
steve-chavez committed Jun 23, 2022
1 parent d87b622 commit 86c40e8
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 110 deletions.
12 changes: 0 additions & 12 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -150,17 +150,13 @@ jobs:
cache: |
~/.stack
.stack-work
test: true
pgdir: /usr/lib/postgresql
artifact: postgrest-ubuntu-x64

- name: MacOS & test
runs-on: macos-latest
cache: |
~/.stack
.stack-work
test: true
pgdir: /usr/local/Cellar/postgresql
artifact: postgrest-macos-x64

- name: Windows
Expand All @@ -170,8 +166,6 @@ jobs:
~\AppData\Local\Programs\stack
.stack-work
deps: Add-Content $env:GITHUB_PATH $env:PGBIN
# We'd need to make test/with_tmp_db run on Windows first
# test: true
artifact: postgrest-windows-x64

name: Build ${{ matrix.name }} (Stack)
Expand All @@ -188,12 +182,6 @@ jobs:
run: ${{ matrix.deps }}
- name: Build with Stack
run: stack build --local-bin-path result --copy-bins
- name: Run Spec tests with Stack
if: ${{ matrix.test }}
run: |
postgresql_bin="$(find ${{ matrix.pgdir }} -maxdepth 2 -type d -name bin | head -n 1)"
echo "Using PostgreSQL binaries at $postgresql_bin ..."
PATH="$postgresql_bin:$PATH" test/with_tmp_db stack test
- name: Save built executable as artifact
uses: actions/upload-artifact@v3
with:
Expand Down
3 changes: 1 addition & 2 deletions nix/tools/style.nix
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,7 @@ let
echo "Linting bash scripts..."
${shellcheck}/bin/shellcheck \
.github/get_cirrusci_freebsd \
.github/release \
test/with_tmp_db
.github/release
echo "Linting workflows..."
${actionlint}/bin/actionlint
Expand Down
96 changes: 0 additions & 96 deletions test/with_tmp_db

This file was deleted.

0 comments on commit 86c40e8

Please sign in to comment.