appmap 0.72.4 → 0.72.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: df8cfae5642145ffdf039e46a97b4dc2cf748e35658611237e4f8c86fcfdeb4b
-  data.tar.gz: af8324a534ba90df3ba87467581936e3ce3109e67fcc1952fd3e25912b003be6
+  metadata.gz: e6b34c9af8d6d392be8211d9667daaccd2156c41a9c3c040bcdcfbfcbcba9332
+  data.tar.gz: 16bc9b9fe0a4288af913b3b6038ea4866913db72c0a49e5b91b11f08eebf06b0
 SHA512:
-  metadata.gz: c8428c0343bb9a856d96fa332ca6217c9cabb518df36e48562f5855714ef6291c2b2ea7a14f7e8b5ea2a4a01ae3e1ac685a95e7889cca705ab451989c83f226b
-  data.tar.gz: ea54394ee1feb68949a4b92764585a1e170ac35d0fd0b2fb5ef744a0499831f23d1e179f85110b21a7dec029ca0a4cb7258ad64f081fa5273423e6fe178bbbcb
+  metadata.gz: b5d3870cecadc652deb2c7dfa682d54e4ca166eb3ed8bd10df40d7d03a45f43dc1737190541c22a28e605207ab27925bd4ce2ea80fa67ab9e5d1090cfdeb253b
+  data.tar.gz: 3a2f3c10dec5548ad30a4ffb3c32ca94b1efcc3848fcb9b050a76f33bcb0b084d1f26a69c74269da77264c11f85b130fcdcce9c53c3a7bb52742301b864b251d

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [0.72.5](https://github.com/applandinc/appmap-ruby/compare/v0.72.4...v0.72.5) (2022-02-17)
+
+
+### Bug Fixes
+
+* Override method accessors to provide the correct signature ([#223](https://github.com/applandinc/appmap-ruby/issues/223)) ([936bba4](https://github.com/applandinc/appmap-ruby/commit/936bba470c5360ee313e0b3e45a65d83acd0b53d))
+
 ## [0.72.4](https://github.com/applandinc/appmap-ruby/compare/v0.72.3...v0.72.4) (2022-02-17)
 
 

data/lib/appmap/hook/method.rb

@@ -7,6 +7,23 @@ module AppMap
   end
 
   class Hook
+    SIGNATURES = {}
+
+    LOOKUP_SIGNATURE = lambda do |id|
+      method = super(id)
+    
+      signature = SIGNATURES[[ method.owner, method.name ]]
+      if signature
+        method.singleton_class.module_eval do
+          define_method(:parameters) do
+            signature
+          end
+        end
+      end
+    
+      method
+    end
+    
     class Method
       attr_reader :hook_package, :hook_class, :hook_method
 
@@ -100,12 +117,10 @@ module AppMap
         hook_method_def = hook_method_def.ruby2_keywords if hook_method_def.respond_to?(:ruby2_keywords)
 
         hook_method_parameters = hook_method.parameters.dup.freeze
+        SIGNATURES[[ hook_class, hook_method.name ]] = hook_method_parameters
+
         hook_class.ancestors.first.tap do |cls|
           cls.define_method_with_arity(hook_method.name, hook_method.arity, hook_method_def)
-          redefined_method = cls.instance_method(hook_method.name)
-          redefined_method.singleton_class.module_eval do
-            define_method(:parameters) { hook_method_parameters }
-          end
         end
       end
 
@@ -136,4 +151,27 @@ module AppMap
       end
     end
   end
+
+  module ObjectMethods
+    define_method(:method, AppMap::Hook::LOOKUP_SIGNATURE)
+    define_method(:public_method, AppMap::Hook::LOOKUP_SIGNATURE)
+    define_method(:singleton_method, AppMap::Hook::LOOKUP_SIGNATURE)
+  end
+
+  module ModuleMethods
+    define_method(:instance_method, AppMap::Hook::LOOKUP_SIGNATURE)
+    define_method(:public_instance_method, AppMap::Hook::LOOKUP_SIGNATURE)
+  end
+end
+
+unless ENV['APPMAP_NO_PATCH_OBJECT'] == 'true'
+  class Object
+    prepend AppMap::ObjectMethods
+  end
+end
+
+unless ENV['APPMAP_NO_PATCH_MODULE'] == 'true'
+  class Module
+    prepend AppMap::ModuleMethods
+  end
 end

data/lib/appmap/version.rb

@@ -3,7 +3,7 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.72.4'
+  VERSION = '0.72.5'
 
   APPMAP_FORMAT_VERSION = '1.5.1'
 

data/spec/fixtures/hook/report_parameters.rb

@@ -0,0 +1,8 @@
+
+class ReportParameters
+  def to_s; self.class.name; end
+
+  def report_parameters(*args, kw1:, kw2: 'kw2', **kws)
+    method(:report_parameters).parameters
+  end
+end

data/spec/hook_spec.rb

@@ -225,6 +225,12 @@ describe 'AppMap class Hooking', docker: false do
         :value: default
     YAML
     test_hook_behavior 'spec/fixtures/hook/instance_method.rb', events_yaml do
+      expect(InstanceMethod.instance_method(:say_default).parameters).to eq([])
+      expect(InstanceMethod.public_instance_method(:say_default).parameters).to eq([])
+      expect(InstanceMethod.new.method(:say_default).parameters).to eq([])
+      expect(InstanceMethod.new.public_method(:say_default).parameters).to eq([])
+      expect { InstanceMethod.new.singleton_method(:say_default) }.to raise_error(NameError)
+
       expect(InstanceMethod.new.say_default).to eq('default')
     end
   end
@@ -315,7 +321,22 @@ describe 'AppMap class Hooking', docker: false do
         :class: String
         :value: protected
     YAML
+    parameters = []
     test_hook_behavior 'spec/fixtures/hook/protected_method.rb', events_yaml do
+      expect(ProtectedMethod.singleton_method(:call_protected).parameters).to eq(parameters)
+      expect(ProtectedMethod.instance_method(:call_protected).parameters).to eq(parameters)
+      expect(ProtectedMethod.public_instance_method(:call_protected).parameters).to eq(parameters)
+      expect(ProtectedMethod.new.method(:call_protected).parameters).to eq(parameters)
+      expect(ProtectedMethod.new.public_method(:call_protected).parameters).to eq(parameters)
+      expect { ProtectedMethod.new.singleton_method(:call_protected) }.to raise_error(NameError)
+
+      expect(ProtectedMethod.singleton_method(:protected_method).parameters).to eq(parameters)
+      expect(ProtectedMethod.instance_method(:protected_method).parameters).to eq(parameters)
+      expect(ProtectedMethod.public_instance_method(:protected_method).parameters).to eq(parameters)
+      expect(ProtectedMethod.new.method(:protected_method).parameters).to eq(parameters)
+      expect(ProtectedMethod.new.public_method(:protected_method).parameters).to eq(parameters)
+      expect { ProtectedMethod.new.singleton_method(:protected_method) }.to raise_error(NameError)
+
       expect(ProtectedMethod.new.call_protected).to eq('protected')
     end
   end
@@ -358,6 +379,7 @@ describe 'AppMap class Hooking', docker: false do
         :class: String
         :value: self.protected
     YAML
+    parameters = []
     test_hook_behavior 'spec/fixtures/hook/protected_method.rb', events_yaml do
       expect(ProtectedMethod.call_protected).to eq('self.protected')
     end
@@ -1100,6 +1122,51 @@ describe 'AppMap class Hooking', docker: false do
     end
   end
 
+  it 'preserves the sighnature of hooked methods' do
+    parameters = []
+    events = <<~YAML
+    ---
+    - :id: 1
+      :event: :call
+      :defined_class: ReportParameters
+      :method_id: report_parameters
+      :path: spec/fixtures/hook/report_parameters.rb
+      :lineno: 5
+      :static: false
+      :parameters:
+      - :name: :args
+        :class: Array
+        :value: '["foo"]'
+        :kind: :rest
+      - :name: :kw1
+        :class: String
+        :value: kw1
+        :kind: :keyreq
+      - :name: :kw2
+        :class: NilClass
+        :value: null
+        :kind: :key
+      - :name: :kws
+        :class: Hash
+        :value: "{}"
+        :kind: :keyrest
+      :receiver:
+        :class: ReportParameters
+        :value: ReportParameters
+    - :id: 2
+      :event: :return
+      :parent_id: 1
+      :return_value:
+        :class: Array
+        :value: "[[:rest, :args], [:keyreq, :kw1], [:key, :kw2], [:keyrest, :kws]]"
+    YAML
+    parameters = [[:rest, :args], [:keyreq, :kw1], [:key, :kw2], [:keyrest, :kws]]
+    test_hook_behavior 'spec/fixtures/hook/report_parameters.rb', events do
+      expect(ReportParameters.instance_method(:report_parameters).parameters).to eq(parameters)
+      expect(ReportParameters.new.report_parameters('foo', kw1: 'kw1')).to eq(parameters)
+    end
+  end
+
   describe 'kwargs handling' do
     if ruby_2?
       # https://github.com/applandinc/appmap-ruby/issues/153

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.72.4
+  version: 0.72.5
 platform: ruby
 authors:
 - Kevin Gilpin
@@ -447,6 +447,7 @@ files:
 - spec/fixtures/hook/pkg_a/a.rb
 - spec/fixtures/hook/prepended_override.rb
 - spec/fixtures/hook/protected_method.rb
+- spec/fixtures/hook/report_parameters.rb
 - spec/fixtures/hook/revoke_api_key.appmap.json
 - spec/fixtures/hook/singleton_method.rb
 - spec/fixtures/hook/spec/api_spec.rb