From 52ec480724d3f0f3b442ad8115ec4ee488cb7d1b Mon Sep 17 00:00:00 2001
From: Mikhail Cheshkov <mcheshkov@gmail.com>
Date: Thu, 23 Jan 2025 19:49:18 +0200
Subject: [PATCH] [WIP] try to combine llvm-cov and insta test run

---
 .github/workflows/rust-cubesql.yml | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/rust-cubesql.yml b/.github/workflows/rust-cubesql.yml
index aac7a8b11bd41..3b6eb0504976d 100644
--- a/.github/workflows/rust-cubesql.yml
+++ b/.github/workflows/rust-cubesql.yml
@@ -104,18 +104,11 @@ jobs:
           CUBESQL_REWRITE_TIMEOUT: 60
         run: |
           cd rust/cubesql
-          cargo llvm-cov --all-features --workspace --lcov --output-path lcov.info
-      - name: Check unreferenced insta snapshots in tests (Rewrite Engine)
-        env:
-          CUBESQL_TESTING_CUBE_TOKEN: ${{ secrets.CUBESQL_TESTING_CUBE_TOKEN }}
-          CUBESQL_TESTING_CUBE_URL: ${{ secrets.CUBESQL_TESTING_CUBE_URL }}
-          CUBESQL_SQL_PUSH_DOWN: true
-          CUBESQL_TOP_DOWN_EXTRACTOR: ${{ matrix.top-down-extractor }}
-          CUBESQL_REWRITE_CACHE: true
-          CUBESQL_REWRITE_TIMEOUT: 60
-        run: |
-          cd rust/cubesql
+          # See https://github.com/taiki-e/cargo-llvm-cov/blob/main/README.md#get-coverage-of-external-tests
+          # shellcheck source=/dev/null
+          source <(cargo llvm-cov show-env --export-prefix)
           cargo insta test --all-features --workspace --unreferenced reject
+          cargo llvm-cov report --all-features --workspace --lcov --output-path lcov.info
       - name: Upload code coverage
         uses: codecov/codecov-action@v4
         with: