Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

play around with codecov #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

real-or-random
Copy link
Owner

No description provided.

@real-or-random real-or-random force-pushed the 202106-wip-codecov branch 2 times, most recently from 8cf57f5 to 5c97e4b Compare June 17, 2021 09:05
@codecov
Copy link

codecov bot commented Jun 17, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@8ae56e3). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 1507148 differs from pull request most recent head 6b8dd8a. Consider uploading reports for the commit 6b8dd8a to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   46.28%           
=========================================
  Files             ?       45           
  Lines             ?     6888           
  Branches          ?        0           
=========================================
  Hits              ?     3188           
  Misses            ?     3700           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ae56e3...6b8dd8a. Read the comment docs.

@real-or-random real-or-random force-pushed the 202106-wip-codecov branch 7 times, most recently from 9447601 to 0965b2f Compare June 22, 2021 10:29
real-or-random added a commit that referenced this pull request Mar 10, 2025
… names

87384f5 cmake, test: Add `secp256k1_` prefix to test names (Hennadii Stepanov)

Pull request description:

  This PR improves regex matching options when using `ctest` in downstream projects, such as Bitcoin Core.

  For instance, a downstream project users can filter their tests like that:
  ```
  ctest --tests-regex "secp256k1"
  ```
  or
  ```
  ctest --exclude-regex "secp256k1"
  ```

  A `ctest` log with this PR:
  ```
  $ ctest --test-dir build -j 16
  Internal ctest changing into directory: /home/hebasto/git/secp256k1/secp256k1/build
  Test project /home/hebasto/git/secp256k1/secp256k1/build
      Start 1: secp256k1_noverify_tests
      Start 2: secp256k1_tests
      Start 3: secp256k1_exhaustive_tests
      Start 4: secp256k1_ecdsa_example
      Start 5: secp256k1_ecdh_example
      Start 6: secp256k1_schnorr_example
      Start 7: secp256k1_ellswift_example
      Start 8: secp256k1_musig_example
  1/8 Test bitcoin-core#4: secp256k1_ecdsa_example ..........   Passed    0.00 sec
  2/8 Test bitcoin-core#5: secp256k1_ecdh_example ...........   Passed    0.00 sec
  3/8 Test bitcoin-core#6: secp256k1_schnorr_example ........   Passed    0.00 sec
  4/8 Test bitcoin-core#7: secp256k1_ellswift_example .......   Passed    0.00 sec
  5/8 Test bitcoin-core#8: secp256k1_musig_example ..........   Passed    0.00 sec
  6/8 Test bitcoin-core#3: secp256k1_exhaustive_tests .......   Passed    6.19 sec
  7/8 Test #1: secp256k1_noverify_tests .........   Passed   38.83 sec
  8/8 Test bitcoin-core#2: secp256k1_tests ..................   Passed   91.66 sec

  100% tests passed, 0 tests failed out of 8

  Total Test time (real) =  91.67 sec
  ```

ACKs for top commit:
  theuni:
    utACK 87384f5
  real-or-random:
    utACK 87384f5

Tree-SHA512: d8e46558cf58c9c660544b7bdfed24c991eb3e120b6511aa3968f509190130e498749a3c4dbabc87a7f22f0aa0056c6bcd3fc6c44f5eb131588945d593546840
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant