appsignal 3.3.5 → 3.3.7

data/spec/lib/appsignal/logger_spec.rb

@@ -1,23 +1,22 @@
 describe Appsignal::Logger do
-  let(:level) { ::Logger::DEBUG }
-  let(:logger) { Appsignal::Logger.new("group", level) }
+  let(:logger) { Appsignal::Logger.new("group", :level => ::Logger::DEBUG) }
 
   it "should not create a logger with a nil group" do
     expect do
-      Appsignal::Logger.new(nil, level)
+      Appsignal::Logger.new(nil)
     end.to raise_error(TypeError)
   end
 
   describe "#add" do
     it "should log with a level and message" do
       expect(Appsignal::Extension).to receive(:log)
-        .with("group", 3, "Log message", instance_of(Appsignal::Extension::Data))
+        .with("group", 3, 0, "Log message", instance_of(Appsignal::Extension::Data))
       logger.add(::Logger::INFO, "Log message")
     end
 
     it "should log with a block" do
       expect(Appsignal::Extension).to receive(:log)
-        .with("group", 3, "Log message", instance_of(Appsignal::Extension::Data))
+        .with("group", 3, 0, "Log message", instance_of(Appsignal::Extension::Data))
       logger.add(::Logger::INFO) do
         "Log message"
       end
@@ -25,7 +24,7 @@ describe Appsignal::Logger do
 
     it "should log with a level, message and group" do
       expect(Appsignal::Extension).to receive(:log)
-        .with("other_group", 3, "Log message", instance_of(Appsignal::Extension::Data))
+        .with("other_group", 3, 0, "Log message", instance_of(Appsignal::Extension::Data))
       logger.add(::Logger::INFO, "Log message", "other_group")
     end
 
@@ -34,8 +33,8 @@ describe Appsignal::Logger do
       logger.add(::Logger::INFO, nil)
     end
 
-    context "with debug log level" do
-      let(:level) { ::Logger::INFO }
+    context "with info log level" do
+      let(:logger) { Appsignal::Logger.new("group", :level => ::Logger::INFO) }
 
       it "should skip logging if the level is too low" do
         expect(Appsignal::Extension).not_to receive(:log)
@@ -43,6 +42,16 @@ describe Appsignal::Logger do
       end
     end
 
+    context "with a format set" do
+      let(:logger) { Appsignal::Logger.new("group", :format => Appsignal::Logger::LOGFMT) }
+
+      it "should log and pass the format flag" do
+        expect(Appsignal::Extension).to receive(:log)
+          .with("group", 3, 1, "Log message", instance_of(Appsignal::Extension::Data))
+        logger.add(::Logger::INFO, "Log message")
+      end
+    end
+
     context "with a formatter set" do
       before do
         logger.formatter = proc do |_level, _timestamp, _appname, message|
@@ -52,7 +61,7 @@ describe Appsignal::Logger do
 
       it "should log with a level, message and group" do
         expect(Appsignal::Extension).to receive(:log)
-          .with("other_group", 3, "formatted: 'Log message'", instance_of(Appsignal::Extension::Data))
+          .with("other_group", 3, 0, "formatted: 'Log message'", instance_of(Appsignal::Extension::Data))
         logger.add(::Logger::INFO, "Log message", "other_group")
       end
     end
@@ -71,7 +80,7 @@ describe Appsignal::Logger do
           .with({})
           .and_call_original
         expect(Appsignal::Extension).to receive(:log)
-          .with("group", method[1], "Log message", instance_of(Appsignal::Extension::Data))
+          .with("group", method[1], 0, "Log message", instance_of(Appsignal::Extension::Data))
 
         logger.send(method[0], "Log message")
       end
@@ -81,7 +90,7 @@ describe Appsignal::Logger do
           .with({})
           .and_call_original
         expect(Appsignal::Extension).to receive(:log)
-          .with("group", method[1], "Log message", instance_of(Appsignal::Extension::Data))
+          .with("group", method[1], 0, "Log message", instance_of(Appsignal::Extension::Data))
 
         logger.send(method[0]) do
           "Log message"
@@ -95,7 +104,7 @@ describe Appsignal::Logger do
 
       if method[2]
         context "with a lower log level" do
-          let(:level) { method[2] }
+          let(:logger) { Appsignal::Logger.new("group", :level => method[2]) }
 
           it "should skip logging if the level is too low" do
             expect(Appsignal::Extension).not_to receive(:log)
@@ -113,7 +122,7 @@ describe Appsignal::Logger do
 
         it "should log with a level, message and group" do
           expect(Appsignal::Extension).to receive(:log)
-            .with("group", method[1], "formatted: 'Log message'", instance_of(Appsignal::Extension::Data))
+            .with("group", method[1], 0, "formatted: 'Log message'", instance_of(Appsignal::Extension::Data))
           logger.send(method[0], "Log message")
         end
       end

data/spec/support/fixtures/projects/valid/config/appsignal.yml

@@ -52,3 +52,9 @@ old_config_mixed_with_new_config:
     "REQUEST_METHOD", "REQUEST_URI", "SERVER_NAME", "SERVER_PORT",
     "SERVER_PROTOCOL", "HTTP_USER_AGENT"
   ]
+
+rack_env:
+  <<: *defaults
+
+rails_env:
+  <<: *defaults

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 3.3.5
+  version: 3.3.7
 platform: ruby
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-13 00:00:00.000000000 Z
+date: 2023-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -438,7 +438,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.2
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com