Fixed a list construction.

Added equality method for metadata sources. Checks that they contain the same information or not.

Author: Roland Hedberg

src/saml2/mdstore.py

@@ -345,6 +345,26 @@ def entity_categories(self, entity_id):
 
         return res
 
+    def __eq__(self, other):
+        try:
+            assert isinstance(other, MetaData)
+        except AssertionError:
+            return False
+
+        if len(self.entity) != len(other.entity):
+            return False
+
+        if set(self.entity.keys()) != set(other.entity.keys()):
+            return False
+
+        for key, item in self.entity.items():
+            try:
+                assert item == other[key]
+            except AssertionError:
+                return False
+
+        return True
+
 
 class MetaDataFile(MetaData):
     """

src/saml2/server.py

@@ -307,8 +307,9 @@ def _authn_response(self, in_response_to, consumer_url,
 
         if authn:  # expected to be a dictionary
             # Would like to use dict comprehension but ...
-            authn_args = dict([(AUTHN_DICT_MAP[k],
-                                v) for k, v in authn.items()])
+            authn_args = dict([
+                (AUTHN_DICT_MAP[k], v) for k, v in authn.items()
+                if k in AUTHN_DICT_MAP])
 
             assertion = ast.construct(sp_entity_id, in_response_to,
                                       consumer_url, name_id,