Skip to content

Commit de99d94

Browse files
committed
[WIP] try to combine llvm-cov and insta test run
1 parent 1848739 commit de99d94

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

.github/workflows/rust-cubesql.yml

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -104,18 +104,11 @@ jobs:
104104
CUBESQL_REWRITE_TIMEOUT: 60
105105
run: |
106106
cd rust/cubesql
107-
cargo llvm-cov --all-features --workspace --lcov --output-path lcov.info
108-
- name: Check unreferenced insta snapshots in tests (Rewrite Engine)
109-
env:
110-
CUBESQL_TESTING_CUBE_TOKEN: ${{ secrets.CUBESQL_TESTING_CUBE_TOKEN }}
111-
CUBESQL_TESTING_CUBE_URL: ${{ secrets.CUBESQL_TESTING_CUBE_URL }}
112-
CUBESQL_SQL_PUSH_DOWN: true
113-
CUBESQL_TOP_DOWN_EXTRACTOR: ${{ matrix.top-down-extractor }}
114-
CUBESQL_REWRITE_CACHE: true
115-
CUBESQL_REWRITE_TIMEOUT: 60
116-
run: |
117-
cd rust/cubesql
107+
# See https://github.com/taiki-e/cargo-llvm-cov/blob/main/README.md#get-coverage-of-external-tests
108+
# shellcheck source=/dev/null
109+
source <(cargo llvm-cov show-env --export-prefix)
118110
cargo insta test --all-features --workspace --unreferenced reject
111+
cargo llvm-cov report --workspace --lcov --output-path lcov.info
119112
- name: Upload code coverage
120113
uses: codecov/codecov-action@v4
121114
with:

0 commit comments

Comments
 (0)