adobe / aem-modernize-tools

A suite of tools to modernize your AEM Sites implementations off legacy features.
http://opensource.adobe.com/aem-modernize-tools/
Apache License 2.0
64 stars 42 forks source link

Protect against infinite loops. #100

Closed bstopp closed 2 years ago

bstopp commented 2 years ago

Description

Manage a state of rules which were applied to paths.

Related Issue

99

How Has This Been Tested?

Added unit tests

Screenshots (if appropriate):

Types of changes

Checklist:

codecov[bot] commented 2 years ago

Codecov Report

Merging #100 (c0306cd) into main (b96ccf9) will increase coverage by 0.16%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #100      +/-   ##
============================================
+ Coverage     79.04%   79.21%   +0.16%     
- Complexity      427      430       +3     
============================================
  Files            34       34              
  Lines          1928     1934       +6     
  Branches        355      356       +1     
============================================
+ Hits           1524     1532       +8     
+ Misses          296      295       -1     
+ Partials        108      107       -1     
Flag Coverage Δ
unittests 79.21% <100.00%> (+0.16%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...odernize/component/impl/ComponentTreeRewriter.java 92.30% <100.00%> (+7.45%) :arrow_up:
.../aem/modernize/policy/impl/PolicyTreeImporter.java 81.39% <100.00%> (ø)
...ze/policy/rule/impl/NodeBasedPolicyImportRule.java 100.00% <100.00%> (ø)
.../aem/modernize/rule/impl/NodeBasedRewriteRule.java 93.33% <100.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 084f418...c0306cd. Read the comment docs.