appsignal 0.11.13.beta.0 → 0.11.13.beta.1

data/CHANGELOG.md

@@ -1,6 +1,7 @@
 # 0.11.13
 * Configure Net::HTTP to only use TLS
 * Add http proxy support
+* Don't send queue if there is no content
 
 # 0.11.12
 * Sanitizer will no longer inspect unknown objects, since implementations of inspect sometimes trigger unexpected behavior.

data/lib/appsignal/agent.rb

@@ -103,6 +103,9 @@ module Appsignal
 
     def send_queue
       Appsignal.logger.debug('Sending queue')
+      unless aggregator.has_transactions? || aggregator_queue.any?
+        return
+      end
       # Replace aggregator while making sure no thread
       # is adding to it's queue
       aggregator_to_be_sent = nil

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.11.13.beta.0'
+  VERSION = '0.11.13.beta.1'
 end

data/spec/lib/appsignal/agent_spec.rb

@@ -271,39 +271,61 @@ describe Appsignal::Agent do
   describe "#send_queue" do
     let(:zipped_payload) { double(:body => :zip) }
 
-    it "adds Zipped Payload to queue" do
-      subject.aggregator.stub(:post_processed_queue! => :foo)
+    context "with something in the aggregator" do
+      before do
+        subject.aggregator.stub(:has_transactions? => true)
+      end
 
-      Appsignal::ZippedPayload
-        .should_receive(:new)
-        .with(:foo)
-        .and_return(zipped_payload)
+      it "adds Zipped Payload to queue" do
+        subject.aggregator.stub(:post_processed_queue! => :foo)
 
-      subject.should_receive(:add_to_aggregator_queue).with(zipped_payload)
-    end
+        Appsignal::ZippedPayload
+          .should_receive(:new)
+          .with(:foo)
+          .and_return(zipped_payload)
 
-    it "sends aggregators" do
-      subject.should_receive(:send_aggregators)
-    end
+        subject.should_receive(:add_to_aggregator_queue).with(zipped_payload)
+      end
 
-    it "handle exceptions in post processing" do
-      subject.aggregator.stub(:post_processed_queue!).and_raise(
-        PostProcessingException.new('Message')
-      )
+      it "sends aggregators" do
+        subject.should_receive(:send_aggregators)
+      end
 
-      Appsignal.logger.should_receive(:error).
-        with(kind_of(String)).
-        once
+      it "handle exceptions in post processing" do
+        subject.aggregator.stub(:post_processed_queue!).and_raise(
+          PostProcessingException.new('Message')
+        )
+
+        Appsignal.logger.should_receive(:error).
+          with(kind_of(String)).
+          once
+      end
+
+      it "handles exceptions in transmit" do
+        subject.stub(:send_aggregators).and_raise(
+          Exception.new('Message')
+        )
+
+        Appsignal.logger.should_receive(:error).
+          with(kind_of(String)).
+          once
+      end
     end
 
-    it "handles exceptions in transmit" do
-      subject.stub(:send_aggregators).and_raise(
-        Exception.new('Message')
-      )
+    context "with something in the aggregator queue" do
+      before do
+        subject.aggregator_queue.stub(:any? => true)
+      end
 
-      Appsignal.logger.should_receive(:error).
-        with(kind_of(String)).
-        once
+      it "sends aggregators" do
+        subject.should_receive(:send_aggregators)
+      end
+    end
+
+    context "with empty aggregator and aggregator queue" do
+      it "does not send aggregators" do
+        subject.should_not_receive(:send_aggregators)
+      end
     end
 
     after { subject.send_queue }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.11.13.beta.0
+  version: 0.11.13.beta.1
   prerelease: 8
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-08 00:00:00.000000000 Z
+date: 2015-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack