judges 0.21.0 → 0.22.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f826572e575f1c40c556cb5c0de0a2ca5aaf39554ef9d4219e517c6d8492236b
-  data.tar.gz: 7f16f70af23a1fb4903934939133015289f06d4a276c32a567356dd60e3fa75f
+  metadata.gz: cbef0d45d39354c46895af943ef6180bed47a28b4738bcca9755f9b66a282756
+  data.tar.gz: 8dec5ca451e4499509f7158f9c1b115f301713d3e065becd53be78fc82e75c87
 SHA512:
-  metadata.gz: f3b4e89fcd538caddf490ca8769c2142e6c06bec83eb99582bf426f48ce35617ce169d08d604a9d4ee2780d88f0cc877651fabd863efffc5cb2d1ee96cf6a6ec
-  data.tar.gz: 4dd669683164fcd9402f9f4043b5fab0faa086cc68de9f0a6dc1f6e78f9959f5a8a2f18eb3c8fed23986d735010e71c964313090acba222966803b02bd8ba9e3
+  metadata.gz: 074313a259b93c9979a941231251335481cca8212ecea88df3a25d8c70eefed71b55064e11f48e9f63b2c3d8f15982d9647193515a4ec57a1b6120afdf302485
+  data.tar.gz: 3b5628e0ae419cb8197ab4845165ddcde7de4e02bd6174ca6057dd76b3982475bc036d74a778dbca7f6c0b97f605a836e824d1fd1377d0e92a47212cde879db0

data/.rubocop.yml

@@ -58,3 +58,7 @@ Naming/MethodParameterName:
   MinNameLength: 2
 Layout/EndOfLine:
   EnforcedStyle: lf
+Layout/MultilineAssignmentLayout:
+  Enabled: true
+Layout/FirstHashElementIndentation:
+  EnforcedStyle: consistent

data/Gemfile

@@ -30,7 +30,7 @@ gem 'net-ping', '2.0.8', require: false
 gem 'rake', '13.2.1', require: false
 gem 'random-port', '~>0.0', require: false
 gem 'rspec-rails', '6.1.3', require: false
-gem 'rubocop', '1.65.0', require: false
+gem 'rubocop', '1.65.1', require: false
 gem 'rubocop-performance', '1.21.1', require: false
 gem 'rubocop-rspec', '3.0.3', require: false
 gem 'simplecov', '0.22.0', require: false

data/Gemfile.lock

@@ -71,7 +71,7 @@ GEM
       multi_test (~> 1.1)
       sys-uname (~> 1.2)
     cucumber-ci-environment (10.0.1)
-    cucumber-core (13.0.2)
+    cucumber-core (13.0.3)
       cucumber-gherkin (>= 27, < 28)
       cucumber-messages (>= 20, < 23)
       cucumber-tag-expressions (> 5, < 7)
@@ -79,18 +79,18 @@ GEM
       bigdecimal
     cucumber-gherkin (27.0.0)
       cucumber-messages (>= 19.1.4, < 23)
-    cucumber-html-formatter (21.4.1)
+    cucumber-html-formatter (21.5.0)
       cucumber-messages (> 19, < 25)
     cucumber-messages (22.0.0)
     cucumber-tag-expressions (6.1.0)
     decoor (0.0.1)
     diff-lcs (1.5.1)
-    docile (1.4.0)
+    docile (1.4.1)
     drb (2.2.1)
     erubi (1.13.0)
     ethon (0.16.0)
       ffi (>= 1.15.0)
-    factbase (0.2.0)
+    factbase (0.2.1)
       backtrace (> 0)
       decoor (> 0)
       json (~> 2.7)
@@ -107,7 +107,7 @@ GEM
     ffi (1.17.0-x86_64-darwin)
     ffi (1.17.0-x86_64-linux-gnu)
     gli (2.21.5)
-    hashdiff (1.1.0)
+    hashdiff (1.1.1)
     i18n (1.14.5)
       concurrent-ruby (~> 1.0)
     io-console (0.7.2)
@@ -120,7 +120,7 @@ GEM
     loofah (2.22.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.12.0)
-    loog (0.5.2)
+    loog (0.6.0)
     mini_mime (1.1.5)
     minitest (5.24.1)
     minitest-reporters (1.7.1)
@@ -132,19 +132,19 @@ GEM
     multi_test (1.1.0)
     mutex_m (0.2.0)
     net-ping (2.0.8)
-    nokogiri (1.16.6-aarch64-linux)
+    nokogiri (1.16.7-aarch64-linux)
       racc (~> 1.4)
-    nokogiri (1.16.6-arm-linux)
+    nokogiri (1.16.7-arm-linux)
       racc (~> 1.4)
-    nokogiri (1.16.6-arm64-darwin)
+    nokogiri (1.16.7-arm64-darwin)
       racc (~> 1.4)
-    nokogiri (1.16.6-x64-mingw-ucrt)
+    nokogiri (1.16.7-x64-mingw-ucrt)
       racc (~> 1.4)
-    nokogiri (1.16.6-x86-linux)
+    nokogiri (1.16.7-x86-linux)
       racc (~> 1.4)
-    nokogiri (1.16.6-x86_64-darwin)
+    nokogiri (1.16.7-x86_64-darwin)
       racc (~> 1.4)
-    nokogiri (1.16.6-x86_64-linux)
+    nokogiri (1.16.7-x86_64-linux)
       racc (~> 1.4)
     others (0.0.3)
     parallel (1.25.1)
@@ -153,8 +153,8 @@ GEM
       racc
     psych (5.1.2)
       stringio
-    public_suffix (6.0.0)
-    racc (1.8.0)
+    public_suffix (6.0.1)
+    racc (1.8.1)
     rack (3.1.7)
     rack-session (2.0.0)
       rack (>= 3.0.0)
@@ -187,7 +187,7 @@ GEM
     reline (0.5.9)
       io-console (~> 0.5)
     retries (0.0.5)
-    rexml (3.3.2)
+    rexml (3.3.4)
       strscan
     rspec-core (3.13.0)
       rspec-support (~> 3.13.0)
@@ -206,7 +206,7 @@ GEM
       rspec-mocks (~> 3.13)
       rspec-support (~> 3.13)
     rspec-support (3.13.1)
-    rubocop (1.65.0)
+    rubocop (1.65.1)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
@@ -252,7 +252,7 @@ GEM
     webrick (1.8.1)
     yaml (0.3.0)
     yard (0.9.36)
-    zeitwerk (2.6.16)
+    zeitwerk (2.6.17)
 
 PLATFORMS
   aarch64-linux
@@ -272,7 +272,7 @@ DEPENDENCIES
   rake (= 13.2.1)
   random-port (~> 0.0)
   rspec-rails (= 6.1.3)
-  rubocop (= 1.65.0)
+  rubocop (= 1.65.1)
   rubocop-performance (= 1.21.1)
   rubocop-rspec (= 3.0.3)
   simplecov (= 0.22.0)
@@ -281,4 +281,4 @@ DEPENDENCIES
   yard (= 0.9.36)
 
 BUNDLED WITH
-   2.5.6
+   2.5.16

data/assets/index.xsl

@@ -66,6 +66,12 @@ SOFTWARE.
           article { border: none; }
           header img { width: 3em; height: 3em; }
           .sorter { cursor: pointer; }
+          .S { color: #196F3D; }
+          .T { color: #2471A3; }
+          .I { color: #212F3C; }
+          .F { color: #E74C3C; }
+          .BR { color: gray; }
+          .hidden { color: gray; }
         </style>
         <script type="text/javascript">
           $(function() {
@@ -97,6 +103,12 @@ SOFTWARE.
               <xsl:value-of select="$date"/>
               <xsl:text>.</xsl:text>
               <br/>
+              <xsl:text>Properties with columns: </xsl:text>
+              <xsl:value-of select="$columns"/>
+              <br/>
+              <xsl:text>Hidden properties: </xsl:text>
+              <xsl:value-of select="$hidden"/>
+              <br/>
               <a href="https://github.com/yegor256/factbase">
                 <xsl:text>Factbase</xsl:text>
               </a>
@@ -177,14 +189,14 @@ SOFTWARE.
         <td>
           <xsl:for-each select="$f/*">
             <xsl:text> </xsl:text>
-            <xsl:variable name="visible" select="string-length(substring-before(concat(',', $hidden, ','), concat(',', name(), ','))) = 0"/>
+            <xsl:variable name="visible" select="string-length(substring-before(concat(' ,', $hidden, ','), concat(',', name(), ','))) = 0"/>
             <xsl:if test="string-length(substring-before(concat(',', $columns, ','), concat(',', name(), ','))) = 0">
               <xsl:choose>
                 <xsl:when test="$visible">
                   <xsl:value-of select="name()"/>
                 </xsl:when>
                 <xsl:otherwise>
-                  <span style="color:gray;">
+                  <span class="hidden" title="{.}">
                     <xsl:value-of select="name()"/>
                   </span>
                 </xsl:otherwise>
@@ -218,25 +230,30 @@ SOFTWARE.
       </xsl:when>
       <xsl:otherwise>
         <span>
-          <xsl:attribute name="style">
-            <xsl:text>color:</xsl:text>
-            <xsl:choose>
-              <xsl:when test="$v/@t = 'S'">
-                <xsl:text>#196F3D</xsl:text>
-              </xsl:when>
-              <xsl:when test="$v/@t = 'T'">
-                <xsl:text>#2471A3</xsl:text>
-              </xsl:when>
-              <xsl:when test="$v/@t = 'I'">
-                <xsl:text>#212F3C</xsl:text>
-              </xsl:when>
-              <xsl:when test="$v/@t = 'F'">
-                <xsl:text>#E74C3C</xsl:text>
-              </xsl:when>
-            </xsl:choose>
+          <xsl:attribute name="class">
+            <xsl:value-of select="$v/@t"/>
           </xsl:attribute>
-          <xsl:value-of select="$v"/>
+          <xsl:call-template name="just-value">
+            <xsl:with-param name="v" select="$v"/>
+          </xsl:call-template>
+        </span>
+      </xsl:otherwise>
+    </xsl:choose>
+  </xsl:template>
+  <xsl:template name="just-value">
+    <xsl:param name="v"/>
+    <xsl:choose>
+      <xsl:when test="string-length($v) &gt; 64">
+        <xsl:value-of select="substring($v, 0, 64)"/>
+        <span class="BR">
+          <xsl:text>-</xsl:text>
         </span>
+        <xsl:call-template name="just-value">
+          <xsl:with-param name="v" select="substring($v, 64)"/>
+        </xsl:call-template>
+      </xsl:when>
+      <xsl:otherwise>
+        <xsl:value-of select="$v"/>
       </xsl:otherwise>
     </xsl:choose>
   </xsl:template>

data/bin/judges

@@ -32,7 +32,15 @@ Encoding.default_internal = Encoding::UTF_8
 
 class JudgesGLI extend GLI::App
 
-  loog = Loog::REGULAR
+  def self.run_it(cmd, ruby)
+    cmd.action do |global, options, args|
+      require_relative "../lib/judges/commands/#{ruby}"
+      @@loog.debug("Running '#{ruby}' command...")
+      Object.const_get("Judges::#{ruby.capitalize}").new(@@loog).run(options, args)
+    end
+  end
+
+  @@loog = Loog::REGULAR
 
   program_desc('Automated executor of judges for a factbase')
 
@@ -48,13 +56,13 @@ class JudgesGLI extend GLI::App
 
   pre do |global, command, options, args|
     if global[:verbose]
-      loog = Loog::VERBOSE
+      @@loog = Loog::VERBOSE
     end
-    loog.debug("Judges #{Judges::VERSION}")
-    loog.debug("Factbase #{Factbase::VERSION}")
-    loog.debug("Ruby: #{RUBY_VERSION}-p#{RUBY_PATCHLEVEL}")
-    loog.debug("Current directory: #{Dir.getwd}")
-    loog.debug("Time: #{Time.now.utc.iso8601}")
+    @@loog.debug("Judges #{Judges::VERSION}")
+    @@loog.debug("Factbase #{Factbase::VERSION}")
+    @@loog.debug("Ruby: #{RUBY_VERSION}-p#{RUBY_PATCHLEVEL}")
+    @@loog.debug("Current directory: #{Dir.getwd}")
+    @@loog.debug("Time: #{Time.now.utc.iso8601}")
     true
   end
 
@@ -72,44 +80,29 @@ class JudgesGLI extend GLI::App
     c.switch([:summary], default_value: false)
     c.desc 'Use default logging facility'
     c.switch([:log], default_value: true)
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/update'
-      Judges::Update.new(loog).run(options, args)
-    end
+    run_it(c, 'update')
   end
 
   desc 'Evaluate a single Ruby expression on the factbase'
   command :eval do |c|
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/eval'
-      Judges::Eval.new(loog).run(options, args)
-    end
+    run_it(c, 'eval')
   end
 
   desc 'Join two factbases'
   command :join do |c|
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/join'
-      Judges::Join.new(loog).run(options, args)
-    end
+    run_it(c, 'join')
   end
 
   desc 'Import YAML into a factbase'
   command :import do |c|
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/import'
-      Judges::Import.new(loog).run(options, args)
-    end
+    run_it(c, 'import')
   end
 
   desc 'Remove the facts that are too old'
   command :trim do |c|
     c.desc 'Only the facts that match the expression are deleted'
     c.flag([:query], default_value: '(never)')
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/trim'
-      Judges::Trim.new(loog).run(options, args)
-    end
+    run_it(c, 'trim')
   end
 
   desc 'Print the factbase into a human-readable format (YAML, JSON, etc.)'
@@ -128,18 +121,12 @@ class JudgesGLI extend GLI::App
     c.flag([:hidden], default_value: '_id,_time,_version')
     c.desc 'Print even if target file already exists and is older than the factbase'
     c.switch([:force], default_value: false)
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/print'
-      Judges::Print.new(loog).run(options, args)
-    end
+    run_it(c, 'print')
   end
 
   desc 'Inspect the factbase and print all its possible meta-data'
   command :inspect do |c|
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/inspect'
-      Judges::Inspect.new(loog).run(options, args)
-    end
+    run_it(c, 'inspect')
   end
 
   desc 'Run automated tests for all judges'
@@ -160,10 +147,7 @@ class JudgesGLI extend GLI::App
     c.switch([:quiet], default_value: false)
     c.desc 'Use default logging facility'
     c.switch([:log], default_value: true)
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/test'
-      Judges::Test.new(loog).run(options, args)
-    end
+    run_it(c, 'test')
   end
 
   desc 'Push the factbase to the server'
@@ -184,10 +168,9 @@ class JudgesGLI extend GLI::App
     c.flag([:meta], type: String, multiple: true)
     c.desc 'How many times to retry'
     c.flag([:retries], type: Integer, default_value: 3)
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/push'
-      Judges::Push.new(loog).run(options, args)
-    end
+    c.desc 'Turn on the package compression'
+    c.switch([:zip], default_value: true)
+    run_it(c, 'push')
   end
 
   desc 'Pull the factbase from the server'
@@ -208,10 +191,7 @@ class JudgesGLI extend GLI::App
     c.flag([:owner], default_value: 'default', type: String)
     c.desc 'How many times to retry'
     c.flag([:retries], type: Integer, default_value: 3)
-    c.action do |global, options, args|
-      require_relative '../lib/judges/commands/pull'
-      Judges::Pull.new(loog).run(options, args)
-    end
+    run_it(c, 'pull')
   end
 end
 

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.21.0'
+  s.version = '0.22.1'
   s.license = 'MIT'
   s.summary = 'Command-Line Tool for a Factbase'
   s.description =

data/lib/judges/baza.rb

@@ -38,7 +38,7 @@ require_relative '../judges/elapsed'
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
 class Judges::Baza
-  def initialize(host, port, token, ssl: true, timeout: 30, retries: 3, loog: Loog::NULL)
+  def initialize(host, port, token, ssl: true, timeout: 30, retries: 3, loog: Loog::NULL, compression: true)
     @host = host
     @port = port
     @ssl = ssl
@@ -46,6 +46,7 @@ class Judges::Baza
     @timeout = timeout
     @loog = loog
     @retries = retries
+    @compression = compression
   end
 
   # Push factbase to the server.
@@ -62,18 +63,22 @@ class Judges::Baza
     unless meta.empty?
       hdrs = hdrs.merge('X-Zerocracy-Meta' => meta.map { |v| Base64.encode64(v).gsub("\n", '') }.join(' '))
     end
+    params = {
+      connecttimeout: @timeout,
+      timeout: @timeout,
+      body: data,
+      headers: hdrs
+    }
     elapsed(@loog) do
-      ret = with_retries(max_tries: @retries) do
-        checked(
-          Typhoeus::Request.put(
-            home.append('push').append(name).to_s,
-            body: data,
-            headers: hdrs,
-            connecttimeout: @timeout,
-            timeout: @timeout
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.put(
+              home.append('push').append(name).to_s,
+              @compression ? zipped(params) : params
+            )
           )
-        )
-      end
+        end
       id = ret.body.to_i
       throw :"Pushed #{data.size} bytes to #{@host}, job ID is ##{id}"
     end
@@ -117,20 +122,63 @@ class Judges::Baza
   def finished?(id)
     finished = false
     elapsed(@loog) do
-      ret = with_retries(max_tries: @retries) do
-        checked(
-          Typhoeus::Request.get(
-            home.append('finished').append(id).to_s,
-            headers:
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.get(
+              home.append('finished').append(id).to_s,
+              headers:
+            )
           )
-        )
-      end
+        end
       finished = ret.body == 'yes'
       throw :"The job ##{id} is #{finished ? '' : 'not yet '}finished at #{@host}"
     end
     finished
   end
 
+  # Read and return the stdout of the job.
+  # @param [Integer] id The ID of the job on the server
+  # @return [String] The stdout, as a text
+  def stdout(id)
+    stdout = ''
+    elapsed(@loog) do
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.get(
+              home.append('stdout').append("#{id}.txt").to_s,
+              headers:
+            )
+          )
+        end
+      stdout = ret.body
+      throw :"The stdout of the job ##{id} has #{stdout.split("\n")} lines"
+    end
+    stdout
+  end
+
+  # Read and return the exit code of the job.
+  # @param [Integer] id The ID of the job on the server
+  # @return [Integer] The exit code
+  def exit_code(id)
+    code = 0
+    elapsed(@loog) do
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.get(
+              home.append('exit').append("#{id}.txt").to_s,
+              headers:
+            )
+          )
+        end
+      code = ret.body.to_i
+      throw :"The exit code of the job ##{id} is #{code}"
+    end
+    code
+  end
+
   # Lock the name.
   # @param [String] name The name of the job on the server
   # @param [String] owner The owner of the lock (any string)
@@ -171,14 +219,15 @@ class Judges::Baza
   def recent(name)
     job = 0
     elapsed(@loog) do
-      ret = with_retries(max_tries: @retries) do
-        checked(
-          Typhoeus::Request.get(
-            home.append('recent').append("#{name}.txt").to_s,
-            headers:
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.get(
+              home.append('recent').append("#{name}.txt").to_s,
+              headers:
+            )
           )
-        )
-      end
+        end
       job = ret.body.to_i
       throw :"The recent \"#{name}\" job's ID is ##{job} at #{@host}"
     end
@@ -191,14 +240,15 @@ class Judges::Baza
   def name_exists?(name)
     exists = 0
     elapsed(@loog) do
-      ret = with_retries(max_tries: @retries) do
-        checked(
-          Typhoeus::Request.get(
-            home.append('exists').append(name).to_s,
-            headers:
+      ret =
+        with_retries(max_tries: @retries) do
+          checked(
+            Typhoeus::Request.get(
+              home.append('exists').append(name).to_s,
+              headers:
+            )
           )
-        )
-      end
+        end
       exists = ret.body == 'yes'
       throw :"The name \"#{name}\" #{exists ? 'exists' : "doesn't exist"} at #{@host}"
     end
@@ -215,6 +265,27 @@ class Judges::Baza
     }
   end
 
+  def zipped(params)
+    body = gzip(params.fetch(:body))
+    headers = params
+      .fetch(:headers)
+      .merge({
+        'Content-Type' => 'application/zip',
+        'Content-Encoding' => 'gzip',
+        'Content-Length' => body.size
+      })
+    params.merge(body:, headers:)
+  end
+
+  def gzip(data)
+    ''.dup.tap do |result|
+      io = StringIO.new(result)
+      gz = Zlib::GzipWriter.new(io)
+      gz.write(data)
+      gz.close
+    end
+  end
+
   def home
     Iri.new('')
       .host(@host)

data/lib/judges/commands/pull.rb

@@ -51,9 +51,14 @@ class Judges::Pull
     )
     name = args[0]
     elapsed(@loog) do
+      baza.lock(name, opts['owner'])
       if baza.name_exists?(name)
-        baza.lock(name, opts['owner'])
-        fb.import(baza.pull(wait(name, baza, baza.recent(name), opts['wait'])))
+        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)
         throw :"Pulled #{fb.size} facts by the name '#{name}'"
       else

data/lib/judges/commands/push.rb

@@ -48,7 +48,8 @@ class Judges::Push
       ssl: opts['ssl'],
       timeout: (opts['timeout'] || 30).to_i,
       loog: @loog,
-      retries: (opts['retries'] || 3).to_i
+      retries: (opts['retries'] || 3).to_i,
+      compression: opts.fetch('zip', true)
     )
     elapsed(@loog) do
       baza.lock(name, opts['owner'])

data/lib/judges/commands/update.rb

@@ -64,7 +64,7 @@ class Judges::Update
       loop do
         c += 1
         if c > 1
-          @loog.info("\n\nStarting cycle ##{c}#{opts['max-cycles'] ? " (out of #{opts['max-cycles']})" : ''}...")
+          @loog.info("\nStarting cycle ##{c}#{opts['max-cycles'] ? " (out of #{opts['max-cycles']})" : ''}...")
         end
         delta = cycle(opts, judges, fb, options)
         churn += delta
@@ -103,17 +103,18 @@ class Judges::Update
     churn = Judges::Churn.new(0, 0)
     global = {}
     elapsed(@loog) do
-      done = judges.each_with_index do |p, i|
-        @loog.info("\n👉 Running #{p.name} (##{i}) at #{p.dir.to_rel}...")
-        elapsed(@loog) do
-          c = one_judge(fb, p, global, options)
-          churn += c
-          throw :"👍 The judge #{p.name} modified #{c} facts out of #{fb.size}"
+      done =
+        judges.each_with_index do |p, i|
+          @loog.info("\n👉 Running #{p.name} (##{i}) at #{p.dir.to_rel}...")
+          elapsed(@loog) do
+            c = one_judge(fb, p, global, options)
+            churn += c
+            throw :"👍 The judge #{p.name} modified #{c} facts out of #{fb.size}"
+          end
+        rescue StandardError, SyntaxError => e
+          @loog.warn(Backtrace.new(e))
+          churn << e.message
         end
-      rescue StandardError, SyntaxError => e
-        @loog.warn(Backtrace.new(e))
-        churn << e.message
-      end
       throw :"👍 #{done} judge(s) processed" if churn.errors.empty?
       throw :"❌ #{done} judge(s) processed with #{churn.errors.size} errors"
     end

data/lib/judges/options.rb

@@ -52,32 +52,33 @@ class Judges::Options
       v = v.to_s
       v = "#{v[0..3]}#{'*' * (v.length - 8)}#{v[-4..]}" if v.length > 8
       "#{k} → \"#{v}\""
-    end.join("\n")
+    end.sort.join("\n")
   end
 
   def to_h
-    @to_h ||= begin
-      pp = @pairs || []
-      pp = pp.split(',') if pp.is_a?(String)
-      if pp.is_a?(Array)
-        pp = pp
-          .compact
-          .map(&:strip)
-          .reject(&:empty?)
-          .map { |s| s.split('=', 2) }
-          .map { |a| a.size == 1 ? [a[0], nil] : a }
-          .reject { |a| a[0].empty? }
+    @to_h ||=
+      begin
+        pp = @pairs || []
+        pp = pp.split(',') if pp.is_a?(String)
+        if pp.is_a?(Array)
+          pp = pp
+            .compact
+            .map(&:strip)
+            .reject(&:empty?)
+            .map { |s| s.split('=', 2) }
+            .map { |a| a.size == 1 ? [a[0], nil] : a }
+            .reject { |a| a[0].empty? }
+            .to_h
+        end
+        pp
+          .reject { |k, _| k.nil? }
+          .reject { |k, _| k.is_a?(String) && k.empty? }
           .to_h
+          .transform_values { |v| v.nil? ? 'true' : v }
+          .transform_values { |v| v.is_a?(String) ? v.strip : v }
+          .transform_values { |v| v.is_a?(String) && v.match?(/^[0-9]+$/) ? v.to_i : v }
+          .transform_keys { |k| k.to_s.strip.upcase.to_sym }
       end
-      pp
-        .reject { |k, _| k.nil? }
-        .reject { |k, _| k.is_a?(String) && k.empty? }
-        .to_h
-        .transform_values { |v| v.nil? ? 'true' : v }
-        .transform_values { |v| v.is_a?(String) ? v.strip : v }
-        .transform_values { |v| v.is_a?(String) && v.match?(/^[0-9]+$/) ? v.to_i : v }
-        .transform_keys { |k| k.to_s.strip.upcase.to_sym }
-    end
   end
 
   # Get option by name.

data/lib/judges.rb

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

data/test/commands/test_print.rb

@@ -50,11 +50,13 @@ class TestPrint < Minitest::Test
     fb = Factbase.new
     10.times do
       f = fb.insert
+      f._id = 44
       f.what = SecureRandom.hex(10)
       f.when = Time.now
       f.details = 'hey, друг'
       f.ticket = 42
       f.ticket = 55
+      f.long_property = 'test_' * 100
     end
     Dir.mktmpdir do |d|
       f = File.join(d, 'base.fb')

data/test/commands/test_pull.rb

@@ -38,6 +38,7 @@ class TestPull < Minitest::Test
     stub_request(:get, 'http://example.org/exists/foo').to_return(body: 'yes')
     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')
     fb = Factbase.new
     fb.insert.foo = 42
     stub_request(:get, 'http://example.org/pull/42.fb').to_return(body: fb.export)
@@ -58,4 +59,32 @@ class TestPull < Minitest::Test
       fb.import(File.binread(file))
     end
   end
+
+  def test_fail_pull_when_job_is_broken
+    WebMock.disable_net_connect!
+    stub_request(:get, 'http://example.org/lock/foo?owner=none').to_return(status: 302)
+    stub_request(:get, 'http://example.org/exists/foo').to_return(body: 'yes')
+    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: '1')
+    stub_request(:get, 'http://example.org/stdout/42.txt').to_return(body: 'oops, some trouble here')
+    Dir.mktmpdir do |d|
+      file = File.join(d, 'base.fb')
+      e =
+        assert_raises do
+          Judges::Pull.new(Loog::NULL).run(
+            {
+              'token' => '000',
+              'host' => 'example.org',
+              'port' => 80,
+              'ssl' => false,
+              'wait' => 10,
+              'owner' => 'none'
+            },
+            ['foo', file]
+          )
+        end
+      assert(e.message.include?('expire it'), e)
+    end
+  end
 end

data/test/test_baza.rb

@@ -22,8 +22,10 @@
 
 require 'minitest/autorun'
 require 'webmock/minitest'
+require 'webrick'
 require 'loog'
 require 'socket'
+require 'stringio'
 require 'random-port'
 require_relative '../lib/judges'
 require_relative '../lib/judges/baza'
@@ -65,6 +67,26 @@ class TestBaza < Minitest::Test
     )
   end
 
+  def test_exit_code_check
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://example.org/exit/42.txt').to_return(
+      status: 200, body: '0'
+    )
+    assert(
+      Judges::Baza.new('example.org', 443, '000').exit_code(42).zero?
+    )
+  end
+
+  def test_stdout_read
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://example.org/stdout/42.txt').to_return(
+      status: 200, body: 'hello!'
+    )
+    assert(
+      !Judges::Baza.new('example.org', 443, '000').stdout(42).empty?
+    )
+  end
+
   def test_simple_pull
     WebMock.disable_net_connect!
     stub_request(:get, 'https://example.org/pull/333.fb').to_return(
@@ -76,53 +98,76 @@ class TestBaza < Minitest::Test
   end
 
   def test_real_http
-    req = with_http_server(200, 'yes') do |baza|
-      baza.name_exists?('simple')
-    end
-    assert(req.include?("User-Agent: judges #{Judges::VERSION}\r\n"))
+    req =
+      with_http_server(200, 'yes') do |baza|
+        baza.name_exists?('simple')
+      end
+    assert_equal("judges #{Judges::VERSION}", req['user-agent'])
   end
 
   def test_push_with_meta
-    req = with_http_server(200, 'yes') do |baza|
-      baza.push('simple', 'hello, world!', ['boom!', 'хей!'])
-    end
-    assert(req.include?("X-Zerocracy-Meta: Ym9vbSE= 0YXQtdC5IQ==\r\n"))
+    req =
+      with_http_server(200, 'yes') do |baza|
+        baza.push('simple', 'hello, world!', ['boom!', 'хей!'])
+      end
+    assert_equal('Ym9vbSE= 0YXQtdC5IQ==', req['x-zerocracy-meta'])
   end
 
   def test_push_with_big_meta
-    req = with_http_server(200, 'yes') do |baza|
-      baza.push(
-        'simple',
-        'hello, world!',
-        [
-          'pages_url:https://zerocracy.github.io/zerocracy.html',
-          'others:https://zerocracy.github.io/zerocracy.html',
-          'duration:59595'
-        ]
-      )
-    end
-    assert(req.join.include?('X-Zerocracy-Meta: '))
+    req =
+      with_http_server(200, 'yes') do |baza|
+        baza.push(
+          'simple',
+          'hello, world!',
+          [
+            'pages_url:https://zerocracy.github.io/zerocracy.html',
+            'others:https://zerocracy.github.io/zerocracy.html',
+            'duration:59595'
+          ]
+        )
+      end
+    assert(req['x-zerocracy-meta'])
+  end
+
+  def test_push_compressed_content
+    skip # this test is not stable, see https://github.com/yegor256/judges/issues/105
+    req =
+      with_http_server(200, 'yes') do |baza|
+        baza.push('simple', 'hello, world!', %w[meta1 meta2 meta3])
+      end
+    assert_equal('application/zip', req.content_type)
+    assert_equal('gzip', req['content-encoding'])
+    body = Zlib::GzipReader.zcat(StringIO.new(req.body))
+    assert_equal('hello, world!', body)
+  end
+
+  def test_push_compression_disabled
+    req =
+      with_http_server(200, 'yes', compression: false) do |baza|
+        baza.push('simple', 'hello, world!', %w[meta1 meta2 meta3])
+      end
+    assert_equal('application/octet-stream', req.content_type)
+    assert_equal('hello, world!', req.body)
   end
 
   private
 
-  def with_http_server(code, response)
+  def with_http_server(code, response, opts = {})
+    opts = { ssl: false, timeout: 1 }.merge(opts)
     WebMock.enable_net_connect!
-    req = []
+    req = WEBrick::HTTPRequest.new(WEBrick::Config::HTTP)
     host = '127.0.0.1'
     RandomPort::Pool::SINGLETON.acquire do |port|
       server = TCPServer.new(host, port)
-      t = Thread.new do
-        socket = server.accept
-        loop do
-          line = socket.gets
-          break if line == "\r\n"
-          req << line
+      t =
+        Thread.new do
+          socket = server.accept
+          req.parse(socket)
+          req.body
+          socket.puts "HTTP/1.1 #{code} OK\r\nContent-Length: #{response.length}\r\n\r\n#{response}"
+          socket.close
         end
-        socket.puts "HTTP/1.1 #{code} OK\r\nContent-Length: #{response.length}\r\n\r\n#{response}"
-        socket.close
-      end
-      yield Judges::Baza.new(host, port, '0000', ssl: false, timeout: 1)
+      yield Judges::Baza.new(host, port, '0000', **opts)
       t.join
     end
     req

metadata

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