[MGNLACTIVATION-72] Merge activation tools app and activation monitor into one. Created: 12/Dec/13  Updated: 29/Aug/14  Resolved: 26/Aug/14

Status: Closed
Project: Activation
Component/s: None
Affects Version/s: 5.2.6
Fix Version/s: 5.3

Type: Task Priority: Neutral
Reporter: Jaroslav Simak Assignee: Evzen Fochr
Resolution: Fixed Votes: 0
Labels: acceptance
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
relation
is related to MGNLACTIVATION-15 Create UI for activation monitor Closed
is related to MGNLXAA-85 Merge exchange-transactional app into... Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty
Date of First Response:

 Comments   
Comment by Roman Kovařík [ 25/Aug/14 ]
  • Please wrap all related tasks into one ArrayDelegateTask and you can also assign this task to a member to make the code more readable (private final Task dawe=new ArrayDelegateTask(...)). Don't forgot to add proper name and description at least to this parent task.
  • new CheckAndModifyPropertyValueTask("", "", RepositoryConstants.CONFIG, "/modules/activation/apps/activation", "appClass", "info.magnolia.ui.framework.app.BaseApp", "info.magnolia.module.activation.app.ActivationBaseApp"))));
    Please use class names instead of plain Strings if possible (BaseApp.class.getName()).
  • Not necessary to fix it now, but for next time:
    assertTrue(session.nodeExists("/modules/activation/apps/activation/subApps/monitor"));
    We have couple of useful methods to test nodes: info.magnolia.test.hamcrest.NodeMatchers.*
Generated at Sun Feb 11 22:59:24 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.