ONNX test code cleanup - part 2 (#13738)

* Common test caller

* Remove incorrect comment

* Make corrections to CI

* fix ci script

ci/docker/runtime_functions.sh

@@ -939,8 +939,7 @@ unittest_centos7_gpu() {
 integrationtest_ubuntu_cpu_onnx() {
 	set -ex
 	export PYTHONPATH=./python/
-	pytest tests/python-pytest/onnx/gluon_backend_test.py
-	pytest tests/python-pytest/onnx/mxnet_backend_test.py
+	python tests/python-pytest/onnx/backend_test.py
 	pytest tests/python-pytest/onnx/mxnet_export_test.py
 	pytest tests/python-pytest/onnx/test_models.py
 	pytest tests/python-pytest/onnx/test_node.py

tests/python-pytest/onnx/backend_test.py

@@ -22,34 +22,67 @@
     raise ImportError("Onnx and protobuf need to be installed")
 
 import test_cases
+import unittest
+import backend as mxnet_backend
+import logging
 
+operations = ['import', 'export']
+backends = ['mxnet', 'gluon']
+# This is a pytest magic variable to load extra plugins
+pytest_plugins = "onnx.backend.test.report",
 
-def prepare_tests(backend, operation):
+
+def test_suite(backend_tests):  # type: () -> unittest.TestSuite
+    '''
+    TestSuite that can be run by TestRunner
+    This has been borrowed from onnx/onnx/backend/test/runner/__init__.py,
+    since Python3 cannot sort objects of type 'Type' as Runner.test_suite()
+    expects.
+    '''
+    suite = unittest.TestSuite()
+    for case in backend_tests.test_cases.values():
+        suite.addTests(unittest.defaultTestLoader.loadTestsFromTestCase(case))
+    return suite
+
+
+def prepare_tests(backend, oper):
     """
     Prepare the test list
     :param backend: mxnet/gluon backend
-    :param operation: str. export or import
+    :param oper: str. export or import
     :return: backend test list
     """
     BACKEND_TESTS = onnx.backend.test.BackendTest(backend, __name__)
     implemented_ops = test_cases.IMPLEMENTED_OPERATORS_TEST.get('both', []) + \
-                      test_cases.IMPLEMENTED_OPERATORS_TEST.get(operation, [])
+                      test_cases.IMPLEMENTED_OPERATORS_TEST.get(oper, [])
 
     for op_test in implemented_ops:
         BACKEND_TESTS.include(op_test)
 
     basic_models = test_cases.BASIC_MODEL_TESTS.get('both', []) + \
-                   test_cases.BASIC_MODEL_TESTS.get(operation, [])
+                   test_cases.BASIC_MODEL_TESTS.get(oper, [])
 
     for basic_model_test in basic_models:
         BACKEND_TESTS.include(basic_model_test)
 
     std_models = test_cases.STANDARD_MODEL.get('both', []) + \
-                 test_cases.STANDARD_MODEL.get(operation, [])
+                 test_cases.STANDARD_MODEL.get(oper, [])
 
     for std_model_test in std_models:
         BACKEND_TESTS.include(std_model_test)
 
     BACKEND_TESTS.exclude('.*bcast.*')
 
     return BACKEND_TESTS
+
+
+for bkend in backends:
+    for operation in operations:
+        log = logging.getLogger(bkend + operation)
+        if bkend == 'gluon' and operation == 'export':
+            log.warning('Gluon->ONNX export not implemented. Skipping tests...')
+            continue
+        log.info('Executing tests for ' + bkend + ' backend: ' + operation)
+        mxnet_backend.MXNetBackend.set_params(bkend, operation)
+        BACKEND_TESTS = prepare_tests(mxnet_backend, operation)
+        unittest.TextTestRunner().run(test_suite(BACKEND_TESTS.enable_report()))