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

Copy or Restore Pages - not both #72

Closed bstopp closed 2 years ago

bstopp commented 2 years ago

Description

This change will now allow for copying the page as is. OR restoring the page and processing the restored page.

Motivation and Context

Due to how handling of processing a page occurs, either restoring the page to pre-modernize content OR copying is supported. If both are executed, then what occurs is the page is restored, then the restored page is copied to the new location. This leaves the restored page in the original location.

More fine tuning can be managed by users manually restoring pages to perform a copy, if that is what is desired.

How Has This Been Tested?

Added unit tests.

Types of changes

Checklist:

codecov[bot] commented 2 years ago

Codecov Report

Merging #72 (b6cfc27) into main (152ade3) will decrease coverage by 0.27%. The diff coverage is 65.95%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #72      +/-   ##
============================================
- Coverage     82.74%   82.47%   -0.28%     
- Complexity      449      452       +3     
============================================
  Files            34       34              
  Lines          1907     1923      +16     
  Branches        351      354       +3     
============================================
+ Hits           1578     1586       +8     
- Misses          225      231       +6     
- Partials        104      106       +2     
Flag Coverage Δ
unittests 82.47% <65.95%> (-0.28%) :arrow_down:

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

Impacted Files Coverage Δ
...ernize/job/datasource/ConversionJobDataSource.java 87.09% <0.00%> (ø)
...m/modernize/job/AbstractConversionJobExecutor.java 73.52% <44.44%> (-2.52%) :arrow_down:
...odernize/servlet/ScheduleConversionJobServlet.java 92.91% <66.66%> (+0.11%) :arrow_up:
...ernize/structure/job/PageStructureJobExecutor.java 80.55% <66.66%> (-6.95%) :arrow_down:
...e/aem/modernize/job/FullConversionJobExecutor.java 78.76% <70.58%> (-0.14%) :arrow_down:
...ava/com/adobe/aem/modernize/impl/RewriteUtils.java 89.47% <100.00%> (-1.01%) :arrow_down:
...a/com/adobe/aem/modernize/model/ConversionJob.java 78.94% <100.00%> (+1.58%) :arrow_up:

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 0d35cca...b6cfc27. Read the comment docs.