Bug 52144 - Test copyable user properties
Summary: Test copyable user properties
Status: CLOSED FIXED
Alias: None
Product: UCS Test
Classification: Unclassified
Component: UMC
Version: unspecified
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 4.4-6
Assignee: Florian Best
QA Contact: Christian Castens
URL:
Keywords:
Depends on: 49823
Blocks:
  Show dependency treegraph
 
Reported: 2020-09-28 16:19 CEST by Florian Best
Modified: 2022-03-23 10:00 CET (History)
6 users (show)

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: 0.011
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 Florian Best univentionstaff 2020-09-28 16:19:06 CEST
The test case 86_selenium/153_udm_check_properties_of_copied_users.py uses hardcoded HTML/dojo/dijit ID's which will change/break on any change in the structure of UMC.

1. There should be a method Users.copy() in univention.testing.selenium.udm.Users.
2. The method should use the existing functionality "self.selenium.enter_input()" for inserting text into the input boxes instead of things like:
+       # password 2
+       WebDriverWait(selenium.driver, 30).until(
+               expected_conditions.element_to_be_clickable(
+               (By.XPATH, '//*[@id="umc_widgets_PasswordBox_3"]')
+               )
+       ).send_keys('univention')

+++ This bug was initially created as a clone of Bug #49823 +++
Comment 1 Florian Best univentionstaff 2020-09-28 16:49:59 CEST
The hashbang should contain "-s" so that the output is always printed by pytest.
The test case fails in UCS 5.0.
Comment 2 Florian Best univentionstaff 2020-09-28 17:19:51 CEST
>               _copy_user(selenium, _fqdn, _dn, _username, _copied_username)
153_udm_check_properties_of_copied_users.py:213:
153_udm_check_properties_of_copied_users.py:107: in _copy_user
    (By.XPATH, '//*[@id="dgrid_1-row-%s"]' % (orig_dn,))

>       raise TimeoutException(message, screen, stacktrace)
E       TimeoutException: Message:
Comment 3 Christian Castens univentionstaff 2020-11-30 17:07:55 CET
Package: ucs-test
Version: 9.0.7-2A~4.4.0.202011301644
Branch: ucs_4.4-0
Scope: errata4.4-7

4.4-7 commits:
730fa9c0bd1a764961b72b0b258e1f7e8f42fd35 (changes)
40166745dbe90f6e826087a62ee924df4d125792 (changelog)

merge request to 5.0-0:
https://git.knut.univention.de/univention/ucs/-/merge_requests/44
Comment 5 Christian Castens univentionstaff 2021-01-05 14:13:38 CET
Package: ucs-test
Version: 9.0.7-11A~4.4.0.202101051406
Branch: ucs_4.4-0
Scope: errata4.4-7

4.4-7 commits
4e7536f6dfe7cf683d791b35dfbd37eb584aaf98 (changes)
b46eb2e4d1aee0da489554109f9504e35ee83463 (changelog)
Comment 6 Florian Best univentionstaff 2021-09-24 16:00:19 CEST
Fixed in:
ucs-test (10.0.6-51)
3dd15675c55b | Bug #52144: add generic copy() method