Added options to pytest calls (#23083)

- added more default options to pytest calls in setup.cfg and pytest.ini files.
- pytest-json-report's plugin name in pluggy has changed, so added option for multiple names

cms/conftest.py

@@ -9,6 +9,7 @@ only running cms tests.
 
 import importlib
 import os
+import logging
 import contracts
 import pytest
 
@@ -24,6 +25,11 @@ def pytest_configure(config):
     """
     Do core setup operations from manage.py before collecting tests.
     """
+    if config.pluginmanager.hasplugin("pytest_jsonreport") or config.pluginmanager.hasplugin("json-report"):
+        config.pluginmanager.register(DeferPlugin())
+    else:
+        logging.info("pytest did not register json_report correctly")
+
     if config.getoption('help'):
         return
     enable_contracts = os.environ.get('ENABLE_CONTRACTS', False)
@@ -34,9 +40,6 @@ def pytest_configure(config):
     startup = importlib.import_module(startup_module)
     startup.run()
 
-    if config.pluginmanager.hasplugin("json-report"):
-        config.pluginmanager.register(DeferPlugin())
-
 
 @pytest.fixture(autouse=True, scope='function')
 def _django_clear_site_cache():

cms/pytest.ini

 [pytest]
 DJANGO_SETTINGS_MODULE = cms.envs.test
-addopts = --nomigrations --reuse-db --durations=20 -p no:randomly
+addopts = --nomigrations --reuse-db --durations=20 -p no:randomly --json-report --json-report-omit keywords streams collectors log traceback tests --json-report-file=none
 # Enable default handling for all warnings, including those that are ignored by default;
 # but hide rate-limit warnings (because we deliberately don't throttle test user logins)
 # and field_data deprecation warnings (because fixing them requires a major low-priority refactoring)

common/lib/pytest.ini

 [pytest]
 DJANGO_SETTINGS_MODULE = openedx.tests.settings
-addopts = --nomigrations --reuse-db --durations=20
+addopts = --nomigrations --reuse-db --durations=20 --json-report --json-report-omit keywords streams collectors log traceback tests --json-report-file=none
 # Enable default handling for all warnings, including those that are ignored by default;
 # but hide rate-limit warnings (because we deliberately don't throttle test user logins)
 # and field_data deprecation warnings (because fixing them requires a major low-priority refactoring)

common/test/pytest.ini

 [pytest]
-addopts = -p no:randomly --durations=20
+addopts = -p no:randomly --durations=20 --json-report --json-report-omit keywords streams collectors log traceback tests --json-report-file=none
 # Enable default handling for all warnings, including those that are ignored by default;
 # but hide rate-limit warnings (because we deliberately don't throttle test user logins)
 # and field_data deprecation warnings (because fixing them requires a major low-priority refactoring)

openedx/core/pytest_hooks.py

@@ -75,5 +75,5 @@ class DeferPlugin(object):
 
 
 def pytest_configure(config):
-    if config.pluginmanager.hasplugin("json-report"):
+    if config.pluginmanager.hasplugin("pytest_jsonreport") or config.pluginmanager.hasplugin("json-report"):
         config.pluginmanager.register(DeferPlugin())