appsignal 2.4.3 → 2.5.0.alpha.1

data/spec/lib/appsignal/utils/query_params_sanitizer_spec.rb

@@ -43,7 +43,7 @@ describe Appsignal::Utils::QueryParamsSanitizer do
       end
 
       context "when value is an array" do
-        let(:value) { %w(foo bar) }
+        let(:value) { %w[foo bar] }
 
         it "should only return the first level of the object" do
           expect(subject).to eq("?")
@@ -51,7 +51,7 @@ describe Appsignal::Utils::QueryParamsSanitizer do
 
         it "should not modify source value" do
           subject
-          expect(value).to eq(%w(foo bar))
+          expect(value).to eq(%w[foo bar])
         end
       end
 
@@ -115,7 +115,7 @@ describe Appsignal::Utils::QueryParamsSanitizer do
       end
 
       context "when value is an array" do
-        let(:value) { %w(foo bar) }
+        let(:value) { %w[foo bar] }
 
         it "should sanitize all hash values with a single questionmark" do
           expect(subject).to eq(["?"])

data/spec/lib/appsignal_spec.rb

@@ -87,9 +87,11 @@ describe Appsignal do
           Appsignal.start
         end
 
-        it "should install the allocation event hook" do
-          expect(Appsignal::Extension).to receive(:install_allocation_event_hook)
-          Appsignal.start
+        unless Appsignal::System.jruby?
+          it "installs the allocation event hook" do
+            expect(Appsignal::Extension).to receive(:install_allocation_event_hook)
+            Appsignal.start
+          end
         end
 
         it "should add the gc probe to minutely" do

data/spec/spec_helper.rb

@@ -32,15 +32,27 @@ if DependencyHelper.rails_present?
 end
 require "appsignal"
 
+module Appsignal
+  class << self
+    remove_method :testing?
+
+    def testing?
+      true
+    end
+  end
+end
+
 puts "Running specs in #{RUBY_VERSION} on #{RUBY_PLATFORM}\n\n"
 
 # Add way to clear subscribers between specs
-module ActiveSupport
-  module Notifications
-    class Fanout
-      def clear_subscribers
-        @subscribers.clear
-        @listeners_for.clear
+if defined?(ActiveSupport)
+  module ActiveSupport
+    module Notifications
+      class Fanout
+        def clear_subscribers
+          @subscribers.clear
+          @listeners_for.clear
+        end
       end
     end
   end
@@ -75,9 +87,18 @@ RSpec.configure do |config|
     ENV["PADRINO_ENV"] ||= "test"
 
     # Clean environment
-    appsignal_key_prefixes = %w(APPSIGNAL_ _APPSIGNAL_)
+    appsignal_key_prefixes = %w[APPSIGNAL_ _APPSIGNAL_]
     env_keys = ENV.keys.select { |key| key.start_with?(*appsignal_key_prefixes) }
-    env_keys.each { |key| ENV.delete(key) }
+    env_keys.each do |key|
+      # First set the ENV var to an empty string and then delete the key from
+      # the env. We set the env var to an empty string first as jRuby doesn't
+      # sync `delete` calls to extensions, making our extension think the env
+      # var is still set after calling `ENV.delete`. Setting it to an empty
+      # string will sort of unset it, our extension ignores env vars with an
+      # empty string as a value.
+      ENV[key] = ""
+      ENV.delete(key)
+    end
 
     # Stub system_tmp_dir to something in the project dir for specs
     allow(Appsignal::Config).to receive(:system_tmp_dir).and_return(spec_system_tmp_dir)

data/spec/support/helpers/config_helpers.rb

@@ -5,11 +5,12 @@ module ConfigHelpers
     )
   end
 
-  def project_fixture_config(env = "production", initial_config = {})
+  def project_fixture_config(env = "production", initial_config = {}, logger = Appsignal.logger)
     Appsignal::Config.new(
       project_fixture_path,
       env,
-      initial_config
+      initial_config,
+      logger
     )
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 2.4.3
+  version: 2.5.0.alpha.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-28 00:00:00.000000000 Z
+date: 2017-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -101,14 +101,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.46.0
+        version: 0.49.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.46.0
+        version: 0.49.0
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -146,8 +146,10 @@ files:
 - appsignal.gemspec
 - benchmark.rake
 - bin/appsignal
+- ext/Rakefile
 - ext/agent.yml
 - ext/appsignal_extension.c
+- ext/base.rb
 - ext/extconf.rb
 - gemfiles/capistrano2.gemfile
 - gemfiles/capistrano3.gemfile
@@ -186,6 +188,7 @@ files:
 - lib/appsignal/event_formatter/mongo_ruby_driver/query_formatter.rb
 - lib/appsignal/event_formatter/moped/query_formatter.rb
 - lib/appsignal/extension.rb
+- lib/appsignal/extension/jruby.rb
 - lib/appsignal/garbage_collection_profiler.rb
 - lib/appsignal/hooks.rb
 - lib/appsignal/hooks/action_cable.rb
@@ -258,6 +261,7 @@ files:
 - spec/lib/appsignal/event_formatter/mongo_ruby_driver/query_formatter_spec.rb
 - spec/lib/appsignal/event_formatter/moped/query_formatter_spec.rb
 - spec/lib/appsignal/event_formatter_spec.rb
+- spec/lib/appsignal/extension/jruby_spec.rb
 - spec/lib/appsignal/extension_spec.rb
 - spec/lib/appsignal/garbage_collection_profiler_spec.rb
 - spec/lib/appsignal/hooks/action_cable_spec.rb
@@ -337,7 +341,7 @@ files:
 - spec/support/rails/my_app.rb
 - spec/support/shared_examples/instrument.rb
 - spec/support/stubs/delayed_job.rb
-homepage: https://github.com/appsignal/appsignal
+homepage: https://github.com/appsignal/appsignal-ruby
 licenses:
 - MIT
 metadata: {}
@@ -353,9 +357,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '1.9'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.7.2
@@ -383,6 +387,7 @@ test_files:
 - spec/lib/appsignal/event_formatter/mongo_ruby_driver/query_formatter_spec.rb
 - spec/lib/appsignal/event_formatter/moped/query_formatter_spec.rb
 - spec/lib/appsignal/event_formatter_spec.rb
+- spec/lib/appsignal/extension/jruby_spec.rb
 - spec/lib/appsignal/extension_spec.rb
 - spec/lib/appsignal/garbage_collection_profiler_spec.rb
 - spec/lib/appsignal/hooks/action_cable_spec.rb