Skip to content

Commit d925607

Browse files
committed
Fix coverage test failure: correct Bazel workspace references and LLVM version
Signed-off-by: Matthieu MOREL <[email protected]>
1 parent af228be commit d925607

File tree

4 files changed

+6
-5
lines changed

4 files changed

+6
-5
lines changed

.bazelrc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ build:coverage --experimental_use_llvm_covmap
3737
build:coverage --experimental_generate_llvm_lcov
3838
build:coverage --collect_code_coverage
3939
build:coverage --instrumentation_filter="//source[/:],//cpp2sky[/:]"
40-
build:coverage --coverage_support=@cpp2sky//bazel/coverage:coverage_support
41-
build:coverage --test_env=CC_CODE_COVERAGE_SCRIPT=external/cpp2sky/bazel/coverage/collect_cc_coverage.sh
40+
build:coverage --coverage_support=//bazel/coverage:coverage_support
41+
build:coverage --test_env=CC_CODE_COVERAGE_SCRIPT=bazel/coverage/collect_cc_coverage.sh
4242
build:coverage --strategy=TestRunner=local
4343
build:coverage --strategy=CoverageReport=local
4444
build:coverage --experimental_use_llvm_covmap

.github/workflows/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,8 @@ jobs:
6666
run: |
6767
sudo apt update
6868
sudo apt -y install lcov
69-
sudo ln -s /usr/bin/llvm-profdata-11 /usr/bin/llvm-profdata
70-
sudo ln -s /usr/bin/llvm-cov-11 /usr/bin/llvm-cov
69+
sudo ln -s /usr/bin/llvm-profdata-16 /usr/bin/llvm-profdata
70+
sudo ln -s /usr/bin/llvm-cov-16 /usr/bin/llvm-cov
7171
7272
- name: Run coverage test
7373
run: |

bazel/coverage/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,5 @@ licenses(["notice"]) # Apache 2
33
filegroup(
44
name = "coverage_support",
55
srcs = ["collect_cc_coverage.sh"],
6+
visibility = ["//visibility:public"],
67
)

coverage.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ set -e
44

55
[[ -z "${SRCDIR}" ]] && SRCDIR="${PWD}"
66

7-
OUTPUT_DIR="./coverage_report/"
7+
OUTPUT_DIR="./coverage_report"
88
DATA_DIR="${SRCDIR}/bazel-testlogs/"
99
PROJECT=$(basename "${SRCDIR}")
1010

0 commit comments

Comments
 (0)