Trick-17 / clang-build

Clang-based cross platform build system written in Python
https://clang-build.readthedocs.io
MIT License
8 stars 3 forks source link

Automatic tests and examples #102

Open GPMueller opened 4 years ago

GPMueller commented 4 years ago

Closes #72.

This is currently based on the feature-package branch of PR #98 and will be rebased onto master when PR #98 has been merged.


This design has drawbacks:

Take as an example the following suggestion for how to specify tests:

name = "my_project"

[my_lib]
    # blaaaaaa
some_var = 12

[test]
[test.my_exe_test]
    dependencies = "my_lib"

two differences between the two approaches:

  1. default search for sources: manually specify if tests are present vs. manually specify when all sources should be ingored
  2. organization in the toml: tests in one single block vs. tests each attached to their respective targets
codecov[bot] commented 4 years ago

Codecov Report

Merging #102 into master will increase coverage by 0.20%. The diff coverage is 73.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   80.26%   80.47%   +0.20%     
==========================================
  Files          12       12              
  Lines        1059     1188     +129     
==========================================
+ Hits          850      956     +106     
- Misses        209      232      +23     
Impacted Files Coverage Δ
clang_build/clang_build.py 65.54% <51.72%> (-3.80%) :arrow_down:
clang_build/project.py 73.86% <77.70%> (+3.91%) :arrow_up:
clang_build/dependency_tools.py 68.08% <88.88%> (+4.19%) :arrow_up:
clang_build/io_tools.py 100.00% <100.00%> (ø)
clang_build/target.py 88.92% <100.00%> (ø)
clang_build/single_source.py 89.23% <0.00%> (+21.53%) :arrow_up:

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 3d58a42...37df1db. Read the comment docs.