Bug 30116 - Execute udm users/user list as Administrator
Execute udm users/user list as Administrator
Status: CLOSED FIXED
Product: UCS Test
Classification: Unclassified
Component: UDM
unspecified
Other Linux
: P5 normal (vote)
: UCS 3.1-2
Assigned To: Lukas Walter
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-22 20:02 CET by Stefan Gohmann
Modified: 2023-03-25 06:44 CET (History)
0 users

See Also:
What kind of report is it?: Development Internal
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2013-01-22 20:02:16 CET
It should be checked if 
 udm users/user list --filter uid=Administrator
can be executed as user Administrator.
Comment 1 Lukas Walter univentionstaff 2013-03-26 13:31:30 CET
========================================================================
Administrator@master10:~$ /usr/sbin/udm users/user list --filter uid=Administrator
uid=Administrator
DN: uid=Administrator,cn=users,dc=lwadev,dc=lwaelop
ARG: None
  homedrive: None
  CtxKeyboardLayout: None
  disabled: none
  postcode: None
  CtxWFProfilePath: None
[...]
=========================================================================

The user "Administrator" can execute "udm users/user list" (with or without filter).
Comment 2 Stefan Gohmann univentionstaff 2013-04-02 06:12:42 CEST
(In reply to comment #1)
> The user "Administrator" can execute "udm users/user list" (with or without
> filter).

Yes, that should be tested by a test script in ucs-test.
Comment 3 Lukas Walter univentionstaff 2013-04-05 14:15:46 CEST
(In reply to comment #2)
> (In reply to comment #1)
> > The user "Administrator" can execute "udm users/user list" (with or without
> > filter).
> 
> Yes, that should be tested by a test script in ucs-test.

Test has been added: 61_udm-users/99_udm_users_as_administrator.


ucs-test (3.0.32-1) unstable; urgency=low
  * added test to determine whether 'Administrator' may execute udm/users
  list with filter "uid=Administrator" (which he should) (Bug #30116)

svn 39889


Changelog entry for 3.1-2 has been added.
Comment 4 Stefan Gohmann univentionstaff 2013-04-09 06:29:28 CEST
The test case should test if the command was successful. Currently the test case checks only if "Permission denied" was printed.
Comment 5 Lukas Walter univentionstaff 2013-04-15 11:53:02 CEST
OK, the test now also regards the UDM return code.


ucs-test (3.0.34-1) unstable; urgency=low
  * modified test 99_udm_users_as_administrator in order to also make it fail
    when the UDM return code is not equal to zero (Bug #30116)

svn 39989
Comment 6 Stefan Gohmann univentionstaff 2013-04-19 06:58:32 CEST
I've modified the package so that the test is now part of ucs-test-udm-users.

Please remove the dependency to python-pexpect. I don't see why expect should be used for such test case, for example:

root@master611:~# su - Administrator -c '/usr/sbin/univention-directory-manager users/user list --filter uid=Administrator | grep "^DN: uid=Administrator"'
DN: uid=Administrator,cn=users,dc=deadlock61,dc=local
root@master611:~#
Comment 7 Lukas Walter univentionstaff 2013-04-19 09:53:12 CEST
The test does not use pexpect anymore.


ucs-test (3.0.36-1) unstable; urgency=low
  * do not use pexpect in 99_udm_users_as_administrator (Bug #30116)

svn 40068
Comment 8 Stefan Gohmann univentionstaff 2016-10-12 07:48:25 CEST
For this bug is no separate QA needed.