grape-msgpack 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cd48218f961f5483fe6ee240467e0c94f98c6063
-  data.tar.gz: 29de7ae2759c8f0bfdd97a7c0a23259e6db0c920
+  metadata.gz: 8d57fed528cc20e364e3a0f876d892891c3a4d5d
+  data.tar.gz: cf68377db5ba1fd4d26a131d563e59d3b8a99b31
 SHA512:
-  metadata.gz: a25e281c2bc76cfb875609ddc37fdb6989cf371e9eb109b94bad133eb3ed3b3b52c560e01df2cf48d393d710c8a984fe44df6760838d3dd64f2f285d458940a2
-  data.tar.gz: bb42efd5b3ced23b92e2690afd34177c3c042515b21e482d2a9e7b947e8f56e374b27443be5bb1b46bd592fc9bb826aba3ad6064bdf7b675d75a5fdc10713aa7
+  metadata.gz: c983d61542f1adbc1fe08e158bb999757c773db64a6cde0f4c96939c71414ca9a0a3dc0cfda1343821dda47de962aa3208b633a6345ccb709641f7be56936f9c
+  data.tar.gz: 5f0c6637a5528e5723877a05056bbeb22147e22f838af2dcabbc3981cea52389f8967dd6f0703eb05c7decfb180cd162cd8e32da068ab755c30cb8cc5461ac67

data/.coveralls.yml

@@ -0,0 +1 @@
+service_name: travis-ci

data/Gemfile

@@ -2,3 +2,5 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in grape-msgpack.gemspec
 gemspec
+
+gem 'coveralls', require: false

data/README.md

@@ -1,6 +1,6 @@
 # Grape::Msgpack
 
-[![Build Status](https://travis-ci.org/rosylilly/grape-msgpack.png?branch=master)](https://travis-ci.org/rosylilly/grape-msgpack)
+[![Gem Version](https://badge.fury.io/rb/grape-msgpack.png)](http://badge.fury.io/rb/grape-msgpack) [![Build Status](https://travis-ci.org/rosylilly/grape-msgpack.png?branch=master)](https://travis-ci.org/rosylilly/grape-msgpack) [![Coverage Status](https://coveralls.io/repos/rosylilly/grape-msgpack/badge.png?branch=master)](https://coveralls.io/r/rosylilly/grape-msgpack?branch=master)
 
 Message pack formatter for grape.
 

data/lib/grape/msgpack.rb

@@ -5,21 +5,35 @@ require 'grape/msgpack/version'
 module Grape
   # Message pack formatter for Grape
   module Msgpack
-    class << self
-      def call(obj, env)
-        return obj.to_msgpack if obj.respond_to?(:to_msgpack)
-        MessagePack.pack(obj)
+    module Formatter
+      class << self
+        def call(obj, env)
+          return obj.to_msgpack if obj.respond_to?(:to_msgpack)
+          MessagePack.pack(obj)
+        end
+      end
+    end
+
+    module ErrorFormatter
+      class << self
+        def call(message, backtrace, options = {}, env = nil)
+          result = message.is_a?(Hash) ? message : { error: message }
+          if (options[:rescue_options] || {})[:backtrace] && backtrace && !backtrace.empty?
+            result = result.merge(backtrace: backtrace)
+          end
+          MessagePack.pack(result)
+        end
       end
     end
   end
 end
 
 class << Grape::Formatter::Base
-  FORMATTERS[:msgpack] = Grape::Msgpack
+  FORMATTERS[:msgpack] = Grape::Msgpack::Formatter
 end
 
 class << Grape::ErrorFormatter::Base
-  FORMATTERS[:msgpack] = Grape::Msgpack
+  FORMATTERS[:msgpack] = Grape::Msgpack::ErrorFormatter
 end
 
 Grape::ContentTypes::CONTENT_TYPES[:msgpack] = 'application/x-msgpack'

data/lib/grape/msgpack/version.rb

@@ -1,5 +1,5 @@
 module Grape
   module Msgpack
-    VERSION = '0.0.1'
+    VERSION = '0.1.0'
   end
 end

data/spec/grape/msgpack_spec.rb

@@ -16,6 +16,8 @@ class MockEntity < Grape::Entity
 end
 
 class MockAPI < Grape::API
+  rescue_from :all
+
   default_format :msgpack
 
   get :test do
@@ -25,6 +27,10 @@ class MockAPI < Grape::API
   get :model do
     present MockModel.new('test_user', 21), with: MockEntity
   end
+
+  get :exception do
+    raise StandardError, 'an error occurred'
+  end
 end
 
 describe MockAPI do
@@ -41,7 +47,7 @@ describe MockAPI do
     end
 
     it { expect(response.status).to eq(200) }
-    it { expect(response.headers["Content-Type"]).to eq('application/x-msgpack') }
+    it { expect(response.headers['Content-Type']).to eq('application/x-msgpack') }
     it { expect(MessagePack.unpack(response.body)).to eq([1, 2, 3]) }
   end
 
@@ -52,7 +58,18 @@ describe MockAPI do
     end
 
     it { expect(response.status).to eq(200) }
-    it { expect(response.headers["Content-Type"]).to eq('application/x-msgpack') }
-    it { expect(MessagePack.unpack(response.body)).to eq("name" => "test_user") }
+    it { expect(response.headers['Content-Type']).to eq('application/x-msgpack') }
+    it { expect(MessagePack.unpack(response.body)).to eq('name' => 'test_user') }
+  end
+
+  describe 'GET /exception' do
+    subject(:response) do
+      get '/exception'
+      last_response
+    end
+
+    it { expect(response.status).to eq(403) }
+    it { expect(response.headers['Content-Type']).to eq('application/x-msgpack') }
+    it { expect(MessagePack.unpack(response.body)).to eq('error' => 'an error occurred') }
   end
 end

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 RSpec.configure do |config|
   config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grape-msgpack
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - Sho Kusano
@@ -115,6 +115,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .coveralls.yml
 - .gitignore
 - .rspec
 - .travis.yml