grape-appsignal 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: eca276a652910634f9bc78d02cea3c704f9322be
-  data.tar.gz: 8516f43b625fcf4bb519d03cb6a8d2a1b6488cd4
+  metadata.gz: 7bacb69629bb0ce1b5d12fe5b7530f6e3b0da479
+  data.tar.gz: 39df3b28bb87097d8a32d2a9d4bdba7f5a48187f
 SHA512:
-  metadata.gz: b4b817a504e242701e9b087158679dd4d41d5c29535c5bf355829ea1a23640a6e1b1870c450a7ee73abb7646f232b50deca4fed7e088336d411c7e5b31e88671
-  data.tar.gz: 5f6be88e8678dfbede37021e9f7d33a5fccaab04819885029c3cce38cfaeb4ee83ca8ada2327920af5932a7a8db1ee1ba43ca7b716be207b48d2d72f9335ea0e
+  metadata.gz: fcfce259d320d53a1743b25eb7c8aa8355171f40e8afbb3b26c8dac30e3dea285d12840f3e1f917002c07a74abdb9f6704f200acc37c725bb563e283bcd5fbd8
+  data.tar.gz: 8cc64653b024fea9229daa715dfbf66483b431f0d262970a6fb4fc48a8580443f7348ff177c486578a3550230eb2cf386e221bf1bedb33007f33eae0e278102b

data/lib/grape-appsignal/middleware.rb

@@ -14,9 +14,9 @@ module Appsignal
 
         api_endpoint = env['api.endpoint']
 
-        puts api_endpoint.method_name
 
         method_name = api_endpoint.method_name.gsub(/ ?[ \/]/, '/') if api_endpoint && api_endpoint.method_name
+        method_name = method_name.gsub(/\/\//, '/') if method_name
 
         request_path = api_endpoint.routes.first.route_path[1..-1].sub(/\(\.:format\)\z/, "")
 

data/lib/grape-appsignal/version.rb

@@ -1,5 +1,5 @@
 module Appsignal
   module Grape
-    VERSION = "0.4.0"
+    VERSION = "0.4.1"
   end
 end

data/spec/grape-appsignal/middleware_spec.rb

@@ -1,5 +1,4 @@
 require 'spec_helper'
-
 describe Appsignal::Grape::Middleware do
 
   class ComplexAPI < Grape::API
@@ -11,6 +10,9 @@ describe Appsignal::Grape::Middleware do
       get 'name/:name' do
         "hello #{params['name']}"
       end
+      get '/goodbye' do
+        "hello #{params['name']}"
+      end
     end
   end
 
@@ -88,5 +90,27 @@ describe Appsignal::Grape::Middleware do
         expect(subject.status).to eq(200)
       end
     end
+
+    context "with a complex API" do
+      let(:events){ [] }
+      let(:app){ ComplexAPI }
+      before do
+        ActiveSupport::Notifications.subscribe(/^[^!]/) do |*args|
+          events << ActiveSupport::Notifications::Event.new(*args)
+        end
+      end
+
+      subject { get "/api/v1/hello/goodbye/"; events.last}
+
+      it "delivers a payload consistent with the API call."do
+        expect(subject.payload ).to eq(
+          { method: "GET" , path: "api/:version/hello/goodbye", action: "Grape(v1)(api)::GET/hello/goodbye", class: "API"}
+        )
+      end
+
+      it "names the payload consistent with the API call."do
+        expect(subject.name ).to eq("process_action.grape.GET.api.version.hello.goodbye")
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grape-appsignal
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Mark Madsen