Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

More tests #40

Merged
merged 5 commits into from
Apr 24, 2017
Merged

More tests #40

merged 5 commits into from
Apr 24, 2017

Conversation

pieterbos
Copy link
Member

minor test refactor, some test additions

@codecov
Copy link

codecov bot commented Apr 14, 2017

Codecov Report

Merging #40 into master will increase coverage by 0.17%.
The diff coverage is 57.14%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #40      +/-   ##
============================================
+ Coverage     66.37%   66.55%   +0.17%     
- Complexity     2034     2059      +25     
============================================
  Files           333      333              
  Lines          7198     7209      +11     
  Branches        908      909       +1     
============================================
+ Hits           4778     4798      +20     
+ Misses         1995     1982      -13     
- Partials        425      429       +4
Impacted Files Coverage Δ Complexity Δ
...va/com/nedap/archie/base/MultiplicityInterval.java 66.66% <0%> (-33.34%) 14 <11> (+11)
src/main/java/com/nedap/archie/aom/CAttribute.java 78.94% <100%> (+9.08%) 35 <5> (+9) ⬆️
...nedap/archie/rules/evaluation/AssertionsFixer.java 62.29% <60%> (-0.42%) 22 <0> (ø)
.../java/com/nedap/archie/rminfo/ModelInfoLookup.java 79.33% <0%> (+0.82%) 35% <0%> (+1%) ⬆️
.../adlparser/treewalkers/NumberConstraintParser.java 96.26% <0%> (+7.47%) 45% <0%> (+4%) ⬆️

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 d9c6e4a...997435f. Read the comment docs.

Copy link
Member

@jpleppers jpleppers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 🐷

@pieterbos pieterbos merged commit 90c3bd1 into master Apr 24, 2017
@jpleppers jpleppers deleted the more_tests branch May 11, 2017 06:57
pieterbos added a commit that referenced this pull request Aug 21, 2018
Remove licence field from resource description item. Should not be th…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants