judges 0.19.0 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5dc4d51701430ff754d2fb33b4938d6ad105f0d08590c875c88095b0992bab17
-  data.tar.gz: 7b5b223a403e3d5d82128589a4838ac66b8518644e0c110e5a688e6139fa92d6
+  metadata.gz: f826572e575f1c40c556cb5c0de0a2ca5aaf39554ef9d4219e517c6d8492236b
+  data.tar.gz: 7f16f70af23a1fb4903934939133015289f06d4a276c32a567356dd60e3fa75f
 SHA512:
-  metadata.gz: 5a293823a64b90aa7edb793b4d9fc35c7e06d7012d70073e96a9080232d48b5c12ec0f0f1b20759412eb033a4921cbeef43129101050c784b90ecf11e55d4900
-  data.tar.gz: 5cd0c89a797662738454256c960afa5edf46b77e6109157cb8d8d44c534a130a7dcd0522dfe5091c878f7e9f3c0a7e95c1f47c283696ba4cd0fb7e592e61d3de
+  metadata.gz: f3b4e89fcd538caddf490ca8769c2142e6c06bec83eb99582bf426f48ce35617ce169d08d604a9d4ee2780d88f0cc877651fabd863efffc5cb2d1ee96cf6a6ec
+  data.tar.gz: 4dd669683164fcd9402f9f4043b5fab0faa086cc68de9f0a6dc1f6e78f9959f5a8a2f18eb3c8fed23986d735010e71c964313090acba222966803b02bd8ba9e3

data/.github/workflows/copyrights.yml

@@ -21,7 +21,11 @@
 name: copyrights
 'on':
   push:
+    branches:
+      - master
   pull_request:
+    branches:
+      - master
 jobs:
   copyrights:
     runs-on: ubuntu-22.04

data/.github/workflows/xcop.yml

@@ -19,9 +19,13 @@
 # SOFTWARE.
 ---
 name: xcop
-on:
+'on':
   push:
+    branches:
+      - master
   pull_request:
+    branches:
+      - master
 jobs:
   xcop:
     runs-on: ubuntu-22.04

data/README.md

@@ -48,6 +48,9 @@ options:
   max: 100
 expected:
   - /fb[count(f)=1]
+after:
+  - first.rb
+  - second.rb
 ```
 
 Here, the `input` is an array of facts to be placed into the Factbase before
@@ -65,6 +68,10 @@ be executed. After each execution, all expected XPath expressions are validated.
 The `before` (default: `[]`) is a list of judges that must be executed before
 the current one.
 
+The `after` (default: `[]`) is a list of relative file names
+of Ruby scripts that are executed after the judge
+(`$fb` and `$loog` are passed into them).
+
 ## How to contribute
 
 Read

data/Rakefile

@@ -33,6 +33,8 @@ def version
   Gem::Specification.load(Dir['*.gemspec'].first).version
 end
 
+ENV['RACK_ENV'] = 'test'
+
 task default: %i[clean test features rubocop yard copyright]
 
 require 'rake/testtask'

data/bin/judges

@@ -24,13 +24,13 @@ $stdout.sync = true
 
 require 'gli'
 require 'loog'
+require 'time'
 require 'factbase'
 
 Encoding.default_external = Encoding::UTF_8
 Encoding.default_internal = Encoding::UTF_8
 
-class App
-  extend GLI::App
+class JudgesGLI extend GLI::App
 
   loog = Loog::REGULAR
 
@@ -52,6 +52,9 @@ class App
     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}")
     true
   end
 
@@ -212,4 +215,4 @@ class App
   end
 end
 
-exit App.run(ARGV)
+exit JudgesGLI.run(ARGV) if ENV['GLI_TESTING'].nil?

data/features/pull.feature

@@ -1,12 +1,10 @@
 Feature: Pull
   I want to pull a factbase
 
-  Scenario: Pull a small factbase
+  Scenario: Pull a small factbase, which is absent on the server
     Given We are online
     Given I make a temp directory
-    Then I run bin/judges with "--verbose pull --token 00000000-0000-0000-0000-000000000000 --wait=15 simple simple.fb"
-    Then Stdout contains "Pulled"
-    And Exit code is zero
-    Then I run bin/judges with "inspect simple.fb"
+    Then I run bin/judges with "--verbose pull --token 00000000-0000-0000-0000-000000000000 --wait=15 {FAKE-NAME} simple.fb"
+    Then Stdout contains "doesn't exist at www.zerocracy.com"
     And Exit code is zero
 

data/features/push.feature

@@ -6,6 +6,6 @@ Feature: Push
     Given I make a temp directory
     Then I run bin/judges with "--verbose eval simple.fb '(0..1000).each { $fb.insert.foo = 42 }'"
     And Exit code is zero
-    Then I run bin/judges with "push --token 00000000-0000-0000-0000-000000000000 --meta a:b --meta foo:bar --meta=pages_url:https://zerocracy.github.io/zerocracy.html --meta=duration:1055  simple simple.fb"
+    Then I run bin/judges with "push --token 00000000-0000-0000-0000-000000000000 --meta a:b --meta foo:bar --meta=pages_url:https://zerocracy.github.io/zerocracy.html --meta=duration:1055 {FAKE-NAME} simple.fb"
     Then Stdout contains "Pushed"
     And Exit code is zero

data/features/step_definitions/steps.rb

@@ -23,6 +23,7 @@
 require 'tmpdir'
 require 'net/ping'
 require 'English'
+require 'securerandom'
 
 Before do
   @cwd = Dir.pwd
@@ -52,6 +53,7 @@ end
 
 When(%r{^I run bin/judges with "([^"]*)"$}) do |arg|
   home = File.join(File.dirname(__FILE__), '../..')
+  arg.gsub!('{FAKE-NAME}') { "fake#{SecureRandom.hex(8)}" }
   cmd = "ruby -I#{home}/lib #{home}/bin/judges #{arg}"
   cmd = "GLI_DEBUG=true #{cmd}" unless Gem.win_platform?
   @stdout = `#{cmd} 2>&1`

data/features/test.feature

@@ -28,14 +28,14 @@ Feature: Test
 
   Scenario: Simple test of a few judges, with a lib
     Given I make a temp directory
-    Then I have a "myjudges/myjudge/simple_judge.rb" file with content:
+    Then I have a "myjudges/good/good.rb" file with content:
     """
       $valve.enter('boom', 'some reason') do
         n = $fb.insert
         n.foo = $foo
       end
     """
-    Then I have a "myjudges/myjudge/good.yml" file with content:
+    Then I have a "myjudges/good/good.yml" file with content:
     """
     ---
     category: good
@@ -51,7 +51,7 @@ Feature: Test
 
   Scenario: Simple test with many runs
     Given I make a temp directory
-    Then I have a "foo/simple.rb" file with content:
+    Then I have a "foo/foo.rb" file with content:
     """
       n = $fb.insert
       n.foo = $fb.size
@@ -70,7 +70,7 @@ Feature: Test
 
   Scenario: Simple test with many runs and many asserts
     Given I make a temp directory
-    Then I have a "foo/simple.rb" file with content:
+    Then I have a "foo/foo.rb" file with content:
     """
       n = $fb.insert
       n.foo = $fb.size
@@ -141,3 +141,25 @@ Feature: Test
     """
     Then I run bin/judges with "test mine"
     And Exit code is zero
+
+  Scenario: Test with a post-assert
+    Given I make a temp directory
+    Then I have a "mine/foo/foo.rb" file with content:
+    """
+    n = $fb.insert
+    n.foo = 42
+    """
+    Then I have a "mine/foo/assert.rb" file with content:
+    """
+    raise unless $fb.size == 1
+    """
+    Then I have a "mine/foo/simple.yml" file with content:
+    """
+    ---
+    expected:
+      - /fb[count(f)=1]
+    after:
+      - assert.rb
+    """
+    Then I run bin/judges with "test mine"
+    And Exit code is zero

data/features/update.feature

@@ -3,7 +3,7 @@ Feature: Update
 
   Scenario: Simple run of a few judges
     Given I make a temp directory
-    Then I have a "simple/simple_judge.rb" file with content:
+    Then I have a "simple/simple.rb" file with content:
     """
       n = $fb.insert
       n.kind = 'yes!'
@@ -17,7 +17,7 @@ Feature: Update
 
   Scenario: Simple run of a few judges, with a lib
     Given I make a temp directory
-    Then I have a "mine/judge1/simple_judge.rb" file with content:
+    Then I have a "mine/judge1/judge1.rb" file with content:
     """
       $valve.enter('boom', 'for no particular reason') do
         n = $fb.insert
@@ -35,7 +35,7 @@ Feature: Update
 
   Scenario: The update fails when a bug in a judge
     Given I make a temp directory
-    Then I have a "mine/judge1/broken.rb" file with content:
+    Then I have a "mine/broken/broken.rb" file with content:
     """
     a < 1
     """
@@ -45,10 +45,24 @@ Feature: Update
 
   Scenario: The update fails when a broken Ruby syntax in a judge
     Given I make a temp directory
-    Then I have a "mine/judge1/broken.rb" file with content:
+    Then I have a "mine/broken/broken.rb" file with content:
     """
     invalid$ruby$syntax$here
     """
     Then I run bin/judges with "update mine simple.fb"
     Then Stdout contains "Failed to update correctly"
     And Exit code is not zero
+
+  Scenario: Avoid duplicate insert on error in the middle
+    Given I make a temp directory
+    Then I have a "mine/foo/foo.rb" file with content:
+    """
+    return unless $fb.query('(eq foo 42)').each.to_a.empty?
+    f = $fb.insert
+    f.foo = 42
+    raise 'intentional'
+    """
+    Then I run bin/judges with "update --quiet --max-cycles=3 mine simple.fb"
+    And Exit code is zero
+    Then I run bin/judges with "inspect simple.fb"
+    Then Stdout contains "Facts: 1"

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

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/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,7 +50,7 @@ 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

data/lib/judges.rb

@@ -25,5 +25,5 @@
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
 module Judges
-  VERSION = '0.19.0'
+  VERSION = '0.21.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_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_bin.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+# Copyright (c) 2024 Yegor Bugayenko
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the 'Software'), to deal
+# in the Software without restriction, including without limitation the rights
+# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the Software is
+# furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in all
+# copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+# SOFTWARE.
+
+require 'minitest/autorun'
+
+# Test.
+# Author:: Yegor Bugayenko (yegor256@gmail.com)
+# Copyright:: Copyright (c) 2024 Yegor Bugayenko
+# License:: MIT
+class TestBin < Minitest::Test
+  def test_simple_run
+    ENV.store('GLI_TESTING', 'yes')
+    load File.join(__dir__, '../bin/judges')
+    before = $stdout
+    begin
+      $stdout = StringIO.new
+      JudgesGLI.run(['--version'])
+      s = $stdout.string
+    ensure
+      $stdout = before
+    end
+    assert(s.include?(Judges::VERSION), s)
+  end
+end

data/test/test_judge.rb

@@ -31,10 +31,10 @@ require_relative '../lib/judges/judge'
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
-class TestPack < Minitest::Test
+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 TestPack < 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 TestPack < 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 TestPack < 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 TestPack < 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 TestPack < 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.19.0
+  version: 0.21.0
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-07-22 00:00:00.000000000 Z
+date: 2024-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backtrace
@@ -263,6 +263,7 @@ files:
 - test/commands/test_update.rb
 - test/test__helper.rb
 - test/test_baza.rb
+- test/test_bin.rb
 - test/test_categories.rb
 - test/test_churn.rb
 - test/test_impex.rb