Always use NewModuleTest instead of ModuleTest. (pytorch#44745)

Summary:
Pull Request resolved: pytorch#44745

Much like CriterionTest, NewCriterionTest these are outdated formulations and we should just use the new one.

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D23717808

Pulled By: gchanan

fbshipit-source-id: eb91982eef23452456044381334bfc9a5bbd837e

Author: gchanan

test/test_cpp_api_parity.py

@@ -26,9 +26,9 @@ class TestCppApiParity(common.TestCase):
 expected_test_params_dicts = []
 
 for test_params_dicts, test_instance_class in [
-    (sample_module.module_tests, common_nn.ModuleTest),
+    (sample_module.module_tests, common_nn.NewModuleTest),
     (sample_functional.functional_tests, common_nn.NewModuleTest),
-    (common_nn.module_tests, common_nn.ModuleTest),
+    (common_nn.module_tests, common_nn.NewModuleTest),
     (common_nn.new_module_tests, common_nn.NewModuleTest),
     (common_nn.criterion_tests, common_nn.CriterionTest),
 ]: