judges 0.20.0 → 0.22.0

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/test.rb

@@ -82,6 +82,12 @@ class Judges::Test
               test_one(fb, opts, j, n, yaml, assert: false)
             end
             test_one(fb, opts, judge, tname, yaml)
+            yaml['after']&.each do |rb|
+              @loog.info("Running #{rb} assertion script...")
+              $fb = fb
+              $loog = @loog
+              load(File.join(judge.dir, rb), true)
+            end
             tests += 1
           rescue StandardError => e
             @loog.warn(Backtrace.new(e))

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/judge.rb

@@ -70,9 +70,10 @@ class Judges::Judge
 
   # Get the name of the .rb script in the judge.
   def script
-    s = Dir.glob(File.join(@dir, '*.rb')).first
-    raise "No *.rb scripts in #{@dir.to_rel}" if s.nil?
-    File.basename(s)
+    b = "#{File.basename(@dir)}.rb"
+    files = Dir.glob(File.join(@dir, '*.rb')).map { |f| File.basename(f) }
+    raise "No #{b} script in #{@dir.to_rel} among #{files}" unless files.include?(b)
+    b
   end
 
   # Return all .yml tests files.

data/lib/judges/judges.rb

@@ -24,6 +24,20 @@ require_relative '../judges'
 require_relative 'judge'
 
 # Collection of all judges to run.
+#
+# In the directory +dir+ the following structure must be maintained:
+#
+#  dir/
+#    judge-one/
+#      judge-one.rb
+#      other files...
+#    judge-two/
+#      judge-two.rb
+#      other files...
+#
+# The name of a directory of a judge must be exactly the same as the
+# name of the +.rb+ script inside the directory.
+#
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
@@ -45,9 +59,12 @@ class Judges::Judges
   # Iterate over them all.
   # @yield [Judge]
   def each
-    Dir.glob(File.join(@dir, '**/*.rb')).each do |f|
-      d = File.dirname(File.absolute_path(f))
-      yield Judges::Judge.new(d, @lib, @loog)
+    return to_enum(__method__) unless block_given?
+    Dir.glob(File.join(@dir, '*')).each do |d|
+      next unless File.directory?(d)
+      b = File.basename(d)
+      next unless File.exist?(File.join(d, "#{b}.rb"))
+      yield Judges::Judge.new(File.absolute_path(d), @lib, @loog)
     end
   end
 

data/lib/judges/options.rb

@@ -50,34 +50,35 @@ class Judges::Options
   def to_s
     to_h.map do |k, v|
       v = v.to_s
-      v = "#{v[0..3]}#{'*' * (v.length - 4)}" if v.length > 8
+      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.20.0'
+  VERSION = '0.22.0'
 end

data/test/commands/test_import.rb

@@ -36,7 +36,7 @@ class TestImport < Minitest::Test
     Dir.mktmpdir do |d|
       file = File.join(d, 'base.fb')
       yaml = File.join(d, 'input.yml')
-      File.write(
+      save_it(
         yaml,
         <<-YAML
         -

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/commands/test_test.rb

@@ -32,9 +32,9 @@ require_relative '../../lib/judges/commands/test'
 class TestTest < Minitest::Test
   def test_positive
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), '$fb.query("(eq foo 42)").each { |f| f.bar = 4 }')
-      File.write(
-        File.join(d, 'something.yml'),
+      save_it(File.join(d, 'foo/foo.rb'), '$fb.query("(eq foo 42)").each { |f| f.bar = 4 }')
+      save_it(
+        File.join(d, 'foo/something.yml'),
         <<-YAML
         input:
           -
@@ -51,9 +51,9 @@ class TestTest < Minitest::Test
 
   def test_negative
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), '$fb.query("(eq foo 42)").each { |f| f.bar = 4 }')
-      File.write(
-        File.join(d, 'something.yml'),
+      save_it(File.join(d, 'foo/foo.rb'), '$fb.query("(eq foo 42)").each { |f| f.bar = 4 }')
+      save_it(
+        File.join(d, 'foo/something.yml'),
         <<-YAML
         input:
           -
@@ -72,9 +72,9 @@ class TestTest < Minitest::Test
 
   def test_with_options
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), '$fb.insert.foo = $options.bar')
-      File.write(
-        File.join(d, 'something.yml'),
+      save_it(File.join(d, 'foo/foo.rb'), '$fb.insert.foo = $options.bar')
+      save_it(
+        File.join(d, 'foo/something.yml'),
         <<-YAML
         input: []
         options:
@@ -90,13 +90,10 @@ class TestTest < Minitest::Test
 
   def test_with_before
     Dir.mktmpdir do |d|
-      home = File.join(d, 'judges')
-      FileUtils.mkdir_p(File.join(home, 'first'))
-      File.write(File.join(d, 'judges/first/the-first.rb'), 'x = $fb.size; $fb.insert.foo = x')
-      FileUtils.mkdir_p(File.join(home, 'second'))
-      File.write(File.join(d, 'judges/second/the-second.rb'), '$fb.insert.bar = 55')
-      File.write(
-        File.join(d, 'judges/second/something.yml'),
+      save_it(File.join(d, 'first/first.rb'), 'x = $fb.size; $fb.insert.foo = x')
+      save_it(File.join(d, 'second/second.rb'), '$fb.insert.bar = 55')
+      save_it(
+        File.join(d, 'second/something.yml'),
         <<-YAML
         input:
           -
@@ -110,15 +107,15 @@ class TestTest < Minitest::Test
           - /fb/f[bar=55]
         YAML
       )
-      Judges::Test.new(Loog::NULL).run({}, [home])
+      Judges::Test.new(Loog::NULL).run({}, [d])
     end
   end
 
   def test_one_judge_negative
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), '')
-      File.write(
-        File.join(d, 'x.yml'),
+      save_it(File.join(d, 'foo/foo.rb'), '')
+      save_it(
+        File.join(d, 'foo/x.yml'),
         <<-YAML
         input: []
         expected:
@@ -130,4 +127,20 @@ class TestTest < Minitest::Test
       end
     end
   end
+
+  def test_with_after_assertion
+    Dir.mktmpdir do |d|
+      save_it(File.join(d, 'foo/foo.rb'), '$fb.insert.foo = 42;')
+      save_it(File.join(d, 'foo/assert.rb'), 'raise unless $fb.size == 1')
+      save_it(
+        File.join(d, 'foo/x.yml'),
+        <<-YAML
+        input: []
+        after:
+          - assert.rb
+        YAML
+      )
+      Judges::Test.new(Loog::NULL).run({}, [d])
+    end
+  end
 end

data/test/commands/test_update.rb

@@ -34,7 +34,7 @@ require_relative '../../lib/judges/commands/update'
 class TestUpdate < Minitest::Test
   def test_build_factbase_from_scratch
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), 'return if $fb.size > 2; $fb.insert.zzz = $options.foo_bar + 1')
+      save_it(File.join(d, 'foo/foo.rb'), 'return if $fb.size > 2; $fb.insert.zzz = $options.foo_bar + 1')
       file = File.join(d, 'base.fb')
       Judges::Update.new(Loog::NULL).run({ 'option' => ['foo_bar=42'] }, [d, file])
       fb = Factbase.new
@@ -50,7 +50,7 @@ class TestUpdate < Minitest::Test
       fb = Factbase.new
       fb.insert.foo_bar = 42
       File.binwrite(file, fb.export)
-      File.write(File.join(d, 'foo.rb'), '$fb.insert.tt = 4')
+      save_it(File.join(d, 'foo/foo.rb'), '$fb.insert.tt = 4')
       Judges::Update.new(Loog::NULL).run({ 'max-cycles' => 1 }, [d, file])
       fb = Factbase.new
       fb.import(File.binread(file))
@@ -62,7 +62,7 @@ class TestUpdate < Minitest::Test
 
   def test_update_with_error
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), 'this$is$a$broken$Ruby$script')
+      save_it(File.join(d, 'foo/foo.rb'), 'this$is$a$broken$Ruby$script')
       file = File.join(d, 'base.fb')
       Judges::Update.new(Loog::NULL).run({ 'quiet' => true, 'max-cycles' => 2 }, [d, file])
     end
@@ -71,7 +71,7 @@ class TestUpdate < Minitest::Test
   def test_update_with_error_no_quiet
     assert_raises do
       Dir.mktmpdir do |d|
-        File.write(File.join(d, 'foo.rb'), 'a < 1')
+        save_it(File.join(d, 'foo/foo.rb'), 'a < 1')
         file = File.join(d, 'base.fb')
         Judges::Update.new(Loog::NULL).run({ 'quiet' => false }, [d, file])
       end
@@ -80,7 +80,7 @@ class TestUpdate < Minitest::Test
 
   def test_update_with_error_and_summary
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), 'this$is$a$broken$Ruby$script')
+      save_it(File.join(d, 'foo/foo.rb'), 'this$is$a$broken$Ruby$script')
       file = File.join(d, 'base.fb')
       2.times do
         Judges::Update.new(Loog::NULL).run(

data/test/test__helper.rb

@@ -32,3 +32,11 @@ require 'minitest/autorun'
 
 require 'minitest/reporters'
 Minitest::Reporters.use! [Minitest::Reporters::SpecReporter.new]
+
+class Minitest::Test
+  def save_it(file, content)
+    require 'fileutils'
+    FileUtils.mkdir_p(File.dirname(file))
+    File.binwrite(file, content)
+  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

data/test/test_judge.rb

@@ -34,7 +34,7 @@ require_relative '../lib/judges/judge'
 class TestJudge < Minitest::Test
   def test_basic_run
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), '$fb.insert')
+      save_it(File.join(d, "#{File.basename(d)}.rb"), '$fb.insert')
       judge = Judges::Judge.new(d, nil, Loog::NULL)
       fb = Factbase.new
       judge.run(fb, {}, {}, {})
@@ -44,7 +44,7 @@ class TestJudge < Minitest::Test
 
   def test_run_isolated
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'bar.rb'), '$fb.insert')
+      save_it(File.join(d, "#{File.basename(d)}.rb"), '$fb.insert')
       judge = Judges::Judge.new(d, nil, Loog::NULL)
       fb1 = Factbase.new
       judge.run(fb1, {}, {}, {})
@@ -57,8 +57,8 @@ class TestJudge < Minitest::Test
 
   def test_passes_local_vars_between_tests
     Dir.mktmpdir do |d|
-      File.write(
-        File.join(d, 'x.rb'),
+      save_it(
+        File.join(d, "#{File.basename(d)}.rb"),
         '
         $local[:foo] = 42 if $local[:foo].nil?
         $local[:foo] = $local[:foo] + 1
@@ -77,8 +77,7 @@ class TestJudge < Minitest::Test
     Dir.mktmpdir do |d|
       j = 'this_is_it'
       dir = File.join(d, j)
-      FileUtils.mkdir(dir)
-      File.write(File.join(dir, 'foo.rb'), '$loog.info("judge=" + $judge)')
+      save_it(File.join(dir, "#{j}.rb"), '$loog.info("judge=" + $judge)')
       log = Loog::Buffer.new
       Judges::Judge.new(dir, nil, log).run(Factbase.new, {}, {}, {})
       assert(log.to_s.include?("judge=#{j}"))
@@ -89,8 +88,7 @@ class TestJudge < Minitest::Test
     assert_raises do
       Dir.mktmpdir do |d|
         dir = File.join(d, 'judges')
-        FileUtils.mkdir_p(dir)
-        File.write(File.join(dir, 'x.rb'), 'this$is$broken$syntax')
+        save_it(File.join(dir, "#{File.basename(d)}.rb"), 'this$is$broken$syntax')
         judge = Judges::Judge.new(dir, lib, Loog::NULL)
         judge.run(Factbase.new, {}, {}, {})
       end
@@ -101,8 +99,7 @@ class TestJudge < Minitest::Test
     assert_raises do
       Dir.mktmpdir do |d|
         dir = File.join(d, 'judges')
-        FileUtils.mkdir_p(dir)
-        File.write(File.join(dir, 'x.rb'), 'a < 1')
+        save_it(File.join(dir, "#{File.basename(d)}.rb"), 'a < 1')
         judge = Judges::Judge.new(dir, lib, Loog::NULL)
         judge.run(Factbase.new, {}, {}, {})
       end

data/test/test_judges.rb

@@ -33,8 +33,9 @@ require_relative '../lib/judges/judges'
 class TestJudges < Minitest::Test
   def test_basic
     Dir.mktmpdir do |d|
-      File.write(File.join(d, 'foo.rb'), 'hey')
-      File.write(File.join(d, 'something.yml'), "---\nfoo: 42")
+      dir = File.join(d, 'foo')
+      save_it(File.join(dir, 'foo.rb'), 'hey')
+      save_it(File.join(dir, 'something.yml'), "---\nfoo: 42")
       found = 0
       Judges::Judges.new(d, nil, Loog::NULL).each do |p|
         assert_equal('foo.rb', p.script)
@@ -47,11 +48,32 @@ class TestJudges < Minitest::Test
 
   def test_get_one
     Dir.mktmpdir do |d|
-      f = File.join(d, 'boo/foo.rb')
-      FileUtils.mkdir_p(File.dirname(f))
-      File.write(f, 'hey')
+      save_it(File.join(d, 'boo/boo.rb'), 'hey')
       j = Judges::Judges.new(d, nil, Loog::NULL).get('boo')
-      assert_equal('foo.rb', j.script)
+      assert_equal('boo.rb', j.script)
+    end
+  end
+
+  def test_list_only_direct_subdirs
+    Dir.mktmpdir do |d|
+      save_it(File.join(d, 'first/first.rb'), '')
+      save_it(File.join(d, 'second/second.rb'), '')
+      save_it(File.join(d, 'second/just-file.rb'), '')
+      save_it(File.join(d, 'wrong.rb'), '')
+      save_it(File.join(d, 'another/wrong/wrong.rb'), '')
+      save_it(File.join(d, 'bad/hello.rb'), '')
+      list = Judges::Judges.new(d, nil, Loog::NULL).each.to_a
+      assert_equal(2, list.size)
+    end
+  end
+
+  def test_list_with_empty_dir
+    Dir.mktmpdir do |d|
+      save_it(File.join(d, 'wrong.rb'), '')
+      save_it(File.join(d, 'another/wrong/wrong.rb'), '')
+      save_it(File.join(d, 'bad/hello.rb'), '')
+      list = Judges::Judges.new(d, nil, Loog::NULL).each.to_a
+      assert(list.empty?)
     end
   end
 end

data/test/test_options.rb

@@ -81,7 +81,7 @@ class TestOptions < Minitest::Test
     opts = Judges::Options.new('foo' => 44, 'bar' => 'long-string-maybe-secret')
     s = opts.to_s
     assert(s.include?('FOO → "44"'), s)
-    assert(s.include?('"long********************"'))
+    assert(s.include?('"long****************cret"'), s)
   end
 
   def test_merge

metadata

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