From 0cef6bc5d627524354c47295df7a0aa84a64539e Mon Sep 17 00:00:00 2001
From: Michael Merickel <michael@merickel.org>
Date: Thu, 1 Nov 2018 21:05:04 -0500
Subject: remove deprecated media range support from add_view and add_route

---
 tests/test_config/test_routes.py | 18 +++++-------------
 tests/test_config/test_views.py  | 29 ++++-------------------------
 2 files changed, 9 insertions(+), 38 deletions(-)

(limited to 'tests/test_config')

diff --git a/tests/test_config/test_routes.py b/tests/test_config/test_routes.py
index d6f45608d..32a64b4bd 100644
--- a/tests/test_config/test_routes.py
+++ b/tests/test_config/test_routes.py
@@ -216,17 +216,12 @@ class RoutesConfiguratorMixinTests(unittest.TestCase):
         request.accept = DummyAccept('text/html')
         self.assertEqual(predicate(None, request), False)
 
-    def test_add_route_with_wildcard_accept(self):
+    def test_add_route_with_wildcard_accept_raises(self):
         config = self._makeOne(autocommit=True)
-        config.add_route('name', 'path', accept='text/*')
-        route = self._assertRoute(config, 'name', 'path', 1)
-        predicate = route.predicates[0]
-        request = self._makeRequest(config)
-        request.accept = DummyAccept('text/xml', contains=True)
-        self.assertEqual(predicate(None, request), True)
-        request = self._makeRequest(config)
-        request.accept = DummyAccept('application/json', contains=False)
-        self.assertEqual(predicate(None, request), False)
+        self.assertRaises(
+            ValueError,
+            lambda: config.add_route('name', 'path', accept='text/*'),
+        )
 
     def test_add_route_no_pattern_with_path(self):
         config = self._makeOne(autocommit=True)
@@ -313,6 +308,3 @@ class DummyAccept(object):
             if match in offers:
                 results.append((match, 1.0))
         return results
-
-    def __contains__(self, value):
-        return self.contains
diff --git a/tests/test_config/test_views.py b/tests/test_config/test_views.py
index 5e722c9a0..b72b9b36a 100644
--- a/tests/test_config/test_views.py
+++ b/tests/test_config/test_views.py
@@ -1815,25 +1815,12 @@ class TestViewsConfigurationMixin(unittest.TestCase):
         request.accept = DummyAccept('text/html')
         self._assertNotFound(wrapper, None, request)
 
-    def test_add_view_with_range_accept_match(self):
-        from pyramid.renderers import null_renderer
-
-        view = lambda *arg: 'OK'
-        config = self._makeOne(autocommit=True)
-        config.add_view(view=view, accept='text/*', renderer=null_renderer)
-        wrapper = self._getViewCallable(config)
-        request = self._makeRequest(config)
-        request.accept = DummyAccept('text/html', contains=True)
-        self.assertEqual(wrapper(None, request), 'OK')
-
-    def test_add_view_with_range_accept_nomatch(self):
+    def test_add_view_with_wildcard_accept_raises(self):
         view = lambda *arg: 'OK'
         config = self._makeOne(autocommit=True)
-        config.add_view(view=view, accept='text/*')
-        wrapper = self._getViewCallable(config)
-        request = self._makeRequest(config)
-        request.accept = DummyAccept('application/json', contains=False)
-        self._assertNotFound(wrapper, None, request)
+        self.assertRaises(
+            ValueError, lambda: config.add_view(view=view, accept='text/*')
+        )
 
     def test_add_view_with_containment_true(self):
         from pyramid.renderers import null_renderer
@@ -3142,11 +3129,6 @@ class TestMultiView(unittest.TestCase):
         self.assertEqual(mv.media_views['text/xml'], [(100, 'view5', None)])
         self.assertEqual(set(mv.accepts), set(['text/xml', 'text/html']))
         self.assertEqual(mv.views, [(99, 'view2', None), (100, 'view', None)])
-        mv.add('view6', 98, accept='text/*')
-        self.assertEqual(
-            mv.views,
-            [(98, 'view6', None), (99, 'view2', None), (100, 'view', None)],
-        )
 
     def test_add_with_phash(self):
         mv = self._makeOne()
@@ -4266,9 +4248,6 @@ class DummyAccept(object):
                 results.append((match, 1.0))
         return results
 
-    def __contains__(self, value):
-        return self.contains
-
 
 class DummyConfig:
     def __init__(self):
-- 
cgit v1.2.3