change helper method name restful -> status_data

Author: kuboon

README.md

@@ -26,13 +26,13 @@ RestfulError::BaseError === ex # => true
 
 RestfulError[404] == RestfulError::NotFound # => true # same class
 
-ex.restful # returns Data about status code
+ex.status_data # returns Data about status code
 # => #<data RestfulError::Status
 #      code=404,
 #      reason_phrase="Not Found",
 #      symbol=:not_found,
 #      const_name="NotFound">
-ex.restful.code # => 404
+ex.status_data.code # => 404
 ```
 
 #### Custom error by subclassing
@@ -49,7 +49,7 @@ class User::PermissionError < StandardError
   include RestfulError::Helper
   def http_status = :unauthorized # or 401
 end
-User::PermissionError.new.restful.reason_phrase # => "Unauthorized"
+User::PermissionError.new.status_data.reason_phrase # => "Unauthorized"
 ```
 
 ### With I18n

lib/restful_error.rb

@@ -11,20 +11,20 @@ module RestfulError
   autoload :ExceptionsController, "restful_error/exceptions_controller"
 
   module Helper
-    def restful
-      @restful ||= begin
-        raise NotImplementedError, "http_status must be implemented by including class" unless respond_to?(:http_status)
-        RestfulError.build_status_from_symbol_or_code(http_status)
-      end
+    def status_data
+      @status_data ||= RestfulError.build_status_from_symbol_or_code(http_status)
     end
     def response_message
       return @response_message unless @response_message.nil?
-      @response_message = RestfulError.localized_phrase(self.class.name, restful)
+      @response_message = RestfulError.localized_phrase(self.class.name, status_data)
     end
   end
 
   class BaseError < StandardError
     include RestfulError::Helper
+    def http_status
+      raise NotImplementedError, "http_status must be implemented"
+    end
   end
 
   @cache = {}
@@ -58,7 +58,7 @@ def localized_phrase(class_name, status)
     def build_error_class_for(status)
       klass = Class.new(BaseError) do
         define_method(:http_status) { status.code }
-        define_method(:restful) { status }
+        define_method(:status_data) { status }
       end
       const_set(status.const_name, klass)
       if defined? ActionDispatch::ExceptionWrapper

spec/restful_error_spec.rb

@@ -8,7 +8,7 @@
 
     it do
       expect(subject.http_status).to eq 404
-      expect(subject.restful.reason_phrase).to eq "Not Found"
+      expect(subject.status_data.reason_phrase).to eq "Not Found"
     end
   end
 
@@ -17,7 +17,7 @@
 
     it do
       expect(subject.http_status).to eq 400
-      expect(subject.restful.reason_phrase).to eq "Bad Request"
+      expect(subject.status_data.reason_phrase).to eq "Bad Request"
     end
   end
 
@@ -26,7 +26,7 @@
 
     it do
       expect(subject.http_status).to eq 403
-      expect(subject.restful.reason_phrase).to eq "Forbidden"
+      expect(subject.status_data.reason_phrase).to eq "Forbidden"
     end
   end
 
@@ -41,7 +41,7 @@ def http_status = 404
     end
 
     it do
-      expect(subject.restful.symbol).to eq :not_found
+      expect(subject.status_data.symbol).to eq :not_found
     end
   end
 end

spec/with_rails/exceptions_app_spec.rb

@@ -43,9 +43,15 @@ def app = RestfulError::ExceptionsApp.new
         end
       end
       context 'custom message' do
-        let(:exception) { described_class.new("custom message") }
+        let(:klass) do
+          Class.new(described_class) do
+            def response_message = "custom message"
+          end
+        end
+        let(:exception) { klass.new }
         it do
-          expect(json).to eq({status_code:, reason_phrase: "Not Found", response_message: "custom message"}.stringify_keys)
+          expect(json).to eq({status_code: 404, reason_phrase: "Not Found", response_message: 'custom message'}.stringify_keys)
+          expect(last_response.status).to eq status_code
         end
       end
     end