azeroth 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '09152b8053d40bd19405c5b7165458483d4304310fa5ec64c0d2b5723dda0067'
-  data.tar.gz: 1a1039542352929720e5cc7ae1242932b7d2a65856e2156b14a36d1a8bb1722b
+  metadata.gz: fef3f6d11872a3a25c77900028239d743c809aa1aae8d59526df351a1c47e97c
+  data.tar.gz: ae58f2737c72e939067a86061dbc3ab20f1da0ef0d27615c753aacbebb0591d1
 SHA512:
-  metadata.gz: 2ae5257e2851ccbbadbd79d9a0c0a13be63e392b2af16c6e1f66be399ee00bc51b06f9682f25d88648f3a32d2c07451d456d80035f04b48a4cc471d27a572e59
-  data.tar.gz: d7c01b81ce09615d65cc451d2a1669ac9d843d18d75fbb0c12c2e4356507fce62288145bd30295674619a0649327b1b69568fede8adb452e602073f67659235e
+  metadata.gz: ed3b96c2b672b737239713899bfb8ef5546dd1e66d28327fb3a7a66540eede50e93f5e73a3890dcea0474eda4f4bceaaa6804da47a1ea174606430eac14afe93
+  data.tar.gz: ce7d30afc6594731f31aa0591c6747dd110db2f485557d51fffd14cb4f7199e1ab4ebbfb748109dc42da72bda1764de48bd25821461f1ddf15fc49c607a971be

data/.circleci/config.yml

@@ -7,8 +7,12 @@ workflows:
           filters:
             tags:
               only: /.*/
+      - checks:
+          filters:
+            tags:
+              only: /.*/
       - build-and-release:
-          requires: [test]
+          requires: [test, checks]
           filters:
             tags:
               only: /\d+\.\d+\.\d+/
@@ -18,7 +22,7 @@ workflows:
 jobs:
   test:
     docker:
-      - image: darthjee/circleci_rails_gems:0.6.3
+      - image: darthjee/circleci_rails_gems:0.8.0
         environment:
           PROJECT: azeroth
     steps:
@@ -32,12 +36,22 @@ jobs:
       - run:
           name: RSpec
           command: bundle exec rspec
-      - run:
-          name: Rubocop
-          command: rubocop
       - run:
           name: Coverage Test Report
           command: cc-test-reporter after-build --exit-code $?
+  checks:
+    docker:
+      - image: darthjee/circleci_rails_gems:0.8.0
+        environment:
+          PROJECT: azeroth
+    steps:
+      - checkout
+      - run:
+          name: Bundle Install
+          command: bundle install
+      - run:
+          name: Rubocop
+          command: rubocop
       - run:
           name: Yardstick coverage check
           command: bundle exec rake verify_measurements
@@ -52,7 +66,7 @@ jobs:
           command: check_specs
   build-and-release:
     docker:
-      - image: darthjee/circleci_rails_gems:0.6.3
+      - image: darthjee/circleci_rails_gems:0.8.0
         environment:
           PROJECT: azeroth
     steps:

data/Dockerfile

@@ -1,4 +1,4 @@
-FROM darthjee/rails_gems:0.7.0 as base
+FROM darthjee/rails_gems:0.8.0 as base
 FROM darthjee/scripts:0.1.8 as scripts
 
 ######################################

data/README.md

@@ -11,7 +11,7 @@ Azeroth
 
 Yard Documentation
 -------------------
-[https://www.rubydoc.info/gems/azeroth/0.8.0](https://www.rubydoc.info/gems/azeroth/0.8.0)
+[https://www.rubydoc.info/gems/azeroth/0.9.0](https://www.rubydoc.info/gems/azeroth/0.9.0)
 
 Azeroth has been designed making the coding of controllers easier
 as routes in controllers are usually copy, paste and replace of same
@@ -61,6 +61,7 @@ It accepts options
 - before_save: Method/Proc to be ran before saving the resource on create or update
 - after_save: Method/Proc to be ran after saving the resource on create or update
 - build_with: Method/Block to be ran when building the reource on create
+- update_with: Method/Block to be ran when updating the reource on update
 - paginated: Flag when pagination should be applied
 - per_page: Number of items returned when pagination is active
 

data/azeroth.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'activerecord',              '~> 5.2.x'
   gem.add_development_dependency 'bundler',                   '1.16.1'
   gem.add_development_dependency 'factory_bot',               '5.2.0'
-  gem.add_development_dependency 'nokogiri',                  '1.11.4'
+  gem.add_development_dependency 'nokogiri',                  '1.12.5'
   gem.add_development_dependency 'pry',                       '0.12.2'
   gem.add_development_dependency 'pry-nav',                   '0.3.0'
   gem.add_development_dependency 'rails',                     '~> 5.2.x'

data/config/yardstick.yml

@@ -23,6 +23,7 @@ rules:
   Summary::Presence:
     enabled: true
     exclude:
+      - Azeroth::ControllerInterface#initialize
       - Azeroth::Decorator#initialize
       - Azeroth::Decorator::HashBuilder#initialize
       - Azeroth::Decorator::KeyValueExtractor#initialize

data/lib/azeroth/controller_interface.rb

@@ -0,0 +1,102 @@
+# frozen_string_literal: true
+
+module Azeroth
+  # @api private
+  #
+  # Interface for using the controller
+  class ControllerInterface
+    # @param controller [ApplicationController]
+    def initialize(controller)
+      @controller = controller
+    end
+
+    # Set response headers
+    #
+    # @param headers_hash [Hash] headers to be set
+    #
+    # @return [Hash]
+    def add_headers(headers_hash)
+      controller.instance_eval do
+        headers_hash.each do |key, value|
+          headers[key.to_s] = value
+        end
+      end
+    end
+
+    # Renders response json
+    #
+    # @return [String]
+    def render_json(json, status)
+      controller.instance_eval do
+        render(json: json, status: status)
+      end
+    end
+
+    # Set a variable in the controller
+    #
+    # @param variable [String,Symbol] variable name
+    # @param value [Object] value to be set
+    #
+    # @return [Object]
+    def set(variable, value)
+      controller.instance_variable_set("@#{variable}", value)
+    end
+
+    # Forces a controller to run a block
+    #
+    # When the block is a +Proc+ that is evaluated,
+    # when it is a +Symbol+ or +String+, a method is called
+    #
+    # @return [Object] whatever the block returns
+    def run(block)
+      case block
+      when Proc
+        controller.instance_eval(&block)
+      else
+        controller.send(block)
+      end
+    end
+
+    private
+
+    attr_reader :controller
+    # @method controller
+    # @private
+    # @api private
+    #
+    # Controller where methods will be called
+    #
+    # @return [ApplicationController]
+
+    # @private
+    #
+    # Dispatcher to delegate all methods call to controller
+    #
+    # @param method_name [Symbol] name of the method
+    #   called
+    # @param args [Array<Object>] arguments of the
+    #   method called
+    #
+    # @return [Object]
+    def method_missing(method_name, *args)
+      if controller.respond_to?(method_name, true)
+        controller.send(method_name, *args)
+      else
+        super
+      end
+    end
+
+    # @private
+    #
+    # Checks if a controller responds to a method
+    #
+    # @param method_name [Symbol] name of the method checked
+    # @param include_private [TrueClass,FalseClass] flag
+    #   indicating if private methods should be included
+    #
+    # @return [TrueClass,FalseClass]
+    def respond_to_missing?(method_name, include_private)
+      controller.respond_to?(method_name, include_private)
+    end
+  end
+end

data/lib/azeroth/decorator/method_builder.rb

@@ -11,7 +11,7 @@ module Azeroth
       #
       # reaader delegate method calls to @object
       #
-      # @return (see Sinclair#build)
+      # @return [Array<Sinclair::MethodDefinition>]
       def self.build_reader(klass, attribute)
         new(klass).build_reader(attribute)
       end

data/lib/azeroth/options.rb

@@ -21,6 +21,7 @@ module Azeroth
       before_save: nil,
       after_save: nil,
       build_with: nil,
+      update_with: nil,
       paginated: false,
       per_page: 20
     }.freeze
@@ -110,6 +111,13 @@ module Azeroth
     #
     # @return [Symbol,Proc]
 
+    # @method update_with
+    # @api private
+    #
+    # Block or method name to be ran when updating the resource
+    #
+    # @return [Symbol,Proc]
+
     # @method paginated
     # @api private
     #

data/lib/azeroth/request_handler/create.rb

@@ -29,7 +29,7 @@ module Azeroth
       # @return [Object]
       def build_and_save_resource
         @resource = build_resource
-        controller.instance_variable_set("@#{model.name}", resource)
+        controller.set(model.name, resource)
 
         trigger_event(:save) do
           resource.tap(&:save)
@@ -47,12 +47,7 @@ module Azeroth
       def build_resource
         return collection.build(attributes) unless build_with
 
-        case build_with
-        when Proc
-          controller.instance_eval(&build_with)
-        else
-          controller.send(build_with)
-        end
+        controller.run(build_with)
       end
 
       # @private

data/lib/azeroth/request_handler/index.rb

@@ -21,6 +21,21 @@ module Azeroth
         paginated_entries
       end
 
+      # @private
+      #
+      # returns pagination headers
+      #
+      # @return [Hash] heders
+      def headers
+        return {} unless paginated?
+
+        {
+          pages: pages,
+          page: current_page,
+          per_page: limit
+        }
+      end
+
       # @private
       #
       # paginated collection of the model
@@ -39,8 +54,7 @@ module Azeroth
       #
       # @return [Integer]
       def offset
-        page = (params[:page] || 1).to_i - 1
-        page * limit
+        (current_page - 1) * limit
       end
 
       # @private
@@ -62,6 +76,24 @@ module Azeroth
       def scoped_entries
         controller.send(model.plural)
       end
+
+      # @private
+      #
+      # calculates current page
+      #
+      # @return [Integer]
+      def current_page
+        (params[:page] || 1).to_i
+      end
+
+      # @private
+      #
+      # calculates how many pages are there
+      #
+      # @return [Integer]
+      def pages
+        (scoped_entries.count.to_f / limit).ceil
+      end
     end
   end
 end

data/lib/azeroth/request_handler/update.rb

@@ -8,6 +8,8 @@ module Azeroth
     class Update < RequestHandler
       private
 
+      delegate :update_with, to: :options
+
       # @private
       #
       # Updates and return an instance of the model
@@ -17,20 +19,33 @@ module Azeroth
       #
       # @return [Object]
       def resource
-        @resource ||= update_resource
+        @resource ||= perform_update
       end
 
-      # build resource for update
+      # Update a resource saving it to the database
       #
-      # @return [Object]
-      def update_resource
-        controller.send(model.name).tap do |entry|
+      # @return [Object] updated resource
+      def perform_update
+        @resource = controller.send(model.name)
+        resource.tap do
           trigger_event(:save) do
-            entry.update(attributes)
+            update_and_save_resource
           end
         end
       end
 
+      # @private
+      #
+      # Update the resource, either by running update_with
+      # or directly updating the attributes in the object
+      #
+      # @return [Object] updated resource
+      def update_and_save_resource
+        return resource.update(attributes) unless update_with
+
+        controller.run(update_with)
+      end
+
       # @private
       #
       # Response status

data/lib/azeroth/request_handler.rb

@@ -19,7 +19,7 @@ module Azeroth
     # @param controller [ApplicationController]
     # @param model [Azeroth::Model]
     def initialize(controller, model, options)
-      @controller = controller
+      @controller = ControllerInterface.new(controller)
       @model = model
       @options = options
     end
@@ -35,12 +35,11 @@ module Azeroth
     def process
       return unless json?
 
-      json            = model.decorate(resource)
-      response_status = status
-
-      controller.instance_eval do
-        render(json: json, status: response_status)
-      end
+      controller.add_headers(headers)
+      controller.render_json(
+        model.decorate(resource),
+        status
+      )
     end
 
     private
@@ -141,5 +140,15 @@ module Azeroth
     def collection
       @collection = controller.send(model.plural)
     end
+
+    # @private
+    # @abstract
+    #
+    # Headers to be added
+    #
+    # @return [Hash]
+    def headers
+      {}
+    end
   end
 end

data/lib/azeroth/resourceable/builder.rb

@@ -25,6 +25,7 @@ module Azeroth
         add_params
         add_resource
         add_routes
+        add_helpers
       end
 
       private
@@ -127,6 +128,14 @@ module Azeroth
         RoutesBuilder.new(model, builder, options).append
       end
 
+      # Add helpers to render objects on template
+      #
+      # @return [String]
+      def add_helpers
+        clazz.public_send(:helper_method, model.name)
+        clazz.public_send(:helper_method, model.plural)
+      end
+
       # Returns all updatable attributes
       #
       # @return [Array<String>]

data/lib/azeroth/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Azeroth
-  VERSION = '0.8.0'
+  VERSION = '0.9.0'
 end

data/lib/azeroth.rb

@@ -11,12 +11,13 @@ require 'jace'
 #
 # @see Resourceable
 module Azeroth
-  autoload :Decorator,       'azeroth/decorator'
-  autoload :DummyDecorator,  'azeroth/dummy_decorator'
-  autoload :Model,           'azeroth/model'
-  autoload :RequestHandler,  'azeroth/request_handler'
-  autoload :Resourceable,    'azeroth/resourceable'
-  autoload :ResourceBuilder, 'azeroth/resource_builder'
-  autoload :RoutesBuilder,   'azeroth/routes_builder'
-  autoload :Options,         'azeroth/options'
+  autoload :ControllerInterface, 'azeroth/controller_interface'
+  autoload :Decorator,           'azeroth/decorator'
+  autoload :DummyDecorator,      'azeroth/dummy_decorator'
+  autoload :Model,               'azeroth/model'
+  autoload :RequestHandler,      'azeroth/request_handler'
+  autoload :Resourceable,        'azeroth/resourceable'
+  autoload :ResourceBuilder,     'azeroth/resource_builder'
+  autoload :RoutesBuilder,       'azeroth/routes_builder'
+  autoload :Options,             'azeroth/options'
 end

data/spec/controllers/paginated_documents_controller_spec.rb

@@ -38,6 +38,18 @@ describe PaginatedDocumentsController do
         it 'renders documents json' do
           expect(parsed_response).to eq(expected_json)
         end
+
+        it 'adds total pages header' do
+          expect(response.headers['pages']).to eq(1)
+        end
+
+        it 'adds current page header' do
+          expect(response.headers['page']).to eq(1)
+        end
+
+        it 'adds per page header' do
+          expect(response.headers['per_page']).to eq(20)
+        end
       end
 
       context 'when there are more documents than expected pagination' do
@@ -54,6 +66,18 @@ describe PaginatedDocumentsController do
           expect(parsed_response)
             .to have(20).elements
         end
+
+        it 'adds total pages header' do
+          expect(response.headers['pages']).to eq(2)
+        end
+
+        it 'adds current page header' do
+          expect(response.headers['page']).to eq(1)
+        end
+
+        it 'adds per page header' do
+          expect(response.headers['per_page']).to eq(20)
+        end
       end
 
       context 'when page is given' do
@@ -72,6 +96,18 @@ describe PaginatedDocumentsController do
             .to have(documents_count - 20)
             .elements
         end
+
+        it 'adds total pages header' do
+          expect(response.headers['pages']).to eq(2)
+        end
+
+        it 'adds current page header' do
+          expect(response.headers['page']).to eq(2)
+        end
+
+        it 'adds per page header' do
+          expect(response.headers['per_page']).to eq(20)
+        end
       end
 
       context 'when per_page is given' do
@@ -91,6 +127,19 @@ describe PaginatedDocumentsController do
             .to have(per_page)
             .elements
         end
+
+        it 'adds total pages header' do
+          expect(response.headers['pages'])
+            .to eq((documents_count.to_f / per_page).ceil)
+        end
+
+        it 'adds current page header' do
+          expect(response.headers['page']).to eq(1)
+        end
+
+        it 'adds per page header' do
+          expect(response.headers['per_page']).to eq(per_page)
+        end
       end
     end
   end

data/spec/controllers/rendering_controller_spec.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe RenderingController do
+  let(:parsed_response) do
+    JSON.parse(response.body)
+  end
+
+  describe 'GET index' do
+    render_views
+
+    let(:documents_count) { Random.rand(2..5) }
+
+    let(:expected_content) do
+      documents.map do |document|
+        "<li><strong>Name:</strong>#{document.name}</li>"
+      end.join("\n    ")
+    end
+
+    let!(:documents) { create_list(:document, documents_count) }
+
+    context 'when calling on format html' do
+      before do
+        get :index, params: { format: :html }
+      end
+
+      it { expect(response).to be_successful }
+
+      it { expect(response).to render_template('rendering/index') }
+
+      it { expect(response.body).to include(expected_content) }
+    end
+  end
+
+  describe 'GET show' do
+    render_views
+
+    let(:document)    { create(:document) }
+    let(:document_id) { document.id }
+    let(:expected_content) do
+      "<p><strong>Name:</strong>#{document.name}</p>"
+    end
+
+    context 'when calling on format html' do
+      before do
+        get :show, params: { id: document.id, format: :html }
+      end
+
+      it { expect(response).to be_successful }
+
+      it { expect(response).to render_template('rendering/show') }
+
+      it { expect(response.body).to include(expected_content) }
+    end
+  end
+end

data/spec/dummy/app/controllers/rendering_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class RenderingController < ApplicationController
+  include Azeroth::Resourceable
+
+  resource_for :document
+end

data/spec/dummy/app/views/rendering/index.html.erb

@@ -0,0 +1,5 @@
+<ul>
+  <% documents.each do |document| %>
+    <li><strong>Name:</strong><%= document.name %></li>
+  <% end %>
+</ul>

data/spec/dummy/app/views/rendering/show.html.erb

@@ -0,0 +1 @@
+<p><strong>Name:</strong><%= document.name %></p>

data/spec/dummy/config/routes.rb

@@ -2,6 +2,7 @@
 
 Rails.application.routes.draw do
   resources :documents
+  resources :rendering
   resources :public_documents, controller: :index_documents
   resources :create_documents, controller: :documents_with_error
   resources :paginated_documents

data/spec/lib/azeroth/controller_interface_spec.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::ControllerInterface do
+  subject(:interface)    { described_class.new(controller) }
+
+  let(:controller)       { controller_class.new }
+  let(:controller_class) { RequestHandlerController }
+
+  describe '#add_headers' do
+    let(:controller_headers) do
+      {
+        'old_key': 100
+      }
+    end
+
+    let(:headers) do
+      {
+        key1: 10,
+        key2: 20
+      }
+    end
+
+    before do
+      allow(controller)
+        .to receive(:headers)
+        .and_return(controller_headers)
+    end
+
+    it 'sets controller headers' do
+      expect { interface.add_headers(headers) }
+        .to change { controller_headers }
+        .from(controller_headers)
+        .to(controller_headers.merge(headers.stringify_keys))
+    end
+  end
+
+  describe '#render_json' do
+    let(:json)   { { key: 'value' } }
+    let(:status) { 200 }
+
+    before do
+      allow(controller)
+        .to receive(:render)
+        .with(json: json, status: status)
+    end
+
+    it 'renders json' do
+      interface.render_json(json, status)
+
+      expect(controller).to have_received(:render)
+    end
+  end
+
+  describe '#set' do
+    it 'sets instance variable' do
+      expect { interface.set(:name, 20) }
+        .to change { controller.instance_variable_get('@name') }
+        .from(nil)
+        .to(20)
+    end
+  end
+
+  describe 'method missing' do
+    before do
+      create(:document)
+    end
+
+    it 'delegates call to controller' do
+      expect(interface.documents)
+        .to eq(Document.all)
+    end
+
+    context 'when method is not defined' do
+      it do
+        expect { interface.pokemons }
+          .to raise_error(NoMethodError)
+      end
+    end
+  end
+end

data/spec/lib/azeroth/request_handler/index_spec.rb

@@ -14,6 +14,21 @@ describe Azeroth::RequestHandler::Index do
         let(:documents_count)   { Random.rand(21..30) }
         let(:expected_resource) { Document.all.limit(20) }
         let(:options_hash)      { { paginated: true } }
+
+        it 'adds total pages header' do
+          handler.process
+          expect(controller_headers['pages']).to eq(2)
+        end
+
+        it 'adds current page header' do
+          handler.process
+          expect(controller_headers['page']).to eq(1)
+        end
+
+        it 'adds per page header' do
+          handler.process
+          expect(controller_headers['per_page']).to eq(20)
+        end
       end
 
       context 'when page is given' do
@@ -22,6 +37,21 @@ describe Azeroth::RequestHandler::Index do
           let(:expected_resource) { Document.all.offset(20).limit(20) }
           let(:options_hash)      { { paginated: true } }
           let(:extra_params)      { { page: '2' } }
+
+          it 'adds total pages header' do
+            handler.process
+            expect(controller_headers['pages']).to eq(3)
+          end
+
+          it 'adds current page header' do
+            handler.process
+            expect(controller_headers['page']).to eq(2)
+          end
+
+          it 'adds per page header' do
+            handler.process
+            expect(controller_headers['per_page']).to eq(20)
+          end
         end
       end
 
@@ -31,6 +61,21 @@ describe Azeroth::RequestHandler::Index do
           let(:expected_resource) { Document.all.offset(40) }
           let(:options_hash)      { { paginated: true } }
           let(:extra_params)      { { page: '3' } }
+
+          it 'adds total pages header' do
+            handler.process
+            expect(controller_headers['pages']).to eq(3)
+          end
+
+          it 'adds current page header' do
+            handler.process
+            expect(controller_headers['page']).to eq(3)
+          end
+
+          it 'adds per page header' do
+            handler.process
+            expect(controller_headers['per_page']).to eq(20)
+          end
         end
       end
 
@@ -39,15 +84,45 @@ describe Azeroth::RequestHandler::Index do
           let(:documents_count)   { Random.rand(21..30) }
           let(:expected_resource) { Document.all.limit(10) }
           let(:options_hash)      { { paginated: true, per_page: 10 } }
+
+          it 'adds total pages header' do
+            handler.process
+            expect(controller_headers['pages']).to eq(3)
+          end
+
+          it 'adds current page header' do
+            handler.process
+            expect(controller_headers['page']).to eq(1)
+          end
+
+          it 'adds per page header' do
+            handler.process
+            expect(controller_headers['per_page']).to eq(10)
+          end
         end
       end
 
       context 'when per page is given in params' do
         it_behaves_like 'a request handler' do
           let(:documents_count)   { Random.rand(41..50) }
-          let(:expected_resource) { Document.all.offset(15).limit(15) }
-          let(:options_hash)      { { paginated: true, per_page: 10 } }
-          let(:extra_params)      { { page: '2', per_page: '15' } }
+          let(:expected_resource) { Document.all.offset(10).limit(10) }
+          let(:options_hash)      { { paginated: true, per_page: 15 } }
+          let(:extra_params)      { { page: '2', per_page: '10' } }
+
+          it 'adds total pages header' do
+            handler.process
+            expect(controller_headers['pages']).to eq(5)
+          end
+
+          it 'adds current page header' do
+            handler.process
+            expect(controller_headers['page']).to eq(2)
+          end
+
+          it 'adds per page header' do
+            handler.process
+            expect(controller_headers['per_page']).to eq(10)
+          end
         end
       end
     end

data/spec/lib/azeroth/request_handler/update_spec.rb

@@ -30,6 +30,118 @@ describe Azeroth::RequestHandler::Update do
       end
     end
 
+    context 'when update_with option is given ' do
+      context 'with block' do
+        it_behaves_like 'a request handler' do
+          let(:block) do
+            proc do
+              document.assign_attributes(document_params)
+              document.name = "#{document.name}!"
+              document.save
+            end
+          end
+
+          let(:options_hash) do
+            {
+              update_with: block
+            }
+          end
+
+          let(:expected_resource) { document }
+
+          let(:extra_params) do
+            {
+              id: document.id,
+              document: {
+                name: 'New Name'
+              }
+            }
+          end
+
+          let(:expected_json) do
+            {
+              'name' => 'New Name!'
+            }
+          end
+
+          it 'updates the values given by request' do
+            expect { handler.process }
+              .to change { document.reload.name }
+              .from(document.name)
+              .to('New Name!')
+          end
+        end
+      end
+
+      context 'with symbol' do
+        it_behaves_like 'a request handler' do
+          let(:options_hash) do
+            {
+              update_with: :add_bang_name
+            }
+          end
+
+          let(:expected_resource) { document }
+
+          let(:extra_params) do
+            {
+              id: document.id,
+              document: {
+                name: 'New Name'
+              }
+            }
+          end
+
+          let(:expected_json) do
+            {
+              'name' => 'New Name!'
+            }
+          end
+
+          it 'updates the values given by request' do
+            expect { handler.process }
+              .to change { document.reload.name }
+              .from(document.name)
+              .to('New Name!')
+          end
+        end
+      end
+
+      context 'with string' do
+        it_behaves_like 'a request handler' do
+          let(:options_hash) do
+            {
+              update_with: 'add_bang_name'
+            }
+          end
+
+          let(:expected_resource) { document }
+
+          let(:extra_params) do
+            {
+              id: document.id,
+              document: {
+                name: 'New Name'
+              }
+            }
+          end
+
+          let(:expected_json) do
+            {
+              'name' => 'New Name!'
+            }
+          end
+
+          it 'updates the values given by request' do
+            expect { handler.process }
+              .to change { document.reload.name }
+              .from(document.name)
+              .to('New Name!')
+          end
+        end
+      end
+    end
+
     context 'with before_save block option' do
       it_behaves_like 'a request handler' do
         let(:block) do

data/spec/lib/azeroth/request_handler_spec.rb

@@ -67,5 +67,15 @@ describe Azeroth::RequestHandler do
         expect(controller).not_to have_received(:render)
       end
     end
+
+    context 'with format js' do
+      let(:format) { 'js' }
+
+      it do
+        handler.process
+
+        expect(controller).not_to have_received(:render)
+      end
+    end
   end
 end

data/spec/support/app/controllers/request_handler_controller.rb

@@ -28,4 +28,10 @@ class RequestHandlerController < ActionController::Base
     documents.where(reference: 'X-MAGIC-15')
              .build(document_params)
   end
+
+  def add_bang_name
+    document.assign_attributes(document_params)
+    document.name = "#{document.name}!"
+    document.save
+  end
 end

data/spec/support/shared_examples/request_handler.rb

@@ -24,6 +24,8 @@ shared_examples 'a request handler' do |status: :ok|
     { format: format }.merge(extra_params)
   end
 
+  let(:controller_headers) { {} }
+
   before do
     create_list(:document, documents_count)
 
@@ -33,6 +35,9 @@ shared_examples 'a request handler' do |status: :ok|
     allow(controller).to receive(:render)
       .with(json: expected_json, status: status)
       .and_return(expected_json)
+
+    allow(controller).to receive(:headers)
+      .and_return(controller_headers)
   end
 
   it 'returns all documents json' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: azeroth
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Darthjee
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-08-11 00:00:00.000000000 Z
+date: 2022-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -128,14 +128,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.11.4
+        version: 1.12.5
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.11.4
+        version: 1.12.5
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -456,6 +456,7 @@ files:
 - config/yardstick.yml
 - docker-compose.yml
 - lib/azeroth.rb
+- lib/azeroth/controller_interface.rb
 - lib/azeroth/decorator.rb
 - lib/azeroth/decorator/hash_builder.rb
 - lib/azeroth/decorator/key_value_extractor.rb
@@ -484,6 +485,7 @@ files:
 - spec/controllers/paginated_documents_controller_spec.rb
 - spec/controllers/pokemon_masters_controller_spec.rb
 - spec/controllers/pokemons_controller_spec.rb
+- spec/controllers/rendering_controller_spec.rb
 - spec/dummy/.ruby-version
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/config/manifest.js
@@ -504,6 +506,7 @@ files:
 - spec/dummy/app/controllers/pokemon_masters_controller.rb
 - spec/dummy/app/controllers/pokemons_controller.rb
 - spec/dummy/app/controllers/publishers_controller.rb
+- spec/dummy/app/controllers/rendering_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/jobs/application_job.rb
 - spec/dummy/app/mailers/application_mailer.rb
@@ -537,6 +540,8 @@ files:
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/views/layouts/mailer.html.erb
 - spec/dummy/app/views/layouts/mailer.text.erb
+- spec/dummy/app/views/rendering/index.html.erb
+- spec/dummy/app/views/rendering/show.html.erb
 - spec/dummy/bin/bundle
 - spec/dummy/bin/rails
 - spec/dummy/bin/rake
@@ -583,6 +588,7 @@ files:
 - spec/integration/readme/controllers/games_controller_spec.rb
 - spec/integration/yard/azeroth/decorator_spec.rb
 - spec/integration/yard/controllers/games_controller_spec.rb
+- spec/lib/azeroth/controller_interface_spec.rb
 - spec/lib/azeroth/decorator/hash_builder_spec.rb
 - spec/lib/azeroth/decorator/key_value_extractor_spec.rb
 - spec/lib/azeroth/decorator/method_builder_spec.rb
@@ -649,6 +655,7 @@ test_files:
 - spec/controllers/paginated_documents_controller_spec.rb
 - spec/controllers/pokemon_masters_controller_spec.rb
 - spec/controllers/pokemons_controller_spec.rb
+- spec/controllers/rendering_controller_spec.rb
 - spec/dummy/.ruby-version
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/config/manifest.js
@@ -669,6 +676,7 @@ test_files:
 - spec/dummy/app/controllers/pokemon_masters_controller.rb
 - spec/dummy/app/controllers/pokemons_controller.rb
 - spec/dummy/app/controllers/publishers_controller.rb
+- spec/dummy/app/controllers/rendering_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/jobs/application_job.rb
 - spec/dummy/app/mailers/application_mailer.rb
@@ -702,6 +710,8 @@ test_files:
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/views/layouts/mailer.html.erb
 - spec/dummy/app/views/layouts/mailer.text.erb
+- spec/dummy/app/views/rendering/index.html.erb
+- spec/dummy/app/views/rendering/show.html.erb
 - spec/dummy/bin/bundle
 - spec/dummy/bin/rails
 - spec/dummy/bin/rake
@@ -748,6 +758,7 @@ test_files:
 - spec/integration/readme/controllers/games_controller_spec.rb
 - spec/integration/yard/azeroth/decorator_spec.rb
 - spec/integration/yard/controllers/games_controller_spec.rb
+- spec/lib/azeroth/controller_interface_spec.rb
 - spec/lib/azeroth/decorator/hash_builder_spec.rb
 - spec/lib/azeroth/decorator/key_value_extractor_spec.rb
 - spec/lib/azeroth/decorator/method_builder_spec.rb