tcell_agent 0.4.0 → 1.0.0

data/spec/apps/rails-3.2/log/test.log

@@ -0,0 +1,12 @@
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL
+Connecting to database specified by DATABASE_URL

data/spec/lib/tcell_agent/agent/fork_pipe_manager_spec.rb

@@ -1,21 +1,19 @@
 require 'spec_helper'
 
 module TCellAgent
-
   describe Agent::ForkPipeManager do
-    describe ".send_to_metrics_pipe" do
-      context "incrementing a route" do
-        it "should call increment_route on the agent" do
-          expect(TCellAgent).to receive(:increment_route).with("/ma_route", 50)
+    describe '.send_to_metrics_pipe' do
+      context 'incrementing a route' do
+        it 'should call increment_route on the agent' do
+          expect(TCellAgent).to receive(:increment_route).with('/ma_route', 50)
           expect(TCellAgent).to_not receive(:discover_database_fields)
           expect(TCellAgent).to_not receive(:increment_session_info)
 
-
-          child_process = ForkBreak::Process.new do |breakpoints|
+          child_process = ForkBreak::Process.new do |_breakpoints|
             TCellAgent::Agent.send_to_metrics_pipe(
-              "_type"=>"increment_route",
-              "route_id"=>"/ma_route",
-              "response_time"=>50
+              '_type' => 'increment_route',
+              'route_id' => '/ma_route',
+              'response_time' => 50
             )
           end
 
@@ -23,71 +21,75 @@ module TCellAgent
         end
       end
 
-      context "discover database fields" do
-        it "should call discover_database_fields on the agent" do
+      context 'discover database fields' do
+        it 'should call discover_database_fields on the agent' do
           expect(TCellAgent).to_not receive(:increment_route)
           expect(TCellAgent).to receive(:discover_database_fields).with(
-            "route_id", "database", "schema", "table", "fields"
+            'route_id', 'database', 'schema', 'table', 'fields'
           )
           expect(TCellAgent).to_not receive(:increment_session_info)
 
-
-          child_process = ForkBreak::Process.new do |breakpoints|
-            TCellAgent::Agent.send_to_metrics_pipe({
-              "_type"=>"discover_database_fields",
-              "route_id"=>"route_id",
-              "database"=>"database",
-              "schema"=>"schema",
-              "table"=>"table",
-              "fields"=>"fields"
-            })
+          child_process = ForkBreak::Process.new do |_breakpoints|
+            TCellAgent::Agent.send_to_metrics_pipe(
+              {
+                '_type' => 'discover_database_fields',
+                'route_id' => 'route_id',
+                'database' => 'database',
+                'schema' => 'schema',
+                'table' => 'table',
+                'fields' => 'fields'
+              }
+            )
           end
 
           child_process.finish.wait
         end
       end
 
-      context "increment session info" do
-        it "should call increment_session_info on the agent" do
+      context 'increment session info' do
+        it 'should call increment_session_info on the agent' do
           expect(TCellAgent).to_not receive(:increment_route)
           expect(TCellAgent).to_not receive(:discover_database_fields)
           expect(TCellAgent).to receive(:increment_session_info).with(
-            "hmac_session_id", "user_id", "ip_address", "user_agent"
+            'hmac_session_id', 'user_id', 'ip_address', 'user_agent'
           )
 
-          child_process = ForkBreak::Process.new do |breakpoints|
-            TCellAgent::Agent.send_to_metrics_pipe({
-              "_type"=>"increment_session_info",
-              "hmac_session_id" => "hmac_session_id",
-              "user_id" => "user_id",
-              "ip_address" => "ip_address",
-              "user_agent" => "user_agent"
-            })
+          child_process = ForkBreak::Process.new do |_breakpoints|
+            TCellAgent::Agent.send_to_metrics_pipe(
+              {
+                '_type' => 'increment_session_info',
+                'hmac_session_id' => 'hmac_session_id',
+                'user_id' => 'user_id',
+                'ip_address' => 'ip_address',
+                'user_agent' => 'user_agent'
+              }
+            )
           end
 
           child_process.finish.wait
         end
       end
 
-      context "unkown metric" do
-        it "should raise an exception" do
+      context 'unknown metric' do
+        it 'should raise an exception' do
           expect(TCellAgent).to_not receive(:increment_route)
           expect(TCellAgent).to_not receive(:discover_database_fields)
           expect(TCellAgent).to_not receive(:increment_session_info)
 
-          logger = double("logger")
+          logger = double('logger')
           expect(TCellAgent).to receive(:logger).and_return(logger)
           expect(TCellAgent).to receive(:logger).and_return(logger)
+
           expect(logger).to receive(:debug).with(
-            "Exception in safe_block Handling metrics_pipe_block: Exception happened, message" +
-            " is Metrics Pipe Manager received unknown metric: dunno_what_this_is"
+            'Exception in safe_block Handling metrics_pipe_block: StandardError happened, message' \
+            ' is Metrics Pipe Manager received unknown metric: dunno_what_this_is'
           )
-          expect(logger).to receive(:debug).with(anything())
+          expect(logger).to receive(:debug).with(anything)
 
-          child_process = ForkBreak::Process.new do |breakpoints|
-            TCellAgent::Agent.send_to_metrics_pipe({
-              "_type"=>"dunno_what_this_is"
-            })
+          child_process = ForkBreak::Process.new do |_breakpoints|
+            TCellAgent::Agent.send_to_metrics_pipe(
+              { '_type' => 'dunno_what_this_is' }
+            )
           end
 
           child_process.finish.wait
@@ -95,5 +97,4 @@ module TCellAgent
       end
     end
   end
-
 end