Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Agnul97
Copy link
Contributor

@Agnul97 Agnul97 commented Feb 28, 2023

Brief description of the PR.
Modifications to the "find" function inside the endpoint service to correctly retrieve specific endpoints.

Description of the solution adopted
The query function of the service has been re-used in order to retrieve the available endpoints in the scope (aka, the nearest defined proceeding upwards in the hierarchy). This doesn't mean a performance penalty with respect to a specific traversal to retrieve the same data because actually the steps performed are the same.

@Agnul97 Agnul97 changed the title Correct retrieve of specific endpoints Correct retrieval of specific endpoints Feb 28, 2023
@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Merging #3727 (ab78689) into develop (0abf65f) will increase coverage by 0.35%.
The diff coverage is 0.00%.

❗ Current head ab78689 differs from pull request most recent head a69f478. Consider uploading reports for the commit a69f478 to get more accurate results

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #3727      +/-   ##
=============================================
+ Coverage      55.47%   55.82%   +0.35%     
  Complexity         8        8              
=============================================
  Files           1835     1835              
  Lines          34054    34060       +6     
  Branches        2721     2723       +2     
=============================================
+ Hits           18890    19014     +124     
+ Misses         14166    14049     -117     
+ Partials         998      997       -1     
Impacted Files Coverage Δ
...ice/endpoint/internal/EndpointInfoServiceImpl.java 80.29% <0.00%> (-3.68%) ⬇️
...a/service/device/registry/internal/DeviceImpl.java 72.95% <0.00%> (+0.62%) ⬆️
...tication/shiro/AuthenticationServiceShiroImpl.java 47.70% <0.00%> (+0.70%) ⬆️
...g/eclipse/kapua/service/user/internal/UserDAO.java 85.00% <0.00%> (+5.00%) ⬆️
...egistry/event/internal/DeviceEventServiceImpl.java 83.63% <0.00%> (+7.27%) ⬆️
...tication/shiro/realm/KapuaAuthenticatingRealm.java 78.04% <0.00%> (+7.31%) ⬆️
...e/kapua/service/user/internal/UserServiceImpl.java 55.97% <0.00%> (+8.95%) ⬆️
...e/kapua/service/user/internal/UserFactoryImpl.java 63.63% <0.00%> (+9.09%) ⬆️
...rtz/persistence/KapuaQuartzConnectionProvider.java 100.00% <0.00%> (+12.50%) ⬆️
...ion/credential/mfa/shiro/MfaOptionServiceImpl.java 51.74% <0.00%> (+31.46%) ⬆️
... and 3 more

@Agnul97 Agnul97 requested a review from Coduz March 1, 2023 15:09
@Coduz Coduz added the Bug This is a bug or an unexpected behaviour. Fix it! label Mar 1, 2023
@Coduz Coduz merged commit a3a8f53 into eclipse-kapua:develop Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug This is a bug or an unexpected behaviour. Fix it!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants