Issue #5228: fixed violations for test makeup

[rnveach]

Issue #5228

This fixes all violations found in test besides ones for #5157. I only had to suppress 4 cases.

[rnveach]

One big change was in AbstractModuleTestSupport with the identification of the root module so we could just send a configuration to it with properties set instead of constantly recreating all the other properties (Thread, listener, etc) in every test.
Some places just needed this to set the cache file or add multiple children.

[codecov-io]

Codecov Report

Merging #5273 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5273   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         296     296           
  Lines       16105   16105           
  Branches     3665    3665           
======================================
  Hits        16105   16105

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ca6e3a...0378f0e. Read the comment docs.