appsignal 2.11.0.alpha.2-java → 2.11.0.beta.5-java

data/spec/lib/appsignal/hooks_spec.rb

@@ -78,6 +78,63 @@ describe Appsignal::Hooks do
     expect(Appsignal::Hooks.hooks[:mock_error_hook].installed?).to be_falsy
     Appsignal::Hooks.hooks.delete(:mock_error_hook)
   end
+
+  describe "missing constants" do
+    let(:err_stream) { std_stream }
+    let(:stderr) { err_stream.read }
+    let(:log_stream) { std_stream }
+    let(:log) { log_contents(log_stream) }
+    before do
+      Appsignal.logger = test_logger(log_stream)
+    end
+
+    def call_constant(&block)
+      capture_std_streams(std_stream, err_stream, &block)
+    end
+
+    describe "SidekiqProbe" do
+      it "logs a deprecation message and returns the new constant" do
+        constant = call_constant { Appsignal::Hooks::SidekiqProbe }
+
+        expect(constant).to eql(Appsignal::Probes::SidekiqProbe)
+        expect(constant.name).to eql("Appsignal::Probes::SidekiqProbe")
+
+        deprecation_message =
+          "The constant Appsignal::Hooks::SidekiqProbe has been deprecated. " \
+          "Please update the constant name to Appsignal::Probes::SidekiqProbe " \
+          "in the following file to remove this message.\n#{__FILE__}:"
+        expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
+        expect(log).to contains_log :warn, deprecation_message
+      end
+    end
+
+    describe "PumaProbe" do
+      it "logs a deprecation message and returns the new constant" do
+        constant = call_constant { Appsignal::Hooks::PumaProbe }
+
+        expect(constant).to eql(Appsignal::Probes::PumaProbe)
+        expect(constant.name).to eql("Appsignal::Probes::PumaProbe")
+
+        deprecation_message =
+          "The constant Appsignal::Hooks::PumaProbe has been deprecated. " \
+          "Please update the constant name to Appsignal::Probes::PumaProbe " \
+          "in the following file to remove this message.\n#{__FILE__}:"
+        expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
+        expect(log).to contains_log :warn, deprecation_message
+      end
+    end
+
+    describe "other constant" do
+      it "raises a NameError like Ruby normally does" do
+        expect do
+          call_constant { Appsignal::Hooks::Unknown }
+        end.to raise_error(NameError)
+
+        expect(stderr).to be_empty
+        expect(log).to be_empty
+      end
+    end
+  end
 end
 
 describe Appsignal::Hooks::Helpers do

data/spec/lib/appsignal/integrations/que_spec.rb

@@ -14,7 +14,6 @@ if DependencyHelper.que_present?
           :error_count => 0
         }
       end
-
       let(:env) do
         {
           :class => "MyQueJob",
@@ -29,7 +28,6 @@ if DependencyHelper.que_present?
           :params => %w[1 birds]
         }
       end
-
       let(:job) do
         Class.new(::Que::Job) do
           def run(*args)
@@ -37,7 +35,6 @@ if DependencyHelper.que_present?
         end
       end
       let(:instance) { job.new(job_attrs) }
-
       before do
         allow(Que).to receive(:execute)
 
@@ -46,10 +43,14 @@ if DependencyHelper.que_present?
       end
       around { |example| keep_transactions { example.run } }
 
+      def perform_job(job)
+        job._run
+      end
+
       context "success" do
         it "creates a transaction for a job" do
           expect do
-            instance._run
+            perform_job(instance)
           end.to change { created_transactions.length }.by(1)
 
           expect(last_transaction).to be_completed
@@ -95,7 +96,7 @@ if DependencyHelper.que_present?
 
           expect do
             expect do
-              instance._run
+              perform_job(instance)
             end.to raise_error(ExampleException)
           end.to change { created_transactions.length }.by(1)
 
@@ -130,7 +131,7 @@ if DependencyHelper.que_present?
         it "reports errors and not re-raise them" do
           allow(instance).to receive(:run).and_raise(error)
 
-          expect { instance._run }.to change { created_transactions.length }.by(1)
+          expect { perform_job(instance) }.to change { created_transactions.length }.by(1)
 
           expect(last_transaction).to be_completed
           transaction_hash = last_transaction.to_h
@@ -156,6 +157,24 @@ if DependencyHelper.que_present?
           )
         end
       end
+
+      context "when action set in job" do
+        let(:job) do
+          Class.new(::Que::Job) do
+            def run(*_args)
+              Appsignal.set_action("MyCustomJob#perform")
+            end
+          end
+        end
+
+        it "uses the custom action" do
+          perform_job(instance)
+
+          expect(last_transaction).to be_completed
+          transaction_hash = last_transaction.to_h
+          expect(transaction_hash).to include("action" => "MyCustomJob#perform")
+        end
+      end
     end
   end
 end

data/spec/lib/appsignal/integrations/resque_active_job_spec.rb

@@ -1,187 +1,28 @@
-if DependencyHelper.active_job_present?
-  require "active_job"
-  require File.expand_path("lib/appsignal/integrations/resque_active_job.rb")
+require "appsignal/integrations/resque_active_job"
 
-  class TestActiveJob < ActiveJob::Base
-    include Appsignal::Integrations::ResqueActiveJobPlugin
+describe "Legacy Resque ActiveJob integration" do
+  let(:err_stream) { std_stream }
+  let(:stderr) { err_stream.read }
+  let(:log_stream) { std_stream }
+  let(:log) { log_contents(log_stream) }
 
-    def perform(_)
-    end
-  end
-
-  describe Appsignal::Integrations::ResqueActiveJobPlugin do
-    let(:args) { "argument" }
-    let(:job) { TestActiveJob.new(args) }
-    before { start_agent }
-
-    def perform
-      keep_transactions do
-        job.perform_now
-      end
-    end
-
-    context "without error" do
-      it "creates a new transaction" do
-        expect { perform }.to change { created_transactions.length }.by(1)
-
-        expect(last_transaction.to_h).to include(
-          "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-          "action" => "TestActiveJob#perform",
-          "error" => nil,
-          "events" => [
-            hash_including(
-              "name" => "perform_job.resque",
-              "title" => "",
-              "body" => "",
-              "body_format" => Appsignal::EventFormatter::DEFAULT,
-              "count" => 1,
-              "duration" => kind_of(Float)
-            )
-          ],
-          "sample_data" => hash_including(
-            "params" => ["argument"],
-            "metadata" => {
-              "id" => kind_of(String),
-              "queue" => "default"
-            }
-          )
-        )
-      end
-    end
-
-    context "with error" do
-      let(:job) do
-        class BrokenTestActiveJob < ActiveJob::Base
-          include Appsignal::Integrations::ResqueActiveJobPlugin
-
-          def perform(_)
-            raise ExampleException, "my error message"
-          end
-        end
-
-        BrokenTestActiveJob.new(args)
-      end
-
-      it "creates a new transaction with an error" do
-        expect do
-          expect { perform }.to raise_error(ExampleException, "my error message")
-        end.to change { created_transactions.length }.by(1)
+  it "logs and prints a deprecation message on extend" do
+    Appsignal.logger = test_logger(log_stream)
 
-        expect(last_transaction.to_h).to include(
-          "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-          "action" => "BrokenTestActiveJob#perform",
-          "error" => {
-            "name" => "ExampleException",
-            "message" => "my error message",
-            "backtrace" => kind_of(String)
-          },
-          "sample_data" => hash_including(
-            "params" => ["argument"],
-            "metadata" => {
-              "id" => kind_of(String),
-              "queue" => "default"
-            }
-          )
-        )
+    capture_std_streams(std_stream, err_stream) do
+      Class.new do
+        include Appsignal::Integrations::ResqueActiveJobPlugin
       end
     end
 
-    context "with complex arguments" do
-      context "with too long values" do
-        let(:args) do
-          {
-            :foo => "Foo",
-            :bar => "a" * 2001
-          }
-        end
-
-        it "truncates large argument values" do
-          perform
-          expect(last_transaction.to_h).to include(
-            "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-            "action" => "TestActiveJob#perform",
-            "error" => nil,
-            "sample_data" => hash_including(
-              "params" => ["foo" => "Foo", "bar" => "#{"a" * 2000}..."],
-              "metadata" => {
-                "id" => kind_of(String),
-                "queue" => "default"
-              }
-            )
-          )
-        end
-      end
-
-      context "with parameter filtering" do
-        let(:args) do
-          {
-            :foo => "Foo",
-            :bar => "Bar"
-          }
-        end
-        before { Appsignal.config[:filter_parameters] = ["foo"] }
-
-        it "filters selected arguments" do
-          perform
-          expect(last_transaction.to_h).to include(
-            "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-            "action" => "TestActiveJob#perform",
-            "error" => nil,
-            "sample_data" => hash_including(
-              "params" => ["foo" => "[FILTERED]", "bar" => "Bar"],
-              "metadata" => {
-                "id" => kind_of(String),
-                "queue" => "default"
-              }
-            )
-          )
-        end
-      end
-    end
-
-    context "without queue time" do
-      it "does not add queue time to transaction" do
-        # TODO: Not available in transaction.to_h yet.
-        # https://github.com/appsignal/appsignal-agent/issues/293
-        expect(Appsignal).to receive(:monitor_single_transaction).with(
-          "perform_job.resque",
-          a_hash_including(:queue_start => nil)
-        ).and_call_original
-
-        perform
-        expect(last_transaction.to_h).to include(
-          "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-          "action" => "TestActiveJob#perform",
-          "events" => [
-            hash_including("name" => "perform_job.resque")
-          ]
-        )
-      end
-    end
-
-    if DependencyHelper.rails6_present?
-      context "with queue time" do
-        it "adds queue time to transction" do
-          queue_start = "2017-01-01 10:01:00UTC"
-          queue_start_time = Time.parse(queue_start)
-          # TODO: Not available in transaction.to_h yet.
-          # https://github.com/appsignal/appsignal-agent/issues/293
-          expect(Appsignal).to receive(:monitor_single_transaction).with(
-            "perform_job.resque",
-            a_hash_including(:queue_start => queue_start_time)
-          ).and_call_original
-          job.enqueued_at = queue_start
-
-          perform
-          expect(last_transaction.to_h).to include(
-            "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-            "action" => "TestActiveJob#perform",
-            "events" => [
-              hash_including("name" => "perform_job.resque")
-            ]
-          )
-        end
-      end
-    end
+    deprecation_message =
+      "The AppSignal ResqueActiveJobPlugin is deprecated and does " \
+      "nothing on extend. In this version of the AppSignal Ruby gem " \
+      "the integration with Resque is automatic on all Resque workers. " \
+      "Please remove the following line from this file to remove this " \
+      "message: include Appsignal::Integrations::ResqueActiveJobPlugin\n" \
+      "#{__FILE__}:"
+    expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
+    expect(log).to contains_log :warn, deprecation_message
   end
 end

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

@@ -1,93 +1,28 @@
-if DependencyHelper.resque_present?
-  describe "Resque integration" do
-    let(:file) { File.expand_path("lib/appsignal/integrations/resque.rb") }
+require "appsignal/integrations/resque"
 
-    context "with resque" do
-      before do
-        load file
-        start_agent
+describe "Legacy Resque integration" do
+  let(:err_stream) { std_stream }
+  let(:stderr) { err_stream.read }
+  let(:log_stream) { std_stream }
+  let(:log) { log_contents(log_stream) }
 
-        class TestJob
-          extend Appsignal::Integrations::ResquePlugin
+  it "logs and prints a deprecation message on extend" do
+    Appsignal.logger = test_logger(log_stream)
 
-          def self.perform
-          end
-        end
-
-        class BrokenTestJob
-          extend Appsignal::Integrations::ResquePlugin
-
-          def self.perform
-            raise ExampleException, "my error message"
-          end
-        end
-      end
-
-      describe :around_perform_resque_plugin do
-        let(:job) { ::Resque::Job.new("default", "class" => "TestJob") }
-        before { expect(Appsignal).to receive(:stop) }
-
-        context "without exception" do
-          it "creates a new transaction" do
-            expect do
-              keep_transactions { job.perform }
-            end.to change { created_transactions.length }.by(1)
-
-            expect(last_transaction).to be_completed
-            expect(last_transaction.to_h).to include(
-              "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-              "action" => "TestJob#perform",
-              "error" => nil,
-              "events" => [
-                hash_including(
-                  "name" => "perform_job.resque",
-                  "title" => "",
-                  "body" => "",
-                  "body_format" => Appsignal::EventFormatter::DEFAULT,
-                  "count" => 1,
-                  "duration" => kind_of(Float)
-                )
-              ]
-            )
-          end
-        end
-
-        context "with exception" do
-          let(:job) { ::Resque::Job.new("default", "class" => "BrokenTestJob") }
-
-          def perform
-            keep_transactions do
-              expect do
-                job.perform
-              end.to raise_error(ExampleException, "my error message")
-            end
-          end
-
-          it "sets the exception on the transaction" do
-            expect do
-              perform
-            end.to change { created_transactions.length }.by(1)
-
-            expect(last_transaction).to be_completed
-            expect(last_transaction.to_h).to include(
-              "namespace" => Appsignal::Transaction::BACKGROUND_JOB,
-              "action" => "BrokenTestJob#perform",
-              "error" => {
-                "name" => "ExampleException",
-                "message" => "my error message",
-                "backtrace" => kind_of(String)
-              }
-            )
-          end
-        end
+    capture_std_streams(std_stream, err_stream) do
+      Class.new do
+        extend Appsignal::Integrations::ResquePlugin
       end
     end
 
-    context "without resque" do
-      before(:context) { Object.send(:remove_const, :Resque) }
-
-      it { expect { ::Resque }.to raise_error(NameError) }
-      it { expect { load file }.to_not raise_error }
-    end
+    deprecation_message =
+      "The AppSignal ResquePlugin is deprecated and does " \
+      "nothing on extend. In this version of the AppSignal Ruby gem " \
+      "the integration with Resque is automatic on all Resque workers. " \
+      "Please remove the following line from this file to remove this " \
+      "message: extend Appsignal::Integrations::ResquePlugin\n" \
+      "#{__FILE__}:"
+    expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
+    expect(log).to contains_log :warn, deprecation_message
   end
 end