Add backwards compatibility with Rails 4

Most of the consuming apps are on Rails 4, so it is important that this
gem continue to work with them.

There are two main changed in this commit.

First to fix the monkey patch to ActionController's
`request_http_token_authentication` because it changes airty between
Rails 4 and Rails 5. It adds a second optional param, but rails won't
let you override a method unless the signature matches -- so this
required two different `def`s to set this based on the Rails version
number. I looked for a way to get the airty of a module method but after
20 min gave up and just checked the rails version number instead.

The second is to use the different syntax with ActionDispatch's
integration tests syntax. The `get` method now expects `params` and
`headers` parameters to be named parameters. Again, I just gave up and
checked the Rails version and made the decision. This was needed because
I was juggling the rails versions in the gemspec to test compatibility.

Author: csexton

lib/kracken/controllers/token_authenticatable.rb

@@ -13,10 +13,16 @@ def self.included(base)
         end
       end
 
-      # NOTE: Monkey-patch until this is merged into the gem
-      def request_http_token_authentication(realm = 'Application', message = nil)
-        headers["WWW-Authenticate"] = %(Token realm="#{realm.gsub(/"/, "")}")
-        raise TokenUnauthorized, "Invalid Credentials"
+      if Rails::VERSION::MAJOR >= 5
+        def request_http_token_authentication(realm = 'Application', message = nil)
+          headers["WWW-Authenticate"] = %(Token realm="#{realm.gsub(/"/, "")}")
+          raise TokenUnauthorized, "Invalid Credentials"
+        end
+      else
+        def request_http_token_authentication(realm = 'Application')
+          headers["WWW-Authenticate"] = %(Token realm="#{realm.gsub(/"/, "")}")
+          raise TokenUnauthorized, "Invalid Credentials"
+        end
       end
 
     private

spec/dummy/config/environments/test.rb

@@ -14,8 +14,15 @@
   config.eager_load = false
 
   # Configure static asset server for tests with Cache-Control for performance.
-  config.public_file_server.enabled = true
-  config.public_file_server.headers = { 'Cache-Control' => 'public, max-age=3600' }
+  if config.respond_to? :public_file_server
+    # Hot new Rails 5 Settings
+    config.public_file_server.enabled = true
+    config.public_file_server.headers = { 'Cache-Control' => 'public, max-age=3600' }
+  else
+    # Deprecated Rails 4 Settings
+    config.serve_static_files = true
+    config.static_cache_control = "public, max-age=3600"
+  end
 
   # Show full error reports and disable caching.
   config.consider_all_requests_local       = true

spec/requests/api_spec.rb

@@ -18,7 +18,12 @@ def headers_with_token(token)
         stub_request(:get, "https://account.radiusnetworks.com/auth/radius/user.json?oauth_token=123")
           .to_return(status: 200, body: json)
 
-        get api_index_path, params: {}, headers: headers_with_token("123")
+        # Temporary work around while we support versions of Rails before 4
+        if Rails::VERSION::MAJOR >= 5
+          get api_index_path, params: {}, headers: headers_with_token("123")
+        else
+          get api_index_path, {}, headers_with_token("123")
+        end
 
         expect(response.status).to be 200
       end