From 45e64eacf9bc5c66b954d31521366f8288ea4a7b Mon Sep 17 00:00:00 2001
From: arvidn <arvid@libtorrent.org>
Date: Fri, 16 Apr 2021 14:14:50 +0200
Subject: [PATCH] measure test coverage and post to coveralls

---
 .github/workflows/coverage.yml | 77 ++++++++++++++++++++++++++++++++++
 src/number.rs                  |  8 ----
 2 files changed, 77 insertions(+), 8 deletions(-)
 create mode 100644 .github/workflows/coverage.yml

diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
new file mode 100644
index 00000000..445a85f7
--- /dev/null
+++ b/.github/workflows/coverage.yml
@@ -0,0 +1,77 @@
+
+name: Test Coverage
+
+on:
+  push:
+    branches:
+      - main
+      - dev
+    tags:
+        - '**'
+  pull_request:
+    branches:
+      - '**'
+
+jobs:
+  coverage:
+    name: Test Coverage
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        fetch-depth: 1
+
+    - uses: actions/setup-python@v2
+      name: Install Python 3.9
+      with:
+        python-version: 3.9
+
+    - name: Update pip
+      run: |
+          python -m pip install --upgrade pip
+
+    - name: Set up rust
+      uses: actions-rs/toolchain@v1
+      with:
+        toolchain: nightly
+        default: true
+        profile: minimal
+        components: llvm-tools-preview
+
+    - name: Install dependencies
+      run: |
+        python -m venv venv
+        ln -s venv/bin/activate
+        . ./activate
+
+        git clone https://github.com/Chia-Network/clvm_tools.git --branch=develop --single-branch
+        git clone https://github.com/Chia-Network/clvm.git --branch=develop --single-branch
+        python -m pip install ./clvm_tools
+        python -m pip install ./clvm
+        python -m pip install colorama
+        python -m pip install maturin
+        python -m pip install pytest
+
+        rustup target add x86_64-unknown-linux-musl
+        cargo install grcov
+        cargo install cargo-binutils
+
+    - name: Build
+      run: |
+        . ./activate
+        RUSTFLAGS="-Zinstrument-coverage" maturin develop
+
+    - name: Run tests
+      run: |
+        . ./activate
+        LLVM_PROFILE_FILE="pytest.profraw" pytest clvm/tests
+        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
+
+    - name: Coveralls
+      uses: coverallsapp/github-action@master
+      with:
+        path-to-lcov: ./lcov.info
+        github-token: ${{ secrets.GITHUB_TOKEN }}
+
diff --git a/src/number.rs b/src/number.rs
index af9b12e8..4cd69ec9 100644
--- a/src/number.rs
+++ b/src/number.rs
@@ -1,5 +1,4 @@
 use crate::allocator::Allocator;
-use crate::node::Node;
 use crate::reduction::EvalErr;
 
 use num_bigint::BigInt;
@@ -22,13 +21,6 @@ pub fn ptr_from_number<T: Allocator>(
     allocator.new_atom(&slice)
 }
 
-impl<T: Allocator> From<&Node<'_, T>> for Option<Number> {
-    fn from(item: &Node<T>) -> Self {
-        let v: &[u8] = &item.atom()?;
-        Some(number_from_u8(v))
-    }
-}
-
 pub fn number_from_u8(v: &[u8]) -> Number {
     let len = v.len();
     if len == 0 {