Skip to content

Conversation

wilzbach
Copy link
Member

A start at fixing #141

The script from Phobos only extracts public top-level unittests (so tests inside a struct are not tested for now).
Also the mapping back from the error message is a bit annoying because it doesn't add empty whitespace, so the line numbers are off.

Anyhow, it's a start.

@wilzbach wilzbach force-pushed the tests_exctractor branch 2 times, most recently from a97b8dc to 3e17172 Compare June 12, 2018 17:52
@codecov-io
Copy link

Codecov Report

Merging #142 into master will decrease coverage by <.01%.
The diff coverage is 50%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #142 +/- ## ========================================== - Coverage 95.14% 95.14% -0.01%  ========================================== Files 46 46 Lines 7513 7512 -1 ========================================== - Hits 7148 7147 -1  Misses 365 365
Impacted Files Coverage Δ
source/mir/ndslice/fuse.d 88.75% <ø> (ø) ⬆️
source/mir/math/sum.d 99.4% <ø> (-0.01%) ⬇️
source/mir/ndslice/concatenation.d 99.04% <ø> (ø) ⬆️
source/mir/ndslice/connect/cpython.d 3.12% <0%> (ø) ⬆️
source/mir/ndslice/topology.d 99.75% <100%> (ø) ⬆️

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 f024f2b...96e35d4. Read the comment docs.

@9il 9il merged commit 865a3a9 into libmir:master Jun 13, 2018
@wilzbach wilzbach deleted the tests_exctractor branch June 13, 2018 11:22
@wilzbach
Copy link
Member Author

Wow thanks for fixing the CI. I ran out of time yesterday :/

@wilzbach
Copy link
Member Author

FYI it's now on code.dlang.org: https://code.dlang.org/packages/dtools

dub fetch dtools dub run dtools:tests_extractor -- ... 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants