sexp_processor 4.17.5 → 4.17.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2441d551c221cb6fcc415bb348cbfc7aeb37ead23e747fa061cfe09b72bf5666
-  data.tar.gz: e8653fe553c3a3271a73aa85e646e2ee0961d97f66d5056c6d1ea48ab5b74c82
+  metadata.gz: c0529dac14d4e7c036927a03dd94c13292ee9164f79ed71feb08885aae10620b
+  data.tar.gz: b983642d3019a24ec0d49afd65290331de4703998507d8fe185c0583e4820a45
 SHA512:
-  metadata.gz: 3457f325545dc6e549739d7b03aba6497953e82f16bde2d96ef02ee714e5a08a98a8a0ddfdc5202c71386528f7eb417931e3d5dde8d4f9e87e15010323a0f273
-  data.tar.gz: 21b21ab7f5f7a3b2ec530deaf8a03092819334d278e86ac1cbb8d6edfbb562ca80356109ea0b797e2a04e8f2dd2fe3b79477e3a3d557372e93611c1556ba2aa3
+  metadata.gz: 89352cb2e01400cbdef0082441419eac1b3593986b91a18e6066306a7e14ecf518fa20dea4cc0feb4ca8d91f4788abd1e8003e506ca8ea445dd1cfd8a4cc2f2b
+  data.tar.gz: 2824574101c1ca17b682360c9e7cd6132002a0de13027c521a247eee937b8b997a48df2a0b3c62755015dcf46ffc8cc80add6fb25ce04281549fcc4002a789b4

data/History.rdoc

@@ -1,3 +1,10 @@
+=== 4.17.6 / 2026-09-20
+
+* 2 bug fixes:
+
+  * Improved coverage to 100% for some modules. More to go. Fixed some minor problems.
+  * Minor cleanup in SexpProcessor.
+
 === 4.17.5 / 2025-12-24
 
 * 1 minor enhancement:

data/Rakefile

@@ -5,6 +5,7 @@ require 'hoe'
 
 Hoe.plugin :seattlerb
 Hoe.plugin :rdoc
+Hoe.plugin :cov
 
 Hoe.add_include_dirs("../../ruby_parser/dev/lib")
 

data/lib/sexp_processor.rb

@@ -34,7 +34,7 @@ require "sexp"
 class SexpProcessor
 
   # duh
-  VERSION = "4.17.5"
+  VERSION = "4.17.6"
 
   ##
   # Automatically shifts off the Sexp type before handing the
@@ -218,7 +218,7 @@ class SexpProcessor
     return nil if exp.nil?
 
     unless Sexp === exp then
-      raise SexpTypeError, "exp must be a Sexp, was #{exp.class}:#{exp.inspect}"
+      raise SexpTypeError, "exp must be a Sexp, was %s:%p" % [exp.class, exp]
     end
 
     if self.context.empty? then
@@ -296,7 +296,7 @@ class SexpProcessor
             end
             # result << sub_result
             result = result.class.new(*result, sub_result) # HACK
-          end
+          end # until empty
 
           # NOTE: this is costly, but we are in the generic processor
           # so we shouldn't hit it too much with RubyToC stuff at least.
@@ -325,8 +325,8 @@ class SexpProcessor
   rescue StandardError => err
     return @exceptions[type].call self, exp, err if @exceptions.key? type
 
-    warn "#{err.class} Exception thrown while processing #{type} for sexp #{exp.inspect} #{caller.inspect}" if
-      $DEBUG
+    warn "%s Exception thrown while processing %s for sexp %p %p" % \
+      [err.class, type, exp, caller] if $DEBUG
 
     raise
   end

data/test/test_composite_sexp_processor.rb

@@ -20,29 +20,30 @@ class FakeProcessor1 < SexpProcessor # ZenTest SKIP
 end
 
 class TestCompositeSexpProcessor < Minitest::Test
+  attr_accessor :p
 
   def setup
-    @p = CompositeSexpProcessor.new
+    self.p = CompositeSexpProcessor.new
   end
 
   def test_process_default
     data = s(1, 2, 3)
-    result = @p.process(data.dup)
+    result = p.process(data.dup)
     assert_equal(data.dup, result)
   end
 
   def test_process_fake1
     data = s(:x, 1, 2, 3)
-    @p << FakeProcessor1.new
-    result = @p.process(data.dup)
+    p << FakeProcessor1.new
+    result = p.process(data.dup)
     assert_equal [:x, "1 woot", "2 woot", "3 woot"], result
   end
 
   def test_process_fake1_twice
     data = s(:x, 1, 2, 3)
-    @p << FakeProcessor1.new
-    @p << FakeProcessor1.new
-    result = @p.process(data.dup)
+    p << FakeProcessor1.new
+    p << FakeProcessor1.new
+    result = p.process(data.dup)
     assert_equal [:x, "1 woot woot", "2 woot woot", "3 woot woot"], result
   end
 
@@ -52,15 +53,15 @@ class TestCompositeSexpProcessor < Minitest::Test
   end
 
   def test_append
-    assert_equal([], @p.processors)
+    assert_equal([], p.processors)
 
     assert_raises(ArgumentError) do
-      @p << 42
+      p << 42
     end
 
     fp1 = FakeProcessor1.new
-    @p << fp1
-    assert_equal([fp1], @p.processors)
+    p << fp1
+    assert_equal([fp1], p.processors)
   end
 
 end

data/test/test_sexp.rb

@@ -1,14 +1,5 @@
 $TESTING = true
 
-if ENV["COV"]
-  require "simplecov"
-  SimpleCov.start do
-    add_filter "lib/sexp_processor"
-    add_filter "test"
-  end
-  warn "Running simplecov"
-end
-
 require "minitest/autorun"
 require "minitest/hell" # beat these up
 require "minitest/benchmark" if ENV["BENCH"]
@@ -159,6 +150,12 @@ class TestSexp < SexpTestCase # ZenTest FULL
     assert_equal exp, Sexp.from_array(input)
   end
 
+  def test_cls_s
+    assert_output "", /DEPRECATED/ do
+      assert_equal Sexp.q(42), Sexp.s(42)
+    end
+  end
+
   def test_class_from_array
     assert_from_array s(),                            []
     assert_from_array s(:s),                          [:s]
@@ -694,6 +691,16 @@ class TestSexp < SexpTestCase # ZenTest FULL
     skip "TODO?"
     assert_equal M::Not.new(s(:a)), !s(:a)
   end
+
+  def test_value
+    assert_equal 42, s(:lit, 42).value
+  end
+
+  def test_value__multi
+    assert_raises do
+      s(:whatevs, :nope, 42).value
+    end
+  end
 end # TestSexp
 
 class TestSexpMatcher < SexpTestCase
@@ -747,8 +754,13 @@ class TestSexpMatcher < SexpTestCase
   end
 
   def test_cls_k
-    assert_equal M::Klass.new(Float),       s{ k(Float)     }
+    k = s { k(Float) }
+
+    assert_equal    M::Klass.new(Float), k
     assert_operator M::Klass.new(Float), :===, 6.28
+
+    assert_equal "k(Float)", k.inspect
+    assert_equal "k(Float)", k.pretty_inspect.chomp
   end
 
   def test_amp
@@ -1267,16 +1279,22 @@ class TestSexpSearch < SexpTestCase
 
   def test_satisfy_eh_any_capture # TODO: remove
     sexp = s(:add, :a, :b)
-    assert_satisfy s{ any(q(:add, :a, :b), q(:sub, :a, :b)) }, sexp
 
+    assert_satisfy s{ any(q(:add, :a, :b), q(:sub, :a, :b)) }, sexp
     assert_satisfy s{ any(q(atom, :a, :b), q(:sub, :a, :b)) }, sexp
+
+    assert_satisfy sexp, s{ any(q(:add, :a, :b), q(:sub, :a, :b)) }
+    assert_satisfy sexp, s{ any(q(atom, :a, :b), q(:sub, :a, :b)) }
   end
 
   def test_satisfy_eh_all_capture # TODO: remove
     sexp = s(:add, :a, :b)
-    assert_satisfy s{ all(q(_, :a, :b), q(atom, :a, :b)) }, sexp
 
     assert_satisfy s{ all(q(_, :a, :b), q(atom, :a, :b)) }, sexp
+    assert_satisfy s{ all(q(_, :a, :b), q(atom, :a, :b)) }, sexp
+
+    assert_satisfy sexp, s{ all(q(_, :a, :b), q(atom, :a, :b)) }
+    assert_satisfy sexp, s{ all(q(_, :a, :b), q(atom, :a, :b)) }
 
     assert_search 1, sexp, s{ all(q(_, :a, :b), q(atom, :a, :b)) }
   end

data/test/test_sexp_processor.rb

@@ -125,22 +125,38 @@ class TestSexpProcessor < Minitest::Test
     assert_equal(@processor.expected.new(42), @processor.process(a))
   end
 
-  def test_process_not_sexp
+  def test_process__return_not_sexp
     @processor = TestProcessor.new
     @processor.warn_on_default = false
 
-    assert_raises SexpTypeError do
+    e = assert_raises SexpTypeError do
       @processor.process(s(:broken, 1, 2, 3))
     end
+
+    assert_equal "Result of broken must be a Sexp, was Array:[:broken, 1, 2, 3]", e.message
+  end
+
+  def test_process__input_not_sexp
+    @processor = TestProcessor.new
+    @processor.warn_on_default = false
+
+    e = assert_raises SexpTypeError do
+      @processor.process(42)
+    end
+
+    assert_equal "exp must be a Sexp, was Integer:42", e.message
   end
 
+
   def test_process_unsupported_wrong
     @processor = TestProcessor.new
     @processor.unsupported << :strip
 
-    assert_raises UnsupportedNodeError do
+    e = assert_raises UnsupportedNodeError do
       @processor.process(s(:whatever))
     end
+
+    assert_equal "[:strip] shouldn't be in @unsupported", e.message
   end
 
   def test_unsupported_equal

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sexp_processor
 version: !ruby/object:Gem::Version
-  version: 4.17.5
+  version: 4.17.6
 platform: ruby
 authors:
 - Ryan Davis
@@ -9,9 +9,9 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDPjCCAiagAwIBAgIBCTANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
+  MIIDPjCCAiagAwIBAgIBCjANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
   ZC1ydWJ5MRkwFwYKCZImiZPyLGQBGRYJemVuc3BpZGVyMRMwEQYKCZImiZPyLGQB
-  GRYDY29tMB4XDTI1MDEwNjIzMjcwMVoXDTI2MDEwNjIzMjcwMVowRTETMBEGA1UE
+  GRYDY29tMB4XDTI2MDEwNzAxMDkxNFoXDTI3MDEwNzAxMDkxNFowRTETMBEGA1UE
   AwwKcnlhbmQtcnVieTEZMBcGCgmSJomT8ixkARkWCXplbnNwaWRlcjETMBEGCgmS
   JomT8ixkARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALda
   b9DCgK+627gPJkB6XfjZ1itoOQvpqH1EXScSaba9/S2VF22VYQbXU1xQXL/WzCkx
@@ -21,12 +21,12 @@ cert_chain:
   qhtV7HJxNKuPj/JFH0D2cswvzznE/a5FOYO68g+YCuFi5L8wZuuM8zzdwjrWHqSV
   gBEfoTEGr7Zii72cx+sCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAw
   HQYDVR0OBBYEFEfFe9md/r/tj/Wmwpy+MI8d9k/hMA0GCSqGSIb3DQEBCwUAA4IB
-  AQAC0WQJcPOWPFwkojhzweilRVjTJ19UiLhiBTw3C1wJO3LVdBkWDmnnhAmKuX4D
-  r7vjQvESlABGIPdutI1Yl7mrHQzTkfLfXvNN6MT0nLChPyIYauT6SZZxubwJrUfA
-  7R0c2CJTIboZ0XaGpLsXqHEF1c29H7TV1QvVuqKAN2mCjh4N82QVn+ZKtys28AwT
-  6GfQX2fqLoi4KSc7xIzHKaNzqxeOICmJofk9w5VZ2rRN6yes8jvFYwz9HR41wdj8
-  bwfinv7Yp5fA6AysuZLhCykyfDuZVRrUp0Vb68YCKsLjJly/Theak+euNTxvHsB+
-  al9oSgPPHICMEX65qvLywitx
+  AQA/X8/PKTTc/IkYQEUL6XWtfK8fAfbuLJzmLcz6f2ZWrtBvPsYvqRuwI1bWUtil
+  2ibJEfIuSIHX6BsUTX18+hlaIussf6EWq/YkE7e2BKmgQE42vSOZMce0kCEAPbx0
+  rQtqonfWTHQ8UbQ7GqCL3gDQ0QDD2B+HUlb4uaCZ2icxqa/eOtxMvHC2tj+h0xKY
+  xL84ipM5kr0bHGf9/MRZJWcw51urueNycBXu1Xh+pEN8qBmYsFRR4SIODLClybAO
+  6cbm2PyPQgKNiqE4H+IQrDVHd9bJs1XgLElk3qoaJBWXc/5fy0J1imYb25UqmiHG
+  snGe1hrppvBRdcyEzvhfIPjI
   -----END CERTIFICATE-----
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
@@ -36,45 +36,72 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '9'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
 - !ruby/object:Gem::Dependency
   name: hoe
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: '4.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: '4.7'
 description: |-
   sexp_processor branches from ParseTree bringing all the generic sexp
   processing tools with it. Sexp, SexpProcessor, Environment, etc... all
   for your language processing pleasure.
+
+  == Features/Problems:
+
+  * Includes SexpProcessor and CompositeSexpProcessor.
+
+    * Allows you to write very clean filters.
+
+  * Includes MethodBasedSexpProcessor
+
+    * Makes writing language processors even easier!
+
+  * Sexp provides a simple and clean interface to creating and manipulating ASTs.
+
+    * Includes new pattern matching system.
 email:
 - ryand-ruby@zenspider.com
 executables: []
 extensions: []
 extra_rdoc_files:
 - History.rdoc
-- Manifest.txt
 - README.rdoc
 files:
 - History.rdoc
@@ -97,6 +124,7 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/seattlerb/sexp_processor
+  documentation_uri: http://docs.seattlerb.org/sexp_processor
 rdoc_options:
 - "--main"
 - README.rdoc