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

Bug 564420 conditions | move HC condition miner to new interfaces #268

Merged
merged 5 commits into from
Jun 27, 2020

Conversation

eparovyshnaya
Copy link
Contributor

  • rework appealing to licensing server coordinate definition settings: hide property name string literal
  • implement settings retrieval service
  • extend available PassageFileExtensions with .settings files required to store licensing server connection credentials

Signed-off-by: elena.parovyshnaya [email protected]

 - rework appealing to licensing server coordinate definition settings:
hide property name string literal
 - implement settings retrieval service
 - extend available PassageFileExtensions with `.settings` files
required to store licensing server connection credentials

Signed-off-by: elena.parovyshnaya <[email protected]>
 - add missed license header

Signed-off-by: elena.parovyshnaya <[email protected]>
@eparovyshnaya eparovyshnaya requested a review from ruspl-afed June 26, 2020 15:22
@codecov
Copy link

codecov bot commented Jun 26, 2020

Codecov Report

Merging #268 into master will increase coverage by 0.31%.
The diff coverage is 68.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
+ Coverage   11.67%   11.98%   +0.31%     
==========================================
  Files         619      624       +5     
  Lines       15409    15484      +75     
  Branches     1174     1178       +4     
==========================================
+ Hits         1799     1856      +57     
- Misses      13565    13583      +18     
  Partials       45       45              
Impacted Files Coverage Δ
...lipse/passage/lic/internal/hc/i18n/HcMessages.java 0.00% <ø> (ø)
...internal/hc/remote/LicensingServerCoordinates.java 0.00% <0.00%> (ø)
.../passage/lic/internal/net/LicensingServerHost.java 0.00% <0.00%> (ø)
.../passage/lic/internal/net/LicensingServerPort.java 0.00% <0.00%> (ø)
.../src/org/eclipse/passage/lic/net/LicensingNet.java 0.00% <ø> (ø)
...age/lic/internal/base/io/PassageFileExtension.java 50.00% <57.14%> (+50.00%) ⬆️
...eclipse/passage/lic/internal/base/io/Settings.java 96.15% <96.15%> (ø)
...ssage/lic/internal/base/tests/io/SettingsTest.java 100.00% <100.00%> (ø)
... and 4 more

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 9893417...c04cb06. Read the comment docs.

 - add tests for Settings read service

Signed-off-by: elena.parovyshnaya <[email protected]>
 - fix typos in test names

Signed-off-by: elena.parovyshnaya <[email protected]>
 - add missed license header

Signed-off-by: elena.parovyshnaya <[email protected]>
@eparovyshnaya eparovyshnaya requested a review from ruspl-afed June 27, 2020 06:27
@eparovyshnaya eparovyshnaya merged commit d9e2310 into master Jun 27, 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.

2 participants