onlyoffice_webdriver_wrapper 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 00ff6d75b8882a1088614c55c5543866ff4628df3b26c177757891059b5c69f6
-  data.tar.gz: 490cb05d3e59bf86fcfc3665bbbe02c7333c4b06ec003ca1ac75771058c1c7bc
+  metadata.gz: 365ea413e047186c7bb5597541c1458d22c63d1e83a158abaed307cf0409b98c
+  data.tar.gz: b3ba5fab97ce89e527e939035eb4cc74e2418e5cedc2ee4f08570f8d3ffeb2a8
 SHA512:
-  metadata.gz: 57c04a87025f30da7161703eee3aa9aee4408b3f6daf1f4c2aa11b7a0053a926186ae8c03b0bedbaa08d4283bc0ba9f20b196d94a6f5e7a05f5edd3933f3bc0c
-  data.tar.gz: 6a87c48c0c767cc44244f68c5fc96c7622b82cfdb217cfb8b91931fd55d67a50a781d70f89b5fbf88f091860559789136a67a631d132528ec492affb3bd5c35f
+  metadata.gz: f035f8d66699a0f2ea615448bfa251c7aafd35b57a0d32a4ae995c4484ea5b94b760fb03b87b6916c5c8d43095365d2a3f801a4267b0c5c1c9569497e59d99f1
+  data.tar.gz: ef77f826a28f3c9f88251f7f295ec0c6176e6b1b35a28ccd33f0a96a7e837f28c52651dc035925e783b5de39f857667bf593e17fbf55b7d5792ffc65a00941e0

data/lib/onlyoffice_webdriver_wrapper/helpers/chrome_helper.rb

@@ -24,12 +24,12 @@ module OnlyofficeWebdriverWrapper
     def start_chrome_driver
       prefs = {
         download: {
-          prompt_for_download: false,
-          default_directory: download_directory
+          'prompt_for_download' => false,
+          'default_directory' => download_directory
         },
         profile: {
-          default_content_setting_values: {
-            'automatic_downloads': 1
+          'default_content_setting_values' => {
+            'automatic_downloads' => 1
           }
         },
         credentials_enable_service: false
@@ -38,31 +38,15 @@ module OnlyofficeWebdriverWrapper
       caps[:logging_prefs] = { browser: 'ALL' }
       caps[:proxy] = Selenium::WebDriver::Proxy.new(ssl: "#{@proxy.proxy_address}:#{@proxy.proxy_port}") if @proxy
       caps['chromeOptions'] = { w3c: false }
-      if ip_of_remote_server.nil?
-        switches = add_useragent_to_switches(DEFAULT_CHROME_SWITCHES)
-        options = Selenium::WebDriver::Chrome::Options.new(args: switches,
-                                                           prefs: prefs)
-        webdriver_options = { options: options,
-                              desired_capabilities: caps,
-                              service: chrome_service }
-        begin
-          driver = Selenium::WebDriver.for :chrome, webdriver_options
-        rescue Selenium::WebDriver::Error::WebDriverError,
-               Net::ReadTimeout,
-               Errno::ECONNREFUSED => e
-          OnlyofficeLoggerHelper.log("Starting chrome failed with error: #{e.backtrace}")
-          sleep 10
-          driver = Selenium::WebDriver.for :chrome, webdriver_options
-        end
-        maximize_chrome(driver)
-        driver
-      else
-        caps['chromeOptions'] = {
-          profile: data['zip'],
-          extensions: data['extensions']
-        }
-        Selenium::WebDriver.for(:remote, url: 'http://' + remote_server + ':4444/wd/hub', desired_capabilities: caps)
-      end
+      switches = add_useragent_to_switches(DEFAULT_CHROME_SWITCHES)
+      options = Selenium::WebDriver::Chrome::Options.new(args: switches,
+                                                         prefs: prefs)
+      webdriver_options = { options: options,
+                            desired_capabilities: caps,
+                            service: chrome_service }
+      driver = Selenium::WebDriver.for :chrome, webdriver_options
+      maximize_chrome(driver)
+      driver
     end
 
     private

data/lib/onlyoffice_webdriver_wrapper/helpers/firefox_helper.rb

@@ -12,21 +12,26 @@ module OnlyofficeWebdriverWrapper
     def start_firefox_driver
       profile = Selenium::WebDriver::Firefox::Profile.new
       profile['browser.download.folderList'] = 2
-      profile['browser.helperApps.neverAsk.saveToDisk'] = 'application/doct, application/mspowerpoint, application/msword, application/octet-stream, application/oleobject, application/pdf, application/powerpoint, application/pptt, application/rtf, application/vnd.ms-excel, application/vnd.ms-powerpoint, application/vnd.oasis.opendocument.spreadsheet, application/vnd.oasis.opendocument.text, application/vnd.openxmlformats-officedocument.presentationml.presentation, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet, application/vnd.openxmlformats-officedocument.wordprocessingml.document, application/x-compressed, application/x-excel, application/xlst, application/x-msexcel, application/x-mspowerpoint, application/x-rtf, application/x-zip-compressed, application/zip, image/jpeg, image/pjpeg, image/pjpeg, image/x-jps, message/rfc822, multipart/x-zip, text/csv, text/html, text/html, text/plain, text/richtext'
+      profile['browser.helperApps.neverAsk.saveToDisk'] = read_firefox_files_to_save
       profile['browser.download.dir'] = @download_directory
       profile['browser.download.manager.showWhenStarting'] = false
       profile['dom.disable_window_move_resize'] = false
       options = Selenium::WebDriver::Firefox::Options.new(profile: profile)
       caps = Selenium::WebDriver::Remote::Capabilities.firefox
       caps[:proxy] = Selenium::WebDriver::Proxy.new(ssl: "#{@proxy.proxy_address}:#{@proxy.proxy_port}") if @proxy
-      if ip_of_remote_server.nil?
-        driver = Selenium::WebDriver.for :firefox, options: options, service: firefox_service, desired_capabilities: caps
-        driver.manage.window.size = Selenium::WebDriver::Dimension.new(headless.resolution_x, headless.resolution_y) if headless.running?
-        driver
-      else
-        capabilities = Selenium::WebDriver::Remote::Capabilities.firefox(firefox_profile: profile, desired_capabilities: caps)
-        Selenium::WebDriver.for :remote, desired_capabilities: capabilities, url: 'http://' + ip_of_remote_server + ':4444/wd/hub'
-      end
+      driver = Selenium::WebDriver.for :firefox, options: options, service: firefox_service, desired_capabilities: caps
+      driver.manage.window.size = Selenium::WebDriver::Dimension.new(headless.resolution_x, headless.resolution_y) if headless.running?
+      driver
+    end
+
+    private
+
+    # @return [Array<String>] list of formats to save
+    def read_firefox_files_to_save
+      path_to_file = "#{Dir.pwd}/lib/onlyoffice_webdriver_wrapper/"\
+                     'helpers/firefox_helper/save_to_disk_files.list'
+      OnlyofficeFileHelper::FileHelper.read_array_from_file(path_to_file)
+                                      .join(', ')
     end
   end
 end

data/lib/onlyoffice_webdriver_wrapper/helpers/firefox_helper/save_to_disk_files.list

@@ -0,0 +1,35 @@
+application/doct
+application/mspowerpoint
+application/msword
+application/octet-stream
+application/oleobject
+application/pdf
+application/powerpoint
+application/pptt
+application/rtf
+application/vnd.ms-excel
+application/vnd.ms-powerpoint
+application/vnd.oasis.opendocument.spreadsheet
+application/vnd.oasis.opendocument.text
+application/vnd.openxmlformats-officedocument.presentationml.presentation
+application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
+application/vnd.openxmlformats-officedocument.wordprocessingml.document
+application/x-compressed
+application/x-excel
+application/xlst
+application/x-msexcel
+application/x-mspowerpoint
+application/x-rtf
+application/x-zip-compressed
+application/zip
+image/jpeg
+image/pjpeg
+image/pjpeg
+image/x-jps
+message/rfc822
+multipart/x-zip
+text/csv
+text/html
+text/html
+text/plain
+text/richtext

data/lib/onlyoffice_webdriver_wrapper/helpers/headless_helper.rb

@@ -42,7 +42,7 @@ module OnlyofficeWebdriverWrapper
                                           dimensions: "#{@resolution_x + 1}x#{@resolution_y + 1}x24")
       rescue Exception => e
         OnlyofficeLoggerHelper.log("xvfb not started with problem #{e}")
-        RspecHelper.clean_up(true)
+        WebDriver.clean_up(true)
         @headless_instance = Headless.new(reuse: false,
                                           destroy_at_exit: true,
                                           dimensions: "#{@resolution_x + 1}x#{@resolution_y + 1}x24")

data/lib/onlyoffice_webdriver_wrapper/helpers/headless_helper/real_display_tools.rb

@@ -4,11 +4,7 @@ module OnlyofficeWebdriverWrapper
   # module for getting info about real display
   module RealDisplayTools
     def xrandr_result
-      begin
-        result = `xrandr 2>&1`
-      rescue Exception
-        result = 'xrandr throw an exception'
-      end
+      result = `xrandr 2>&1`
       OnlyofficeLoggerHelper.log("xrandr answer: #{result}".delete("\n"))
       result
     end

data/lib/onlyoffice_webdriver_wrapper/name.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module OnlyofficeWebdriverWrapper
+  module Name
+    STRING = 'onlyoffice_webdriver_wrapper'
+  end
+end

data/lib/onlyoffice_webdriver_wrapper/version.rb

@@ -2,6 +2,6 @@
 
 module OnlyofficeWebdriverWrapper
   module Version
-    STRING = '0.1.2'
+    STRING = '0.2.0'
   end
 end

data/lib/onlyoffice_webdriver_wrapper/webdriver.rb

@@ -46,17 +46,14 @@ module OnlyofficeWebdriverWrapper
     attr_reader :browser_running
     # @return [Symbol] device of which we try to simulate, default - :desktop_linux
     attr_accessor :device
-    attr_accessor :ip_of_remote_server
     attr_accessor :download_directory
     attr_accessor :server_address
     attr_accessor :headless
     # @return [Net::HTTP::Proxy] connection proxy
     attr_accessor :proxy
 
-    singleton_class.class_eval { attr_accessor :web_console_error }
-
     def initialize(browser = :firefox,
-                   remote_server = nil,
+                   _remote_server = nil,
                    device: :desktop_linux,
                    proxy: nil)
       raise WebdriverSystemNotSupported, 'Your OS is not 64 bit. It is not supported' unless os_64_bit?
@@ -67,7 +64,6 @@ module OnlyofficeWebdriverWrapper
 
       @download_directory = Dir.mktmpdir('webdriver-download')
       @browser = browser
-      @ip_of_remote_server = remote_server
       @proxy = proxy
       case browser
       when :firefox
@@ -80,10 +76,6 @@ module OnlyofficeWebdriverWrapper
       @browser_running = true
     end
 
-    def add_web_console_error(log)
-      WebDriver.web_console_error = log
-    end
-
     def open(url)
       url = 'http://' + url unless url.include?('http') || url.include?('file://')
       loop do
@@ -112,7 +104,7 @@ module OnlyofficeWebdriverWrapper
       end
       alert_confirm
       @headless.stop
-      FileUtils.remove_dir(@download_directory) if Dir.exist?(@download_directory)
+      cleanup_download_folder
       @browser_running = false
     end
 
@@ -124,11 +116,9 @@ module OnlyofficeWebdriverWrapper
       nil
     end
 
-    def set_text_to_iframe(element, text)
-      element.click
-      @driver.action.send_keys(text).perform
-    end
-
+    # Get text from all elements with specified xpath
+    # @param array_elements [String] xpath of elements
+    # @return [Array<String>] values of elements
     def get_text_array(array_elements)
       get_elements(array_elements).map { |current_element| get_text(current_element) }
     end
@@ -147,17 +137,6 @@ module OnlyofficeWebdriverWrapper
       webdriver_error("click_and_wait: Wait for element: #{element_to_click} for 30 seconds") if count == 30
     end
 
-    def select_from_list(xpath_value, value)
-      @driver.find_element(:xpath, xpath_value).find_elements(tag_name: 'li').each do |element|
-        next unless element.text == value.to_s
-
-        element.click
-        return true
-      end
-
-      webdriver_error("select_from_list: Option #{value} in list #{xpath_value} not found")
-    end
-
     def select_from_list_elements(value, elements_value)
       index = get_element_index(value, elements_value)
       elements_value[index].click
@@ -206,11 +185,11 @@ module OnlyofficeWebdriverWrapper
     end
 
     def remove_class_by_jquery(selector, class_name)
-      execute_javascript("#{jquery_selector_by_xpath(selector)}.removeClass('#{class_name}');")
+      execute_javascript("$(#{dom_element_by_xpath(selector)}).removeClass('#{class_name}');")
     end
 
     def add_class_by_jquery(selector, class_name)
-      execute_javascript("#{jquery_selector_by_xpath(selector)}.addClass('#{class_name}');")
+      execute_javascript("$(#{dom_element_by_xpath(selector)}).addClass('#{class_name}');")
     end
 
     # Perform drag'n'drop action in one element (for example on big canvas area)
@@ -307,15 +286,6 @@ module OnlyofficeWebdriverWrapper
       @driver.action.context_click(@driver.find_element(:xpath, xpath_name)).perform
     end
 
-    def context_click(xpath, x_coord, y_coord)
-      element = get_element(xpath)
-      if browser == :firefox
-        element.send_keys %i[shift f10]
-      else
-        @driver.action.move_to(element, x_coord.to_i, y_coord.to_i).context_click.perform
-      end
-    end
-
     def click_on_displayed(xpath_name)
       element = get_element_by_display(xpath_name)
       begin
@@ -325,10 +295,6 @@ module OnlyofficeWebdriverWrapper
       end
     end
 
-    def click_on_locator_by_action(xpath)
-      @driver.action.move_to(get_element(xpath)).click.perform
-    end
-
     def click_on_locator_coordinates(xpath_name, right_by, down_by)
       wait_until_element_visible(xpath_name)
       element = @driver.find_element(:xpath, xpath_name)
@@ -383,36 +349,6 @@ module OnlyofficeWebdriverWrapper
       false
     end
 
-    def click_on_one_of_several_with_display_by_text(xpath_several_elements, text_to_click)
-      @driver.find_elements(:xpath, xpath_several_elements).each do |current_element|
-        if current_element.displayed? && text_to_click == current_element.text
-          current_element.click
-          return true
-        end
-      end
-      false
-    end
-
-    def right_click_on_one_of_several_by_text(xpath_several_elements, text_to_click)
-      @driver.find_elements(:xpath, xpath_several_elements).each do |current_element|
-        if text_to_click == current_element.text
-          @driver.action.context_click(current_element).perform
-          return true
-        end
-      end
-      false
-    end
-
-    def click_on_one_of_several_with_display_by_number(xpath_several_elements, number)
-      @driver.find_elements(:xpath, "#{xpath_several_elements}[#{number}]").each do |current_element|
-        if current_element.displayed?
-          current_element.click
-          return true
-        end
-      end
-      false
-    end
-
     def click_on_one_of_several_by_parameter(xpath_several_elements, parameter_name, parameter_value)
       @driver.find_elements(:xpath, xpath_several_elements).each do |current_element|
         if current_element.attribute(parameter_name).include? parameter_value
@@ -423,16 +359,6 @@ module OnlyofficeWebdriverWrapper
       false
     end
 
-    def click_on_one_of_several_by_parameter_and_text(xpath_several_elements, parameter_name, parameter_value, text_to_click)
-      @driver.find_elements(:xpath, xpath_several_elements).each do |current_element|
-        if current_element.attribute(parameter_name).include?(parameter_value) && text_to_click == current_element.text
-          current_element.click
-          return true
-        end
-      end
-      false
-    end
-
     def click_on_one_of_several_xpath_by_number(xpath, number_of_element)
       click_on_locator("(#{xpath})[#{number_of_element}]")
     end
@@ -448,16 +374,6 @@ module OnlyofficeWebdriverWrapper
       OnlyofficeLoggerHelper.log("Moved mouse to element: #{xpath_name}")
     end
 
-    def move_to_one_of_several_displayed_element(xpath_several_elements)
-      get_elements(xpath_several_elements).each do |current_element|
-        if current_element.displayed?
-          move_to_element(current_element)
-          return true
-        end
-      end
-      false
-    end
-
     def mouse_over(xpath_name, x_coordinate = 0, y_coordinate = 0)
       wait_until_element_present(xpath_name)
       @driver.action.move_to(@driver.find_element(:xpath, xpath_name), x_coordinate.to_i, y_coordinate.to_i).perform
@@ -494,32 +410,6 @@ module OnlyofficeWebdriverWrapper
       end
     end
 
-    def get_elements_from_array_before_some(xpath_several_elements, xpath_for_some)
-      elements = get_elements(xpath_several_elements)
-      result = []
-      some_element = get_element(xpath_for_some)
-      return result if some_element.nil?
-
-      elements.each do |current|
-        break if current == some_element
-
-        result << current
-      end
-      result
-    end
-
-    def get_elements_from_array_after_some(xpath_several_elements, xpath_for_some)
-      elements = get_elements(xpath_several_elements)
-      some_element = get_element(xpath_for_some)
-      return elements if some_element.nil?
-
-      elements.each do |current|
-        elements.delete(current)
-        break if current == some_element
-      end
-      elements
-    end
-
     def get_element_by_display(xpath_name)
       @driver.find_elements(:xpath, xpath_name).each do |element|
         return element if element.displayed?
@@ -528,10 +418,14 @@ module OnlyofficeWebdriverWrapper
       webdriver_error("get_element_by_display(#{xpath_name}): invalid selector: Unable to locate an element with the xpath expression")
     end
 
+    # Return count of elements (visible and not visible)
+    # @param xpath_name [String] xpath to find
+    # @param only_visible [True, False] count only visible elements?
+    # @return [Integer] element count
     def get_element_count(xpath_name, only_visible = true)
       if element_present?(xpath_name)
         elements = @driver.find_elements(:xpath, xpath_name)
-        only_visible ? elements.delete_if { |element| @browser == :firefox && !element.displayed? }.length : elements.length
+        only_visible ? elements.delete_if { |element| !element.displayed? }.length : elements.length
       else
         0
       end
@@ -579,6 +473,9 @@ module OnlyofficeWebdriverWrapper
       webdriver_error("Element #{xpath_name} not visible for #{timeout} seconds")
     end
 
+    # Check if any element of xpath is displayed
+    # @param xpath_several_elements [String] xpath to check
+    # @return [True, False] result of visibility
     def one_of_several_elements_displayed?(xpath_several_elements)
       @driver.find_elements(:xpath, xpath_several_elements).each do |current_element|
         return true if current_element.displayed?
@@ -598,10 +495,6 @@ module OnlyofficeWebdriverWrapper
       end
     end
 
-    def remove_element(xpath)
-      execute_javascript("element = document.evaluate(\"#{xpath}\", document, null, XPathResult.ANY_TYPE, null).iterateNext();if (element !== null) {element.parentNode.removeChild(element);};")
-    end
-
     # Select frame as current
     # @param [String] xpath_name name of current xpath
     def select_frame(xpath_name = '//iframe', count_of_frames = 1)
@@ -655,12 +548,6 @@ module OnlyofficeWebdriverWrapper
                              "singleNodeValue.removeAttribute('#{attribute}');")
     end
 
-    def select_text_from_page(xpath_name)
-      wait_until_element_visible(xpath_name)
-      elem = get_element xpath_name
-      @driver.action.key_down(:control).click(elem).send_keys('a').key_up(:control).perform
-    end
-
     def select_combo_box(xpath_name, select_value, select_by = :value)
       wait_until_element_visible(xpath_name)
       option = Selenium::WebDriver::Support::Select.new(get_element(xpath_name))
@@ -671,13 +558,8 @@ module OnlyofficeWebdriverWrapper
       end
     end
 
-    def get_element_number_by_text(xpath_list, element_text)
-      @driver.find_elements(:xpath, xpath_list).each_with_index do |current_element, index|
-        return index if element_text == current_element.attribute('innerHTML')
-      end
-      nil
-    end
-
+    # Get page source
+    # @return [String] all page source
     def get_page_source
       @driver.execute_script('return document.documentElement.innerHTML;')
     end
@@ -725,5 +607,18 @@ module OnlyofficeWebdriverWrapper
       source = get_page_source
       source.include?('Error 503')
     end
+
+    # Perform cleanup if something went wrong during tests
+    # @param forced [True, False] should cleanup run in all cases
+    def self.clean_up(forced = false)
+      return unless OnlyofficeFileHelper::LinuxHelper.user_name.include?('nct-at') ||
+                    OnlyofficeFileHelper::LinuxHelper.user_name.include?('ubuntu') ||
+                    forced == true
+
+      OnlyofficeFileHelper::LinuxHelper.kill_all('chromedriver')
+      OnlyofficeFileHelper::LinuxHelper.kill_all('geckodriver')
+      OnlyofficeFileHelper::LinuxHelper.kill_all('Xvfb')
+      OnlyofficeFileHelper::LinuxHelper.kill_all_browsers
+    end
   end
 end

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_attributes_helper.rb

@@ -26,13 +26,6 @@ module OnlyofficeWebdriverWrapper
       end
     end
 
-    def get_attribute_from_displayed_element(xpath_name, attribute)
-      @driver.find_elements(:xpath, xpath_name).each do |current_element|
-        return current_element.attribute(attribute) if current_element.displayed?
-      end
-      false
-    end
-
     def get_attributes_of_several_elements(xpath_several_elements, attribute)
       elements = @driver.find_elements(:xpath, xpath_several_elements)
 

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_helper.rb

@@ -16,9 +16,22 @@ module OnlyofficeWebdriverWrapper
     def download(file_url)
       data = Kernel.open(file_url, &:read)
       file = Tempfile.new('onlyoffice-downloaded-file')
-      file.write(data)
+      file.write(data.force_encoding('UTF-8'))
       file.close
       file.path
     end
+
+    # Perform safe cleanup of download folder
+    # @return [Nothing]
+    def cleanup_download_folder
+      return unless Dir.exist?(@download_directory)
+
+      if @download_directory.start_with?(Dir.tmpdir)
+        FileUtils.remove_dir(@download_directory)
+      else
+        OnlyofficeLoggerHelper.log("Download directory #{@download_directory} is not at tmp dir. "\
+                                   'It will be not deleted')
+      end
+    end
   end
 end

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_js_methods.rb

@@ -3,17 +3,24 @@
 module OnlyofficeWebdriverWrapper
   # Methods for webdriver for calling Javascript
   module WebdriverJsMethods
-    def execute_javascript(script)
+    # Execute javascript
+    # @param script [String] code to execute
+    # @param wait_timeout [Integer] wait after JS is executed.
+    #   Some code require some time to execute
+    def execute_javascript(script, wait_timeout: 0)
       result = @driver.execute_script(script)
       OnlyofficeLoggerHelper.log("Executed js: `#{script}` with result: `#{result}`")
+      sleep(wait_timeout)
       result
     rescue Exception => e
       webdriver_error("Exception #{e} in execute_javascript: #{script}")
     end
 
-    # @return [String] string to select by xpath
-    def jquery_selector_by_xpath(xpath)
-      "$(document.evaluate('#{xpath}', document, null, XPathResult.ANY_TYPE, null).iterateNext())"
+    # @param [String] xpath element to select
+    # @return [String] string to select dom by xpath
+    def dom_element_by_xpath(xpath)
+      escaped_xpath = xpath.tr('"', "'")
+      "document.evaluate(\"#{escaped_xpath}\", document, null, XPathResult.ANY_TYPE, null).iterateNext()"
     end
 
     def type_to_locator_by_javascript(xpath_name, text)
@@ -29,8 +36,8 @@ module OnlyofficeWebdriverWrapper
     # @param xpath [Sting] xpath of object
     # @return [Dimensions] size of element
     def element_size_by_js(xpath)
-      width = execute_javascript("return document.evaluate(\"#{xpath.tr('"', "'")}\",document, null, XPathResult.ANY_TYPE, null ).iterateNext().offsetWidth")
-      height = execute_javascript("return document.evaluate(\"#{xpath.tr('"', "'")}\",document, null, XPathResult.ANY_TYPE, null ).iterateNext().offsetHeight")
+      width = execute_javascript("return #{dom_element_by_xpath(xpath)}.offsetWidth")
+      height = execute_javascript("return #{dom_element_by_xpath(xpath)}.offsetHeight")
       Dimensions.new(width, height)
     end
 
@@ -38,8 +45,9 @@ module OnlyofficeWebdriverWrapper
     # @param xpath [Sting] xpath of object
     # @return [CursorPoint] position of element
     def object_absolute_position(xpath)
-      left = execute_javascript("return document.evaluate(\"#{xpath.tr('"', "'")}\",document, null, XPathResult.ANY_TYPE, null ).iterateNext().getBoundingClientRect().left")
-      top = execute_javascript("return document.evaluate(\"#{xpath.tr('"', "'")}\",document, null, XPathResult.ANY_TYPE, null ).iterateNext().getBoundingClientRect().top")
+      bounding = "#{dom_element_by_xpath(xpath)}.getBoundingClientRect()"
+      left = execute_javascript("return #{bounding}.left")
+      top = execute_javascript("return #{bounding}.top")
       Dimensions.new(left, top)
     end
 
@@ -73,5 +81,15 @@ module OnlyofficeWebdriverWrapper
       result = execute_javascript("return #{full_command}")
       result.delete('"')
     end
+
+    # Remove element by its xpath
+    # @param [String] xpath of element to remove
+    # @return [String] result of javascript execution
+    def remove_element(xpath)
+      script = "element = #{dom_element_by_xpath(xpath)};"\
+               'if (element !== null) '\
+               '{element.parentNode.removeChild(element);};'
+      execute_javascript(script)
+    end
   end
 end

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_style_helper.rb

@@ -7,16 +7,7 @@ module OnlyofficeWebdriverWrapper
       get_attribute(xpath, 'style').split(';').each do |current_param|
         return /:\s(.*);?$/.match(current_param)[1] if current_param.include?(parameter_name)
       end
-    end
-
-    def get_style_attributes_of_several_elements(xpath_several_elements, style)
-      @driver.find_elements(:xpath, xpath_several_elements).map do |element|
-        el_style = element.attribute('style')
-        unless el_style.empty?
-          found_style = el_style.split(';').find { |curr_param| curr_param.include?(style) }
-          found_style&.gsub(/\s?#{style}:/, '')
-        end
-      end.compact
+      nil
     end
 
     def set_style_parameter(xpath, attribute, attribute_value)
@@ -30,7 +21,7 @@ module OnlyofficeWebdriverWrapper
     end
 
     def set_style_show_by_xpath(xpath, move_to_center = false)
-      xpath.tr!("'", '"')
+      xpath = xpath.tr("'", '"')
       execute_javascript('document.evaluate( \'' + xpath.to_s +
                              '\' ,document, null, XPathResult.FIRST_ORDERED_NODE_TYPE, null ).singleNodeValue.style.display = "block";')
       return unless move_to_center

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_tab_helper.rb

@@ -9,11 +9,6 @@ module OnlyofficeWebdriverWrapper
       execute_javascript('window.open()')
     end
 
-    def maximize
-      @driver.manage.window.maximize
-      OnlyofficeLoggerHelper.log('Maximized current window')
-    end
-
     def resize_tab(width, height)
       @driver.manage.window.resize_to(width, height)
       OnlyofficeLoggerHelper.log("Resize current window to #{width}x#{height}")
@@ -62,10 +57,6 @@ module OnlyofficeWebdriverWrapper
       switch_to_main_tab
     end
 
-    def close_window
-      @driver.close
-    end
-
     def close_popup_and_switch_to_main_tab
       switch_to_popup
       close_tab

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_type_helper.rb

@@ -20,7 +20,10 @@ module OnlyofficeWebdriverWrapper
         begin
           element.clear
         rescue Exception => e
-          webdriver_error("Error in element.clear #{e} for type_to_locator(#{xpath_name}, #{text_to_send}, #{clear_content}, #{click_on_it}, #{by_action}, #{by_element_send_key})")
+          webdriver_error(e.class, "Error in element.clear #{e} for "\
+                          "type_to_locator(#{xpath_name}, #{text_to_send}, "\
+                          "#{clear_content}, #{click_on_it}, "\
+                          "#{by_action}, #{by_element_send_key})")
         end
       end
 
@@ -37,21 +40,25 @@ module OnlyofficeWebdriverWrapper
             @driver.action.send_keys(symbol).perform
           end
         else
-          @driver.action.send_keys(text_to_send).perform
+          webdriver_bug_8179_workaround(text_to_send)
         end
       end
     end
 
     def type_to_input(xpath_name, text_to_send, clear_content = false, click_on_it = true)
       element = get_element(xpath_name)
-      webdriver_error(Selenium::WebDriver::Error::NoSuchElementError, "type_to_input(#{xpath_name}, #{text_to_send}, #{clear_content}, #{click_on_it}): element not found") if element.nil?
+      if element.nil?
+        webdriver_error(Selenium::WebDriver::Error::NoSuchElementError,
+                        "type_to_input(#{xpath_name}, #{text_to_send}, "\
+                        "#{clear_content}, #{click_on_it}): element not found")
+      end
       element.clear if clear_content
       sleep 0.2
       if click_on_it
         begin
           element.click
         rescue Exception => e
-          webdriver_error("type_to_input(#{xpath_name}, #{text_to_send}, #{clear_content}, #{click_on_it}) error in 'element.click' error: #{e}")
+          webdriver_error(e.class, "type_to_input(#{xpath_name}, #{text_to_send}, #{clear_content}, #{click_on_it}) error in 'element.click' error: #{e}")
         end
         sleep 0.2
       end
@@ -80,10 +87,35 @@ module OnlyofficeWebdriverWrapper
 
     def key_down(xpath, key)
       @driver.action.key_down(get_element(xpath), key).perform
+      sleep(1) # for some reason quick key_down select text in control
     end
 
     def key_up(xpath, key)
       @driver.action.key_up(get_element(xpath), key).perform
     end
+
+    private
+
+    # Workaround for bug with typing with :control
+    # See https://github.com/SeleniumHQ/selenium/issues/8179
+    # for more details
+    def webdriver_bug_8179_workaround(text_to_send)
+      text_to_send = [text_to_send].flatten
+
+      key_modifiers = text_to_send.select { |i| i.is_a?(Symbol) }
+      letters = text_to_send - key_modifiers
+
+      key_modifiers.each do |modifier|
+        @driver.action.key_down(modifier).perform
+      end
+
+      letters.each do |letter|
+        @driver.action.send_keys(letter.to_s).perform
+      end
+
+      key_modifiers.each do |modifier|
+        @driver.action.key_up(modifier).perform
+      end
+    end
   end
 end

data/lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_user_agent_helper.rb

@@ -3,10 +3,18 @@
 module OnlyofficeWebdriverWrapper
   # Module for wokring with webdriver useragent
   module WebdriverUserAgentHelper
-    USERAGENT_ANDROID_PHONE = 'Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MDB08M) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.81 Mobile Safari/537.36'
-    USERAGENT_IPHONE = 'Mozilla/5.0 (iPhone; CPU iPhone OS 9_3_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13F69 Safari/601.1'
-    USERAGENT_IPAD_AIR_2_SAFARI = 'Mozilla/5.0 (iPad; CPU OS 10_0 like Mac OS X) AppleWebKit/602.1.50 (KHTML, like Gecko) Version/10.0 Mobile/14A5346a Safari/602.1'
-    USERAGENT_NEXUS_10_CHROME = 'Mozilla/5.0 (Linux; Android 4.3; Nexus 10 Build/JSS15Q) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.23 Safari/537.36'
+    USERAGENT_ANDROID_PHONE = 'Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MDB08M) '\
+                              'AppleWebKit/537.36 (KHTML, like Gecko) '\
+                              'Chrome/51.0.2704.81 Mobile Safari/537.36'
+    USERAGENT_IPHONE = 'Mozilla/5.0 (iPhone; CPU iPhone OS 9_3_2 like Mac OS X) '\
+                       'AppleWebKit/601.1.46 (KHTML, like Gecko) '\
+                       'Version/9.0 Mobile/13F69 Safari/601.1'
+    USERAGENT_IPAD_AIR_2_SAFARI = 'Mozilla/5.0 (iPad; CPU OS 10_0 like Mac OS X) '\
+                                  'AppleWebKit/602.1.50 (KHTML, like Gecko) '\
+                                  'Version/10.0 Mobile/14A5346a Safari/602.1'
+    USERAGENT_NEXUS_10_CHROME = 'Mozilla/5.0 (Linux; Android 4.3; Nexus 10 Build/JSS15Q) '\
+                                'AppleWebKit/537.36 (KHTML, like Gecko) '\
+                                'Chrome/48.0.2564.23 Safari/537.36'
 
     # @return [String] user agent string for current device
     def user_agent_for_device

metadata

@@ -1,16 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: onlyoffice_webdriver_wrapper
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
+- ONLYOFFICE
 - Pavel Lobashov
 - Oleg Nazarov
 - Dmitry Rotaty
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-18 00:00:00.000000000 Z
+date: 2020-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: headless
@@ -110,10 +111,23 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '6'
-description: onlyoffice_webdriver_wrapper
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+description: ONLYOFFICE Webdriver Wrapper Gem. Used in QA
 email:
 - shockwavenn@gmail.com
-- dafttrick@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -125,10 +139,12 @@ files:
 - lib/onlyoffice_webdriver_wrapper/helpers/bin/geckodriver
 - lib/onlyoffice_webdriver_wrapper/helpers/chrome_helper.rb
 - lib/onlyoffice_webdriver_wrapper/helpers/firefox_helper.rb
+- lib/onlyoffice_webdriver_wrapper/helpers/firefox_helper/save_to_disk_files.list
 - lib/onlyoffice_webdriver_wrapper/helpers/headless_helper.rb
 - lib/onlyoffice_webdriver_wrapper/helpers/headless_helper/real_display_tools.rb
 - lib/onlyoffice_webdriver_wrapper/helpers/headless_helper/ruby_helper.rb
 - lib/onlyoffice_webdriver_wrapper/helpers/os_helper.rb
+- lib/onlyoffice_webdriver_wrapper/name.rb
 - lib/onlyoffice_webdriver_wrapper/version.rb
 - lib/onlyoffice_webdriver_wrapper/webdriver.rb
 - lib/onlyoffice_webdriver_wrapper/webdriver/webdriver_alert_helper.rb
@@ -146,7 +162,12 @@ files:
 homepage: https://github.com/onlyoffice-testing-robot/onlyoffice_webdriver_wrapper
 licenses:
 - AGPL-3.0
-metadata: {}
+metadata:
+  bug_tracker_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_webdriver_wrapper/issues
+  changelog_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_webdriver_wrapper/blob/master/CHANGELOG.md
+  documentation_uri: https://www.rubydoc.info/gems/onlyoffice_webdriver_wrapper
+  homepage_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_webdriver_wrapper
+  source_code_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_webdriver_wrapper
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -155,14 +176,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: ONLYOFFICE Webdriver Wrapper Gem