- 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.*
|