banditcpp / bandit

Human-friendly unit testing for C++11
https://banditcpp.github.io/bandit/
Other
260 stars 37 forks source link

Some restructuring #118

Closed sbeyer closed 7 years ago

sbeyer commented 7 years ago

Restructure bandit a little. Many classes are put into separate namespaces instead of populating the bandit::detail namespace. These new namespaces are designed in a consistent way, for example, instead of bandit::detail::listener and bandit::detail::dots_reporter (that implements listener), we now have bandit::reporter::interface and bandit::reporter::dots.

While at it

codecov-io commented 7 years ago

Codecov Report

Merging #118 into master will increase coverage by 0.05%. The diff coverage is 87.63%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
+ Coverage   89.64%   89.69%   +0.05%     
==========================================
  Files          30       31       +1     
  Lines         869      864       -5     
  Branches       73       73              
==========================================
- Hits          779      775       -4     
+ Misses         90       89       -1
Impacted Files Coverage Δ
bandit/run_policies/filter_chain.h 100% <ø> (ø)
bandit/colorizers/colorizer.h 100% <ø> (ø)
bandit/registration/spec_registry.h 100% <ø> (ø) :arrow_up:
bandit/reporters/spec.h 0% <0%> (ø)
bandit/reporters/interface.h 100% <100%> (ø)
bandit/grammar.h 98.63% <100%> (ø) :arrow_up:
bandit/adapters.h 100% <100%> (ø)
bandit/run_policies/never.h 100% <100%> (ø)
bandit/run_policies/always.h 100% <100%> (ø)
bandit/failure_formatters/interface.h 100% <100%> (ø)
... and 29 more

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 1af830a...3df96d4. Read the comment docs.