Skip to content

Commit

Permalink
AbstractModelIntegrationTest: fixed TODOs from intest refactoring
Browse files Browse the repository at this point in the history
  • Loading branch information
virgo47 committed Mar 2, 2020
1 parent a928cc1 commit f4c6d08
Showing 1 changed file with 1 addition and 4 deletions.
Expand Up @@ -1123,8 +1123,6 @@ protected <F extends AssignmentHolderType> void modifyFocusAssignment(Class<F> f
Consumer<AssignmentType> modificationBlock, boolean add, ModelExecuteOptions options, OperationResult result)
throws ObjectNotFoundException, SchemaException, ExpressionEvaluationException, CommunicationException, ConfigurationException, ObjectAlreadyExistsException, PolicyViolationException, SecurityViolationException {
ObjectDelta<F> focusDelta = createAssignmentFocusDelta(focusClass, focusOid, elementName, roleOid, refType, relation, modificationBlock, add);
// TODO inttest: seems useless and doesn't change the problem with TestSecurityAdvanced 320
Collection<ObjectDelta<? extends ObjectType>> deltas = MiscSchemaUtil.createCollection(focusDelta);
executeChanges(focusDelta, options, task, result);
}

Expand Down Expand Up @@ -5603,8 +5601,7 @@ protected Consumer<Task> createShowTaskTreeConsumer(long period) {
return task -> {
try {
if (lastTimeShown.get() + period < System.currentTimeMillis()) {
// TODO inttest: task. (for result) is missing in master, is it good?
dumpTaskTree(task.getOid(), task.getResult());
dumpTaskTree(task.getOid(), createOperationalResult());
lastTimeShown.set(System.currentTimeMillis());
}
} catch (CommonException e) {
Expand Down

0 comments on commit f4c6d08

Please sign in to comment.