UI tests for "Automember": check search filter

https://pagure.io/freeipa/issue/7721

Reviewed-By: Mohammad Rizwan Yusuf <myusuf@redhat.com>
Reviewed-By: Petr Cech <pcech@redhat.com>
Reviewed-By: Petr Vobornik <pvoborni@redhat.com>

ipatests/test_webui/test_automember.py

@@ -21,8 +21,7 @@
 Automember tests
 """
 
-from ipatests.test_webui.ui_driver import UI_driver
+from ipatests.test_webui.ui_driver import UI_driver, screenshot
-from ipatests.test_webui.ui_driver import screenshot
 import ipatests.test_webui.data_hostgroup as hostgroup
 from ipatests.test_webui.test_host import host_tasks
 import pytest
@@ -54,12 +53,63 @@ HOST_GROUP_DATA = {
 @pytest.mark.tier1
 class test_automember(UI_driver):
 
+    def setup(self):
+        super(test_automember, self).setup()
+        self.init_app()
+
+    def add_user_group_rule(self, pkey):
+        self.add_record(
+            ENTITY,
+            {'pkey': pkey, 'add': [('combobox', 'cn', pkey)]},
+            facet='searchgroup'
+        )
+
+    def add_host_group_rule(self, pkey):
+        self.add_record(
+            ENTITY,
+            {'pkey': pkey, 'add': [('combobox', 'cn', pkey)]},
+            facet='searchhostgroup'
+        )
+
+    def add_user(self, pkey, name, surname):
+        self.add_record('user', {
+            'pkey': pkey,
+            'add': [
+                ('textbox', 'uid', pkey),
+                ('textbox', 'givenname', name),
+                ('textbox', 'sn', surname),
+            ]
+        })
+
+    def add_user_group(self, pkey, description):
+        self.add_record('group', {
+            'pkey': pkey,
+            'add': [
+                ('textbox', 'cn', pkey),
+                ('textarea', 'description', description),
+            ]
+        })
+
+    def add_host_group(self, pkey, description):
+        self.add_record('hostgroup', {
+            'pkey': pkey,
+            'add': [
+                ('textbox', 'cn', pkey),
+                ('textarea', 'description', description),
+            ]
+        })
+
+    def delete_user_group_rule(self, pkey):
+        self.delete(ENTITY, [{'pkey': pkey}], facet='searchgroup')
+
+    def delete_host_group_rule(self, pkey):
+        self.delete(ENTITY, [{'pkey': pkey}], facet='searchhostgroup')
+
     @screenshot
     def test_crud(self):
         """
         Basic CRUD: automember
         """
-        self.init_app()
 
         # user group rule
         self.basic_crud(ENTITY, USER_GROUP_DATA,
@@ -92,7 +142,6 @@ class test_automember(UI_driver):
         """
         Test automember rebuild membership feature for hosts
         """
-        self.init_app()
 
         host_util = host_tasks()
         host_util.driver = self.driver
@@ -102,24 +151,14 @@ class test_automember(UI_driver):
         host2 = 'web2.%s' % domain
 
         # Add a hostgroup
-        self.add_record('hostgroup', {
+        self.add_host_group('webservers', 'webservers')
-            'pkey': 'webservers',
-            'add': [
-                ('textbox', 'cn', 'webservers'),
-                ('textarea', 'description', 'webservers'),
-            ]
-        })
 
         # Add hosts
         self.add_record('host', host_util.get_data("web1", domain))
         self.add_record('host', host_util.get_data("web2", domain))
 
         # Add an automember rule
-        self.add_record(
+        self.add_host_group_rule('webservers')
-            'automember',
-            {'pkey': 'webservers', 'add': [('combobox', 'cn', 'webservers')]},
-            facet='searchhostgroup'
-        )
 
         # Add a condition for automember rule
         self.navigate_to_record('webservers')
@@ -172,51 +211,25 @@ class test_automember(UI_driver):
         # Delete hostgroup, hosts and automember rule
         self.delete('hostgroup', [{'pkey': 'webservers'}])
         self.delete('host', [{'pkey': host1}, {'pkey': host2}])
-        self.delete('automember', [{'pkey': 'webservers'}],
+        self.delete_host_group_rule('webservers')
-                    facet='searchhostgroup')
 
     @screenshot
     def test_rebuild_membership_users(self):
         """
         Test automember rebuild membership feature for users
         """
-        self.init_app()
 
         # Add a group
-        self.add_record('group', {
+        self.add_user_group('devel', 'devel')
-            'pkey': 'devel',
-            'add': [
-                ('textbox', 'cn', 'devel'),
-                ('textarea', 'description', 'devel'),
-            ]
-        })
 
         # Add a user
-        self.add_record('user', {
+        self.add_user('dev1', 'Dev', 'One')
-            'pkey': 'dev1',
-            'add': [
-                ('textbox', 'uid', 'dev1'),
-                ('textbox', 'givenname', 'Dev'),
-                ('textbox', 'sn', 'One'),
-            ]
-        })
 
         # Add another user
-        self.add_record('user', {
+        self.add_user('dev2', 'Dev', 'Two')
-            'pkey': 'dev2',
-            'add': [
-                ('textbox', 'uid', 'dev2'),
-                ('textbox', 'givenname', 'Dev'),
-                ('textbox', 'sn', 'Two'),
-            ]
-        })
 
         # Add an automember rule
-        self.add_record(
+        self.add_user_group_rule('devel')
-            'automember',
-            {'pkey': 'devel', 'add': [('combobox', 'cn', 'devel')]},
-            facet='searchgroup'
-        )
 
         # Add a condition for automember rule
         self.navigate_to_record('devel')
@@ -269,4 +282,42 @@ class test_automember(UI_driver):
         # Delete group, users and automember rule
         self.delete('group', [{'pkey': 'devel'}])
         self.delete('user', [{'pkey': 'dev1'}, {'pkey': 'dev2'}])
-        self.delete('automember', [{'pkey': 'devel'}], facet='searchgroup')
+        self.delete_user_group_rule('devel')
+
+    @screenshot
+    def test_search_user_group_rule(self):
+        pkey = 'search123'
+        self.add_user_group(pkey, '')
+        self.add_user_group_rule(pkey)
+
+        for text in ['search123', 'search', ' search ', 'SEARCH', '123']:
+            self.apply_search_filter(text)
+            self.wait_for_request()
+            self.assert_record(pkey)
+
+        for text in ['search1234', '321']:
+            self.apply_search_filter(text)
+            self.wait_for_request()
+            self.assert_record(pkey, negative=True)
+
+        self.delete_user_group_rule(pkey)
+        self.delete('group', [{'pkey': pkey}])
+
+    @screenshot
+    def test_search_host_group_rule(self):
+        pkey = 'search123'
+        self.add_host_group(pkey, '')
+        self.add_host_group_rule(pkey)
+
+        for text in ['search123', 'search', ' search ', 'SEARCH', '123']:
+            self.apply_search_filter(text)
+            self.wait_for_request()
+            self.assert_record(pkey)
+
+        for text in ['search1234', '321']:
+            self.apply_search_filter(text)
+            self.wait_for_request()
+            self.assert_record(pkey, negative=True)
+
+        self.delete_host_group_rule(pkey)
+        self.delete('hostgroup', [{'pkey': pkey}])

ipatests/test_webui/ui_driver.py

@@ -42,6 +42,7 @@ try:
     from selenium.common.exceptions import StaleElementReferenceException
     from selenium.common.exceptions import WebDriverException
     from selenium.common.exceptions import ElementClickInterceptedException
+    from selenium.webdriver.common.action_chains import ActionChains
     from selenium.webdriver.common.desired_capabilities import DesiredCapabilities
     from selenium.webdriver.common.keys import Keys
     from selenium.webdriver.common.by import By
@@ -807,6 +808,11 @@ class UI_driver:
                 parent = self.get_form()
         self.fill_text(search_field_s, value, parent)
 
+    def apply_search_filter(self, value):
+        self.fill_search_filter(value)
+        actions = ActionChains(self.driver)
+        actions.send_keys(Keys.ENTER).perform()
+
     def add_multivalued(self, name, value, parent=None):
         """
         Add new value to multivalued textbox