judges 0.13.0 → 0.13.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '098f6126af6c35b9ee74dca41bbfb5b8e6b1357aee5dcec215a2819929e859d5'
-  data.tar.gz: 139a4b55ea82a090e385c4f54be682dd74d0922c66d77824c8d33cecaa799a84
+  metadata.gz: 423404090687b2214385439c28e9d5aa3e916d3eab7fcad7b406e9c795750ed0
+  data.tar.gz: 8f5c5b7f86625a2d57f3419010ea06891ca34ae3e1332a6858bd0f3edd467f8f
 SHA512:
-  metadata.gz: 0e173739bc5341c55f869375fef959ff9e6ca57c1e0b814a29d4920700f640fc788a948932221c9fbf6f9e7d9d90501e274645f3de213b7f9b3211549652ad1c
-  data.tar.gz: 52f4c44351e5f3c9efb873913f26c8a7c2c5c08dbf490434145cc691968eec240ef95bb5007568b623c043f48ab658624a573664598df587282cb1cfb43ef94b
+  metadata.gz: 56b32e90fb490e8664960b113b1f4aea5ff8b5be8778aab308c3d2cfdfade484d1efbe1550f523303ff8372f6065dd581ad01c19a2b48e38b4b250052ae74332
+  data.tar.gz: '0578ac5a5a17b870e887e7ba6aaafafdaf51ad02b3d32fbb77857718b427550419e04c5d9711e907f2cb2b6eb7c35d58a0b5f4bc3ead36854fe6f5733c90bb8c'

data/features/update.feature

@@ -9,8 +9,8 @@ Feature: Update
       n.kind = 'yes!'
     """
     Then I run bin/judges with "--verbose update --quiet -o foo=1 -o bar=2 --max-cycles 3 . simple.fb"
-    Then Stdout contains "foo → "
-    Then Stdout contains "bar → "
+    Then Stdout contains "FOO → "
+    Then Stdout contains "BAR → "
     Then Stdout contains "1 judge(s) processed"
     Then Stdout contains "Update finished in 3 cycle(s), modified 3/0 fact(s)"
     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.13.0'
+  s.version = '0.13.2'
   s.license = 'MIT'
   s.summary = 'Command-Line Tool for a Factbase'
   s.description =

data/lib/judges/commands/update.rb

@@ -47,7 +47,11 @@ class Judges::Update
     fb = impex.import(strict: false)
     fb = Factbase::Looged.new(fb, @loog) if opts['log']
     options = Judges::Options.new(opts['option'])
-    @loog.debug("The following options provided:\n\t#{options.to_s.gsub("\n", "\n\t")}")
+    if options.empty?
+      @loog.debug('No options provided by the --option flag')
+    else
+      @loog.debug("The following options provided:\n\t#{options.to_s.gsub("\n", "\n\t")}")
+    end
     judges = Judges::Judges.new(dir, opts['lib'], @loog)
     c = 0
     churn = Judges::Churn.new(0, 0)

data/lib/judges/options.rb

@@ -34,43 +34,54 @@ class Judges::Options
     @pairs = pairs
   end
 
+  def empty?
+    to_h.empty?
+  end
+
   def +(other)
-    touch # this will trigger method_missing() method, which will create @hash
-    h = @hash.dup
-    other.touch # this will trigger method_missing() method, which will create @hash
-    other.instance_variable_get(:@hash).each do |k, v|
+    h = to_h
+    other.to_h.each do |k, v|
       h[k] = v
     end
-    Judges::Options.new(h.map { |k, v| "#{k}=#{v}" })
+    Judges::Options.new(h)
   end
 
   # Convert them all to a string (printable in a log).
   def to_s
-    touch # this will trigger method_missing() method, which will create @hash
-    @hash.map do |k, v|
+    to_h.map do |k, v|
       v = v.to_s
       v = "#{v[0..3]}#{'*' * (v.length - 4)}" if v.length > 8
       "#{k} → \"#{v}\""
     end.join("\n")
   end
 
-  # Get option by name.
-  others do |*args|
-    @hash ||= begin
+  def to_h
+    @to_h ||= begin
       pp = @pairs || []
-      pp = @pairs.map { |k, v| "#{k}=#{v}" } if pp.is_a?(Hash)
       pp = pp.split(',') if pp.is_a?(String)
-      pp.compact!
-      pp.reject!(&:empty?)
-      pp.to_h do |pair|
-        p = pair.split('=', 2)
-        k = p[0].strip
-        v = p[1]
-        v = v.nil? ? 'true' : v.strip
-        [k.to_sym, v.match?(/^[0-9]+$/) ? v.to_i : v]
+      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
-    k = args[0].downcase
-    @hash[k]
+  end
+
+  # Get option by name.
+  others do |*args|
+    to_h[args[0].upcase.to_sym]
   end
 end

data/lib/judges.rb

@@ -25,5 +25,5 @@
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
 module Judges
-  VERSION = '0.13.0'
+  VERSION = '0.13.2'
 end

data/test/test_options.rb

@@ -41,19 +41,27 @@ class TestOptions < Minitest::Test
     assert_equal(42, opts.max)
   end
 
+  def test_case_insensitive
+    opts = Judges::Options.new(['aBcDeF=1', 'aBCDEf=2'])
+    assert_equal(2, opts.abcdef)
+  end
+
   def test_with_nil
     opts = Judges::Options.new(nil)
     assert(opts.foo.nil?)
+    assert(opts.empty?)
   end
 
   def test_with_empty_string
     opts = Judges::Options.new('   ')
     assert(opts.foo.nil?)
+    assert(opts.empty?, opts)
   end
 
   def test_with_empty_strings
     opts = Judges::Options.new(['', nil])
     assert(opts.foo.nil?)
+    assert(opts.empty?)
   end
 
   def test_with_string
@@ -65,14 +73,14 @@ class TestOptions < Minitest::Test
   def test_with_hash
     opts = Judges::Options.new('foo' => 42, 'bar' => 'hello')
     assert_equal(42, opts.foo)
-    assert_equal('hello', opts.bar)
+    assert_equal('hello', opts.Bar)
     assert(opts.xxx.nil?)
   end
 
   def test_converts_to_string
     opts = Judges::Options.new('foo' => 44, 'bar' => 'long-string-maybe-secret')
     s = opts.to_s
-    assert(s.include?('foo → "44"'))
+    assert(s.include?('FOO → "44"'), s)
     assert(s.include?('"long********************"'))
   end
 
@@ -83,4 +91,10 @@ class TestOptions < Minitest::Test
     assert_equal(44, opts.a)
     assert_equal(3, opts.c)
   end
+
+  def test_merge_by_symbols
+    opts = Judges::Options.new(a: 42) + Judges::Options.new(b: 7)
+    assert_equal(42, opts.a, opts)
+    assert_equal(7, opts.b, opts)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: judges
 version: !ruby/object:Gem::Version
-  version: 0.13.0
+  version: 0.13.2
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-06-26 00:00:00.000000000 Z
+date: 2024-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backtrace