rules_fuzzing icon indicating copy to clipboard operation
rules_fuzzing copied to clipboard

Add coverage support to C++ regression tests

Open fmeum opened this issue 3 years ago • 1 comments

Co-authored-by: Zhen Yu Ding [email protected]

Closes #174

fmeum avatar Sep 28 '22 09:09 fmeum

@stefanbucur Could you review this? This differs from #174 in that the LCOV merger is no longer built without bazel coverage.

@zhenyudg I added you as a coauthor.

fmeum avatar Sep 28 '22 10:09 fmeum

@stefanbucur Friendly ping

fmeum avatar Nov 10 '22 16:11 fmeum

Thanks so much for the ping, it totally fell through the cracks. I'll take a look by EOD.

(PS: In general, please feel free to ping me if you don't get a review within a few days. Nowadays I'm spending less time on Github so things may easily slip off my attention.)

stefanbucur avatar Nov 10 '22 16:11 stefanbucur

LGTM, but we seem to have new test failures?

Looks like ~Bazel CI~ OSS-Fuzz replaced python with python3, which is incompatible with Bazel 4. I will try raising the minimum version and see whether that fixes the failure.

fmeum avatar Nov 14 '22 18:11 fmeum

@stefanbucur I updated the Bazel version to 5.0.0, which makes Bazel look for python3 in PATH. The CI run still fails, but when I run /usr/bin/env python3 in base-builder locally, that passes. Not sure what's wrong here.

fmeum avatar Nov 15 '22 03:11 fmeum

@stefanbucur I updated the Bazel version to 5.0.0, which makes Bazel look for python3 in PATH. The CI run still fails, but when I run /usr/bin/env python3 in base-builder locally, that passes. Not sure what's wrong here.

Interesting indeed 🤔 FWIW, the CI fuzzing test has been broken for about 3 weeks at head too: https://github.com/bazelbuild/rules_fuzzing/actions/workflows/oss_fuzz.yml

stefanbucur avatar Nov 16 '22 16:11 stefanbucur

@stefanbucur Tests are passing now.

fmeum avatar Dec 02 '22 08:12 fmeum