parallel_report_portal 3.0.1 → 3.0.2.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa816eee089ddc4b187164a08791c5d2f1d6a1558b85574313bdd080c7caec25
-  data.tar.gz: 4e2b2123fe5a51e3e583975a19847e02c2c088b171952a93ab6a8a2566733d2d
+  metadata.gz: e1d5cb1312e5c4b45d5581eea41e02df63266535ee5e3b58412f5d9fde5fd0dd
+  data.tar.gz: a10beda80c2270a68c06ff13a18701cf1cd0734eae91bd0c81c37cf0ecfd0623
 SHA512:
-  metadata.gz: 4dce56b66318bdcd43d41e0f8c7a14062f22022438d2c8b4e64a539af3c39b6f56fc8ba5d5af91f65dbcca7407633aea5f5ba19392fa999c04c81dbda57e4eb6
-  data.tar.gz: 807457edccdbf26c310ce41d667ecfc313bd58f85aa502daad3d06b5585f354c6820a144df7307e5bc2227d3d945b1fdbc0eba2eeaf3e200026b75ebe2543c0c
+  metadata.gz: aa2e9ce388cd7de1df78bbcd958c44f6bd329e8ebc7c40c0f550b5e79febd82b2a5913e141fc74f3e80206968f801c9e885539b7df2a4f5c06ae4508b9df11a6
+  data.tar.gz: 02b361da3143afa20ecd5d2111cd74fef9655b848a56174b2624d60193b2f2ee40b996ccd0dbf29abefa165e271836cac7a8c08bcab63a71d08a7c80306be85d

data/lib/parallel_report_portal/cucumber/report.rb

@@ -23,11 +23,10 @@ module ParallelReportPortal
         unknown: 'UNKNOWN'
       }
 
-
       # Create a new instance of the report
       def initialize(ast_lookup = nil)
         @feature = nil
-        @tree = Tree::TreeNode.new( 'root' )
+        @tree = Tree::TreeNode.new('root')
         @ast_lookup = ast_lookup
         check_faraday_compatibility
       end
@@ -38,7 +37,7 @@ module ParallelReportPortal
         end
       end
 
-      # Issued to start a launch. It is possilbe that this method could be called
+      # Issued to start a launch. It is possible that this method could be called
       # from multiple processes for the same launch if this is being run with
       # parallel tests enabled. A temporary launch file will be created (using
       # exclusive locking). The first time this method is called it will write the
@@ -48,13 +47,13 @@ module ParallelReportPortal
       # @param start_time [Integer] the millis from the epoch
       # @return [String] the UUID of this launch
       def launch_started(start_time)
-        ParallelReportPortal.file_open_exlock_and_block(ParallelReportPortal.launch_id_file, 'a+' ) do |file|
+        ParallelReportPortal.file_open_exlock_and_block(ParallelReportPortal.launch_id_file, 'a+') do |file|
           if file.size == 0
             @launch_id = ParallelReportPortal.req_launch_started(start_time)
             file.write(@launch_id)
             file.flush
           else
-             @launch_id = file.readline
+            @launch_id = file.readline
           end
           @launch_id
         end
@@ -65,12 +64,8 @@ module ParallelReportPortal
       # @param clock [Integer] the millis from the epoch
       def launch_finished(clock)
         @tree.postordered_each do |node|
-          response = ParallelReportPortal.req_feature_finished(node.content, clock) unless node.is_root?
-          parse_report_link_from_response(response)
+          ParallelReportPortal.http_repeater { ParallelReportPortal.req_feature_finished(node.content, clock) } unless node.is_root?
         end
-        response = ParallelReportPortal.req_launch_finished(launch_id, clock)
-        parse_report_link_from_response(response)
-        ParallelReportPortal.launch_finished_block.call if ParallelReportPortal.launch_finished_block
       end
 
       # Called to indicate that a feature has started.
@@ -114,7 +109,7 @@ module ParallelReportPortal
           if (using_cucumber_messages? ? test_step : step_source).multiline_arg.doc_string?
             detail << %(\n"""\n#{(using_cucumber_messages? ? test_step : step_source).multiline_arg.content}\n""")
           elsif (using_cucumber_messages? ? test_step : step_source).multiline_arg.data_table?
-            detail << (using_cucumber_messages? ? test_step : step_source).multiline_arg.raw.reduce("\n") {|acc, row| acc << "| #{row.join(' | ')} |\n"}
+            detail << (using_cucumber_messages? ? test_step : step_source).multiline_arg.raw.reduce("\n") { |acc, row| acc << "| #{row.join(' | ')} |\n" }
           end
 
           ParallelReportPortal.req_log(@test_case_id, detail, status_to_level(:trace), clock)
@@ -155,13 +150,14 @@ module ParallelReportPortal
                           else
                             feature.location.file.split(File::SEPARATOR)
                           end
-        ParallelReportPortal.file_open_exlock_and_block(ParallelReportPortal.hierarchy_file, 'a+b' ) do |file|
+
+        ParallelReportPortal.file_open_exlock_and_block(ParallelReportPortal.hierarchy_file, 'a+b') do |file|
           @tree = Marshal.load(File.read(file)) if file.size > 0
           node = @tree.root
           path_components[0..-2].each do |component|
             next_node = node[component]
             unless next_node
-              id = ParallelReportPortal.req_hierarchy(launch_id, "Folder: #{component}", node.content, 'SUITE', [], nil, clock )
+              id = ParallelReportPortal.req_hierarchy(launch_id, "Folder: #{component}", node.content, 'SUITE', [], nil, clock)
               next_node = Tree::TreeNode.new(component, id)
               node << next_node
               node = next_node
@@ -219,7 +215,7 @@ module ParallelReportPortal
         if using_cucumber_messages?
           test_step.hook?
         else
-          ! test_step.source.last.respond_to?(:keyword)
+          !test_step.source.last.respond_to?(:keyword)
         end
       end
 
@@ -235,13 +231,6 @@ module ParallelReportPortal
           LOG_LEVELS.fetch(status, LOG_LEVELS[:info])
         end
       end
-
-      def parse_report_link_from_response(response)
-        if response
-          json = JSON.parse(response.body)
-          ParallelReportPortal.report_url = json['link'] if json['link']
-        end
-      end
     end
   end
 end

data/lib/parallel_report_portal/http.rb

@@ -6,6 +6,8 @@ module ParallelReportPortal
   # REST interface.
   module HTTP
 
+    class HTTPBadGatewayError < StandardError; end
+
     # Creating class level logger and setting log level
     @@logger = Logger.new(STDOUT)
     @@logger.level = Logger::ERROR
@@ -93,9 +95,30 @@ module ParallelReportPortal
     # Send a request to Report Portal to finish a launch.
     # It will bubble up any Faraday connection exceptions.
     def req_launch_finished(launch_id, time)
-      ParallelReportPortal.http_connection.put("launch/#{launch_id}/finish") do |req|
+      @@logger.debug { "Launch finish with ID: '#{launch_id}'" }
+
+      response = ParallelReportPortal.http_connection.put("launch/#{launch_id}/finish") do |req|
         req.body = { end_time: time }.to_json
       end
+
+      @@logger.error { "Launch finish failed with response code #{response.status} -- message #{response.body}" } unless response.success?
+      response
+    end
+
+    def force_stop(launch_id, time)
+      resp = ParallelReportPortal.http_connection.put("launch/#{launch_id}/stop") do |req|
+        req.body = { end_time: time, status: :stopped }.to_json
+      end
+      @@logger.warn { "Failed to force stop: response code #{resp.status} -- message #{resp.body}" } unless resp.success?
+
+      resp
+    end
+
+    def parse_report_link_from_response(response)
+      if response
+        json = JSON.parse(response.body)
+        ParallelReportPortal.report_url = json['link'] if json['link']
+      end
     end
 
     # Send a request to ReportPortal to start a feature.
@@ -145,9 +168,14 @@ module ParallelReportPortal
 
     # Send a request to Report Portal that a feature has completed.
     def req_feature_finished(feature_id, time)
-      ParallelReportPortal.http_connection.put("item/#{feature_id}") do |req|
+      response = ParallelReportPortal.http_connection.put("item/#{feature_id}") do |req|
         req.body = { end_time: time }.to_json
       end
+      @@logger.debug { "Feature finish with ID: '#{feature_id}'" }
+
+
+      @@logger.error { "Feature finish failed with response code #{response.status} -- message #{response.body}" } unless response.success?
+      response
     end
 
     # Send a request to ReportPortal to start a test case.
@@ -236,5 +264,23 @@ module ParallelReportPortal
         end
       end
     end
+
+
+
+    def http_repeater(&block)
+      tries = 0
+      begin
+        tries += 1
+        response = block.call
+        raise HTTPBadGatewayError if response && !response.success? && response.status == 502
+      rescue HTTPBadGatewayError => _e
+        if tries <= 3
+          sleep(1)
+          @@logger.warn { 'HTTP call failed, retrying...' }
+          retry
+        end
+      end
+      response
+    end
   end
 end

data/lib/parallel_report_portal/version.rb

@@ -1,3 +1,3 @@
 module ParallelReportPortal
-  VERSION = "3.0.1"
+  VERSION = "3.0.2.beta.1"
 end

data/lib/parallel_report_portal.rb

@@ -33,6 +33,11 @@ module ParallelReportPortal
     if ParallelReportPortal.parallel?
       if ParallelTests.first_process?
         ParallelTests.wait_for_other_processes_to_finish
+
+        launch_id = File.read(launch_id_file)
+        response = http_repeater { req_launch_finished(launch_id, clock) }
+        response.success? ? parse_report_link_from_response(response) : force_stop(launch_id, clock)
+
         delete_file(launch_id_file)
         delete_file(hierarchy_file)
       end

data/parallel_report_portal.gemspec

@@ -41,6 +41,6 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'faraday-net_http_persistent', '~> 2.1'
   spec.add_runtime_dependency 'faraday-multipart', '~> 1.0', '>= 1.0.4'
   spec.add_runtime_dependency 'parallel_tests', '>= 2.29.1'
-  spec.add_runtime_dependency 'rubytree', '~> 1.0'
+  spec.add_runtime_dependency 'rubytree', '~> 2.0'
   spec.add_runtime_dependency 'net-http-persistent', '~> 4.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: parallel_report_portal
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.0.2.beta.1
 platform: ruby
 authors:
 - Nigel Brookes-Thomas
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-01-02 00:00:00.000000000 Z
+date: 2024-01-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: appraisal
@@ -177,14 +177,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: net-http-persistent
   requirement: !ruby/object:Gem::Requirement