Syncleus / Ferma

An ORM / OGM for the TinkerPop graph stack.
http://syncleus.com/Ferma
Apache License 2.0
136 stars 26 forks source link

Use lambdas in FramedGraph traversals #19

Closed joshsh closed 7 years ago

codecov-io commented 7 years ago

Codecov Report

Merging #19 into master will increase coverage by 0.66%. The diff coverage is 35.29%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
+ Coverage     46.26%   46.93%   +0.66%     
  Complexity      191      191              
============================================
  Files            22       22              
  Lines          1126     1076      -50     
  Branches        204      204              
============================================
- Hits            521      505      -16     
+ Misses          496      462      -34     
  Partials        109      109
Impacted Files Coverage Δ Complexity Δ
...cleus/ferma/typeresolvers/UntypedTypeResolver.java 36.36% <0%> (+3.03%) 4 <0> (ø) :arrow_down:
...java/com/syncleus/ferma/DelegatingFramedGraph.java 45.86% <10%> (+4.41%) 23 <1> (ø) :arrow_down:
...n/java/com/syncleus/ferma/AbstractVertexFrame.java 68.68% <100%> (-0.32%) 32 <1> (ø)
...ain/java/com/syncleus/ferma/AbstractEdgeFrame.java 20.83% <100%> (-3.17%) 5 <1> (ø)
...mefactories/annotation/OutVertexMethodHandler.java 63.63% <100%> (-5.6%) 4 <0> (ø)
...amefactories/annotation/InVertexMethodHandler.java 63.63% <100%> (-5.6%) 4 <0> (ø)
...mefactories/annotation/IncidenceMethodHandler.java 24.66% <33.33%> (-1.2%) 12 <0> (ø)
...mefactories/annotation/AdjacencyMethodHandler.java 48.11% <37.5%> (ø) 25 <0> (ø) :arrow_down:

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 31b2636...fed2ee2. Read the comment docs.

freemo commented 7 years ago

Gonna run this tomorrow and verify then likely merge. Thanks

freemo commented 7 years ago

@joshsh Thanks very helpful! Merging now.