Fix bug in users enumeration

Wrong user could be retrieved
This refs #4

CHANGES.rst

@@ -5,6 +5,9 @@ Changelog
 2.0.8 (unreleased)
 ------------------
 
+- Fix bug in users enumeration : wrong user could be retrieved (#4)
+  [laulaz]
+
 - Finally check certs on JWT call (except for test).
   [bsuttor]
 

src/pas/plugins/imio/plugin.py

@@ -208,7 +208,7 @@ def enumerateUsers(
           return at most one mapping per supplied ID ('id' and 'login'
           may be sequences).
 
-        o If 'exact_match' is False, then 'id' and / or login may be
+        o If 'exact_match' is False, then 'id' and / or login and / or email may be
           treated by the plugin as "contains" searches (more complicated
           searches may be supported by some plugins using other keyword
           arguments).
@@ -260,6 +260,8 @@ def enumerateUsers(
                 identity = self._useridentities_by_userid[search_id]
             elif search_id in self._useridentities_by_login:
                 identity = self._useridentities_by_login[search_id]
+            else:
+                return []
         if identity is not None:
             identity_userid = identity.userid
             if hasattr(identity, "login"):

src/pas/plugins/imio/tests/test_plugin.py

@@ -60,6 +60,11 @@ def test_enumerate_users(self):
             self.plugin.enumerateUsers(login="bond"),
             [{"login": "jamesbond", "pluginid": "authentic", "id": "123456"}],
         )
+        self.assertEqual(self.plugin.enumerateUsers(login="bond", exact_match=True), [])
+        self.assertEqual(
+            self.plugin.enumerateUsers(login="jamesbond", exact_match=True),
+            [{"login": "jamesbond", "pluginid": "authentic", "id": "123456"}],
+        )
 
     def test_search_all_users(self):
         count_users = 1