appmap 0.54.2 → 0.57.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5368ab640de6fdf14f35502a5c756b7c6f681ee8d9a87c5fd9cd13d2b1036d9
-  data.tar.gz: 15806ff9cfa6256bb29c9ed2d8cb7d4ff5f62cfc3354e313e3b6f52b4cd96c12
+  metadata.gz: 442ca8673e42cd8bea0db1ba97a533e7a38965f3d27707bd8e4379b578ee85cb
+  data.tar.gz: 1174d299f1446c8c3e16403437682196ce794bad27b7c6dd69d712f9bd59e36a
 SHA512:
-  metadata.gz: d078fcb6e1bf1942462be38f2547b73e34c6f9a0313aabdd8919070218701937bd53bec4207c34c48c37e7b67f69500fbd6339f6fc26ad11eed86e2027ca4acf
-  data.tar.gz: f5493735794a3385287721a30021996bc66ac44e19bc4e2214823eb78a730f07213989c93dadf8f38f0163c6c21fc39d375511b4ccd87b33974a54e667898b22
+  metadata.gz: 73bb23b5ac1401565c57ffbd833fe266ddebfe71271926ca0afaa08b92c88ffd47f5c92f6e91f213b8066729e6607a1b0f99731449eecf7bd68ab125a0f5de3f
+  data.tar.gz: f72d956ed6c6a46aebe12724a910d5cbdec1aee3d8b61f300e8b2a679873b25db245cca4407c02a03df1588fd3734d616472aed69c2741898e60bc04f6d04b33

data/.travis.yml

@@ -34,27 +34,29 @@ before_install:
   - |
     nvm install --lts \
       && nvm use --lts
- 
+
 # GEM_ALTERNATIVE_NAME only needed for deployment 
 jobs:
   include:
   - stage: test
     script:
+    - set -e
     - mkdir tmp
-    - GEM_ALTERNATIVE_NAME='' bundle exec rake test
+    - npm i -g yarn
+    - GEM_ALTERNATIVE_NAME='' bundle exec rake gem:build test
 
 before_deploy:
   - |
-    nvm install --lts \
-      && nvm use --lts \
-      && npm i -g \
-        semantic-release \
-        @semantic-release/git \
-        @semantic-release/changelog \
-        semantic-release-rubygem
+    npm i -g \
+      yarn \
+      semantic-release \
+      @semantic-release/git \
+      @semantic-release/changelog \
+      semantic-release-rubygem
 
 deploy:
   - provider: script
     script: ./release.sh
     on:
       branch: master
+      condition: "$TRAVIS_RUBY_VERSION = 2.7"

data/CHANGELOG.md

@@ -1,3 +1,47 @@
+# [0.57.0](https://github.com/applandinc/appmap-ruby/compare/v0.56.0...v0.57.0) (2021-06-29)
+
+
+### Features
+
+* Update init command to return JSON ([1f93e89](https://github.com/applandinc/appmap-ruby/commit/1f93e8909684e1018f513d69adfde2a5d0bf6bc9))
+
+# [0.56.0](https://github.com/applandinc/appmap-ruby/compare/v0.55.0...v0.56.0) (2021-06-28)
+
+
+### Features
+
+* add appmap-agent-status executable with config/project properties ([043f845](https://github.com/applandinc/appmap-ruby/commit/043f8453a2533a6e172d1cd23fcde04f19e73173))
+
+# [0.55.0](https://github.com/applandinc/appmap-ruby/compare/v0.54.4...v0.55.0) (2021-06-28)
+
+
+### Bug Fixes
+
+* Avoid calling == ([f30ed9f](https://github.com/applandinc/appmap-ruby/commit/f30ed9f309753252df35e372d925db3b914260d4))
+* Log dynamic loading of appmap helpers at info level ([15dcd3c](https://github.com/applandinc/appmap-ruby/commit/15dcd3c913fa1c32aea034b28ddae59668efa217))
+* Remove dynamic loading of rake and rspec helpers ([6790970](https://github.com/applandinc/appmap-ruby/commit/67909702f3c8a52081ef1e23a87c292908883334))
+
+
+### Features
+
+* APPMAP_PROFILE_DISPLAY_STRING and APPMAP_OBJECT_STRING ([3f5daa8](https://github.com/applandinc/appmap-ruby/commit/3f5daa890bfbfd39b7f825794d0c43da509b3201))
+* Package name to require can be specified when hooking a gem ([fcc5eb6](https://github.com/applandinc/appmap-ruby/commit/fcc5eb691a0330444560eb4c2afe7fc3c4c8afa8))
+* Profile packaging hooking ([c020a31](https://github.com/applandinc/appmap-ruby/commit/c020a312f4545348ec7cc302443269c57a7fc026))
+
+## [0.54.4](https://github.com/applandinc/appmap-ruby/compare/v0.54.3...v0.54.4) (2021-06-27)
+
+
+### Bug Fixes
+
+* Only allow trace_end once per location ([10e48cf](https://github.com/applandinc/appmap-ruby/commit/10e48cf855907f9029479b4b7b63bc4d25d664ab))
+
+## [0.54.3](https://github.com/applandinc/appmap-ruby/compare/v0.54.2...v0.54.3) (2021-06-25)
+
+
+### Bug Fixes
+
+* Get deployment working with packaging of NodeJS code ([733c5b8](https://github.com/applandinc/appmap-ruby/commit/733c5b85ec1a0c17ada81be524fa572f78f52500))
+
 ## [0.54.2](https://github.com/applandinc/appmap-ruby/compare/v0.54.1...v0.54.2) (2021-06-25)
 
 

data/exe/appmap-agent-init

@@ -5,10 +5,10 @@ require 'optparse'
 require 'appmap'
 require 'appmap/command/agent_setup/init'
 
-@options = {:config_file => AppMap::DEFAULT_CONFIG_FILE_PATH}
+@options = { config_file: AppMap::DEFAULT_CONFIG_FILE_PATH }
 
 OptionParser.new do |parser|
-  parser.banner = 'Usage: bundle exec exe/appmap-agent-init [options]'
+  parser.banner = 'Usage: appmap-agent-init [options]'
 
   description = "AppMap configuration file path (default: #{AppMap::DEFAULT_CONFIG_FILE_PATH})"
   parser.on('-c', '--config=FILEPATH', description) do |filepath|

data/exe/appmap-agent-status

@@ -0,0 +1,19 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'optparse'
+require 'appmap'
+require 'appmap/command/agent_setup/status'
+
+@options = { config_file: AppMap::DEFAULT_CONFIG_FILE_PATH }
+
+OptionParser.new do |parser|
+  parser.banner = 'Usage: appmap-agent-status [options]'
+
+  description = "AppMap configuration file path (default: #{AppMap::DEFAULT_CONFIG_FILE_PATH})"
+  parser.on('-c', '--config=FILEPATH', description) do |filepath|
+    @options[:config_file] = filepath
+  end
+end.parse!
+
+AppMap::Command::AgentSetup::Status.new(@options[:config_file]).perform

data/lib/appmap.rb

@@ -28,11 +28,9 @@ lambda do
   INITIALIZERS = {
     # In a Rails app, Rails is always defined by the time the other gems are loaded. Therefore, we
     # don't try and trap the loading of Rails itself here.
-    'RSpec' => Initializer.new('AppMap::RSpec', 'appmap/rspec', 'rspec-core'),
+    # Emperically, Rake and RSpec are also defined before appmap is loaded whenever a Rake task or
+    # RSpec tests are being run. Therefore, the only hook we need here is Minitest.
     'Minitest::Unit::TestCase' => Initializer.new('AppMap::Minitest', 'appmap/minitest', 'minitest'),
-    'Rake' => [
-      Initializer.new('AppMap::Swagger', 'appmap/swagger', 'rake')
-    ]
   }
 
   TracePoint.new(:class) do |tp|

data/lib/appmap/command/agent_setup/init.rb

@@ -1,8 +1,7 @@
 # frozen_string_literal: true
 
-require 'fileutils'
+require 'json'
 require 'appmap/service/guesser'
-require 'appmap/util'
 
 module AppMap
   module Command
@@ -11,32 +10,19 @@ module AppMap
 
       class Init < InitStruct
         def perform
-          if File.exist?(config_file)
-            puts AppMap::Util.color(%(The AppMap config file #{config_file} already exists.), :magenta)
-            return
-          end
-
-          ensure_directory_exists
-
           config = {
             'name' => Service::Guesser.guess_name,
             'packages' => Service::Guesser.guess_paths.map { |path| { 'path' => path } }
           }
-          content = YAML.dump(config).gsub("---\n", '')
 
-          File.write(config_file, content)
-          puts AppMap::Util.color(
-            %(The following AppMap config file #{config_file} has been created:),
-            :green
-          )
-          puts content
-        end
-
-        private
+          result = {
+            configuration: {
+              filename: config_file,
+              contents: YAML.dump(config)
+            }
+          }
 
-        def ensure_directory_exists
-          dirname = File.dirname(config_file)
-          FileUtils.mkdir_p(dirname) unless File.directory?(dirname)
+          puts JSON.pretty_generate(result)
         end
       end
     end

data/lib/appmap/command/agent_setup/status.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require 'json'
+require 'appmap/service/config_analyzer'
+
+module AppMap
+  module Command
+    module AgentSetup
+      StatusStruct = Struct.new(:config_file)
+
+      class Status < StatusStruct
+        def perform
+          status = {
+            :properties => {
+              :config => {
+                :app => config_analyzer.app_name,
+                :present => config_analyzer.present?,
+                :valid => config_analyzer.valid?
+              },
+              :project => {
+                :agentVersionProject => AppMap::VERSION,
+                :language => 'ruby',
+                :remoteRecordingCapable => Gem.loaded_specs.has_key?('rails'),
+                :integrationTests => false #TODO
+              }
+            }
+          }
+
+          puts status.to_json
+        end
+
+        private
+
+        def config_analyzer
+          @config_analyzer ||= Service::ConfigAnalyzer.new(config_file)
+        end
+      end
+    end
+  end
+end

data/lib/appmap/config.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'set'
 require 'yaml'
 require 'appmap/util'
 require 'appmap/handler/net_http'
@@ -343,7 +344,7 @@ module AppMap
                 shallow = package['shallow']
                 # shallow is true by default for gems
                 shallow = true if shallow.nil?
-                Package.build_from_gem(gem, exclude: package['exclude'] || [], shallow: shallow)
+                Package.build_from_gem(gem, package_name: package['package'], exclude: package['exclude'] || [], shallow: shallow)
               else
                 Package.build_from_path(path, exclude: package['exclude'] || [], shallow: package['shallow'])
               end

data/lib/appmap/event.rb

@@ -29,11 +29,31 @@ module AppMap
 
         # Gets a display string for a value. This is not meant to be a machine deserializable value.
         def display_string(value)
-          return nil unless value
+          return nil if value.equal?(nil)
 
+          # With setting APPMAP_PROFILE_DISPLAY_STRING, stringifying this class is shown to take 9 seconds(!) of a 17 second test run.
+          return nil if best_class_name(value) == 'ActiveSupport::Callbacks::Filters::Environment'
+
+          if @times.nil? && ENV['APPMAP_PROFILE_DISPLAY_STRING'] == 'true'
+            @times = Hash.new {|memo,key| memo[key] = 0}
+            Thread.new do
+              sleep 0.5
+              while true
+                warn @times.to_a.sort{|a,b| b[1] <=> a[1]}[0..9].join("\n")
+                sleep 3
+              end
+            end
+          end
+
+          start = Time.now
           value_string = custom_display_string(value) || default_display_string(value)
 
-          (value_string||'')[0...LIMIT].encode('utf-8', invalid: :replace, undef: :replace, replace: '_')
+          if @times
+            elapsed = Time.now - start
+            @times[best_class_name(value)] += elapsed
+          end
+
+          encode_dislay_string(value_string)
         end
 
         def object_properties(hash_like)
@@ -57,8 +77,16 @@ module AppMap
           value_cls.name
         end
 
+        def encode_dislay_string(value)
+          (value||'')[0...LIMIT].encode('utf-8', invalid: :replace, undef: :replace, replace: '_')
+        end
+
         def custom_display_string(value)
           case value
+          when NilClass, TrueClass, FalseClass, Numeric, Time, Date
+            value.to_s
+          when String
+            value[0...LIMIT].encode('utf-8', invalid: :replace, undef: :replace, replace: '_')
           when File
             "#{value.class}[path=#{value.path}]"
           when Net::HTTP
@@ -71,6 +99,8 @@ module AppMap
         end
 
         def default_display_string(value)
+          return nil if ENV['APPMAP_OBJECT_STRING'] == 'false'
+
           last_resort_string = lambda do
             warn "AppMap encountered an error inspecting a #{value.class.name}: #{$!.message}"
             '*Error inspecting variable*'

data/lib/appmap/hook.rb

@@ -9,6 +9,7 @@ module AppMap
 
     OBJECT_INSTANCE_METHODS = %i[! != !~ <=> == === =~ __id__ __send__ class clone define_singleton_method display dup enum_for eql? equal? extend freeze frozen? hash inspect instance_eval instance_exec instance_of? instance_variable_defined? instance_variable_get instance_variable_set instance_variables is_a? itself kind_of? method methods nil? object_id private_methods protected_methods public_method public_methods public_send remove_instance_variable respond_to? send singleton_class singleton_method singleton_methods taint tainted? tap then to_enum to_s to_h to_a trust untaint untrust untrusted? yield_self].freeze
     OBJECT_STATIC_METHODS = %i[! != !~ < <= <=> == === =~ > >= __id__ __send__ alias_method allocate ancestors attr attr_accessor attr_reader attr_writer autoload autoload? class class_eval class_exec class_variable_defined? class_variable_get class_variable_set class_variables clone const_defined? const_get const_missing const_set constants define_method define_singleton_method deprecate_constant display dup enum_for eql? equal? extend freeze frozen? hash include include? included_modules inspect instance_eval instance_exec instance_method instance_methods instance_of? instance_variable_defined? instance_variable_get instance_variable_set instance_variables is_a? itself kind_of? method method_defined? methods module_eval module_exec name new nil? object_id prepend private_class_method private_constant private_instance_methods private_method_defined? private_methods protected_instance_methods protected_method_defined? protected_methods public_class_method public_constant public_instance_method public_instance_methods public_method public_method_defined? public_methods public_send remove_class_variable remove_instance_variable remove_method respond_to? send singleton_class singleton_class? singleton_method singleton_methods superclass taint tainted? tap then to_enum to_s trust undef_method untaint untrust untrusted? yield_self].freeze
+    SLOW_PACKAGE_THRESHOLD = 0.05
 
     @unbound_method_arity = ::UnboundMethod.instance_method(:arity)
     @method_arity = ::Method.instance_method(:arity)
@@ -37,8 +38,6 @@ module AppMap
     def initialize(config)
       @config = config
       @trace_enabled = []
-      # Paths that are known to be non-tracing
-      @notrace_paths = Set.new
     end
 
     # Observe class loading and hook all methods which match the config.
@@ -47,6 +46,32 @@ module AppMap
 
       hook_builtins
 
+      # Paths that are known to be non-tracing.
+      @notrace_paths = Set.new
+      # Locations that have already been visited.
+      @trace_locations = Set.new
+      @module_load_times = Hash.new {|memo,k| memo[k] = 0}
+      @slow_packages = Set.new
+
+      if ENV['APPMAP_PROFILE_HOOK'] == 'true'
+        Thread.new do
+          sleep 1
+          while true
+            @module_load_times
+              .keys
+              .select { |key| !@slow_packages.member?(key) }
+              .each do |key|
+              elapsed = @module_load_times[key]
+              if elapsed >= SLOW_PACKAGE_THRESHOLD
+                @slow_packages.add(key)
+                warn "AppMap: Package #{key} took #{@module_load_times[key]} seconds to hook"
+              end
+            end
+            sleep 5
+          end
+        end
+      end
+
       @trace_end = TracePoint.new(:end, &method(:trace_end))
       @trace_end.enable(&block)
     end
@@ -99,7 +124,9 @@ module AppMap
     end
 
     def trace_end(trace_point)
-      warn "Class or module ends at location #{trace_location(trace_point)}" if Hook::LOG || Hook::LOG_HOOK
+      location = trace_location(trace_point)
+      warn "Class or module ends at location #{location}" if Hook::LOG || Hook::LOG_HOOK
+      return unless @trace_locations.add?(location)
 
       path = trace_point.path
       enabled = !@notrace_paths.member?(path) && config.path_enabled?(path)
@@ -152,6 +179,7 @@ module AppMap
         end
       end
 
+      start = Time.now
       instance_methods.each(&hook.(cls))
       begin
         # NoMethodError: private method `singleton_class' called for Rack::MiniProfiler:Class
@@ -161,6 +189,11 @@ module AppMap
         #   uninitialized constant Faraday::Connection
         warn "NameError in #{__FILE__}: #{$!.message}"
       end
+      elapsed = Time.now - start
+      if location.index(Bundler.bundle_path.to_s) == 0
+        package_tokens = location[Bundler.bundle_path.to_s.length + 1..-1].split('/')
+        @module_load_times[package_tokens[1]] += elapsed
+      end
     end
   end
 end

data/lib/appmap/hook/method.rb

@@ -49,9 +49,14 @@ module AppMap
 
         hook_method_def = Proc.new do |*args, &block|
           instance_method = hook_method.bind(self).to_proc
+
+          is_array_containing_empty_hash = ->(obj) {
+            obj.is_a?(Array) && obj.length == 1 && obj[0].is_a?(Hash) && obj[0].size == 0
+          }
+
           call_instance_method = -> {
             # https://github.com/applandinc/appmap-ruby/issues/153
-            if Util.ruby_minor_version >= 2.7 && args == ARRAY_OF_EMPTY_HASH && hook_method.arity == 1
+            if Util.ruby_minor_version >= 2.7 && is_array_containing_empty_hash.(args) && hook_method.arity == 1
               instance_method.call({}, &block)
             else
               instance_method.call(*args, &block)

data/lib/appmap/service/config_analyzer.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module AppMap
+  module Service
+    class ConfigAnalyzer
+      attr_reader :config_error
+
+      def initialize(config_file)
+        @config_file = config_file
+        @config = load_config
+      end
+
+      def app_name
+        @config.to_h[:name] if present?
+      end
+
+      def present?
+        File.exist?(@config_file)
+      end
+
+      def valid?
+        present? && @config.to_h.key?(:name) && @config.to_h.key?(:packages)
+      end
+
+      private
+
+      def load_config
+        AppMap::Config.load_from_file @config_file if present?
+      rescue
+        nil
+      end
+    end
+  end
+end

data/lib/appmap/util.rb

@@ -185,7 +185,7 @@ module AppMap
 
       def startup_message(msg)
         if defined?(::Rails) && defined?(::Rails.logger) && ::Rails.logger
-          ::Rails.logger.debug msg
+          ::Rails.logger.info msg
         elsif ENV['DEBUG'] == 'true'
           warn msg
         end

data/lib/appmap/version.rb

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

data/release.sh

@@ -14,4 +14,3 @@ fi
 
 set -x
 exec semantic-release $RELEASE_FLAGS
-

data/spec/fixtures/config/incomplete_config.yml

@@ -0,0 +1 @@
+name: app

data/spec/fixtures/config/invalid_config.yml

@@ -0,0 +1,2 @@
+name: -123- %
+packages: ~{}~ ''

data/spec/fixtures/config/valid_config.yml

@@ -0,0 +1,3 @@
+name: appmap
+packages:
+  - path: app/models

data/spec/hook_spec.rb

@@ -970,7 +970,7 @@ describe 'AppMap class Hooking', docker: false do
         tz = ENV['TZ']
         ENV['TZ'] = 'UTC'
         Timecop.freeze(Time.utc('2020-01-01')) do
-          expect(Time).to receive(:now).exactly(3).times.and_call_original
+          expect(Time).to receive(:now).at_least(3).times.and_call_original
           expect(InstanceMethod.new.say_the_time).to be
         end
       ensure

data/spec/service/config_analyzer_spec.rb

@@ -0,0 +1,96 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'appmap/service/config_analyzer'
+
+describe AppMap::Service::ConfigAnalyzer do
+  subject { described_class.new(config_file) }
+
+  context 'with non-existing config' do
+    let(:config_file) { 'spec/fixtures/config/non_existing_config.yml'}
+
+    describe '.app_name' do
+      it 'returns nil' do
+        expect(subject.app_name).to be_nil
+      end
+    end
+
+    describe '.is_valid?' do
+      it 'returns false' do
+        expect(subject.valid?).to be_falsey
+      end
+    end
+
+    describe '.is_present?' do
+      it 'returns false' do
+        expect(subject.present?).to be_falsey
+      end
+    end
+  end
+
+  context 'with valid config' do
+    let(:config_file) { 'spec/fixtures/config/valid_config.yml'}
+
+    describe '.app_name' do
+      it 'returns app name value from config' do
+        expect(subject.app_name).to eq('appmap')
+      end
+    end
+
+    describe '.is_valid?' do
+      it 'returns true' do
+        expect(subject.valid?).to be_truthy
+      end
+    end
+
+    describe '.is_present?' do
+      it 'returns true' do
+        expect(subject.present?).to be_truthy
+      end
+    end
+  end
+
+  context 'with invalid YAML config' do
+    let(:config_file) { 'spec/fixtures/config/invalid_config.yml'}
+
+    describe '.app_name' do
+      it 'returns app name value from config' do
+        expect(subject.app_name).to be_nil
+      end
+    end
+
+    describe '.is_valid?' do
+      it 'returns false' do
+        expect(subject.valid?).to be_falsey
+      end
+    end
+
+    describe '.is_present?' do
+      it 'return true' do
+        expect(subject.present?).to be_truthy
+      end
+    end
+  end
+
+  context 'with incomplete config' do
+    let(:config_file) { 'spec/fixtures/config/incomplete_config.yml'}
+
+    describe '.app_name' do
+      it 'returns nil' do
+        expect(subject.app_name).to eq('app')
+      end
+    end
+
+    describe '.is_valid?' do
+      it 'guesses paths and returns true ' do
+        expect(subject.valid?).to be_truthy
+      end
+    end
+
+    describe '.is_present?' do
+      it 'returns true' do
+        expect(subject.present?).to be_truthy
+      end
+    end
+  end
+end

data/test/agent_setup_init_test.rb

@@ -5,8 +5,8 @@ require 'test_helper'
 
 class AgentSetupInitTest < Minitest::Test
   CONFIG_FILENAME = '123.yml'
-  SUBFOLDER_CONFIG_FILEPATH = 'conf/123.yml'
-  EXPECTED_CONFIG_CONTENT = %(name: appmap-ruby
+  EXPECTED_CONFIG_CONTENT = %(---
+name: appmap-ruby
 packages:
 - path: lib
 )
@@ -14,24 +14,24 @@ packages:
   def test_init_when_config_exists
     output = `./exe/appmap-agent-init`
     assert_equal 0, $CHILD_STATUS.exitstatus
-    assert_includes output, 'The AppMap config file appmap.yml already exists.'
+    expected = JSON.pretty_generate({
+      configuration: {
+        filename: 'appmap.yml',
+        contents: EXPECTED_CONFIG_CONTENT
+      }
+    })
+    assert_equal expected, output.strip
   end
 
   def test_init_with_custom_config_filename
     output = `./exe/appmap-agent-init -c #{CONFIG_FILENAME}`
     assert_equal 0, $CHILD_STATUS.exitstatus
-    assert_includes output, "The following AppMap config file #{CONFIG_FILENAME} has been created:"
-    assert_equal EXPECTED_CONFIG_CONTENT, File.read(CONFIG_FILENAME)
-  ensure
-    File.delete(CONFIG_FILENAME) if File.exist?(CONFIG_FILENAME)
-  end
-
-  def test_init_with_custom_config_file_in_subfolder
-    output = `./exe/appmap-agent-init --config=#{SUBFOLDER_CONFIG_FILEPATH}`
-    assert_equal 0, $CHILD_STATUS.exitstatus
-    assert_includes output, "The following AppMap config file #{SUBFOLDER_CONFIG_FILEPATH} has been created:"
-    assert_equal EXPECTED_CONFIG_CONTENT, File.read(SUBFOLDER_CONFIG_FILEPATH)
-  ensure
-    File.delete(SUBFOLDER_CONFIG_FILEPATH) if File.exist?(SUBFOLDER_CONFIG_FILEPATH)
+    expected = JSON.pretty_generate({
+      configuration: {
+        filename: CONFIG_FILENAME,
+        contents: EXPECTED_CONFIG_CONTENT
+      }
+    })
+    assert_equal expected, output.strip
   end
 end

data/test/agent_setup_status_test.rb

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class AgentSetupInitTest < Minitest::Test
+  def test_status
+    output = `./exe/appmap-agent-status`
+    assert_equal 0, $CHILD_STATUS.exitstatus
+    expected = {
+      :properties => {
+        :config => {
+          :app => 'AppMap Rubygem',
+          :present => true,
+          :valid => true
+        },
+        :project => {
+          :agentVersionProject => AppMap::VERSION,
+          :language => 'ruby',
+          :remoteRecordingCapable => false,
+          :integrationTests => false
+        }
+      }
+    }.to_json
+    assert_equal expected, output.strip
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.54.2
+  version: 0.57.0
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-06-25 00:00:00.000000000 Z
+date: 2021-06-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -295,6 +295,7 @@ email:
 - kgilpin@gmail.com
 executables:
 - appmap-agent-init
+- appmap-agent-status
 - appmap-inspect
 extensions:
 - ext/appmap/extconf.rb
@@ -324,6 +325,7 @@ files:
 - examples/mock_webapp/lib/mock_webapp/request.rb
 - examples/mock_webapp/lib/mock_webapp/user.rb
 - exe/appmap-agent-init
+- exe/appmap-agent-status
 - exe/appmap-inspect
 - ext/appmap/appmap.c
 - ext/appmap/extconf.rb
@@ -331,6 +333,7 @@ files:
 - lib/appmap/agent.rb
 - lib/appmap/class_map.rb
 - lib/appmap/command/agent_setup/init.rb
+- lib/appmap/command/agent_setup/status.rb
 - lib/appmap/command/inspect.rb
 - lib/appmap/command_error.rb
 - lib/appmap/config.rb
@@ -351,6 +354,7 @@ files:
 - lib/appmap/railtie.rb
 - lib/appmap/record.rb
 - lib/appmap/rspec.rb
+- lib/appmap/service/config_analyzer.rb
 - lib/appmap/service/guesser.rb
 - lib/appmap/swagger.rb
 - lib/appmap/swagger/configuration.rb
@@ -364,6 +368,9 @@ files:
 - release.sh
 - spec/class_map_spec.rb
 - spec/config_spec.rb
+- spec/fixtures/config/incomplete_config.yml
+- spec/fixtures/config/invalid_config.yml
+- spec/fixtures/config/valid_config.yml
 - spec/fixtures/hook/.gitignore
 - spec/fixtures/hook/app/controllers/api/api_keys_controller.rb
 - spec/fixtures/hook/app/controllers/organizations_controller.rb
@@ -545,10 +552,12 @@ files:
 - spec/record_net_http_spec.rb
 - spec/record_sql_rails_pg_spec.rb
 - spec/remote_recording_spec.rb
+- spec/service/config_analyzer_spec.rb
 - spec/spec_helper.rb
 - spec/swagger/swagger_spec.rb
 - spec/util_spec.rb
 - test/agent_setup_init_test.rb
+- test/agent_setup_status_test.rb
 - test/bundle_vendor_test.rb
 - test/cucumber_test.rb
 - test/expectations/openssl_test_key_sign1.json