onlyoffice_rspec_result_parser 0.1.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e0498075e6fc9529b3ab53e7b99fef574e7e02a8a185e59d7889010402c97a5
-  data.tar.gz: ad609118fa567af873dcbc09875942b01d306bf91ea80ef4dfe2346e51d3f271
+  metadata.gz: fca51873554ec893514f458b6b5efbf163208383283601dee566dffe10b8204e
+  data.tar.gz: d3941700b839dd7e52d6d1e6f5614c3d002739189db734fcc8e614ab718ec024
 SHA512:
-  metadata.gz: 3060d19a305e4fa19ee1b1c7c9371f141d93f345fcb2dfa9178b3b71f7d3a332a767d70691b6d7a457eed737342f07ccce632d851ceed5af31e6b0630bc44b7a
-  data.tar.gz: f455696715c4921e44dc463c68e28ca075860fd09a63418da47649a377f096846f4dd7ba65ec92a09910f73c79200a414736e100a8262d86eebcf33dab7148ef
+  metadata.gz: 9d3f5b9169f457d29eb9fa2d94a6aea725f41b95eea798d6218faa41fed472095713e4cd739500d31d1337c38b2af90acf9254087d1cacb3b2ecbfadcea832a1
+  data.tar.gz: f7f2309be7ae1e2ee1a00721fb00518e7a0c5a3b895a9aec87fd374ee5ac9802a9fa039a867411e59e370d3b348d7cba25c44716dda265f97767a7e97c23f075

data/lib/onlyoffice_rspec_result_parser/helper/string_helper.rb

@@ -3,11 +3,18 @@
 module OnlyofficeRspecResultParser
   # Class for string methods
   class StringHelper
+    # Get style param from string
+    # @param [String] string with style
+    # @param [String] param name
+    # @return [String] result
     def self.get_style_param(string, param)
       m = string.match(/#{param}:\s(.*);/)
       m[1]
     end
 
+    # Delete pixel names from string
+    # @param [String] string to delete
+    # @return [String] result
     def self.delete_px(string)
       string.gsub 'px', ''
     end

data/lib/onlyoffice_rspec_result_parser/rspec_result_parser/describe.rb

@@ -11,6 +11,9 @@ module OnlyofficeRspecResultParser
       @result = result
     end
 
+    # Find last info on level
+    # @param [Integer] level
+    # @return [Object] result
     def find_last_on_lvl(level)
       level.zero? ? self : @child.last.find_last_on_lvl(level - 1)
     end

data/lib/onlyoffice_rspec_result_parser/rspec_result_parser/example.rb

@@ -19,10 +19,11 @@ module OnlyofficeRspecResultParser
 
     def fetch_data(data)
       @text = data.css('span').first.text
-      @passed = data[:class].split(' ')[1]
-      if @passed == 'failed'
+      @passed = data[:class].split[1]
+      case @passed
+      when 'failed'
         fetch_failed_data(data)
-      elsif @passed == 'passed'
+      when 'passed'
         @duration = data.css('span')[1].text
       end
     end
@@ -50,7 +51,7 @@ module OnlyofficeRspecResultParser
     # @return [String] text with clickable link
     def format_link(text)
       links = URI.extract(text)
-      links.each do |current_link|
+      links.uniq.each do |current_link|
         if current_link.end_with?('png', 'jpg')
           text.gsub!(current_link, link_url_50_percent(current_link))
         elsif current_link.start_with?('http')

data/lib/onlyoffice_rspec_result_parser/rspec_result_parser/result_creator.rb

@@ -6,6 +6,10 @@ module OnlyofficeRspecResultParser
     # @return [Describe] result describe
     attr_reader :final_result
 
+    # Push result to end
+    # @param [String] describe to pusth
+    # @param [Integer] level to check
+    # @return [Object] result
     def push_to_end(describe, level)
       if level.zero?
         @final_result = describe

data/lib/onlyoffice_rspec_result_parser/rspec_result_parser/rspec_result.rb

@@ -3,6 +3,7 @@
 module OnlyofficeRspecResultParser
   # class for storing rspec result
   class RspecResult
+    # @return [Integer] margin value
     LEVEL_MARGIN = 15
 
     attr_accessor :processing, :result, :time, :total, :describe
@@ -12,31 +13,49 @@ module OnlyofficeRspecResultParser
     attr_accessor :passed_count
     # @return [Integer] how much tests pending
     attr_accessor :pending_count
+    # @return [String] html page of RspecResult
+    attr_reader :page
+
+    def initialize(page)
+      @page = page
+    end
 
     # @return [Integer] total test result count
     def total_tests_count
       failed_count + passed_count
     end
 
-    def parse_page(page, with_describe_info = true)
-      @describe = get_describe(page) if with_describe_info
-      @processing = get_processing(page)
-      @result = get_total_result(page)
-      @time = get_total_time(page)
-      @total = get_totals(page)
-      @failed_count = get_failed_count(page)
-      @passed_count = get_passed_count(page)
-      @pending_count = get_pending_count(page)
+    # Parse current page
+    # @param [Boolean] with_describe_info should info be parsed
+    # @return [RspecResult] result of parse
+    def parse_page(with_describe_info: true)
+      return self unless valid_html?
+
+      @describe = fetch_describe if with_describe_info
+      @processing = fetch_processing
+      @result = fetch_total_result
+      @time = fetch_total_time
+      @total = fetch_totals
+      @failed_count = fetch_failed_count
+      @passed_count = fetch_passed_count
+      @pending_count = fetch_pending_count
       self
     end
 
+    # @return [True, False] is html code is valid
+    def valid_html?
+      return false unless page.at_css('div.results')
+
+      true
+    end
+
     private
 
-    def get_describe(page)
+    def fetch_describe
       results = ResultCreator.new
       page.at_css('div.results').xpath('./div').each do |current|
         results.push_to_end(parse_describe(current),
-                            get_describe_level(current))
+                            fetch_describe_level(current))
       end
       results.final_result
     end
@@ -52,14 +71,14 @@ module OnlyofficeRspecResultParser
       describe_obj
     end
 
-    def get_describe_level(describe)
+    def fetch_describe_level(describe)
       style_string = describe.xpath('./dl')[0][:style]
       style_parameter = StringHelper.get_style_param(style_string,
                                                      'margin-left')
       StringHelper.delete_px(style_parameter).to_i / LEVEL_MARGIN
     end
 
-    def get_processing(page)
+    def fetch_processing
       processing = page.css('script:contains("moveProgressBar")').last
       return '0' unless processing
 
@@ -71,11 +90,11 @@ module OnlyofficeRspecResultParser
       end
     end
 
-    def get_total_result(page)
-      get_totals(page).include?(' 0 failures')
+    def fetch_total_result
+      fetch_totals.include?(' 0 failures')
     end
 
-    def get_totals(page)
+    def fetch_totals
       totals = ''
       total_elem = page.css('script:contains(" example")')
       return totals unless total_elem
@@ -88,19 +107,19 @@ module OnlyofficeRspecResultParser
       end
     end
 
-    def get_failed_count(page)
+    def fetch_failed_count
       page.xpath("//*[@class='example failed']").length
     end
 
-    def get_passed_count(page)
+    def fetch_passed_count
       page.xpath("//*[@class='example passed']").length
     end
 
-    def get_pending_count(page)
+    def fetch_pending_count
       page.xpath("//*[@class='example not_implemented']").length
     end
 
-    def get_total_time(page)
+    def fetch_total_time
       total_time = page.css('script:contains("Finished in")')
                        .text
                        .match(/>(.*?)</)

data/lib/onlyoffice_rspec_result_parser/rspec_result_parser.rb

@@ -17,6 +17,9 @@ module OnlyofficeRspecResultParser
     class << self
       attr_accessor :example_index
 
+      # Parse rspec html
+      # @param [String] html_path path to file
+      # @return [Object] result of parse
       def parse_rspec_html(html_path)
         page = Nokogiri::HTML(read_file(html_path))
         parse_test_result(page)
@@ -31,14 +34,26 @@ module OnlyofficeRspecResultParser
 
       alias parse_rspec_html_string parse_rspec_html
 
+      # Get failed count
+      # @param [String] html_path path to file
+      # @return [Integer] failed count
       def get_failed_cases_count_from_html(html_path)
         page = Nokogiri::HTML(read_file(html_path))
-        RspecResult.new.parse_page(page, true).failed_count
+        result = RspecResult.new(page).parse_page
+        return 0 unless result.valid_html?
+
+        result.failed_count
       end
 
+      # Get total case count
+      # @param [String] html_path path to file
+      # @return [Integer] total count
       def get_total_result_of_rspec_html(html_path)
         page = Nokogiri::HTML(read_file(html_path))
-        RspecResult.new.parse_page(page, true).total
+        result = RspecResult.new(page).parse_page
+        return '' unless result.valid_html?
+
+        result.total
       end
 
       # @param page [String] data in page
@@ -47,7 +62,7 @@ module OnlyofficeRspecResultParser
       # @return [RspecResult] result of parsing
       def parse_test_result(page, with_describe_info: true)
         ResultParser.example_index = 0
-        RspecResult.new.parse_page(page, with_describe_info)
+        RspecResult.new(page).parse_page(with_describe_info: with_describe_info)
       end
 
       private

data/lib/onlyoffice_rspec_result_parser/version.rb

@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
+# Main namespace of gem
 module OnlyofficeRspecResultParser
-  VERSION = '0.1.0'
+  # @return [String] gem version
+  VERSION = '1.0.0'
+  # @return [String] gem name
   NAME = 'onlyoffice_rspec_result_parser'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: onlyoffice_rspec_result_parser
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 1.0.0
 platform: ruby
 authors:
 - ONLYOFFICE
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-20 00:00:00.000000000 Z
+date: 2022-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -26,20 +26,132 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: overcommit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: '13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.49.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.49.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: '1'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.9.20
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.9.20
 description: Gem to parse rspec results. Needed in wrata project
 email:
 - shockwavenn@gmail.com
@@ -55,15 +167,16 @@ files:
 - lib/onlyoffice_rspec_result_parser/rspec_result_parser/result_creator.rb
 - lib/onlyoffice_rspec_result_parser/rspec_result_parser/rspec_result.rb
 - lib/onlyoffice_rspec_result_parser/version.rb
-homepage: https://github.com/onlyoffice-testing-robot/onlyoffice_rspec_result_parser
+homepage: https://github.com/ONLYOFFICE-QA/onlyoffice_rspec_result_parser
 licenses:
 - AGPL-3.0
 metadata:
-  bug_tracker_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_rspec_result_parser/issues
-  changelog_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_rspec_result_parser/blob/master/CHANGELOG.md
+  bug_tracker_uri: https://github.com/ONLYOFFICE-QA/onlyoffice_rspec_result_parser/issues
+  changelog_uri: https://github.com/ONLYOFFICE-QA/onlyoffice_rspec_result_parser/blob/master/CHANGELOG.md
   documentation_uri: https://www.rubydoc.info/gems/onlyoffice_rspec_result_parser
-  homepage_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_rspec_result_parser
-  source_code_uri: https://github.com/onlyoffice-testing-robot/onlyoffice_rspec_result_parser
+  homepage_uri: https://github.com/ONLYOFFICE-QA/onlyoffice_rspec_result_parser
+  source_code_uri: https://github.com/ONLYOFFICE-QA/onlyoffice_rspec_result_parser
+  rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -72,14 +185,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.3.7
 signing_key: 
 specification_version: 4
 summary: Gem to parse rspec results