appsignal 3.4.4 → 3.7.0

data/spec/lib/appsignal/hooks/redis_client_spec.rb

@@ -0,0 +1,238 @@
+describe Appsignal::Hooks::RedisClientHook do
+  before do
+    Appsignal.config = project_fixture_config
+  end
+
+  if DependencyHelper.redis_client_present?
+    context "with redis-client" do
+      context "with instrumentation enabled" do
+        describe "#dependencies_present?" do
+          subject { described_class.new.dependencies_present? }
+
+          context "with gem version new than 0.14.0" do
+            before { stub_const("RedisClient::VERSION", "1.2.3") }
+
+            it { is_expected.to be_truthy }
+          end
+
+          context "with gem version 0.14.0" do
+            before { stub_const("RedisClient::VERSION", "0.14.0") }
+
+            it { is_expected.to be_truthy }
+          end
+
+          context "with gem version older than 0.14.0" do
+            before { stub_const("RedisClient::VERSION", "0.13.9") }
+
+            it { is_expected.to be_falsy }
+          end
+        end
+
+        context "with rest-client gem" do
+          describe "integration" do
+            before do
+              Appsignal.config.config_hash[:instrument_redis] = true
+            end
+
+            context "install" do
+              before do
+                Appsignal::Hooks.load_hooks
+              end
+
+              it "includes the integration for the ruby connection" do
+                # Test if the last included module (prepended module) was our
+                # integration. That's not certain with the assertions below
+                # because we have to overwrite the `process` method for the test.
+                expect(RedisClient::RubyConnection.included_modules.first)
+                  .to eql(Appsignal::Integrations::RedisClientIntegration)
+              end
+            end
+
+            context "requirements" do
+              it "driver should have the write method" do
+                # Since we stub the driver class below, to make sure that we don't
+                # create a real connection, the test won't fail if the method definition
+                # is changed.
+                method = RedisClient::RubyConnection.instance_method(:write)
+                expect(method.arity).to eql(1)
+              end
+            end
+
+            context "instrumentation" do
+              before do
+                start_agent
+                # Stub RedisClient::RubyConnection class so that it doesn't perform an actual
+                # Redis query. This class will be included (prepended) with the
+                # AppSignal Redis integration.
+                stub_const("RedisClient::RubyConnection", Class.new do
+                  def initialize(config)
+                    @config = config
+                  end
+
+                  def write(_commands)
+                    "stub_write"
+                  end
+                end)
+                # Load the integration again for the stubbed RedisClient::RubyConnection class.
+                # Call it directly because {Appsignal::Hooks.load_hooks} keeps
+                # track if it was installed already or not.
+                Appsignal::Hooks::RedisClientHook.new.install
+              end
+              let!(:transaction) do
+                Appsignal::Transaction.create("uuid", Appsignal::Transaction::HTTP_REQUEST, "test")
+              end
+              let!(:client_config) { RedisClient::Config.new(:id => "stub_id") }
+              around { |example| keep_transactions { example.run } }
+
+              it "instrument a redis call" do
+                connection = RedisClient::RubyConnection.new client_config
+                expect(connection.write([:get, "key"])).to eql("stub_write")
+
+                transaction_hash = transaction.to_h
+                expect(transaction_hash["events"]).to include(
+                  hash_including(
+                    "name" => "query.redis",
+                    "body" => "get ?",
+                    "title" => "stub_id"
+                  )
+                )
+              end
+
+              it "instrument a redis script call" do
+                connection = ::RedisClient::RubyConnection.new client_config
+                script = "return redis.call('set',KEYS[1],ARGV[1])"
+                keys = ["foo"]
+                argv = ["bar"]
+                expect(connection.write([:eval, script, keys.size, keys,
+                                         argv])).to eql("stub_write")
+
+                transaction_hash = transaction.to_h
+                expect(transaction_hash["events"]).to include(
+                  hash_including(
+                    "name" => "query.redis",
+                    "body" => "#{script} ? ?",
+                    "title" => "stub_id"
+                  )
+                )
+              end
+            end
+          end
+        end
+
+        if DependencyHelper.hiredis_client_present?
+          context "with hiredis driver" do
+            describe "integration" do
+              before do
+                Appsignal.config.config_hash[:instrument_redis] = true
+              end
+
+              context "install" do
+                before do
+                  Appsignal::Hooks.load_hooks
+                end
+
+                it "includes the integration in the HiredisConnection class" do
+                  # Test if the last included module (prepended module) was our
+                  # integration. That's not certain with the assertions below
+                  # because we have to overwrite the `process` method for the test.
+                  expect(RedisClient::HiredisConnection.included_modules.first)
+                    .to eql(Appsignal::Integrations::RedisClientIntegration)
+                end
+              end
+
+              context "requirements" do
+                it "driver should have the write method" do
+                  # Since we stub the driver class below, to make sure that we don't
+                  # create a real connection, the test won't fail if the method definition
+                  # is changed.
+                  method = RedisClient::HiredisConnection.instance_method(:write)
+                  expect(method.arity).to eql(1)
+                end
+              end
+
+              context "instrumentation" do
+                before do
+                  start_agent
+                  # Stub RedisClient::HiredisConnection class so that it doesn't perform an actual
+                  # Redis query. This class will be included (prepended) with the
+                  # AppSignal Redis integration.
+                  stub_const("RedisClient::HiredisConnection", Class.new do
+                    def initialize(config)
+                      @config = config
+                    end
+
+                    def write(_commands)
+                      "stub_write"
+                    end
+                  end)
+                  # Load the integration again for the stubbed RedisClient::HiredisConnection class.
+                  # Call it directly because {Appsignal::Hooks.load_hooks} keeps
+                  # track if it was installed already or not.
+                  Appsignal::Hooks::RedisClientHook.new.install
+                end
+                let!(:transaction) do
+                  Appsignal::Transaction.create("uuid", Appsignal::Transaction::HTTP_REQUEST,
+                    "test")
+                end
+                let!(:client_config) { RedisClient::Config.new(:id => "stub_id") }
+                around { |example| keep_transactions { example.run } }
+
+                it "instrument a redis call" do
+                  connection = RedisClient::HiredisConnection.new client_config
+                  expect(connection.write([:get, "key"])).to eql("stub_write")
+
+                  transaction_hash = transaction.to_h
+                  expect(transaction_hash["events"]).to include(
+                    hash_including(
+                      "name" => "query.redis",
+                      "body" => "get ?",
+                      "title" => "stub_id"
+                    )
+                  )
+                end
+
+                it "instrument a redis script call" do
+                  connection = ::RedisClient::HiredisConnection.new client_config
+                  script = "return redis.call('set',KEYS[1],ARGV[1])"
+                  keys = ["foo"]
+                  argv = ["bar"]
+                  expect(connection.write([:eval, script, keys.size, keys,
+                                           argv])).to eql("stub_write")
+
+                  transaction_hash = transaction.to_h
+                  expect(transaction_hash["events"]).to include(
+                    hash_including(
+                      "name" => "query.redis",
+                      "body" => "#{script} ? ?",
+                      "title" => "stub_id"
+                    )
+                  )
+                end
+              end
+            end
+          end
+        end
+      end
+
+      context "with instrumentation disabled" do
+        before do
+          Appsignal.config.config_hash[:instrument_redis] = false
+        end
+
+        describe "#dependencies_present?" do
+          subject { described_class.new.dependencies_present? }
+
+          it { is_expected.to be_falsy }
+        end
+      end
+    end
+  else
+    context "without redis-client" do
+      describe "#dependencies_present?" do
+        subject { described_class.new.dependencies_present? }
+
+        it { is_expected.to be_falsy }
+      end
+    end
+  end
+end

data/spec/lib/appsignal/hooks/redis_spec.rb

@@ -5,100 +5,122 @@ describe Appsignal::Hooks::RedisHook do
 
   if DependencyHelper.redis_present?
     context "with redis" do
-      context "with instrumentation enabled" do
-        describe "#dependencies_present?" do
-          subject { described_class.new.dependencies_present? }
+      if DependencyHelper.redis_client_present?
+        context "with redis-client" do
+          context "with instrumentation enabled" do
+            describe "#dependencies_present?" do
+              subject { described_class.new.dependencies_present? }
 
-          it { is_expected.to be_truthy }
+              it { is_expected.to be_falsey }
+            end
+          end
         end
+      else
+        context "with instrumentation enabled" do
+          describe "#dependencies_present?" do
+            subject { described_class.new.dependencies_present? }
 
-        describe "integration" do
-          before do
-            Appsignal.config.config_hash[:instrument_redis] = true
+            it { is_expected.to be_truthy }
           end
 
-          context "install" do
+          describe "integration" do
             before do
-              Appsignal::Hooks.load_hooks
+              Appsignal.config.config_hash[:instrument_redis] = true
             end
 
-            it "does something" do
-              # Test if the last included module (prepended module) was our
-              # integration. That's not certain with the assertions below
-              # because we have to overwrite the `process` method for the test.
-              expect(Redis::Client.included_modules.first)
-                .to eql(Appsignal::Integrations::RedisIntegration)
-            end
-          end
+            context "install" do
+              before do
+                Appsignal::Hooks.load_hooks
+              end
 
-          context "instrumentation" do
-            before do
-              start_agent
-              # Stub Redis::Client class so that it doesn't perform an actual
-              # Redis query. This class will be included (prepended) with the
-              # AppSignal Redis integration.
-              stub_const("Redis::Client", Class.new do
-                def id
-                  "stub_id"
-                end
-
-                def write(_commands)
-                  "stub_write"
-                end
-              end)
-              # Load the integration again for the stubbed Redis::Client class.
-              # Call it directly because {Appsignal::Hooks.load_hooks} keeps
-              # track if it was installed already or not.
-              Appsignal::Hooks::RedisHook.new.install
-            end
-            let!(:transaction) do
-              Appsignal::Transaction.create("uuid", Appsignal::Transaction::HTTP_REQUEST, "test")
+              it "prepends instrumentation module" do
+                # Test if the last included module (prepended module) was our
+                # integration. That's not certain with the assertions below
+                # because we have to overwrite the `process` method for the test.
+                expect(Redis::Client.included_modules.first)
+                  .to eql(Appsignal::Integrations::RedisIntegration)
+              end
             end
-            around { |example| keep_transactions { example.run } }
-
-            it "instrument a redis call" do
-              client = Redis::Client.new
-              expect(client.write([:get, "key"])).to eql("stub_write")
-
-              transaction_hash = transaction.to_h
-              expect(transaction_hash["events"]).to include(
-                hash_including(
-                  "name" => "query.redis",
-                  "body" => "get ?",
-                  "title" => "stub_id"
-                )
-              )
+
+            context "requirements" do
+              it "driver should have the write method" do
+                # Since we stub the client class below, to make sure that we don't
+                # create a real connection, the test won't fail if the method definition
+                # is changed.
+                method = Redis::Client.instance_method(:call)
+                expect(method.arity).to eql(1)
+              end
             end
 
-            it "instrument a redis script call" do
-              client = Redis::Client.new
-              script = "return redis.call('set',KEYS[1],ARGV[1])"
-              keys = ["foo"]
-              argv = ["bar"]
-              expect(client.write([:eval, script, keys.size, keys, argv])).to eql("stub_write")
-
-              transaction_hash = transaction.to_h
-              expect(transaction_hash["events"]).to include(
-                hash_including(
-                  "name" => "query.redis",
-                  "body" => "#{script} ? ?",
-                  "title" => "stub_id"
-                )
-              )
+            context "instrumentation" do
+              before do
+                start_agent
+                # Stub Redis::Client class so that it doesn't perform an actual
+                # Redis query. This class will be included (prepended) with the
+                # AppSignal Redis integration.
+                stub_const("Redis::Client", Class.new do
+                  def id
+                    "stub_id"
+                  end
+
+                  def write(_commands)
+                    "stub_write"
+                  end
+                end)
+                # Load the integration again for the stubbed Redis::Client class.
+                # Call it directly because {Appsignal::Hooks.load_hooks} keeps
+                # track if it was installed already or not.
+                Appsignal::Hooks::RedisHook.new.install
+              end
+              let!(:transaction) do
+                Appsignal::Transaction.create("uuid", Appsignal::Transaction::HTTP_REQUEST, "test")
+              end
+              around { |example| keep_transactions { example.run } }
+
+              it "instrument a redis call" do
+                client = Redis::Client.new
+                expect(client.write([:get, "key"])).to eql("stub_write")
+
+                transaction_hash = transaction.to_h
+                expect(transaction_hash["events"]).to include(
+                                                        hash_including(
+                                                          "name" => "query.redis",
+                                                          "body" => "get ?",
+                                                          "title" => "stub_id"
+                                                        )
+                                                      )
+              end
+
+              it "instrument a redis script call" do
+                client = Redis::Client.new
+                script = "return redis.call('set',KEYS[1],ARGV[1])"
+                keys = ["foo"]
+                argv = ["bar"]
+                expect(client.write([:eval, script, keys.size, keys, argv])).to eql("stub_write")
+
+                transaction_hash = transaction.to_h
+                expect(transaction_hash["events"]).to include(
+                                                        hash_including(
+                                                          "name" => "query.redis",
+                                                          "body" => "#{script} ? ?",
+                                                          "title" => "stub_id"
+                                                        )
+                                                      )
+              end
             end
           end
         end
-      end
 
-      context "with instrumentation disabled" do
-        before do
-          Appsignal.config.config_hash[:instrument_redis] = false
-        end
+        context "with instrumentation disabled" do
+          before do
+            Appsignal.config.config_hash[:instrument_redis] = false
+          end
 
-        describe "#dependencies_present?" do
-          subject { described_class.new.dependencies_present? }
+          describe "#dependencies_present?" do
+            subject { described_class.new.dependencies_present? }
 
-          it { is_expected.to be_falsy }
+            it { is_expected.to be_falsy }
+          end
         end
       end
     end

data/spec/lib/appsignal/hooks/resque_spec.rb

@@ -160,7 +160,7 @@ describe Appsignal::Hooks::ResqueHook do
         end
         after { Object.send(:remove_const, :ActiveJobMock) }
 
-        it "does not set arguments but lets the ActiveJob intergration handle it" do
+        it "does not set arguments but lets the ActiveJob integration handle it" do
           perform_job(
             ResqueTestJob,
             "class" => "ActiveJob::QueueAdapters::ResqueAdapter::JobWrapper",

data/spec/lib/appsignal/hooks_spec.rb

@@ -47,7 +47,7 @@ describe Appsignal::Hooks do
     Appsignal::Hooks.hooks.delete(:mock_present_hook)
   end
 
-  it "should not install if depencies are not present" do
+  it "should not install if dependencies are not present" do
     Appsignal::Hooks::Hook.register(:mock_not_present_hook, MockNotPresentHook)
 
     expect(Appsignal::Hooks.hooks[:mock_not_present_hook]).to be_instance_of(MockNotPresentHook)
@@ -67,12 +67,12 @@ describe Appsignal::Hooks do
     expect(Appsignal::Hooks.hooks[:mock_error_hook]).to be_instance_of(MockErrorHook)
     expect(Appsignal::Hooks.hooks[:mock_error_hook].installed?).to be_falsy
 
-    expect(Appsignal.logger).to receive(:error)
+    expect(Appsignal.internal_logger).to receive(:error)
       .with("Error while installing mock_error_hook hook: error").once
-    expect(Appsignal.logger).to receive(:debug).ordered do |message|
+    expect(Appsignal.internal_logger).to receive(:debug).ordered do |message|
       expect(message).to eq("Installing mock_error_hook hook")
     end
-    expect(Appsignal.logger).to receive(:debug).ordered do |message|
+    expect(Appsignal.internal_logger).to receive(:debug).ordered do |message|
       # Start of the error backtrace as debug log
       expect(message).to start_with(File.expand_path("../../..", __dir__))
     end
@@ -89,7 +89,7 @@ describe Appsignal::Hooks do
     let(:log_stream) { std_stream }
     let(:log) { log_contents(log_stream) }
     before do
-      Appsignal.logger = test_logger(log_stream)
+      Appsignal.internal_logger = test_logger(log_stream)
     end
 
     def call_constant(&block)