From c7795c9847a5ab638bae8b8d75434eaa2a3d27a0 Mon Sep 17 00:00:00 2001 From: arvidn Date: Fri, 16 Apr 2021 16:46:40 +0200 Subject: [PATCH] merge lcov separately, to be able to pass separate binary-path to grcov for unit tests and python binding --- .github/workflows/coverage.yml | 9 ++++++++- src/int_allocator.rs | 6 ------ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 17dd3b0e..e3ef3fcf 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -52,6 +52,7 @@ jobs: python -m pip install colorama python -m pip install maturin python -m pip install pytest + sudo apt install lcov rustup target add x86_64-unknown-linux-musl cargo install grcov @@ -66,8 +67,14 @@ jobs: run: | . ./activate LLVM_PROFILE_FILE="pytest.profraw" pytest clvm/tests + grcov . -s . --binary-path ./venv/lib/python3.9/site-packages/ -t lcov --branch --ignore-not-existing -o pytest.lcov.info --keep-only src + rm pytest.profraw + LLVM_PROFILE_FILE="unittest.profraw" RUSTFLAGS="-Zinstrument-coverage" cargo test --no-default-features - grcov . -s . --binary-path ./venv/lib/python3.9/site-packages/ -t lcov --branch --ignore-not-existing -o lcov.info --keep-only src + grcov . -s . --binary-path ./target/debug/build -t lcov --branch --ignore-not-existing -o unittest.lcov.info --keep-only src + + # merge the lcov files + lcov unittest.lcov.info pytest.lcov.info -o lcov.info - name: Coveralls uses: coverallsapp/github-action@master diff --git a/src/int_allocator.rs b/src/int_allocator.rs index 46efee60..a5483d0e 100644 --- a/src/int_allocator.rs +++ b/src/int_allocator.rs @@ -29,12 +29,6 @@ pub struct IntAllocator { atom_vec: Vec, } -impl Default for IntAllocator { - fn default() -> Self { - Self::new() - } -} - impl IntAllocator { pub fn new() -> Self { let mut r = IntAllocator {