Skip to content

Conversation

@DvirDukhan
Copy link
Contributor

added path support
added equalsverifier for hashCode and equals functions verifications

@DvirDukhan DvirDukhan requested a review from gkorland October 31, 2019 08:18

@Override
public String toString() {
final StringBuilder sb = new StringBuilder("Path{");
Copy link
Contributor

Choose a reason for hiding this comment

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

return "Path{nodes=" + nodes + " edges=" + edges + "}";

Copy link
Contributor Author

Choose a reason for hiding this comment

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

auto-generated. I like string builder better than string appends.

public class RedisGraphAPITest {

private class PathBuilder{
List<Node> nodes;
Copy link
Contributor

Choose a reason for hiding this comment

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

final private?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Path builder is now under test.tuils

gkorland
gkorland previously approved these changes Oct 31, 2019
@codecov
Copy link

codecov bot commented Nov 3, 2019

Codecov Report

Merging #52 into master will increase coverage by 1.91%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   75.86%   77.77%   +1.91%     
==========================================
  Files          22       23       +1     
  Lines         493      522      +29     
  Branches       75       79       +4     
==========================================
+ Hits          374      406      +32     
  Misses         90       90              
+ Partials       29       26       -3
Impacted Files Coverage Δ
...slabs/redisgraph/impl/resultset/ResultSetImpl.java 83.63% <100%> (+1.73%) ⬆️
...edisgraph/impl/resultset/ResultSetScalarTypes.java 92.85% <100%> (+0.54%) ⬆️
.../com/redislabs/redisgraph/graph_entities/Path.java 78.26% <78.26%> (ø)
.../redislabs/redisgraph/graph_entities/Property.java 78.26% <0%> (+4.34%) ⬆️
.../com/redislabs/redisgraph/graph_entities/Edge.java 72% <0%> (+8%) ⬆️
...dislabs/redisgraph/graph_entities/GraphEntity.java 65% <0%> (+10%) ⬆️
.../com/redislabs/redisgraph/graph_entities/Node.java 80% <0%> (+10%) ⬆️

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 9ec317b...06bae07. Read the comment docs.

@DvirDukhan DvirDukhan merged commit 1518a2e into master Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants