Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: improve hugegraph-client ut & stabilize hugegraph-client-ci #355

Merged
merged 2 commits into from
Nov 4, 2022
Merged

chore: improve hugegraph-client ut & stabilize hugegraph-client-ci #355

merged 2 commits into from
Nov 4, 2022

Conversation

simon824
Copy link
Member

@simon824 simon824 commented Nov 3, 2022

refer to https://github.com/apache/incubator-hugegraph-toolchain/actions/runs/3383405590/jobs/5619268055

Error:  Tests run: 100, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 207.147 s <<< FAILURE! - in com.baidu.hugegraph.functional.FuncTestSuite
Error:  testAddVertexLabelWithUserData(com.baidu.hugegraph.functional.VertexLabelTest)  Time elapsed: 0.539 s  <<< FAILURE!
java.lang.AssertionError
	at com.baidu.hugegraph.functional.VertexLabelTest.testAddVertexLabelWithUserData(VertexLabelTest.java:60)

Error:  testAddEdgeLabelWithUserData(com.baidu.hugegraph.functional.EdgeLabelTest)  Time elapsed: 2.563 s  <<< FAILURE!
java.lang.AssertionError
	at com.baidu.hugegraph.functional.EdgeLabelTest.testAddEdgeLabelWithUserData(EdgeLabelTest.java:97)

@codecov
Copy link

codecov bot commented Nov 3, 2022

Codecov Report

Merging #355 (9df7760) into master (6374d5f) will increase coverage by 34.70%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master     #355       +/-   ##
=============================================
+ Coverage      0.00%   34.70%   +34.70%     
- Complexity        0      966      +966     
=============================================
  Files            86      255      +169     
  Lines          4031     9089     +5058     
  Branches        478      837      +359     
=============================================
+ Hits              0     3154     +3154     
- Misses         4031     5756     +1725     
- Partials          0      179      +179     
Impacted Files Coverage Δ
.../hugegraph/loader/flink/HugeGraphOutputFormat.java 0.00% <0.00%> (ø)
...com/baidu/hugegraph/api/schema/VertexLabelAPI.java 86.48% <0.00%> (ø)
...om/baidu/hugegraph/structure/auth/AuthElement.java 100.00% <0.00%> (ø)
...main/java/com/baidu/hugegraph/util/CommonUtil.java 93.75% <0.00%> (ø)
...m/baidu/hugegraph/structure/auth/HugeResource.java 62.50% <0.00%> (ø)
...aidu/hugegraph/api/traverser/TemplatePathsAPI.java 100.00% <0.00%> (ø)
...om/baidu/hugegraph/structure/schema/EdgeLabel.java 68.13% <0.00%> (ø)
...ava/com/baidu/hugegraph/structure/auth/Target.java 62.50% <0.00%> (ø)
...u/hugegraph/structure/traverser/WeightedPaths.java 100.00% <0.00%> (ø)
.../baidu/hugegraph/structure/constant/IndexType.java 73.33% <0.00%> (ø)
... and 160 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@javeme javeme merged commit 6ecc1a2 into apache:master Nov 4, 2022
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