judges 0.25.1 → 0.25.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9985ec7aa091bbab98e33915b990f49cc3d6370ab0949f0b6b2820f848430b52
-  data.tar.gz: a15f89c6a0be6b38fb4d0d2dc6f0c89fcb9149920e02692d9bcd4112271dbf32
+  metadata.gz: 793749ccbd3ca4e609283aa272ccb8d47c0b1c67f6c2dc10b0282844aff12715
+  data.tar.gz: 8c02acf326f92a4819f6a4c8e76f956fa56a68623c0b9c1948ee87466b220566
 SHA512:
-  metadata.gz: b0b6120abac3e27fb11609e1932d7cb5a1d0317572c1f57c9b9642fdd021ca7552b8305ec364056d1d8184c4782f38a74ed20d7016139512627af3d3c1868d37
-  data.tar.gz: 3f55277c36cff1c75798c0b24cb44a9f9a71a96a3be552f449942b49971d4a4be7b95c05f200533d4e476f839cee80f1a343beef942e19ad3bd9a11dea83cedc
+  metadata.gz: bb5f2d0b72d068eeb686f4458e0a1b2241eeb0a54ff5dbf64025170f6607fcf8c840d7bb9ba53c4b43762256d99ceb479e3311094c55db6e66ed30225362911c
+  data.tar.gz: af19e76d102ff80655af90132aea9fa83d985e96a9c7ea5baab107d74baf6045efce82deb5df70277e0a3186a5bdddd303cb8288617d3df7de2a7d140af61b61

data/Gemfile.lock

@@ -93,8 +93,8 @@ GEM
       bigdecimal
     cucumber-gherkin (27.0.0)
       cucumber-messages (>= 19.1.4, < 23)
-    cucumber-html-formatter (21.6.0)
-      cucumber-messages (> 19, < 25)
+    cucumber-html-formatter (21.7.0)
+      cucumber-messages (> 19, < 27)
     cucumber-messages (22.0.0)
     cucumber-tag-expressions (6.1.0)
     decoor (0.0.1)
@@ -104,7 +104,7 @@ GEM
     erubi (1.13.0)
     ethon (0.16.0)
       ffi (>= 1.15.0)
-    factbase (0.3.0)
+    factbase (0.4.0)
       backtrace (> 0)
       decoor (> 0)
       json (~> 2.7)

data/bin/judges

@@ -150,7 +150,7 @@ class JudgesGLI extend GLI::App
     run_it(c, 'test')
   end
 
-  desc 'Push the factbase to the server'
+  desc 'Push the factbase to the server and unlock it over there'
   command :push do |c|
     c.desc 'Authentication token'
     c.flag([:token])
@@ -173,7 +173,7 @@ class JudgesGLI extend GLI::App
     run_it(c, 'push')
   end
 
-  desc 'Pull the factbase from the server'
+  desc 'Pull the factbase from the server and lock it over there'
   command :pull do |c|
     c.desc 'Authentication token'
     c.flag([:token])

data/judges.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |s|
   s.required_rubygems_version = Gem::Requirement.new('>= 0') if s.respond_to? :required_rubygems_version=
   s.required_ruby_version = '>=3.2'
   s.name = 'judges'
-  s.version = '0.25.1'
+  s.version = '0.25.2'
   s.license = 'MIT'
   s.summary = 'Command-Line Tool for a Factbase'
   s.description =

data/lib/judges/commands/print.rb

@@ -41,6 +41,9 @@ class Judges::Print
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'At lease one argument required' if args.empty?
     f = args[0]

data/lib/judges/commands/pull.rb

@@ -39,6 +39,9 @@ class Judges::Pull
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'Exactly two arguments required' unless args.size == 2
     fb = Factbase.new
@@ -51,15 +54,20 @@ class Judges::Pull
     )
     name = args[0]
     elapsed(@loog) do
-      baza.lock(name, opts['owner'])
       if baza.name_exists?(name)
-        jid = baza.recent(name)
-        unless baza.exit_code(jid).zero?
-          @loog.warn("STDOUT of the job ##{jid} (from the server):\n#{baza.stdout(jid)}")
-          raise "The job ##{jid} ('#{name}') is broken, maybe you should expire it"
+        baza.lock(name, opts['owner'])
+        begin
+          jid = baza.recent(name)
+          unless baza.exit_code(jid).zero?
+            @loog.warn("STDOUT of the job ##{jid} (from the server):\n#{baza.stdout(jid)}")
+            raise "The job ##{jid} ('#{name}') is broken, maybe you should expire it"
+          end
+          fb.import(baza.pull(wait(name, baza, jid, opts['wait'])))
+          Judges::Impex.new(@loog, args[1]).export(fb)
+        rescue StandardError => e
+          baza.unlock(name, opts['owner'])
+          raise e
         end
-        fb.import(baza.pull(wait(name, baza, jid, opts['wait'])))
-        Judges::Impex.new(@loog, args[1]).export(fb)
         throw :"Pulled #{fb.size} facts by the name '#{name}'"
       else
         throw :"There is nothing to pull, the name '#{name}' is absent on the server"

data/lib/judges/commands/push.rb

@@ -39,6 +39,9 @@ class Judges::Push
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'Exactly two arguments required' unless args.size == 2
     name = args[0]
@@ -53,9 +56,12 @@ class Judges::Push
     )
     elapsed(@loog) do
       baza.lock(name, opts['owner'])
-      id = baza.push(name, fb.export, opts['meta'] || [])
-      baza.unlock(name, opts['owner'])
-      throw :"Pushed #{fb.size} facts, job ID is #{id}"
+      begin
+        id = baza.push(name, fb.export, opts['meta'] || [])
+        throw :"Pushed #{fb.size} facts, job ID is #{id}"
+      ensure
+        baza.unlock(name, opts['owner'])
+      end
     end
   end
 end

data/lib/judges/commands/test.rb

@@ -45,6 +45,9 @@ class Judges::Test
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'Exactly one argument required' unless args.size == 1
     dir = args[0]

data/lib/judges/commands/trim.rb

@@ -38,6 +38,9 @@ class Judges::Trim
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'Exactly one argument required' unless args.size == 1
     impex = Judges::Impex.new(@loog, args[0])

data/lib/judges/commands/update.rb

@@ -43,6 +43,9 @@ class Judges::Update
     @loog = loog
   end
 
+  # Run it (it is supposed to be called by the +bin/judges+ script.
+  # @param [Hash] opts Command line options (start with '--')
+  # @param [Array] args List of command line arguments
   def run(opts, args)
     raise 'Exactly two arguments required' unless args.size == 2
     dir = args[0]

data/lib/judges.rb

@@ -25,5 +25,5 @@
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
 module Judges
-  VERSION = '0.25.1'
+  VERSION = '0.25.2'
 end

data/test/commands/test_pull.rb

@@ -39,6 +39,7 @@ class TestPull < Minitest::Test
     stub_request(:get, 'http://example.org/recent/foo.txt').to_return(body: '42')
     stub_request(:get, 'http://example.org/finished/42').to_return(body: 'yes')
     stub_request(:get, 'http://example.org/exit/42.txt').to_return(body: '0')
+    stub_request(:get, 'http://example.org/unlock/foo?owner=none').to_return(status: 302)
     fb = Factbase.new
     fb.insert.foo = 42
     stub_request(:get, 'http://example.org/pull/42.fb').to_return(body: fb.export)
@@ -68,6 +69,7 @@ class TestPull < Minitest::Test
     stub_request(:get, 'http://example.org/finished/42').to_return(body: 'yes')
     stub_request(:get, 'http://example.org/exit/42.txt').to_return(body: '1')
     stub_request(:get, 'http://example.org/stdout/42.txt').to_return(body: 'oops, some trouble here')
+    stub_request(:get, 'http://example.org/unlock/foo?owner=none').to_return(status: 302)
     Dir.mktmpdir do |d|
       file = File.join(d, 'base.fb')
       e =

data/test/commands/test_push.rb

@@ -60,6 +60,7 @@ class TestPush < Minitest::Test
     WebMock.disable_net_connect!
     stub_request(:get, 'http://example.org/lock/foo?owner=none').to_return(status: 302)
     stub_request(:put, 'http://example.org/push/foo').to_return(status: 500)
+    stub_request(:get, 'http://example.org/unlock/foo?owner=none').to_return(status: 302)
     Dir.mktmpdir do |d|
       file = File.join(d, 'base.fb')
       fb = Factbase.new

data/test/commands/test_test.rb

@@ -170,7 +170,7 @@ class TestTest < Minitest::Test
         expected_failure: true
         YAML
       )
-      Judges::Test.new(Loog::VERBOSE).run({}, [d])
+      Judges::Test.new(Loog::NULL).run({}, [d])
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: judges
 version: !ruby/object:Gem::Version
-  version: 0.25.1
+  version: 0.25.2
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-08-13 00:00:00.000000000 Z
+date: 2024-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backtrace