Use recwarn everywhere

...instead of custom fixture. Also assert that no warnings are left over
after the test.

tests/conftest.py

@@ -126,8 +126,7 @@ def purge_module(request):
     return inner
 
 
-@pytest.fixture
-def catch_deprecation_warnings():
-    import warnings
-    warnings.simplefilter('default', category=DeprecationWarning)
-    return lambda: warnings.catch_warnings(record=True)
+@pytest.yield_fixture(autouse=True)
+def catch_deprecation_warnings(recwarn):
+    yield
+    assert not recwarn.list

tests/test_basic.py

@@ -1116,9 +1116,10 @@ def test_static_files():
     rv.close()
 
 
-def test_static_path_deprecated():
-    with pytest.deprecated_call():
+def test_static_path_deprecated(recwarn):
     app = flask.Flask(__name__, static_path='/foo')
+    recwarn.pop(DeprecationWarning)
+
     app.testing = True
     rv = app.test_client().get('/foo/index.html')
     assert rv.status_code == 200

tests/test_deprecations.py

@@ -16,7 +16,7 @@ import flask
 
 class TestRequestDeprecation(object):
 
-    def test_request_json(self, catch_deprecation_warnings):
+    def test_request_json(self, recwarn):
         """Request.json is deprecated"""
         app = flask.Flask(__name__)
         app.testing = True
@@ -27,13 +27,11 @@ class TestRequestDeprecation(object):
             print(flask.request.json)
             return 'OK'
 
-        with catch_deprecation_warnings() as captured:
         c = app.test_client()
         c.post('/', data='{"spam": 42}', content_type='application/json')
+        recwarn.pop(DeprecationWarning)
 
-        assert len(captured) == 1
-
-    def test_request_module(self, catch_deprecation_warnings):
+    def test_request_module(self, recwarn):
         """Request.module is deprecated"""
         app = flask.Flask(__name__)
         app.testing = True
@@ -43,8 +41,6 @@ class TestRequestDeprecation(object):
             assert flask.request.module is None
             return 'OK'
 
-        with catch_deprecation_warnings() as captured:
         c = app.test_client()
         c.get('/')
-
-        assert len(captured) == 1
+        recwarn.pop(DeprecationWarning)

tests/test_ext.py

@@ -20,6 +20,18 @@ except ImportError:
 from flask._compat import PY2
 
 
+@pytest.fixture(autouse=True)
+def disable_extwarnings(request, recwarn):
+    from flask.exthook import ExtDeprecationWarning
+
+    def inner():
+        assert set(w.category for w in recwarn.list) \
+            <= set([ExtDeprecationWarning])
+        recwarn.clear()
+
+    request.addfinalizer(inner)
+
+
 @pytest.fixture(autouse=True)
 def importhook_setup(monkeypatch, request):
     # we clear this out for various reasons.  The most important one is

tests/test_helpers.py

@@ -337,7 +337,7 @@ class TestSendfile(object):
                 assert rv.data == f.read()
             rv.close()
 
-    def test_send_file_xsendfile(self):
+    def test_send_file_xsendfile(self, catch_deprecation_warnings):
         app = flask.Flask(__name__)
         app.use_x_sendfile = True
         with app.test_request_context():
@@ -349,9 +349,9 @@ class TestSendfile(object):
             assert rv.mimetype == 'text/html'
             rv.close()
 
-    def test_send_file_object(self, catch_deprecation_warnings):
+    def test_send_file_object(self, recwarn):
         app = flask.Flask(__name__)
-        with catch_deprecation_warnings() as captured:
+
         with app.test_request_context():
             f = open(os.path.join(app.root_path, 'static/index.html'), mode='rb')
             rv = flask.send_file(f)
@@ -360,11 +360,13 @@ class TestSendfile(object):
                 assert rv.data == f.read()
             assert rv.mimetype == 'text/html'
             rv.close()
+
         # mimetypes + etag
-            assert len(captured) == 2
+        assert len(recwarn.list) == 2
+        recwarn.clear()
 
         app.use_x_sendfile = True
-        with catch_deprecation_warnings() as captured:
+
         with app.test_request_context():
             f = open(os.path.join(app.root_path, 'static/index.html'))
             rv = flask.send_file(f)
@@ -373,21 +375,23 @@ class TestSendfile(object):
             assert rv.headers['x-sendfile'] == \
                 os.path.join(app.root_path, 'static/index.html')
             rv.close()
+
         # mimetypes + etag
-            assert len(captured) == 2
+        recwarn.pop()
+        recwarn.pop()
 
         app.use_x_sendfile = False
         with app.test_request_context():
-            with catch_deprecation_warnings() as captured:
             f = StringIO('Test')
             rv = flask.send_file(f)
             rv.direct_passthrough = False
             assert rv.data == b'Test'
             assert rv.mimetype == 'application/octet-stream'
             rv.close()
+
             # etags
-            assert len(captured) == 1
-            with catch_deprecation_warnings() as captured:
+            recwarn.pop()
+
             class PyStringIO(object):
                 def __init__(self, *args, **kwargs):
                     self._io = StringIO(*args, **kwargs)
@@ -400,39 +404,45 @@ class TestSendfile(object):
             assert rv.data == b'Test'
             assert rv.mimetype == 'text/plain'
             rv.close()
+
             # attachment_filename and etags
-            assert len(captured) == 3
-            with catch_deprecation_warnings() as captured:
+            recwarn.pop()
+            recwarn.pop()
+            recwarn.pop()
+
             f = StringIO('Test')
             rv = flask.send_file(f, mimetype='text/plain')
             rv.direct_passthrough = False
             assert rv.data == b'Test'
             assert rv.mimetype == 'text/plain'
             rv.close()
+
             # etags
-            assert len(captured) == 1
+            recwarn.pop()
 
         app.use_x_sendfile = True
-        with catch_deprecation_warnings() as captured:
+
         with app.test_request_context():
             f = StringIO('Test')
             rv = flask.send_file(f)
             assert 'x-sendfile' not in rv.headers
             rv.close()
+
         # etags
-            assert len(captured) == 1
+        recwarn.pop()
 
-    def test_attachment(self, catch_deprecation_warnings):
+    def test_attachment(self, recwarn):
         app = flask.Flask(__name__)
-        with catch_deprecation_warnings() as captured:
         with app.test_request_context():
             f = open(os.path.join(app.root_path, 'static/index.html'))
             rv = flask.send_file(f, as_attachment=True)
             value, options = parse_options_header(rv.headers['Content-Disposition'])
             assert value == 'attachment'
             rv.close()
+
         # mimetypes + etag
-            assert len(captured) == 2
+        assert len(recwarn.list) == 2
+        recwarn.clear()
 
         with app.test_request_context():
             assert options['filename'] == 'index.html'