judges 0.12.0 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fdaaebbfb61ca792443a79d06439a3946f92bf9fdeb46fdf89ab0f702f0a2176
-  data.tar.gz: 617dce64bda23eb2d4987ef45b60987766248d48ee5770b6c4f35964d4f73cdf
+  metadata.gz: '098f6126af6c35b9ee74dca41bbfb5b8e6b1357aee5dcec215a2819929e859d5'
+  data.tar.gz: 139a4b55ea82a090e385c4f54be682dd74d0922c66d77824c8d33cecaa799a84
 SHA512:
-  metadata.gz: 8e0cfe9f3488ed3e67f687400a2d4ab7bd6ccae47985d0eec8681bca3aac61c9bdbef4025ca974adff07e7aadb9a6ab594bc1765f2fccaabc7fa23d0c250201e
-  data.tar.gz: 8148daa02d6d83ddcc6dcd6093108cbe051cd205876a924c65590a02b52e861b9d9517cef775e4e86e7ff007cc040379ce11d86c4e1b758a7a4eff6aa82ec327
+  metadata.gz: 0e173739bc5341c55f869375fef959ff9e6ca57c1e0b814a29d4920700f640fc788a948932221c9fbf6f9e7d9d90501e274645f3de213b7f9b3211549652ad1c
+  data.tar.gz: 52f4c44351e5f3c9efb873913f26c8a7c2c5c08dbf490434145cc691968eec240ef95bb5007568b623c043f48ab658624a573664598df587282cb1cfb43ef94b

data/Gemfile.lock

@@ -3,7 +3,7 @@ PATH
   specs:
     judges (0.0.0)
       backtrace (~> 0.3)
-      concurrent-ruby (= 1.2.3)
+      concurrent-ruby (~> 1.2)
       factbase (~> 0.0)
       gli (~> 2.21)
       iri (~> 0.8)
@@ -51,7 +51,7 @@ GEM
     base64 (0.2.0)
     bigdecimal (3.1.8)
     builder (3.3.0)
-    concurrent-ruby (1.2.3)
+    concurrent-ruby (1.3.3)
     connection_pool (2.4.1)
     crack (1.0.0)
       bigdecimal
@@ -180,7 +180,7 @@ GEM
     reline (0.5.9)
       io-console (~> 0.5)
     retries (0.0.5)
-    rexml (3.3.0)
+    rexml (3.3.1)
       strscan
     rspec-core (3.13.0)
       rspec-support (~> 3.13.0)

data/bin/judges

@@ -65,6 +65,8 @@ class App
     c.flag([:'max-cycles'], default_value: 8, type: Integer)
     c.desc 'Stay quiet even if some judges fail'
     c.switch([:q, :quiet], default_value: false)
+    c.desc 'Place a summarization fact into the factbase'
+    c.switch([:summary], default_value: false)
     c.desc 'Use default logging facility'
     c.switch([:log], default_value: true)
     c.action do |global, options, args|

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.12.0'
+  s.version = '0.13.0'
   s.license = 'MIT'
   s.summary = 'Command-Line Tool for a Factbase'
   s.description =
@@ -43,7 +43,7 @@ Gem::Specification.new do |s|
   s.rdoc_options = ['--charset=UTF-8']
   s.extra_rdoc_files = ['README.md', 'LICENSE.txt']
   s.add_runtime_dependency 'backtrace', '~>0.3'
-  s.add_runtime_dependency 'concurrent-ruby', '1.2.3'
+  s.add_runtime_dependency 'concurrent-ruby', '~>1.2'
   s.add_runtime_dependency 'factbase', '~>0.0'
   s.add_runtime_dependency 'gli', '~>2.21'
   s.add_runtime_dependency 'iri', '~>0.8'

data/lib/judges/churn.rb

@@ -27,34 +27,40 @@ require_relative '../judges'
 # Copyright:: Copyright (c) 2024 Yegor Bugayenko
 # License:: MIT
 class Judges::Churn
-  attr_reader :added, :removed
+  attr_reader :added, :removed, :errors
 
-  def initialize(added, removed)
+  def initialize(added, removed, errors = [])
     @added = added
     @removed = removed
+    @errors = errors
   end
 
   def to_s
-    "#{@added}/#{@removed}"
+    "#{@added}/#{@removed}#{@errors.empty? ? '' : "/#{@errors.size}"}"
   end
 
   def zero?
-    @added.zero? && @removed.zero?
+    @added.zero? && @removed.zero? && @errors.empty?
+  end
+
+  def <<(error)
+    @errors << error
+    nil
   end
 
   def +(other)
     if other.is_a?(Judges::Churn)
-      Judges::Churn.new(@added + other.added, @removed + other.removed)
+      Judges::Churn.new(@added + other.added, @removed + other.removed, @errors + other.errors)
     else
-      Judges::Churn.new(@added + other, @removed)
+      Judges::Churn.new(@added + other, @removed, @errors)
     end
   end
 
   def -(other)
     if other.is_a?(Judges::Churn)
-      Judges::Churn.new(@added - other.added, @removed - other.removed)
+      Judges::Churn.new(@added - other.added, @removed - other.removed, @errors + other.errors)
     else
-      Judges::Churn.new(@added, @removed + other)
+      Judges::Churn.new(@added, @removed + other, @errors)
     end
   end
 end

data/lib/judges/commands/update.rb

@@ -72,6 +72,17 @@ class Judges::Update
       end
       throw :"Update finished in #{c} cycle(s), modified #{churn} fact(s)"
     end
+    return if churn.zero? || !opts['summary']
+    fb.query('(eq what "judges-summary")').delete!
+    f = fb.insert
+    f.what = 'judges-summary'
+    f.when = Time.now
+    f.version = Judges::VERSION
+    f.cycles = c
+    f.added = churn.added.size
+    f.removed = churn.removed.size
+    churn.errors.each { |e| f.error = e }
+    impex.export(fb)
   end
 
   private
@@ -79,7 +90,6 @@ class Judges::Update
   # Run all judges in a full cycle, one by one.
   # @return [Churn] How many modifications have been made
   def cycle(opts, judges, fb, options)
-    errors = []
     churn = Judges::Churn.new(0, 0)
     global = {}
     elapsed(@loog) do
@@ -92,13 +102,13 @@ class Judges::Update
         end
       rescue StandardError, SyntaxError => e
         @loog.warn(Backtrace.new(e))
-        errors << p.script
+        churn << e.message
       end
-      throw :"👍 #{done} judge(s) processed" if errors.empty?
-      throw :"❌ #{done} judge(s) processed with #{errors.size} errors"
+      throw :"👍 #{done} judge(s) processed" if churn.errors.empty?
+      throw :"❌ #{done} judge(s) processed with #{churn.errors.size} errors"
     end
-    unless errors.empty?
-      raise "Failed to update correctly (#{errors.size} errors)" unless opts['quiet']
+    unless churn.errors.empty?
+      raise "Failed to update correctly (#{churn.errors.size} errors)" unless opts['quiet']
       @loog.info('Not failing because of the --quiet flag provided')
     end
     churn

data/lib/judges.rb

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

data/test/commands/test_update.rb

@@ -64,7 +64,7 @@ class TestUpdate < Minitest::Test
     Dir.mktmpdir do |d|
       File.write(File.join(d, 'foo.rb'), 'this$is$a$broken$Ruby$script')
       file = File.join(d, 'base.fb')
-      Judges::Update.new(Loog::NULL).run({ 'quiet' => true }, [d, file])
+      Judges::Update.new(Loog::NULL).run({ 'quiet' => true, 'max-cycles' => 2 }, [d, file])
     end
   end
 
@@ -77,4 +77,23 @@ class TestUpdate < Minitest::Test
       end
     end
   end
+
+  def test_update_with_error_and_summary
+    Dir.mktmpdir do |d|
+      File.write(File.join(d, 'foo.rb'), 'this$is$a$broken$Ruby$script')
+      file = File.join(d, 'base.fb')
+      2.times do
+        Judges::Update.new(Loog::NULL).run(
+          { 'quiet' => true, 'summary' => true, 'max-cycles' => 2 },
+          [d, file]
+        )
+      end
+      fb = Factbase.new
+      fb.import(File.binread(file))
+      sums = fb.query('(eq what "judges-summary")').each.to_a
+      assert_equal(1, sums.size)
+      f = sums.first
+      assert(f.error.include?('unexpected global variable'), f.error)
+    end
+  end
 end

data/test/test_churn.rb

@@ -35,4 +35,12 @@ class TestChurn < Minitest::Test
     assert_equal('42/0', (churn + 42).to_s)
     assert_equal('0/17', (churn - 17).to_s)
   end
+
+  def test_with_errors
+    churn = Judges::Churn.new(0, 0)
+    churn << 'oops'
+    assert_equal('0/0/1', churn.to_s)
+    assert_equal('42/0/1', (churn + 42).to_s)
+    assert_equal('0/0/1', (Judges::Churn.new(0, 0) + churn).to_s)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: judges
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.13.0
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-06-25 00:00:00.000000000 Z
+date: 2024-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backtrace
@@ -28,16 +28,16 @@ dependencies:
   name: concurrent-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.3
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.3
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: factbase
   requirement: !ruby/object:Gem::Requirement