Bug 37655 – add default-umc-users to Domain Users $school-OU

Bug 37655 - add default-umc-users to Domain Users $school-OU


Summary:	add default-umc-users to Domain Users $school-OU

Status:	CLOSED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	ucs-test
Version:	UCS@school 4.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	Ammar Najjar
QA Contact:	Florian Best

URL:
Keywords:

Depends on:	36537
Blocks:
	Show dependency tree / graph

Reported:	2015-01-30 10:43 CET by Florian Best
Modified:	2016-11-03 17:37 CET (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	---
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
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2015-01-30 10:43:23 CET

Please create a test script for this. The ucr variable is named: ucsschool/import/attach/policy/umc (if unset True is assumed).
The policy is "cn=default-umc-users,cn=UMC,cn=policies,%s" % (baseDN,)
It have to be attached at "cn=Domain Users %s,cn=groups,%s" % (schoolNr.lower(), getDN(schoolNr).

+++ This bug was initially created as a clone of Bug #36537 +++

The UMC policy "default-umc-users" should be appended to the "Domain Users $SCHOOL" group by default to allow students and teachers to change their own Password via the UMC module and also to allow UMC single sign on for them.

Comment 1 Florian Best

2015-01-30 12:24:20 CET

Both, ucs-school-lib and ucs-school-import implementations have to be checked.

Comment 2 Florian Best

2015-01-30 12:45:08 CET

renaming UCR variable to ucsschool/import/attach/policy/default-umc-users

Comment 3 Ammar Najjar

2015-02-03 10:59:47 CET

Two new scripts are created:

 110_set_default-umc-users
    Tests the case where ucsschool/import/attach/policy/default-umc-users=no

 110_unset_default-umc-users
    Tests the case where ucsschool/import/attach/policy/default-umc-users is unset

Chengelod entry added, package rebuilt.

Comment 4 Florian Best

2015-02-09 13:46:02 CET

Please make a comment why the import statement is in the main function.
fix sys.exit(main()), as main does not return anything, just call main()
Catch ldap.NO_SUCH_OBJECT also in 110_set_default-umc-users and fail then (it would be raised in case there is a bug in the product).

Comment 5 Ammar Najjar

2015-02-10 10:17:47 CET

(In reply to Florian Best from comment #4)
> Please make a comment why the import statement is in the main function.
> fix sys.exit(main()), as main does not return anything, just call main()

Done.

> Catch ldap.NO_SUCH_OBJECT also in 110_set_default-umc-users and fail then
> (it would be raised in case there is a bug in the product).

Catching the exception is modified to exclusively check the results of this test, and fails if there were a related bug in the product.

Comment 6 Florian Best

2015-02-10 12:07:42 CET

Please also catch ldap.NO_SUCH_OBJECT in 110_unset_default-umc-users in case it fails. svn r57907 only switched the test case names.

Comment 7 Florian Best

2015-02-10 12:08:44 CET

in 110_set_default-umc-users → please log also the found_attr in utils.fail('Incorrect attribute found: %r' % (expected_attr, ))

Comment 8 Ammar Najjar

2015-02-11 10:34:07 CET

(In reply to Florian Best from comment #6)
> Please also catch ldap.NO_SUCH_OBJECT in 110_unset_default-umc-users in case
> it fails.

Done.

(In reply to Florian Best from comment #7)
> in 110_set_default-umc-users → please log also the found_attr in
> utils.fail('Incorrect attribute found: %r' % (expected_attr, ))

Done.

Comment 9 Florian Best

2015-02-11 17:05:21 CET

OK