Skip to content

Add getRoles and getRolePermissions to QueryAPI#59

Merged
Mingela merged 1 commit into
hyperledger-iroha:masterfrom
georgeroman:add-role-queries
Jun 9, 2020
Merged

Add getRoles and getRolePermissions to QueryAPI#59
Mingela merged 1 commit into
hyperledger-iroha:masterfrom
georgeroman:add-role-queries

Conversation

@georgeroman

Copy link
Copy Markdown

Fixes #58

@codecov-commenter

codecov-commenter commented Jun 8, 2020

Copy link
Copy Markdown

Codecov Report

Merging #59 into master will increase coverage by 0.94%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #59      +/-   ##
============================================
+ Coverage     72.60%   73.55%   +0.94%     
- Complexity      345      349       +4     
============================================
  Files            38       38              
  Lines          1610     1622      +12     
  Branches         51       51              
============================================
+ Hits           1169     1193      +24     
+ Misses          358      345      -13     
- Partials         83       84       +1     
Impacted Files Coverage Δ Complexity Δ
...main/java/jp/co/soramitsu/iroha/java/QueryAPI.java 75.80% <100.00%> (+2.59%) 21.00 <2.00> (+2.00)
.../java/jp/co/soramitsu/iroha/java/QueryBuilder.java 60.00% <0.00%> (+5.71%) 26.00% <0.00%> (+2.00%)

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 6de97e1...dd3d114. Read the comment docs.

@Mingela

Mingela commented Jun 9, 2020

Copy link
Copy Markdown
Member

Could you add a test for that as well? Would be perfect

Signed-off-by: George Roman <george.roman.99@gmail.com>
@georgeroman

Copy link
Copy Markdown
Author

Just added the tests

@Mingela Mingela merged commit bb02bbc into hyperledger-iroha:master Jun 9, 2020
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.

QueryAPI misses getRoles and getRolePermissions query types

3 participants