mesa_test 1.1.8 → 1.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cd52a22e9772094f32247bd8785a508f384ceaec1f32d7e6541920d92e730227
-  data.tar.gz: 20b34824551a54e1fd529867d2451972168d663d0f7f799fa19c3e9293477360
+  metadata.gz: b1bc3ecd06c1d2cc95d041b7171dc9c6cda0ccb6536beec798f3969b8f8aa87d
+  data.tar.gz: 4c0109db46572db7e9dc3d2f2aa0dda259b21ba5049448874d5a0d1ec7a85d6a
 SHA512:
-  metadata.gz: 2ee0df9acc559f188b5dcb1dbab9124cc3cb379ad213aff5910943df367f753e8594d4b2bfee597a2c129f46523434a9e7c931707073c340b9db3ccd26fcc9c7
-  data.tar.gz: d1cfbfe89e93881108f0fd5bbe7f0676e2812dbe8310549de9ca971ee044932904ad5062331019bef828d73322ac1aada2c439f0f4533ce2adee7dc51fe022d9
+  metadata.gz: f28bd6704f63e5e37c9fc1bf253458012587febf4c11278e143ec2edf10d826d4c37a183cf0a870c6e92419c2145509407cc3cee37136ee43b02035d7f6254d0
+  data.tar.gz: 92055286204281167d4655278e9b36c35c4bc1c4de04fb8e41180d19cbc21d71f4b50d5f2489628068ffc79bf1fb0794e164b4f6dcdadac2ad46e31cf34c98d8

data/bin/mesa_test

@@ -57,29 +57,31 @@ class MesaTest < Thor
       mod: options[:module].downcase.to_sym,
       github_protocol: s.github_protocol
     )
-    if test_case_name
-      # only testing one test case
-      t = create_and_check_test_case(mesa: m, test_case_name: test_case_name,
-                                     mod: options[:module].downcase.to_sym)
-
-      # run test
-      t.do_one
-
-      # bail out if not submitting
-      return unless options[:submit]
-
-      # submit results
-      shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      s.submit_instance(m, t, force_logs: options['force-logs'])
-    else
-      # run all tests
-      m.each_test_run(mod: options[:module].downcase.to_sym)
-
-      # bail out if not submitting
-      return unless options[:submit]
-
-      shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      s.submit_commit(m, force_logs: options['force-logs'])
+    m.with_mesa_dir do
+      if test_case_name
+        # only testing one test case
+        t = create_and_check_test_case(mesa: m, test_case_name: test_case_name,
+                                       mod: options[:module].downcase.to_sym)
+  
+        # run test
+        t.do_one
+  
+        # bail out if not submitting
+        return unless options[:submit]
+  
+        # submit results
+        shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
+        s.submit_instance(m, t, force_logs: options['force-logs'])
+      else
+        # run all tests
+        m.each_test_run(mod: options[:module].downcase.to_sym)
+  
+        # bail out if not submitting
+        return unless options[:submit]
+  
+        shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
+        s.submit_commit(m, force_logs: options['force-logs'])
+      end
     end
     shell.say "Done.\n", :green
   end
@@ -123,31 +125,32 @@ class MesaTest < Thor
 
     # default value for now; might change to false if we have a bad
     # installation or if a test case is specified
-
-    if test_case_name
-      # just submitting one test case.
-      # this is not an empty submission since there is one test case (the
-      # +empty+ option is ignored in this case)
-
-      t = create_and_check_test_case(mesa: m, test_case_name: test_case_name,
-                                     mod: options[:module].downcase.to_sym)
-      # submit results
-      shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
-      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
-      # no tests (empty)
-      begin
-        m.check_installation
-      rescue MesaDirError
-        shell.say %q{This MESA installation doesn't seem to be compiled } \
-          'properly. Attempting to submit a compilation failure to '\
-          'MESATestHub.', :red
-        empty = true
-      ensure
-        # submit all tests
-        s.submit_commit(m, empty: empty, force_logs: options['force-logs'])
+    m.with_mesa_dir do
+      if test_case_name
+        # just submitting one test case.
+        # this is not an empty submission since there is one test case (the
+        # +empty+ option is ignored in this case)
+  
+        t = create_and_check_test_case(mesa: m, test_case_name: test_case_name,
+                                       mod: options[:module].downcase.to_sym)
+        # submit results
+        shell.say 'Submitting results to ' + s.base_uri + '... ', :blue
+        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
+        # no tests (empty)
+        begin
+          m.check_installation
+        rescue MesaDirError
+          shell.say %q{This MESA installation doesn't seem to be compiled } \
+            'properly. Attempting to submit a compilation failure to '\
+            'MESATestHub.', :red
+          empty = true
+        ensure
+          # submit all tests
+          s.submit_commit(m, empty: empty, force_logs: options['force-logs'])
+        end
       end
     end
   end
@@ -298,7 +301,7 @@ class MesaTest < Thor
 
       # run all tests. Don't be fooled, whether or not a diff happens is set
       # by the diff attribute of the mesa object, set above.
-      m.each_test_run(mod: options[:module].downcase.to_sym)
+      m.with_mesa_dir { m.each_test_run(mod: options[:module].downcase.to_sym) }
     ensure
       # submit all commit and test data
       successfully_submitted = false

data/lib/mesa_test.rb

@@ -609,11 +609,11 @@ class Mesa
       end
     end
 
-    update_mirror
-
     # ensure "work" directory is removed from worktree
     remove
 
+    update_mirror
+
     # create "work" directory with proper commit
     shell.say "\nSetting up worktree repo...", :blue
     FileUtils.mkdir_p mesa_dir
@@ -813,16 +813,6 @@ class Mesa
     File.exist? File.join(mesa_dir, 'testhub.yml')
   end
 
-  private
-
-  # verify that mesa_dir is valid by checking for existence of test_suite
-  # directory for each module (somewhat arbitrary)
-  def check_mesa_dir
-    MesaTestCase.modules.inject(true) do |res, mod|
-      res && dir_or_symlink_exists?(test_suite_dir(mod: mod))
-    end
-  end
-
   # change MESA_DIR for the execution of the block and then revert to the
   # original value
   def with_mesa_dir
@@ -830,7 +820,7 @@ class Mesa
     orig_mesa_dir = ENV['MESA_DIR']
     ENV['MESA_DIR'] = mesa_dir
     shell.say "Temporarily changed MESA_DIR to #{ENV['MESA_DIR']}.", :blue
-
+  
     # do the stuff
     begin
       yield
@@ -841,6 +831,16 @@ class Mesa
     end
   end
 
+  private
+
+  # verify that mesa_dir is valid by checking for existence of test_suite
+  # directory for each module (somewhat arbitrary)
+  def check_mesa_dir
+    MesaTestCase.modules.inject(true) do |res, mod|
+      res && dir_or_symlink_exists?(test_suite_dir(mod: mod))
+    end
+  end
+
   def all_names_ordered(mod: :all)
     load_test_source_data unless @names_to_numbers
     if mod == :all

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mesa_test
 version: !ruby/object:Gem::Version
-  version: 1.1.8
+  version: 1.1.9
 platform: ruby
 authors:
 - William Wolf
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-01 00:00:00.000000000 Z
+date: 2022-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json