judges 0.21.0 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f826572e575f1c40c556cb5c0de0a2ca5aaf39554ef9d4219e517c6d8492236b
-  data.tar.gz: 7f16f70af23a1fb4903934939133015289f06d4a276c32a567356dd60e3fa75f
+  metadata.gz: 17df0b6c88dd19b576bc701737985db26986e9818eb3c87b36180be92fca54f6
+  data.tar.gz: 48ef9ea5656a6bd78264f0b1f7021555ffa92eaf5be71d311d60e59ab0d3c0c8
 SHA512:
-  metadata.gz: f3b4e89fcd538caddf490ca8769c2142e6c06bec83eb99582bf426f48ce35617ce169d08d604a9d4ee2780d88f0cc877651fabd863efffc5cb2d1ee96cf6a6ec
-  data.tar.gz: 4dd669683164fcd9402f9f4043b5fab0faa086cc68de9f0a6dc1f6e78f9959f5a8a2f18eb3c8fed23986d735010e71c964313090acba222966803b02bd8ba9e3
+  metadata.gz: 722ef79a6af057ba1f4cf37860212b6a3c08f7aff838dfb4e8034e7970e27c4efe98e66da9c774e45707999b9e70966eacd19e4a1478a54a798d12b1d03b662b
+  data.tar.gz: c9551f302a6f0f02ef81e4a2ef5631c98bc2ad08a39982563749b29867d0bf06258a5e7a167179f70f899031cf2ffe097a014e9480a0a767192915d68f78479a

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)
@@ -85,12 +85,12 @@ GEM
     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)
@@ -120,7 +120,7 @@ GEM
     loofah (2.22.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.12.0)
-    loog (0.5.2)
+    loog (0.5.3)
     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/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.0'
   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).to_s,
+              headers:
+            )
+          )
+        end
+      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).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.0'
 end

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').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').to_return(body: '1')
+    stub_request(:get, 'http://example.org/stdout/42').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'
@@ -76,53 +78,75 @@ 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
+    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.0
 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