[arrow-adbc] branch main updated: ci: use pytest-error-for-skips for integration tests (#1177)

[lidavidm]

This is an automated email from the ASF dual-hosted git repository.

lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-adbc.git


The following commit(s) were added to refs/heads/main by this push:
     new c78398bb ci: use pytest-error-for-skips for integration tests (#1177)
c78398bb is described below

commit c78398bb7b76edf67ac456fca7aafcaebb514ce8
Author: David Li <li.david...@gmail.com>
AuthorDate: Fri Oct 6 15:05:49 2023 -0400

    ci: use pytest-error-for-skips for integration tests (#1177)
    
    Fixes #999.
---
 .github/workflows/integration.yml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/.github/workflows/integration.yml 
b/.github/workflows/integration.yml
index 4ae42c56..3995db9c 100644
--- a/.github/workflows/integration.yml
+++ b/.github/workflows/integration.yml
@@ -128,6 +128,7 @@ jobs:
           mamba install -c conda-forge \
             --file ci/conda_env_cpp.txt \
             --file ci/conda_env_python.txt
+          pip install pytest-error-for-skips
       - uses: actions/setup-go@v3
         with:
           go-version: 1.19.13
@@ -176,6 +177,7 @@ jobs:
           ADBC_DREMIO_FLIGHTSQL_USER: "dremio"
           ADBC_DREMIO_FLIGHTSQL_PASS: "dremio123"
           ADBC_TEST_FLIGHTSQL_URI: "grpc+tcp://localhost:41414"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           ./ci/scripts/python_test.sh "$(pwd)" "$(pwd)/build"
       - name: Stop SQLite server and Dremio
@@ -213,6 +215,7 @@ jobs:
           mamba install -c conda-forge \
             --file ci/conda_env_cpp.txt \
             --file ci/conda_env_python.txt
+          pip install pytest-error-for-skips
       - name: Build PostgreSQL Driver
         shell: bash -l {0}
         env:
@@ -237,6 +240,7 @@ jobs:
           ADBC_USE_ASAN: "ON"
           ADBC_USE_UBSAN: "ON"
           ADBC_POSTGRESQL_TEST_URI: 
"postgresql://localhost:5432/postgres?user=postgres&password=password"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           env POSTGRES_VERSION=11 docker compose up --wait --detach 
postgres-test
           ./ci/scripts/cpp_test.sh "$(pwd)/build"
@@ -250,6 +254,7 @@ jobs:
           ADBC_USE_ASAN: "ON"
           ADBC_USE_UBSAN: "ON"
           ADBC_POSTGRESQL_TEST_URI: 
"postgresql://localhost:5432/postgres?user=postgres&password=password"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           env POSTGRES_VERSION=12 docker compose up --wait --detach 
postgres-test
           ./ci/scripts/cpp_test.sh "$(pwd)/build"
@@ -263,6 +268,7 @@ jobs:
           ADBC_USE_ASAN: "ON"
           ADBC_USE_UBSAN: "ON"
           ADBC_POSTGRESQL_TEST_URI: 
"postgresql://localhost:5432/postgres?user=postgres&password=password"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           env POSTGRES_VERSION=13 docker compose up --wait --detach 
postgres-test
           ./ci/scripts/cpp_test.sh "$(pwd)/build"
@@ -276,6 +282,7 @@ jobs:
           ADBC_USE_ASAN: "ON"
           ADBC_USE_UBSAN: "ON"
           ADBC_POSTGRESQL_TEST_URI: 
"postgresql://localhost:5432/postgres?user=postgres&password=password"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           env POSTGRES_VERSION=14 docker compose up --wait --detach 
postgres-test
           ./ci/scripts/cpp_test.sh "$(pwd)/build"
@@ -289,6 +296,7 @@ jobs:
           ADBC_USE_ASAN: "ON"
           ADBC_USE_UBSAN: "ON"
           ADBC_POSTGRESQL_TEST_URI: 
"postgresql://localhost:5432/postgres?user=postgres&password=password"
+          PYTEST_ADDOPTS: "--error-for-skips"
         run: |
           env POSTGRES_VERSION=15 docker compose up --wait --detach 
postgres-test
           ./ci/scripts/cpp_test.sh "$(pwd)/build"