Skip to content

Commit

Permalink
Homebrew postgis is pg14 only, so splitting into new test file
Browse files Browse the repository at this point in the history
  • Loading branch information
zachasme committed Oct 2, 2023
1 parent 86c38bd commit 3b178fa
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 2 deletions.
51 changes: 51 additions & 0 deletions .github/workflows/test-macos-12.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
name: test-macos

on:
push:
branches: [main]
pull_request:
branches: [main]

jobs:
macos:
strategy:
matrix:
config: [Release, Debug]

runs-on: macos-12

steps:
- uses: actions/checkout@v3

- name: Cache Cargo
uses: actions/cache@v3
with:
path: ~/.cargo
key: ${{ runner.os }}-cargo-2023

- name: Install pg_validate_extupgrade
run: cargo install --locked --git https://github.com/rjuju/pg_validate_extupgrade.git

- name: Install PostgreSQL server dependencies
run: brew install postgis

- name: Generate
run: cmake -B build -DCMAKE_BUILD_TYPE=${{ matrix.config }}

- name: Build
run: cmake --build build --config ${{ matrix.config }}

- name: Install
run: sudo cmake --install build --component h3-pg --config ${{ matrix.config }}

- name: Prepare PostgreSQL
run: |
pg_ctl start -D /usr/local/var/postgresql@14 --wait
createdb runner
- name: Test
run: ctest --test-dir build --output-on-failure --build-config ${{ matrix.config }}

- name: Print regression diffs
if: ${{ failure() }}
run: cat build/*/regression.diffs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
strategy:
matrix:
config: [Release, Debug]
pg: [16, 15, 14]
pg: [16, 15, 14, 13, 12, 11]

runs-on: macos-13

Expand All @@ -28,7 +28,7 @@ jobs:
run: cargo install --locked --git https://github.com/rjuju/pg_validate_extupgrade.git

- name: Install PostgreSQL server dependencies
run: brew install postgres@${{ matrix.pg }} postgis
run: brew install postgres@${{ matrix.pg }}

- name: Generate
run: cmake -B build -DCMAKE_BUILD_TYPE=${{ matrix.config }}
Expand Down

0 comments on commit 3b178fa

Please sign in to comment.