Treat all warnings as errors (#871)

* Make all warnings errors. * Remove unnecessary warnings filter. * Update changelog.
glotzerlab · Dec 8, 2022 · 35b862f · 35b862f
1 parent 3e0ca40
commit 35b862f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/changelog.txt b/changelog.txt
@@ -34,6 +34,7 @@ Changed
  - Detection of an invalid config will raise an error rather than a debug log (#855).
  - The package namespace has been flattened so that most functionality is directly available in the ``signac`` namespace (#756, #868).
  - The ``calc_id`` function has been moved from the ``hashing`` module to the ``job`` module (#873).
+ - Tests are run with all warnings treated as errors (#871).
 
 Removed
 +++++++

diff --git a/setup.cfg b/setup.cfg
@@ -35,9 +35,7 @@ omit =
 
 [tool:pytest]
 xfail_strict = True
-filterwarnings = 
-	error::FutureWarning:signac.*
-	error::DeprecationWarning:signac.*
+filterwarnings = error
 
 [bumpversion:file:setup.py]
 

diff --git a/tests/test_sync.py b/tests/test_sync.py
@@ -239,7 +239,6 @@ def test_create_doc_dict_with_error_dryrun(self):
         assert len(self.doc) == 0
 
 
-@pytest.mark.filterwarnings("ignore::FutureWarning")
 class TestFileModifyProxyJSONDocBackup(TestFileModifyProxyDocBackup):
     @pytest.fixture(autouse=True)
     def setUp(self, request):