s4-samba-tool: make new samba-tool group listmembers use samAccountName

This is the unique username value.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu May  3 01:57:41 CEST 2012 on sn-devel-104

diff --git a/source4/scripting/python/samba/netcmd/group.py b/source4/scripting/python/samba/netcmd/group.py
index 5cd9ba8701b794468183745f41497578bb72cde8..265170d2d275841ae8b86025532a643e1e3629ce 100644 (file)
--- a/source4/scripting/python/samba/netcmd/group.py
+++ b/source4/scripting/python/samba/netcmd/group.py
@@ -346,13 +346,13 @@ samba-tool group listmembers \"Domain Users\" -H ldap://samba.samdom.example.com
             search_filter = "(|(primaryGroupID=%s)(memberOf=%s))" % (rid, group_dn)
             res = samdb.search(samdb.domain_dn(), scope=ldb.SCOPE_SUBTREE,
                                expression=(search_filter),
-                               attrs=["cn"])
+                               attrs=["samAccountName"])
 
             if (len(res) == 0):
                 return
 
             for msg in res:
-                self.outf.write("%s\n" % msg.get("cn", idx=0))
+                self.outf.write("%s\n" % msg.get("samAccountName", idx=0))
 
         except Exception, e:
             raise CommandError('Failed to list members of "%s" group ' % groupname, e)

diff --git a/source4/scripting/python/samba/tests/samba_tool/group.py b/source4/scripting/python/samba/tests/samba_tool/group.py
index b7f751747dccf60241fd171637a703caefc987a6..2c0c46e5dc8435b81583d83ad92093e49cfd175b 100644 (file)
--- a/source4/scripting/python/samba/tests/samba_tool/group.py
+++ b/source4/scripting/python/samba/tests/samba_tool/group.py
@@ -130,12 +130,12 @@ class GroupCmdTestCase(SambaToolCmdTest):
         grouplist = self.samdb.search(base=self.samdb.domain_dn(),
                                       scope=ldb.SCOPE_SUBTREE,
                                       expression=search_filter,
-                                      attrs=["cn"])
+                                      attrs=["samAccountName"])
 
         self.assertTrue(len(grouplist) > 0, "no groups found in samdb")
 
         for groupobj in grouplist:
-            name = groupobj.get("cn", idx=0)
+            name = groupobj.get("samAccountName", idx=0)
             found = self.assertMatch(out, name, "group '%s' not found" % name)
 
     def _randomGroup(self, base={}):