fix: re-enable Django deprecation warnings (#32277)

* fix: re-enable Django deprecation warnings --------- Co-authored-by: UsamaSadiq <[email protected]>
openedx · Aug 7, 2023 · 2732934 · 2732934
1 parent 4e75a89
commit 2732934
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 14 deletions.
diff --git a/cms/pytest.ini b/cms/pytest.ini
@@ -12,8 +12,6 @@ filterwarnings =
     default
     ignore:No request passed to the backend, unable to rate-limit:UserWarning
     ignore::xblock.exceptions.FieldDataDeprecationWarning
-    ignore::django.utils.deprecation.RemovedInDjango40Warning
-    ignore::django.utils.deprecation.RemovedInDjango41Warning
 norecursedirs = envs
 python_classes =
 python_files = test.py tests.py test_*.py *_tests.py
diff --git a/common/test/pytest.ini b/common/test/pytest.ini
@@ -11,6 +11,4 @@ filterwarnings =
     default
     ignore:No request passed to the backend, unable to rate-limit:UserWarning
     ignore::xblock.exceptions.FieldDataDeprecationWarning
-    ignore::django.utils.deprecation.RemovedInDjango40Warning
-    ignore::django.utils.deprecation.RemovedInDjango41Warning
 norecursedirs = .cache
diff --git a/openedx/core/lib/logsettings.py b/openedx/core/lib/logsettings.py
@@ -126,14 +126,14 @@ def log_python_warnings():
     warnings.filterwarnings('ignore', 'Setting _field_data is deprecated')
     warnings.filterwarnings('ignore', 'Setting _field_data via the constructor is deprecated')
     warnings.filterwarnings('ignore', '.*unclosed.*', category=ResourceWarning)
-    try:
-        # There are far too many of these deprecation warnings in startup to output for every management command;
-        # suppress them until we've fixed at least the most common ones as reported by the test suite
-        from django.utils.deprecation import RemovedInDjango40Warning, RemovedInDjango41Warning
-        warnings.simplefilter('ignore', RemovedInDjango40Warning)
-        warnings.simplefilter('ignore', RemovedInDjango41Warning)
-    except ImportError:
-        pass
+    # try:
+    #     # There are far too many of these deprecation warnings in startup to output for every management command;
+    #     # suppress them until we've fixed at least the most common ones as reported by the test suite
+    #     from django.utils.deprecation import RemovedInDjango50Warning, RemovedInDjango51Warning
+    #     warnings.simplefilter('ignore', RemovedInDjango50Warning)
+    #     warnings.simplefilter('ignore', RemovedInDjango51Warning)
+    # except ImportError:
+    #     pass
     logging.captureWarnings(True)
 
 

diff --git a/setup.cfg b/setup.cfg
@@ -12,8 +12,6 @@ filterwarnings =
     default
     ignore:No request passed to the backend, unable to rate-limit:UserWarning
     ignore::xblock.exceptions.FieldDataDeprecationWarning
-    ignore::django.utils.deprecation.RemovedInDjango40Warning
-    ignore::django.utils.deprecation.RemovedInDjango41Warning
 junit_family = xunit2
 norecursedirs = .* *.egg build conf dist node_modules test_root cms/envs lms/envs
 python_classes =