Skip to content

Commit

Permalink
Add a coverage target
Browse files Browse the repository at this point in the history
Run tests with coverage and report to coveralls.io

For simplicity, this currently only adds a single target in a
configuration that attempts to maximize coverage. The true CI coverage
from all the various builds may be a little larger.

The coverage run has the following configuration:
- no-asm: since we can't track asm coverage anyway, might as well measure the
  non-asm code coverage.
- Enable various disabled-by-default options:
  - rc5
  - md2
  - ec_nistp_64_gcc_128
  - ssl3
  - ssl3-method
  - weak-ssl-ciphers

Finally, observe that no-pic implies no-shared, and therefore running
both builds in the matrix is redundant.

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Kurt Roeckx <[email protected]>
  • Loading branch information
ekasper committed Aug 5, 2016
1 parent 3dc8780 commit ad0a0a3
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
language: c
cache: ccache

before_install:
- pip install --user cpp-coveralls

addons:
apt:
packages:
Expand All @@ -25,12 +28,14 @@ compiler:
env:
- CONFIG_OPTS="" DESTDIR="_install"
- CONFIG_OPTS="--debug no-shared enable-crypto-mdebug enable-rc5 enable-md2"
- CONFIG_OPTS="--strict-warnings no-shared" BUILDONLY="yes"
- CONFIG_OPTS="no-pic --strict-warnings" BUILDONLY="yes"
- CONFIG_OPTS="no-engine no-shared --strict-warnings" BUILDONLY="yes"

matrix:
include:
- os: linux
compiler: gcc
env: CONFIG_OPTS="--debug --coverage no-asm enable-rc5 enable-md2 enable-ec_nistp_64_gcc_128 enable-ssl3 enable-ssl3-method enable-weak-ssl-ciphers" COVERALLS="yes"
- os: linux
compiler: clang-3.6
env: CONFIG_OPTS="no-shared enable-asan"
Expand Down Expand Up @@ -107,6 +112,11 @@ script:
fi
- cd $top

after_success:
- if [ -n "$COVERALLS" ]; then
coveralls -b .;
fi;

notifications:
email:
- [email protected]
Expand Down

0 comments on commit ad0a0a3

Please sign in to comment.