judges 0.0.16 → 0.0.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 583517ec65d24d9415fde952e8ce79daa7fc31662b72642c24e8bafcc211eea5
-  data.tar.gz: 9ab9e0ecbf4f9adec75e07c19053e9c8c0dd28233af16dea85cee57db732fd75
+  metadata.gz: 7312c2f398cbad4906226fbc64c489a95428281003dcab21988185b214bea734
+  data.tar.gz: 7ecf453fcef5028bb43219ddfde277bd1e0e3633059afa69bafe895c8b9fa52a
 SHA512:
-  metadata.gz: '07668e2dde2cfa7290b622e4a683796aaa7913a0591f84c64e4dd0eb5d46b9cd5c5a6e4088caa8f2025f28171aa93d1f44784b6366b63f5ac1da37814b8cc84b'
-  data.tar.gz: 704b1c14632b39c402075fcb5f51c45711b5668a8e7b037b7a2d0a4b6ff209a55f687cdbfa5f0e7b646aefc08e0f17597089f2a632ef7dd89f4b7905782b4111
+  metadata.gz: 8000605816940bd197b284921b1a8218e51537c77f98fd7a932f5f47bac5c366dbe48fa3da0113dfa2db0c3760070d943a5d07a189f29ee87ddb3e0f6cedb18f
+  data.tar.gz: 9692dc49019dab0d0573cec9a95bbdd611328a7e2d95b3efc12ae5cc2e6a250d1ed2801d0c54b954872f53cf5d00139bf12c514612b8a86be72d18cf6dfd3d5e

data/bin/judges

@@ -29,7 +29,7 @@ require 'factbase'
 Encoding.default_external = Encoding::UTF_8
 Encoding.default_internal = Encoding::UTF_8
 
-ver = '0.0.16'
+ver = '0.0.17'
 
 include GLI::App
 
@@ -96,7 +96,9 @@ end
 desc 'Run automated tests for all judges'
 command :test do |c|
   c.desc 'Name of the test pack to run'
-  c.flag([:pack], default_value: '')
+  c.flag([:pack], multiple: true)
+  c.desc 'Fail if no judges tested?'
+  c.flag([:quiet], default_value: false)
   c.action do |global, options, args|
     require_relative '../lib/judges/commands/test'
     Judges::Test.new(loog).run(options, args)

data/features/cli.feature

@@ -27,14 +27,19 @@ Feature: Simple Run
 
   Scenario: Simple test of a few judges
     Given I run bin/judges with "test ./fixtures"
-    Then Stdout contains "judges tested"
+    Then Stdout contains "👉 Testing"
+    Then Stdout contains "judges tested successfully"
     And Exit code is zero
 
   Scenario: Simple test of just one pack
-    Given I run bin/judges with "test --pack absent_for_sure ./fixtures"
-    Then Stdout contains "judges tested"
+    Given I run bin/judges with "test --pack reward_for_good_bug ./fixtures"
+    Then Stdout contains "judges tested successfully"
     And Exit code is zero
 
+  Scenario: Simple test of no packs
+    Given I run bin/judges with "test --pack absent_for_sure ./fixtures"
+    Then Exit code is not zero
+
   Scenario: Simple trimming of a factbase
     Given I make a temp directory
     Then I have a "simple/simple_judge.rb" file with content:

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

data/lib/judges/commands/test.rb

@@ -42,8 +42,9 @@ class Judges::Test
     dir = args[0]
     @loog.info("Testing judges in #{dir.to_rel}...")
     errors = []
-    done = Judges::Packs.new(dir, @loog).each_with_index do |p, i|
-      next if !opts['pack'].nil? && p.name != opts['pack']
+    done = 0
+    Judges::Packs.new(dir, @loog).each_with_index do |p, i|
+      next unless include?(opts, p.name)
       @loog.info("\n👉 Testing #{p.script} (##{i}) in #{p.dir.to_rel}...")
       p.tests.each do |f|
         yaml = YAML.load_file(f, permitted_classes: [Time])
@@ -55,7 +56,9 @@ class Judges::Test
           errors << f
         end
       end
+      done += 1
     end
+    raise 'No judges tested :(' if done.zero? && !opts['quiet']
     if errors.empty?
       @loog.info("\nAll #{done} judges tested successfully")
     else
@@ -66,6 +69,12 @@ class Judges::Test
 
   private
 
+  def include?(opts, name)
+    packs = opts['pack'] || []
+    return true if packs.empty?
+    packs.include?(name)
+  end
+
   def test_one(pack, yaml)
     fb = Factbase.new
     yaml['input'].each do |i|

data/test/commands/test_test.rb

@@ -45,7 +45,7 @@ class TestTest < Minitest::Test
           - /fb/f[bar='4']
         YAML
       )
-      Judges::Test.new(Loog::VERBOSE).run({}, [d])
+      Judges::Test.new(Loog::NULL).run({}, [d])
     end
   end
 
@@ -65,7 +65,7 @@ class TestTest < Minitest::Test
         YAML
       )
       assert_raises do
-        Judges::Test.new(Loog::VERBOSE).run({}, [d])
+        Judges::Test.new(Loog::NULL).run({}, [d])
       end
     end
   end
@@ -84,7 +84,24 @@ class TestTest < Minitest::Test
           - /fb/f[foo='42']
         YAML
       )
-      Judges::Test.new(Loog::VERBOSE).run({}, [d])
+      Judges::Test.new(Loog::NULL).run({}, [d])
+    end
+  end
+
+  def test_one_pack_negative
+    Dir.mktmpdir do |d|
+      File.write(File.join(d, 'foo.rb'), '')
+      File.write(
+        File.join(d, 'x.yml'),
+        <<-YAML
+        input: []
+        expected:
+          - /fb[count(f)=1]
+        YAML
+      )
+      assert_raises do
+        Judges::Test.new(Loog::NULL).run({ 'pack' => [File.basename(dir)] }, [d])
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: judges
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.0.17
 platform: ruby
 authors:
 - Yegor Bugayenko