Use try/finally to ensure chmod is run, filter warning

testing/test_cacheprovider.py

@@ -48,16 +48,21 @@ class TestNewAPI:
         testdir.makeini("[pytest]")
         mode = os.stat(testdir.tmpdir.ensure_dir(".pytest_cache"))[stat.ST_MODE]
         testdir.tmpdir.ensure_dir(".pytest_cache").chmod(0)
+        try:
             config = testdir.parseconfigure()
             cache = config.cache
             cache.set("test/broken", [])
+        finally:
             testdir.tmpdir.ensure_dir(".pytest_cache").chmod(mode)
 
     @pytest.mark.skipif(sys.platform.startswith("win"), reason="no chmod on windows")
-    @pytest.mark.filterwarnings("default")
+    @pytest.mark.filterwarnings(
+        "ignore:could not create cache path:pytest.PytestWarning"
+    )
     def test_cache_failure_warns(self, testdir):
         mode = os.stat(testdir.tmpdir.ensure_dir(".pytest_cache"))[stat.ST_MODE]
         testdir.tmpdir.ensure_dir(".pytest_cache").chmod(0)
+        try:
             testdir.makepyfile(
                 """
                 def test_error():
@@ -66,10 +71,13 @@ class TestNewAPI:
             """
             )
             result = testdir.runpytest("-rw")
-        testdir.tmpdir.ensure_dir(".pytest_cache").chmod(mode)
             assert result.ret == 1
             # warnings from nodeids, lastfailed, and stepwise
-        result.stdout.fnmatch_lines(["*could not create cache path*", "*3 warnings*"])
+            result.stdout.fnmatch_lines(
+                ["*could not create cache path*", "*3 warnings*"]
+            )
+        finally:
+            testdir.tmpdir.ensure_dir(".pytest_cache").chmod(mode)
 
     def test_config_cache(self, testdir):
         testdir.makeconftest(