Skip to content

Commit

Permalink
Adapt TestPlentyOfAssignments to perf improvements
Browse files Browse the repository at this point in the history
We no longer read a role three times during single operation ;)
  • Loading branch information
mederly committed Jul 16, 2019
1 parent 0156597 commit 49ec73b
Showing 1 changed file with 2 additions and 4 deletions.
Expand Up @@ -448,8 +448,7 @@ public void test210AddBob() throws Exception {

display("Inspector", inspector);

// TODO: why *3 ???
inspector.assertRead(RoleType.class, NUMBER_OF_BOB_DUMMY_ROLE_ASSIGNMENTS * 3);
inspector.assertRead(RoleType.class, NUMBER_OF_BOB_DUMMY_ROLE_ASSIGNMENTS);
// assertRepositoryReadCount(xxx); // may be influenced by tasks

assertCounterIncrement(InternalCounters.PRISM_OBJECT_COMPARE_COUNT, 0);
Expand Down Expand Up @@ -608,8 +607,7 @@ public void test220AddAlice() throws Exception {

display("Inspector", inspector);

// TODO: why *3 ???
inspector.assertRead(RoleType.class, NUMBER_OF_GENERATED_DUMMY_GROUPS * 3);
inspector.assertRead(RoleType.class, NUMBER_OF_GENERATED_DUMMY_GROUPS);
// assertRepositoryReadCount(xxx); // may be influenced by tasks

assertCounterIncrement(InternalCounters.PRISM_OBJECT_COMPARE_COUNT, 0);
Expand Down

0 comments on commit 49ec73b

Please sign in to comment.