appmap 0.98.0 → 0.99.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 409ba80fae04609fbed2635342499d743638bf1be48c9a2872931fb21898ffee
-  data.tar.gz: 26dab002cc9e0d616cb803d0d67095c5d5eaf307ae8f8b86f6e4a3446127af99
+  metadata.gz: 463cf482f8d6b1a33721886b5d049ae0ce11c469759562a4ea0b8c6c149abb23
+  data.tar.gz: 727ecbc15e19e486f69ed668d9e077a3045c10b24c0892b66801346d11a02a4d
 SHA512:
-  metadata.gz: 9db98d6ca5f3a620b03ea1b8ecaa2fff7769745ec1b8dda2cee635cca7024f73b90fd6cd6b4c1eacebed5645630e30113d1c771ee07d1af6107d2de228ee58bf
-  data.tar.gz: a777268c6a9edd86113c9f139c010f0987ed60f91ed050bfe9d42bf5dce4e8d4e8acd2bcf9eaef0cade32159ea02ba590998b41cd5520c7d3c24a4652e0d6dad
+  metadata.gz: cec680149e2742d4ed403a73d7283c454e0e561a95207ab426b9996c4056fa93626d0544de0923f575252038869a7e7e6b6f351eaaab4f1b7576ef20467bb527
+  data.tar.gz: 573b7ba81cae478ea658c27937563410bd83143bb0c13b9aeea3402422460ac55debff2adeffc83cb77c4be1372f6e49f1c640d8ff193c18199597a0bff25f58

data/CHANGELOG.md

@@ -1,3 +1,24 @@
+# [0.99.0](https://github.com/getappmap/appmap-ruby/compare/v0.98.1...v0.99.0) (2023-04-13)
+
+
+### Bug Fixes
+
+* Don't report vendored paths as local ([e09cb4f](https://github.com/getappmap/appmap-ruby/commit/e09cb4f9764d275c7ba0858f16b545c2d287b473))
+* Report the first relative path in the backtrace ([f81b346](https://github.com/getappmap/appmap-ruby/commit/f81b346ebc66a4f60ca26ad384d72672893d1333))
+
+
+### Features
+
+* Report metadata.test_failure ([82c87d2](https://github.com/getappmap/appmap-ruby/commit/82c87d2453ce7c040bce138bf55d94d47aca58fc))
+
+## [0.98.1](https://github.com/getappmap/appmap-ruby/compare/v0.98.0...v0.98.1) (2023-03-09)
+
+
+### Bug Fixes
+
+* Prevent AppMaps from being constantly re-indexed ([0311ab4](https://github.com/getappmap/appmap-ruby/commit/0311ab4c2f759e8c471982d47e6038e3aaa9f908))
+* Report test_status from minitest ([659f89a](https://github.com/getappmap/appmap-ruby/commit/659f89aa5a11280eb886abe14ec0c70790cb07a7))
+
 # [0.98.0](https://github.com/getappmap/appmap-ruby/compare/v0.97.0...v0.98.0) (2023-02-22)
 
 

data/README.md

@@ -22,7 +22,7 @@ granular than a full debug trace. It's designed to be optimal for understanding
 Visit the [AppMap for Ruby](https://appland.com/docs/reference/appmap-ruby.html) reference page on AppLand.com for a complete reference guide.
 
 # Development
-[![Build Status](https://travis-ci.com/applandinc/appmap-ruby.svg?branch=master)](https://travis-ci.com/applandinc/appmap-ruby)
+[![Build Status](https://travis-ci.com/getappmap/appmap-ruby.svg?branch=master)](https://travis-ci.com/getappmap/appmap-ruby)
 
 ## Internal architecture
 

data/appmap.yml

@@ -1,2 +1,3 @@
 name: AppMap Rubygem
 packages: []
+appmap_dir: dummy

data/lib/appmap/minitest.rb

@@ -11,7 +11,7 @@ module AppMap
   # be activated around each test.
   module Minitest
     APPMAP_OUTPUT_DIR = 'tmp/appmap/minitest'
-    LOG = ( ENV['APPMAP_DEBUG'] == 'true' || ENV['DEBUG'] == 'true' )
+    LOG = (ENV['APPMAP_DEBUG'] == 'true' || ENV['DEBUG'] == 'true')
 
     def self.metadata
       AppMap.detect_metadata
@@ -26,14 +26,28 @@ module AppMap
       end
 
       def source_location
-        test.method(test_name).source_location.join(':')
+        location = test.method(test_name).source_location
+        [ Util.normalize_path(location.first), location.last ].join(':')
       end
 
-
-      def finish(exception)
-        warn "Finishing recording of test #{test.class}.#{test.name}" if AppMap::Minitest::LOG
+      def finish(failures, exception)
+        failed = failures.any? || exception
+        warn "Finishing recording of #{failed ? 'failed ' : ''} test #{test.class}.#{test.name}" if AppMap::Minitest::LOG
         warn "Exception: #{exception}" if exception && AppMap::Minitest::LOG
 
+        if failed
+          failure_exception = failures.first || exception
+          warn "Failure exception: #{failure_exception}" if AppMap::Minitest::LOG
+
+          first_location = failure_exception.backtrace_locations.find { |location| !Pathname.new(Util.normalize_path(location.absolute_path)).absolute? }
+          failure_location = [ Util.normalize_path(first_location.path), first_location.lineno ].join(':') if first_location
+
+          test_failure = {
+            message: failure_exception.message,
+            location: failure_location,
+          }
+        end
+
         events = []
         AppMap.tracing.delete @trace
 
@@ -45,12 +59,13 @@ module AppMap
 
         feature_group = test.class.name.underscore.split('_')[0...-1].join('_').capitalize
         feature_name = test.name.split('_')[1..-1].join(' ')
-        scenario_name = [ feature_group, feature_name ].join(' ')
+        scenario_name = [feature_group, feature_name].join(' ')
 
         AppMap::Minitest.save name: scenario_name,
                               class_map: class_map,
                               source_location: source_location,
-                              test_status: exception ? 'failed' : 'succeeded',
+                              test_status: failed ? 'failed' : 'succeeded',
+                              test_failure: test_failure,
                               exception: exception,
                               events: events
       end
@@ -80,18 +95,18 @@ module AppMap
         recording = @recordings_by_test.delete(test.object_id)
         return warn "No recording found for #{test}" unless recording
 
-        recording.finish exception
+        recording.finish test.failures || [], exception
       end
 
       def config
-        @config or raise "AppMap is not configured"
+        @config or raise 'AppMap is not configured'
       end
 
       def add_event_methods(event_methods)
         @event_methods += event_methods
       end
 
-      def save(name:, class_map:, source_location:, test_status:, exception:, events:)
+      def save(name:, class_map:, source_location:, test_status:, test_failure:, exception:, events:)
         metadata = AppMap::Minitest.metadata.tap do |m|
           m[:name] = name
           m[:source_location] = source_location
@@ -99,18 +114,16 @@ module AppMap
           m[:frameworks] ||= []
           m[:frameworks] << {
             name: 'minitest',
-            version: Gem.loaded_specs['minitest']&.version&.to_s
+            version: Gem.loaded_specs['minitest']&.version&.to_s,
           }
           m[:recorder] = {
             name: 'minitest',
-            type: 'tests'
+            type: 'tests',
           }
           m[:test_status] = test_status
+          m[:test_failure] = test_failure if test_failure
           if exception
-            m[:exception] = {
-              class: exception.class.name,
-              message: AppMap::Event::MethodEvent.display_string(exception.to_s)
-            }
+            m[:exception] = Util.format_exception(exception)
           end
         end
 
@@ -118,7 +131,7 @@ module AppMap
           version: AppMap::APPMAP_FORMAT_VERSION,
           metadata: metadata,
           classMap: class_map,
-          events: events
+          events: events,
         }.compact
         fname = AppMap::Util.scenario_filename(name)
 

data/lib/appmap/rspec.rb

@@ -8,7 +8,7 @@ require 'fileutils'
 module AppMap
   module RSpec
     APPMAP_OUTPUT_DIR = 'tmp/appmap/rspec'
-    LOG = false
+    LOG = (ENV['APPMAP_DEBUG'] == 'true' || ENV['DEBUG'] == 'true')
 
     def self.metadata
       AppMap.detect_metadata
@@ -96,10 +96,24 @@ module AppMap
         result
       end
 
-      def finish(exception)
-        warn "Finishing recording of example #{example}" if AppMap::RSpec::LOG
+      def finish(failure, exception)
+        failed = true if failure || exception
+        warn "Finishing recording of #{failed ? 'failed ' : ''} example #{example}" if AppMap::RSpec::LOG
         warn "Exception: #{exception}" if exception && AppMap::RSpec::LOG
 
+        if failed
+          failure_exception = failure || exception
+          warn "Failure exception: #{failure_exception}" if AppMap::RSpec::LOG
+
+          first_location = failure_exception.backtrace_locations.find { |location| !Pathname.new(Util.normalize_path(location.absolute_path)).absolute? }
+          failure_location = [ Util.normalize_path(first_location.path), first_location.lineno ].join(':') if first_location
+
+          test_failure = {
+            message: failure_exception.message.strip,
+            location: failure_location,
+          }
+        end
+
         events = []
         AppMap.tracing.delete @trace
 
@@ -134,6 +148,7 @@ module AppMap
                            class_map: class_map,
                            source_location: source_location,
                            test_status: exception ? 'failed' : 'succeeded',
+                           test_failure: test_failure,
                            exception: exception,
                            events: events
       end
@@ -169,7 +184,7 @@ module AppMap
         recording = @recordings_by_example.delete(example.object_id)
         return warn "No recording found for #{example}" unless recording
 
-        recording.finish exception unless recording == :false
+        recording.finish example.execution_result.exception || exception, exception unless recording == :false
       end
 
       def config
@@ -180,7 +195,7 @@ module AppMap
         @event_methods += event_methods
       end
 
-      def save(name:, class_map:, source_location:, test_status:, exception:, events:)
+      def save(name:, class_map:, source_location:, test_status:, test_failure:, exception:, events:)
         metadata = AppMap::RSpec.metadata.tap do |m|
           m[:name] = name
           m[:source_location] = source_location
@@ -195,8 +210,10 @@ module AppMap
             type: 'tests'
           }
           m[:test_status] = test_status
+          m[:test_failure] = test_failure if test_failure
           if exception
-            m[:exception] = {
+            m[:exception] = Util.format_exception(exception)
+            {
               class: exception.class.name,
               message: AppMap::Event::MethodEvent.display_string(exception.to_s)
             }

data/lib/appmap/util.rb

@@ -139,13 +139,24 @@ module AppMap
       end
 
       def normalize_path(path)
-        if path.index(Dir.pwd) == 0 && !path.index(Bundler.bundle_path.to_s)
+        is_local_path    = -> { path.index(Dir.pwd) == 0 }
+        is_bundled_path  = -> { path.index(Bundler.bundle_path.to_s) == 0 }
+        is_vendored_path = -> { path.index(File.join(Dir.pwd, 'vendor/bundle')) == 0 }
+
+        if is_local_path.() && !is_bundled_path.() && !is_vendored_path.()
           path[Dir.pwd.length + 1..-1]
         else
           path
         end
       end
 
+      def format_exception(exception)
+        {
+          class: exception.class.name,
+          message: AppMap::Event::MethodEvent.display_string(exception.to_s),
+        }
+      end
+
       # Convert a Rails-style path from /org/:org_id(.:format)
       # to Swagger-style paths like /org/{org_id}
       def swaggerize_path(path)

data/lib/appmap/version.rb

@@ -3,9 +3,9 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.98.0'
+  VERSION = '0.99.0'
 
-  APPMAP_FORMAT_VERSION = '1.10.0'
+  APPMAP_FORMAT_VERSION = '1.12.0'
 
   SUPPORTED_RUBY_VERSIONS = %w[2.5 2.6 2.7 3.0 3.1 3.2].freeze
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.98.0
+  version: 0.99.0
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-02-22 00:00:00.000000000 Z
+date: 2023-04-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: method_source