Refactor PhysicsTest.cpp: add PhysicsManagerTest feature

[aclegg3]

Motivation and Context

In order to test physics functionality, the scene must first be loaded and initialized. This process would be duplicated for each test. This setup is now abstracted into PhysicsManagerTest feature which can be re-used by each physics test to avoid code duplication.

How Has This Been Tested

C++ tests still run and pass.

Types of changes

• Docs change / refactoring / dependency upgrade
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have completed my CLA (see CONTRIBUTING)
• I have added tests to cover my changes.
• All new and existing tests passed.

[msbaines]

Maybe use a test fixture: https://github.com/google/googletest/blob/master/googletest/docs/primer.md

[aclegg3]

Maybe use a test fixture

Thanks, this is nicer. 😄

[codecov-io]

Codecov Report

Merging #397 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #397      +/-   ##
==========================================
+ Coverage   55.57%   55.58%   +0.01%     
==========================================
  Files         167      167              
  Lines        7862     7865       +3     
  Branches       82       82              
==========================================
+ Hits         4369     4372       +3     
  Misses       3493     3493

Flag	Coverage Δ
#CPP	51.95% <100%> (+0.02%)	⬆️
#JavaScript	5.53% <ø> (ø)	⬆️
#Python	77.42% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
src/tests/PhysicsTest.cpp	100% <100%> (ø)	⬆️

---

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 f43e096...b3dcd9f. Read the comment docs.

[msbaines]

One small nit.