Added content_type :binary, "application/octet-stream"

Updated request_response_spec test for new content_type

Updated CHANGELOG

Added tests for application/octet-stream and other untested content-types

Updated README for undocumented content-types. Flipped :binary and :txt order in content_types.rb

Author: akabraham

CHANGELOG.md

@@ -2,6 +2,7 @@
 ============
 
 * Your contribution here.
+* [#745](https://github.com/intridea/grape/pull/745): Added `:binary, application/octet-stream` content-type - [@contributor](https://github.com/akabraham).
 
 0.9.0 (8/27/2014)
 =================

README.md

@@ -1178,7 +1178,7 @@ end
 
 ## API Formats
 
-By default, Grape supports _XML_, _JSON_, and _TXT_ content-types. The default format is `:txt`.
+By default, Grape supports _XML_, _JSON_, _ATOM_, _RSS_, _BINARY_, and _TXT_ content-types. The default format is `:txt`.
 
 Serialization takes place automatically. For example, you do not have to call `to_json` in each JSON API implementation.
 
@@ -1246,6 +1246,9 @@ Built-in formats are the following.
 * `:xml`: use object's `to_xml` when available, usually via `MultiXml`, otherwise call `to_s`
 * `:txt`: use object's `to_txt` when available, otherwise `to_s`
 * `:serializable_hash`: use object's `serializable_hash` when available, otherwise fallback to `:json`
+* `:atom`
+* `:rss`
+* `:binary`
 
 Use `default_format` to set the fallback format when the format could not be determined from the `Accept` header.
 See below for the order for choosing the API format.

lib/grape/util/content_types.rb

@@ -8,7 +8,8 @@ module ContentTypes
       :jsonapi, 'application/vnd.api+json',
       :atom, 'application/atom+xml',
       :rss,  'application/rss+xml',
-      :txt,  'text/plain',
+      :binary, 'application/octet-stream',
+      :txt,  'text/plain'
    ]
 
     def self.content_types_for(from_settings)

spec/grape/api_spec.rb

@@ -745,25 +745,55 @@ def subject.enable_root_route!
       expect(last_response.headers['Content-Type']).to eql 'text/plain'
     end
 
+    it 'sets content type for xml' do
+      get '/foo.xml'
+      expect(last_response.headers['Content-Type']).to eql 'application/xml'
+    end
+
     it 'sets content type for json' do
       get '/foo.json'
       expect(last_response.headers['Content-Type']).to eql 'application/json'
     end
 
+    it 'sets content type for jsonapi format' do
+      get '/foo.jsonapi'
+      expect(last_response.headers['Content-Type']).to eql 'application/vnd.api+json'
+    end
+
+    it 'sets content type for serializable hash format' do
+      get '/foo.serializable_hash'
+      expect(last_response.headers['Content-Type']).to eql 'application/json'
+    end
+
+    it 'sets content type for atom format' do
+      get '/foo.atom'
+      expect(last_response.headers['Content-Type']).to eql 'application/atom+xml'
+    end
+
+    it 'sets content type for rss format' do
+      get '/foo.rss'
+      expect(last_response.headers['Content-Type']).to eql 'application/rss+xml'
+    end
+
+    it 'sets content type for binary format' do
+      get '/foo.binary'
+      expect(last_response.headers['Content-Type']).to eql 'application/octet-stream'
+    end
+
     it 'sets content type for error' do
       subject.get('/error') { error!('error in plain text', 500) }
       get '/error'
       expect(last_response.headers['Content-Type']).to eql 'text/plain'
     end
 
-    it 'sets content type for error' do
+    it 'sets content type for json error' do
       subject.format :json
       subject.get('/error') { error!('error in json', 500) }
       get '/error.json'
       expect(last_response.headers['Content-Type']).to eql 'application/json'
     end
 
-    it 'sets content type for xml' do
+    it 'sets content type for xml error' do
       subject.format :xml
       subject.get('/error') { error!('error in xml', 500) }
       get '/error.xml'

spec/grape/dsl/request_response_spec.rb

@@ -96,7 +96,8 @@ def self.imbue(key, value)
                                               jsonapi: "application/vnd.api+json",
                                               atom: "application/atom+xml",
                                               rss: "application/rss+xml",
-                                              txt: "text/plain")
+                                              txt: "text/plain",
+                                              binary: "application/octet-stream")
         end
       end