Merged in [19153] from rjsparks@nostrum.com:

    Scroll to buttons before clicking on them in Selenium tests. Fixes ietf-tools#3314.
 - Legacy-Id: 19154
Note: SVN reference [19153] has been migrated to Git commit d61e265

Author: rjsparks

ietf/group/tests_js.py

@@ -45,6 +45,7 @@ def _click_milestone_submit_button(self, label):
             expected_conditions.element_to_be_clickable((By.CSS_SELECTOR, submit_button_selector))
         )
         self.assertIn(label, submit_button.text)
+        self.scroll_to_element(submit_button)
         submit_button.click()
 
     def _assert_milestone_changed(self):
@@ -80,6 +81,7 @@ def test_add_milestone(self):
             expected_conditions.element_to_be_clickable(
                 (By.CSS_SELECTOR, 'button.add-milestone')
             ))
+        self.scroll_to_element(add_milestone_button)
         add_milestone_button.click()
 
         edit_div = self.wait.until(

ietf/meeting/tests_js.py

@@ -921,6 +921,7 @@ def test_agenda_view_group_filter_toggle(self):
 
         # Click the group button
         group_button = self.get_agenda_filter_group_button(wait, group_acronym)
+        self.scroll_to_element(group_button)
         group_button.click()
 
         # Check visibility
@@ -986,6 +987,7 @@ def _schedule_session(meeting, session):
 
         # enable hackathon group
         group_button = self.get_agenda_filter_group_button(wait, 'hackathon')
+        self.scroll_to_element(group_button)
         group_button.click()
         self.assert_agenda_item_visibility(['mars', 'hackathon'])
 
@@ -1027,6 +1029,7 @@ def test_agenda_view_group_filter_toggle_without_replace_state(self):
 
         # Be sure we're at the URL we think we're at before we click
         self.assertEqual(self.driver.current_url, url)
+        self.scroll_to_element(group_button)
         group_button.click()  # click!
 
         expected_url = '%s?show=%s' % (url, group_acronym)
@@ -1136,6 +1139,7 @@ def test_session_materials_modal(self):
             ),
             'Modal open button not found or not clickable',
         )
+        self.scroll_to_element(open_modal_button)
         open_modal_button.click()
         WebDriverWait(self.driver, 2).until(
             expected_conditions.visibility_of(modal_div),
@@ -1168,6 +1172,7 @@ def test_session_materials_modal(self):
             ),
             'Modal close button not found or not clickable',
         )
+        self.scroll_to_element(close_modal_button)
         close_modal_button.click()
         WebDriverWait(self.driver, 2).until(
             expected_conditions.invisibility_of_element(modal_div),
@@ -1187,6 +1192,7 @@ def test_session_materials_modal(self):
             ),
             'Modal open button not found or not clickable for refresh test',
         )
+        self.scroll_to_element(open_modal_button)
         open_modal_button.click()
         WebDriverWait(self.driver, 2).until(
             expected_conditions.visibility_of(modal_div),
@@ -1644,6 +1650,7 @@ def advance_month():
             button = WebDriverWait(self.driver, 2).until(
                 expected_conditions.element_to_be_clickable(
                     (By.CSS_SELECTOR, 'div#calendar button.fc-next-button')))
+            self.scroll_to_element(button)
             button.click()
 
         seen = set()