rewrite of test_mandatory_eys_in_spec use subtest for error assertions

test/test_sams_proxy.py

@@ -26,30 +26,35 @@ class TestSAMSHub(unittest.TestCase):
 
   def test_mandatory_keys_in_spec(self):
     """ ProxySpecKeyMissing is raised if something is mising in spec"""
-    with self.assertRaises(ProxySpecKeyMssing):
-      SAMSProxy(proxySpec = {'out': 'http://baz.foo'})
-      SAMSProxy(proxySpec = {'in': '/foo/bar'})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'token': {'name': 'user'}})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'token': {'value': 'user'}})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'session_passthrough': {}})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'session_passthrough': {'default': {}}})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'session_passthrough': {'default': {'user': {'name': 'u'}}}})
-      SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-        , 'session_passthrough': {'default': {'user': {'param_type': 'body'}}}})
-      for method in ('get', 'post', 'delete', 'put', 'update'):
-        SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-          , 'session_passthrough': {'default': {
-            'user': {'param_type': 'body', 'name': 'u'}
-          }, method: {'user': {'param_type': 'body'}}}})
-        SAMSProxy(proxySpec = {'in': '/foo/bar', 'out': 'http://baz.foo'
-          , 'session_passthrough': {'default': {
-            'user': {'param_type': 'body', 'name': 'u'}
-          }, method: {'user': {'name': 'u'}}}})
+    proxySpecs = [
+      {'out': 'http://baz.foo'},
+      {'in': '/foo/bar'},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'token': {'name': 'user'}},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'token': {'value': 'user'}},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'session_passthrough': {}},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'session_passthrough': {'default': {}}},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'session_passthrough': {'default': {'user': {'name': 'u'}}}},
+      {'in': '/foo/bar', 'out': 'http://baz.foo'
+        , 'session_passthrough': {'default': {'user': {'param_type': 'body'}}}}
+    ]
+    for method in ('get', 'post', 'delete', 'put', 'update'):
+      proxySpecs.append({'in': '/foo/bar', 'out': 'http://baz.foo',
+        'session_passthrough': {'default': {
+          'user': {'param_type': 'body', 'name': 'u'}
+        }, method: {'user': {'param_type': 'body'}}}})
+      proxySpecs.append({'in': '/foo/bar', 'out': 'http://baz.foo',
+        'session_passthrough': {'default': {
+          'user': {'param_type': 'body', 'name': 'u'}
+        }, method: {'user': {'name': 'u'}}}})
+    for proxySpec in proxySpecs:
+      with self.subTest(proxySpec):
+        with self.assertRaises(ProxySpecKeyMssing):
+          SAMSProxy(proxySpec = proxySpec)
 
 class TestSAMSHubWithThreadedAPI(unittest.TestCase):