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 564328 API revision | conditions | rethink key interfaces #252

Merged
merged 2 commits into from Jun 18, 2020

Conversation

eparovyshnaya
Copy link
Contributor

  • contract tests for all the Condition descriptive interfaces
  • TemporaryTest is deleted as there is the first true and
    not-abstract test appeared
  • LicensingVersion is deprecated in the favor of new Condition
    implementation

Signed-off-by: elena.parovyshnaya elena.parovyshnaya@gmail.com

 - contract tests for all the Condition descriptive interfaces
 - `TemporaryTest` is deleted as there is the first true and
not-abstract test appeared
 - `LicensingVersion` is deprecated in the favor of new Condition
implementation


Signed-off-by: elena.parovyshnaya <elena.parovyshnaya@gmail.com>
 - add ending new lines


Signed-off-by: elena.parovyshnaya <elena.parovyshnaya@gmail.com>
@codecov
Copy link

codecov bot commented Jun 18, 2020

Codecov Report

Merging #252 into master will increase coverage by 0.05%.
The diff coverage is 15.62%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #252      +/-   ##
=========================================
+ Coverage    9.66%   9.72%   +0.05%     
=========================================
  Files         575     576       +1     
  Lines       14966   14977      +11     
  Branches     1168    1168              
=========================================
+ Hits         1446    1456      +10     
- Misses      13482   13483       +1     
  Partials       38      38              
Impacted Files Coverage Δ
...rg/eclipse/passage/lic/base/LicensingVersions.java 93.50% <ø> (ø)
...api/conditions/tests/MatchingRuleContractTest.java 0.00% <0.00%> (ø)
...itions/tests/ValidityPeriodClosedContractTest.java 0.00% <0.00%> (ø)
...i/conditions/tests/ValidityPeriodContractTest.java 0.00% <0.00%> (ø)
...nditions/tests/ValidityPeriodOpenContractTest.java 0.00% <0.00%> (ø)
...i/conditions/tests/EvaluationTypeContractTest.java 60.00% <60.00%> (ø)
...conditions/tests/EvaluationTypeOfContractTest.java 100.00% <100.00%> (ø)
...ge/lic/internal/api/conditions/EvaluationType.java 50.00% <0.00%> (+50.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 9746087...5c223f9. Read the comment docs.

@eparovyshnaya eparovyshnaya reopened this Jun 18, 2020
@eparovyshnaya eparovyshnaya merged commit e4c8371 into master Jun 18, 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.

None yet

2 participants