Skip to content

Commit 7f62851

Browse files
mcheshkovKSDaemon
authored andcommitted
[WIP] try to combine llvm-cov and insta test run
1 parent 72fe9c0 commit 7f62851

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
@@ -99,18 +99,11 @@ jobs:
9999
CUBESQL_REWRITE_TIMEOUT: 60
100100
run: |
101101
cd rust/cubesql
102-
cargo llvm-cov --all-features --workspace --lcov --output-path lcov.info
103-
- name: Check unreferenced insta snapshots in tests (Rewrite Engine)
104-
env:
105-
CUBESQL_TESTING_CUBE_TOKEN: ${{ secrets.CUBESQL_TESTING_CUBE_TOKEN }}
106-
CUBESQL_TESTING_CUBE_URL: ${{ secrets.CUBESQL_TESTING_CUBE_URL }}
107-
CUBESQL_SQL_PUSH_DOWN: true
108-
CUBESQL_TOP_DOWN_EXTRACTOR: ${{ matrix.top-down-extractor }}
109-
CUBESQL_REWRITE_CACHE: true
110-
CUBESQL_REWRITE_TIMEOUT: 60
111-
run: |
112-
cd rust/cubesql
102+
# See https://github.com/taiki-e/cargo-llvm-cov/blob/main/README.md#get-coverage-of-external-tests
103+
# shellcheck source=/dev/null
104+
source <(cargo llvm-cov show-env --export-prefix)
113105
cargo insta test --all-features --workspace --unreferenced reject
106+
cargo llvm-cov report --lcov --output-path lcov.info
114107
- name: Upload code coverage
115108
uses: codecov/codecov-action@v5
116109
with:

0 commit comments

Comments
 (0)