judges 0.0.12 → 0.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 63754c5826506b6066688aa2373dafcd87fc20cd193631c15ce67f1496dc361b
-  data.tar.gz: a7c70946e0c24bffc91849d81e15866335ef0ebeb64b7f1cefd2c3d6c2d48506
+  metadata.gz: e45e98c063327eedc9ae2731bb8c8c39358291ca8c687f6cf37997df37cf84aa
+  data.tar.gz: 811ae41514764e68e9cc4fa0a3ea0429bed9128eec2142268e4cdb779ce12b5e
 SHA512:
-  metadata.gz: 6360db23fb5fc573d479a43d40b4c1d1e274bd21320cb4c2acba0e71a6ac8775c3398fef776dc1d7412ccd96782394e53d9631180c5a09314aab9e791738829a
-  data.tar.gz: e5b7a0c2460d589bf9cb69e42183585d13ac6ac1b400f338e3c73057be0d2fdb9c703bcc2896f73d8dbd5f850b22452d8f5a2b147e299a1ed9fd2b1aabe2f564
+  metadata.gz: 0414dfaa83953985ce7186c336eac2c1a53e71f88c73f9b89c8f1410208191c5d1ff106d91045fa7c6f6f0dd9eb59ed8c189138b51d4b546d5a0a9bbe4211546
+  data.tar.gz: d3b3e538af3fa858fd19d67afe3f4b10942c97befa99538fceefcb3eef770cb130bbb1631160191d4e7bac47c0dbe62a184959ba3f1c29956bdf86708f88eefe

data/bin/judges

@@ -35,7 +35,7 @@ loog = Loog::REGULAR
 
 program_desc('Automated executor of judges for a factbase')
 
-version('0.0.12')
+version('0.0.14')
 
 synopsis_format(:full)
 
@@ -54,7 +54,7 @@ end
 desc 'Update the factbase by passing all judges one by one'
 command :update do |c|
   c.desc 'Options to pass to every judge'
-  c.flag([:o, :option], default_value: 'yes', type: Array, arg_name: '<key=value>')
+  c.flag([:o, :option], multiple: true, arg_name: '<key=value>')
   c.action do |global, options, args|
     require_relative '../lib/judges/commands/update'
     Judges::Update.new(loog).run(options, args)

data/features/cli.feature

@@ -19,7 +19,9 @@ Feature: Simple Run
         n.kind = 'yes!'
       end
     """
-    Then I run bin/judges with "update . simple.fb"
+    Then I run bin/judges with "--verbose update -o foo=1 -o bar=2 . simple.fb"
+    Then Stdout contains "foo → "
+    Then Stdout contains "bar → "
     Then Stdout contains "1 judges processed"
     And Exit code is zero
 

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

data/lib/judges/commands/join.rb

@@ -43,11 +43,11 @@ class Judges::Join
     slave = args[1]
     raise "The slave factbase is absent: #{slave.to_rel}" unless File.exist?(slave)
     fb = Factbase.new
-    fb.import(File.read(master))
+    fb.import(File.binread(master))
     @loog.info("Master factbase imported from #{master.to_rel} (#{File.size(master)} bytes)")
-    fb.import(File.read(slave))
+    fb.import(File.binread(slave))
     @loog.info("Slave factbase imported from #{slave.to_rel} (#{File.size(slave)} bytes)")
-    File.write(master, fb.export)
+    File.binwrite(master, fb.export)
     @loog.info("Master factbase exported to #{master.to_rel} (#{File.size(master)} bytes)")
   end
 end

data/lib/judges/commands/print.rb

@@ -46,7 +46,7 @@ class Judges::Print
       o = "#{o}.#{opts[:format]}"
     end
     fb = Factbase.new
-    fb.import(File.read(f))
+    fb.import(File.binread(f))
     @loog.info("Factbase imported from #{f.to_rel} (#{File.size(f)} bytes)")
     FileUtils.mkdir_p(File.dirname(o))
     output =
@@ -58,7 +58,7 @@ class Judges::Print
         when 'xml'
           fb.to_xml
       end
-    File.write(o, output)
+    File.binwrite(o, output)
     @loog.info("Factbase printed to #{o.to_rel} (#{File.size(o)} bytes)")
   end
 end

data/lib/judges/commands/test.rb

@@ -40,13 +40,13 @@ class Judges::Test
   def run(_opts, args)
     raise 'Exactly one argument required' unless args.size == 1
     dir = args[0]
-    @loog.info("Testing judges in '#{dir.to_rel}'...")
+    @loog.info("Testing judges in #{dir.to_rel}...")
     errors = []
     done = Judges::Packs.new(dir, @loog).each_with_index do |p, i|
-      @loog.info("\n👉 Testing '#{p.script}' (##{i}) in '#{p.dir.to_rel}'...")
+      @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])
-        @loog.info("Testing '#{f.to_rel}':")
+        @loog.info("Testing #{f.to_rel}:")
         begin
           test_one(p, yaml)
         rescue StandardError => e

data/lib/judges/commands/update.rb

@@ -40,31 +40,34 @@ class Judges::Update
   def run(opts, args)
     raise 'Exactly two arguments required' unless args.size == 2
     dir = args[0]
-    raise "The directory is absent: #{dir}" unless File.exist?(dir)
+    raise "The directory is absent: #{dir.to_rel}" unless File.exist?(dir)
     file = args[1]
     fb = Factbase.new
     if File.exist?(file)
-      fb.import(File.read(file))
-      @loog.info("Factbase imported from '#{file.to_rel}' (#{File.size(file)} bytes)")
+      fb.import(File.binread(file))
+      @loog.info("Factbase imported from #{file.to_rel} (#{File.size(file)} bytes)")
     else
-      @loog.info("There is no Factbase to import from '#{file.to_rel}' (file is absent)")
+      @loog.info("There is no Factbase to import from #{file.to_rel} (file is absent)")
     end
-    options = Judges::Options.new(opts['options'])
+    options = Judges::Options.new(opts['option'])
     @loog.debug("The following options provided:\n\t#{options.to_s.gsub("\n", "\n\t")}")
     errors = []
     done = Judges::Packs.new(dir, @loog).each_with_index do |p, i|
-      @loog.info("Pack ##{i} found in #{p.dir.to_rel}")
+      @loog.info("Running #{p.dir.to_rel} (##{i})...")
+      before = fb.size
       begin
         p.run(fb, options)
       rescue StandardError => e
         @loog.warn(Backtrace.new(e))
         errors << p.script
       end
+      after = fb.size
+      @loog.info("Pack #{p.dir.to_rel} added #{after - before} facts") if after > before
     end
     @loog.info("#{done} judges processed (#{errors.size} errors)")
     FileUtils.mkdir_p(File.dirname(file))
-    File.write(file, fb.export)
-    @loog.info("Factbase exported to '#{file.to_rel}' (#{File.size(file)} bytes)")
+    File.binwrite(file, fb.export)
+    @loog.info("Factbase exported to #{file.to_rel} (#{File.size(file)} bytes)")
     raise "Failed to update correctly (#{errors.size} errors)" unless errors.empty?
   end
 end

data/lib/judges/options.rb

@@ -39,7 +39,7 @@ class Judges::Options
     @hash.map do |k, v|
       v = v.to_s
       v = "#{v[0..3]}#{'*' * (v.length - 4)}" if v.length > 8
-      "#{k}=#{v}"
+      "#{k} → \"#{v}\""
     end.join("\n")
   end
 
@@ -48,9 +48,13 @@ class Judges::Options
     @hash ||= begin
       pp = @pairs || []
       pp = @pairs.map { |k, v| "#{k}=#{v}" } if pp.is_a?(Hash)
+      pp = pp.split(',') if pp.is_a?(String)
       pp.to_h do |pair|
         p = pair.split('=', 2)
-        [p[0].to_sym, p[1].match?(/^[0-9]+$/) ? p[1].to_i : p[1]]
+        k = p[0].strip
+        v = p[1]
+        v = v.nil? ? 'true' : v.strip
+        [k.to_sym, v.match?(/^[0-9]+$/) ? v.to_i : v]
       end
     end
     k = args[0].downcase

data/lib/judges/to_rel.rb

@@ -31,10 +31,13 @@ class Object
   def to_rel
     s = File.absolute_path(to_s)
     p = Pathname.new(s).relative_path_from(Dir.getwd)
+    t = p.to_s
+    t = s if t.length > s.length
+    t = "\"#{t}\"" if t.include?(' ')
     if p.directory?
-      "#{p}/"
+      "#{t}/"
     else
-      p.to_s
+      t
     end
   end
 end

data/test/commands/test_join.rb

@@ -36,14 +36,14 @@ class TestJoin < Minitest::Test
       master = File.join(d, 'master.fb')
       fb1 = Factbase.new
       fb1.insert.zz = 5
-      File.write(master, fb1.export)
+      File.binwrite(master, fb1.export)
       slave = File.join(d, 'slave.fb')
       fb2 = Factbase.new
       fb2.insert.foo_bar = 42
-      File.write(slave, fb2.export)
+      File.binwrite(slave, fb2.export)
       Judges::Join.new(Loog::VERBOSE).run({}, [master, slave])
       fb = Factbase.new
-      fb.import(File.read(master))
+      fb.import(File.binread(master))
       xml = Nokogiri::XML.parse(fb.to_xml)
       assert(!xml.xpath('/fb/f[zz="5"]').empty?, fb.to_xml)
       assert(!xml.xpath('/fb/f[foo_bar="42"]').empty?, fb.to_xml)

data/test/commands/test_print.rb

@@ -37,7 +37,7 @@ class TestPrint < Minitest::Test
       f = File.join(d, 'base.fb')
       fb = Factbase.new
       fb.insert
-      File.write(f, fb.export)
+      File.binwrite(f, fb.export)
       Judges::Print.new(Loog::VERBOSE).run({ format: 'yaml', auto: true }, [f])
       y = File.join(d, 'base.yaml')
       assert(File.exist?(y))

data/test/commands/test_update.rb

@@ -35,9 +35,9 @@ class TestUpdate < Minitest::Test
     Dir.mktmpdir do |d|
       File.write(File.join(d, 'foo.rb'), '$fb.insert.zzz = $options.foo_bar + 1')
       file = File.join(d, 'base.fb')
-      Judges::Update.new(Loog::VERBOSE).run({ 'options' => ['foo_bar=42'] }, [d, file])
+      Judges::Update.new(Loog::VERBOSE).run({ 'option' => ['foo_bar=42'] }, [d, file])
       fb = Factbase.new
-      fb.import(File.read(file))
+      fb.import(File.binread(file))
       xml = Nokogiri::XML.parse(fb.to_xml)
       assert(!xml.xpath('/fb/f[zzz="43"]').empty?)
     end
@@ -48,11 +48,11 @@ class TestUpdate < Minitest::Test
       file = File.join(d, 'base.fb')
       fb = Factbase.new
       fb.insert.foo_bar = 42
-      File.write(file, fb.export)
+      File.binwrite(file, fb.export)
       File.write(File.join(d, 'foo.rb'), '$fb.insert.tt = 4')
       Judges::Update.new(Loog::VERBOSE).run({}, [d, file])
       fb = Factbase.new
-      fb.import(File.read(file))
+      fb.import(File.binread(file))
       xml = Nokogiri::XML.parse(fb.to_xml)
       assert(!xml.xpath('/fb/f[tt="4"]').empty?)
       assert(!xml.xpath('/fb/f[foo_bar="42"]').empty?)

data/test/test_options.rb

@@ -35,11 +35,23 @@ class TestOptions < Minitest::Test
     assert_equal(42, opts.max)
   end
 
+  def test_stips_spaces
+    opts = Judges::Options.new(['  token=a77   ', 'max  =  42'])
+    assert_equal('a77', opts.token)
+    assert_equal(42, opts.max)
+  end
+
   def test_with_nil
     opts = Judges::Options.new(nil)
     assert(opts.foo.nil?)
   end
 
+  def test_with_string
+    opts = Judges::Options.new('a=1,b=42')
+    assert_equal(1, opts.a)
+    assert_equal(42, opts.b)
+  end
+
   def test_with_hash
     opts = Judges::Options.new('foo' => 42, 'bar' => 'hello')
     assert_equal(42, opts.foo)
@@ -49,6 +61,8 @@ class TestOptions < Minitest::Test
 
   def test_converts_to_string
     opts = Judges::Options.new('foo' => 44, 'bar' => 'long-string-maybe-secret')
-    assert_equal("foo=44\nbar=long********************", opts.to_s)
+    s = opts.to_s
+    assert(s.include?('foo → "44"'))
+    assert(s.include?('"long********************"'))
   end
 end

metadata

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