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

575181 expose License Agreements on license import #882

Merged
merged 5 commits into from
Aug 23, 2021
Merged

Conversation

eparovyshnaya
Copy link
Contributor

@eparovyshnaya eparovyshnaya commented Aug 18, 2021

  • lic.api.requirement interfaces are extended with product agreements support: Requirement supplies list of ResolvedAgreements
  • lic.api.restrictionq interfaces are extended to deliver agreement asessment result with ExaminationCertificate` as the result of access cycle run
  • access cycle implementation is replenished with agreement state assessment: certificate get exhaustive information which state are accepted, which are not, and all the information necessary to acceptance
  • lic.jface facilities are replenished with AgreementsWizard, which currently exposes agreements of plan/text mime type to an end user and allows them to perform active acceptance.
  • AgreementsWizard is plugged to LicenseStatusDialog as one of the intentions.

- invite trouble code for the case when an agreement is discovered which
has not yet been actively accepted by an end user

- remove a copy of BaseAcceptenceCertificate class

Signed-off-by: eparovyshnaya <[email protected]>
start implementing agreements assessment on access cycle run

Signed-off-by: eparovyshnaya <[email protected]>
@eparovyshnaya eparovyshnaya added the pr::wip PR is not ready for review, work is still in progress label Aug 18, 2021
- implement product license state assessment
- remove clones of BaseAccessCycleConfiguration as now 'lic.execute'
provides one

Signed-off-by: eparovyshnaya <[email protected]>
Copy link
Contributor

@ruspl-afed ruspl-afed left a comment

Choose a reason for hiding this comment

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

Intermediate review: minor comments

- Evolve Licensing Certificate structure with the result of
requirement-resident agreements assessment.

- Extend LicenseStatusDialog with the 'expose to accept' functionality

Signed-off-by: eparovyshnaya <[email protected]>
- fix tests according to data structure evolution
- fix agreement-from-requirement resolution according to test results

Signed-off-by: eparovyshnaya <[email protected]>
@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #882 (af8340a) into master (28d22e2) will increase coverage by 0.00%.
The diff coverage is 30.41%.

Impacted file tree graph

@@            Coverage Diff             @@
##             master     #882    +/-   ##
==========================================
  Coverage     33.63%   33.64%            
+ Complexity      346      345     -1     
==========================================
  Files          1099     1106     +7     
  Lines         24474    24617   +143     
  Branches       1520     1528     +8     
==========================================
+ Hits           8233     8283    +50     
- Misses        15754    15846    +92     
- Partials        487      488     +1     
Impacted Files Coverage Δ
...ernal/fls/seal/SealedAccessCycleConfiguration.java 0.00% <ø> (ø)
...ic/base/restrictions/CertificateIsRestrictive.java 50.00% <0.00%> (-16.67%) ⬇️
.../base/restrictions/AgreementAcceptanceService.java 0.00% <0.00%> (ø)
...age/lic/internal/base/restrictions/Assessment.java 0.00% <0.00%> (ø)
...ernal/base/restrictions/BaseAgreementToAccept.java 0.00% <0.00%> (ø)
.../lic/execute/DirectedAccessCycleConfiguration.java 100.00% <ø> (+14.28%) ⬆️
...e/lic/execute/FocusedAccessCycleConfiguration.java 60.00% <ø> (+3.75%) ⬆️
...nternal/jface/dialogs/licensing/AgreementPage.java 0.00% <0.00%> (ø)
...rnal/jface/dialogs/licensing/AgreementsWizard.java 0.00% <0.00%> (ø)
...face/dialogs/licensing/AgreementsWizardDialog.java 0.00% <0.00%> (ø)
... and 27 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 28d22e2...af8340a. Read the comment docs.

@eparovyshnaya eparovyshnaya merged commit dc372eb into master Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr::wip PR is not ready for review, work is still in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants