Bug 53962 - Kelvin ucsschool.lib test creates bad mail domain names
Summary: Kelvin ucsschool.lib test creates bad mail domain names
Status: VERIFIED FIXED
Alias: None
Product: UCS@school
Classification: Unclassified
Component: HTTP-API (Kelvin)
Version: UCS@school 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: ---
Assignee: Daniel Tröder
QA Contact: Tobias Wenzel
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-19 18:38 CEST by Daniel Tröder
Modified: 2021-10-20 15:57 CEST (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):
Customer ID:
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2021-10-19 18:38:41 CEST
The test for the hook feature creates ucsschool.lib.models.misc.Maildomain objects without a dot in the name. This leads to the user bject validation failing, because a dot in the mail domain is expected.
Comment 1 Daniel Tröder univentionstaff 2021-10-19 18:39:49 CEST
The test was fixed to generate proper mail domains.

[feature/kelvin 556dd80fb] Bug #53962: fix mail domain name generation in hook test
Comment 2 Tobias Wenzel univentionstaff 2021-10-20 10:06:11 CEST
QA: looks good

OK Code
OK code already in feature/kelvin
OK test passes on test vm
OK new version in appcenter


before fix all tests pass + maildomains:

udm mail/domain list

DN: cn=fyoung,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: fyoung

DN: cn=robin51,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: robin51

DN: cn=aaron74,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: aaron74

DN: cn=nunezscott,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: nunezscott

DN: cn=rmcguire,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: rmcguire

DN: cn=julieflores,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: julieflores

after fix:

udm mail/domain list

(...)

DN: cn=simpson-young.net,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: simpson-young.net

DN: cn=butler-henderson.com,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: butler-henderson.com

DN: cn=schwartz-edwards.info,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: schwartz-edwards.info

DN: cn=robinson-rose.com,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: robinson-rose.com

DN: cn=jones.com,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: jones.com

DN: cn=pham.com,cn=domain,cn=mail,dc=dc-we,dc=intranet
  name: pham.com



-> verify after jenkins
Comment 3 Tobias Wenzel univentionstaff 2021-10-20 15:57:43 CEST
jenkins is happy -> verify