diff options
author | Jim Hahn <jrh3@att.com> | 2019-09-19 17:58:19 -0400 |
---|---|---|
committer | Jim Hahn <jrh3@att.com> | 2019-09-20 12:10:21 -0400 |
commit | 68b17b2a1ee2b6d1920cfc72f08dae25565bbe6a (patch) | |
tree | 3179d27f33f72f063d4b9fa5be20814eede6ac01 /api-state-management | |
parent | 50cd3fbf4561fe445751d299c53efc499df6dfb6 (diff) |
Address junit failure in active-standby
In testLocking1(), the sleep() call after the final demote() was
allowing the background FPManager thread to re-promote the PDP before
the next test in the test case. Removed the sleep() call, thus
preventing it from being re-promoted before the test. As that's
the last test in that particular test case, it shouldn't cause a
problem.
In the process of diagnosing and fixing the problem, made the
additional changes below.
Made each junit runnable independently so that background threads
could be terminated and started afresh with each test.
Decreased the maximum "real" wait time of the tests, as the
main thread only needs to wait an instant for the other threads
to fire.
Modified code to use common entity manager factories rather than
creating them for each test.
Deleted unnecessary sleeps at the end of tests.
Deleted unnecessary db clean-ups at the start of tests (already
done by the @Before method.
Change-Id: Ia9f1053be564d85f1b29e47379ab184ffdbac235
Issue-ID: POLICY-2080
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'api-state-management')
0 files changed, 0 insertions, 0 deletions