mesa_test 1.1.2 → 1.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ef2c5c38af03ad5ebdf41472063fd0d1255de221e7402befc392d0f94d9d683
-  data.tar.gz: 35afaaa02352b6a9a60759dfff51ad8312c756bdaa92fc7a480ff728a5d71660
+  metadata.gz: 62a113597b8d49b277030dabfb72f46b42bf4c6d33993d86fd3bd98c513bbf44
+  data.tar.gz: 355642660dddb5a4b87934c0f8fcb9c43a9fa4f900d8d809f62cda3cafcb8d89
 SHA512:
-  metadata.gz: d7538fe20d847d901733db1a40dccb6426e5d5b66ee3b727f5394c45babb2baf082ab9576873447e5a25cec46972add717825a27e1a156bf31565d5b569f967d
-  data.tar.gz: 10a7b0356b8b84f7f4e8df45f3d0b18a38f1ef30c71bd12b02947b83c6c23b29c778b3cbb9dfa04087fbf4454bf90bb92ecf7ecf9b2c02abf7bf0dfb765a3a70
+  metadata.gz: 6db7a9183094ce00ca25adc3c51a02570a4d381432be02ee61ebf48d774b4e90904f61171e8530f8db6f2b6dc819bc78ff644ec8e85ff4b0da5d45ea70bf9f37
+  data.tar.gz: a42908b873207b17ed8195f26c4bde8dd479e2c69dc49e3f2a406e4190109f95522aeb466741b2f18fd1d9966356bf22b7539ec2630af37322b7544bc3ac4cfc

data/bin/mesa_test

@@ -36,6 +36,8 @@ class MesaTest < Thor
                        '~/.mesa_test.yml are correct rather than checking ' \
                        'with user.',
                  type: :boolean, default: true, aliases: '-f'
+  option 'force-logs', desc: 'Submit logs for test case(s), even if they pass',
+                       type: :boolean, default: false
   option :submit, desc: 'Submit results (if any) to MesaTestHub.',
                   type: :boolean, default: true
   option :module, desc: 'Which module to test and/or submit, if applicable.',
@@ -68,7 +70,7 @@ class MesaTest < Thor
 
       # submit results
       shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      s.submit_instance(m, t)
+      s.submit_instance(m, t, force_logs: options['force-logs'])
     else
       # run all tests
       m.each_test_run(mod: options[:module].downcase.to_sym)
@@ -77,7 +79,7 @@ class MesaTest < Thor
       return unless options[:submit]
 
       shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      s.submit_commit(m)
+      s.submit_commit(m, force_logs: options['force-logs'])
     end
     shell.say "Done.\n", :green
   end
@@ -98,6 +100,8 @@ class MesaTest < Thor
                        '~/.mesa_test.yml are correct rather than checking ' \
                        'with user.',
                  type: :boolean, default: true, aliases: '-f'
+  option 'force-logs', desc: 'Submit logs for test case(s), even if they pass',
+                       type: :boolean, default: false
   option :module, desc: 'Which module to test and/or submit, if applicable.',
                   type: :string, default: 'all', aliases: '-m'
   option :empty, desc: 'Whether any test cases will be submitted. Useful for '\
@@ -129,7 +133,7 @@ class MesaTest < Thor
                                      mod: options[:module].downcase.to_sym)
       # submit results
       shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      s.submit_instance(m, t)
+      s.submit_instance(m, t, force_logs: options['force-logs'])
       shell.say "Done.\n", :green
     else
       # submitting compilation information and either all tests (entire) or
@@ -143,7 +147,7 @@ class MesaTest < Thor
         empty = true
       ensure
         # submit all tests
-        s.submit_commit(m, empty: empty)
+        s.submit_commit(m, empty: empty, force_logs: options['force-logs'])
       end
     end
   end
@@ -298,7 +302,10 @@ class MesaTest < Thor
     ensure
       # submit all commit and test data
       successfully_submitted = false
-      successfully_submitted = s.submit_commit(m) if options[:submit]
+      if options[:submit]
+        successfully_submitted = s.submit_commit(
+          m, force_logs: options['force-logs'])
+      end
 
       # if requested, installation and submission are successful, destroy the
       # directory

data/lib/mesa_test.rb

@@ -212,7 +212,7 @@ e-mail and password will be stored in plain text.'
   end
 
   def load_computer_data
-    data_hash = YAML.safe_load(File.read(config_file), [Symbol])
+    data_hash = YAML.load(File.read(config_file))
     @computer_name = data_hash['computer_name']
     @email = data_hash['email']
     @password = data_hash['password']
@@ -334,7 +334,7 @@ e-mail and password will be stored in plain text.'
 
   # submit entire commit's worth of test cases, OR submit compilation status
   # and NO test cases
-  def submit_commit(mesa, empty: false)
+  def submit_commit(mesa, empty: false, force_logs: false)
     unless mesa.install_attempted?
       raise MesaDirError, 'No testhub.yml file found in installation; '\
                           'must attempt to install before subitting.'
@@ -386,8 +386,8 @@ e-mail and password will be stored in plain text.'
           test_case_hash.each do |tc_name, test_case|
             # get at each individual test case, see if it failed, and if it
             # did, submit its log files
-            unless test_case.passed?
-              res &&= submit_test_log(test_case)
+            if !test_case.passed? || force_logs
+              res &&= submit_test_log(test_case, skip_passing: !force_logs)
             end
           end
         end
@@ -401,7 +401,7 @@ e-mail and password will be stored in plain text.'
 
   # submit results for a single test case instance. Does *not* report overall
   # compilation status to testhub. Use an empty commit submission for that
-  def submit_instance(mesa, test_case)
+  def submit_instance(mesa, test_case, force_logs: false)
     unless mesa.install_attempted?
       raise MesaDirError, 'No testhub.yml file found in installation; '\
                           'must attempt to install before subitting.'
@@ -436,7 +436,9 @@ e-mail and password will be stored in plain text.'
       shell.say "\nSuccessfully submitted instance of #{test_case.test_name} "\
                 "for commit #{mesa.sha}.", :green
       # submit logs if test failed
-      return submit_test_log(test_case) unless test_case.passed?
+      if !test_case.passed? || force_logs
+        return submit_test_log(test_case, skip_passing: !force_logs)
+      end
       true
     end
   end
@@ -481,9 +483,11 @@ e-mail and password will be stored in plain text.'
   end
 
   # send build log to the logs server
-  def submit_test_log(test_case)
+  def submit_test_log(test_case, skip_passing: true)
     # skip submission if mesa was never installed or if the test passed
-    return true if !test_case.mesa.installed? || test_case.passed?
+    if !test_case.mesa.installed? || (test_case.passed? && skip_passing)
+      return true
+    end
 
     # don't even try unless we have a logs token set
     unless logs_token
@@ -497,14 +501,13 @@ e-mail and password will be stored in plain text.'
 
     # report out results
     if !res.is_a? Net::HTTPOK
-      shell.say "Failed to submit out.txt and mk.txt to the LOGS server for "\
-                "test case #{test_case.test_name} in commit "\
-                "#{test_case.mesa.sha}.", :red
+      shell.say "Failed to submit logs for test case #{test_case.test_name} "\
+                "in commit #{test_case.mesa.sha}.", :red
       false
     else
-      shell.say "Successfully submitted out.txt and mk.txt to the LOGS "\
-                "server for test case #{test_case.test_name} in commit "\
-                "#{test_case.mesa.sha}.", :green
+      shell.say "Successfully submitted logs for test case "\
+                "#{test_case.test_name} in commit #{test_case.mesa.sha}.",
+                :green
       true
     end
   end
@@ -623,7 +626,7 @@ class Mesa
 
   def update_mirror
     shell.say "\nFetching MESA history...", :blue
-    command = "git -C #{mirror_dir} fetch origin"
+    command = "git -C #{mirror_dir} fetch origin --prune"
     shell.say command
     # fail loudly
     return if bash_execute(command)
@@ -990,7 +993,7 @@ class MesaTestCase
       raise TestCaseDirError.new('No results found for test case '\
                                  "#{test_name}.")
     end
-    YAML.safe_load(File.read(testhub_file), [Symbol])
+    YAML.load(File.read(testhub_file))
   end
 
   # whether or not a test case has passed; only has meaning

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mesa_test
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.6
 platform: ruby
 authors:
 - William Wolf
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-19 00:00:00.000000000 Z
+date: 2021-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -81,7 +81,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: Command line tool for running and reporting the MESA test suites.