From c5c59197e69baa8dbef96d03c0669d47a6280b06 Mon Sep 17 00:00:00 2001 From: Michael Spang Date: Fri, 2 Feb 2007 20:29:55 -0500 Subject: [PATCH] Added list_all() to members module. --- pylib/csc/adm/members.py | 27 ++++++++++++++++++++------- 1 file changed, 20 insertions(+), 7 deletions(-) diff --git a/pylib/csc/adm/members.py b/pylib/csc/adm/members.py index d458b2e..63998fb 100644 --- a/pylib/csc/adm/members.py +++ b/pylib/csc/adm/members.py @@ -239,10 +239,7 @@ def list_term(term): # retrieve a list of memberids in term memberlist = connection.select_members_by_term(term) - # convert the list of memberids to a list of dictionaries - memberlist = map(connection.select_member_by_id, memberlist) - - return memberlist + return memberlist.values() def list_name(name): @@ -264,10 +261,20 @@ def list_name(name): # retrieve a list of memberids matching name memberlist = connection.select_members_by_name(name) - # convert the list of memberids to a list of dictionaries - memberlist = map(connection.select_member_by_id, memberlist) + return memberlist.values() - return memberlist + +def list_all(): + """ + Builds a list of all members. + + Returns: a list of member dictionaries + """ + + # retrieve a list of members + memberlist = connection.select_all_members() + + return memberlist.values() def delete(memberid): @@ -481,6 +488,12 @@ if __name__ == '__main__': assert_equal(True, tm2id in [ x['memberid'] for x in list_name(tm2name) ]) success() + test(list_all) + allmembers = list_all() + assert_equal(True, tmid in [ x['memberid'] for x in allmembers ]) + assert_equal(True, tm2id in [ x['memberid'] for x in allmembers ]) + success() + test(register) register(tmid, terms.next(terms.current())) assert_equal(True, registered(tmid, nextterm))