wor-simple_crud 0.2.1 → 0.3.1

data/lib/spec/shared_examples/simple_crud_for_update.rb

@@ -3,9 +3,12 @@
 shared_examples 'simple crud for update' do
   describe 'PUT #update' do
     context 'without authenticated user' do
-      subject!(:req) { put :update, params: attributes_for(model_class).merge(id: model.id) }
+      subject!(:req) do
+        update_params = body_params(params_for(model_class)).merge(record_param(:update, model))
+        put :update, params: with_route_params(update_params), format: request_format(:update)
+      end
 
-      include_examples 'unauthorized when not logged in'
+      include_examples 'unauthorized when not logged in' if check_authenticate(:update)
     end
 
     if check_authorize(:update)
@@ -14,7 +17,8 @@ shared_examples 'simple crud for update' do
 
         before do
           make_policies_fail(:update)
-          put :update, params: model_params.merge(id: model.id)
+          put :update, params: with_route_params(body_params(model_params).merge(record_param(:update, model))),
+                       format: request_format(:update)
         end
 
         it 'fails with forbidden' do
@@ -24,32 +28,48 @@ shared_examples 'simple crud for update' do
     end
 
     context 'when successfully updating an model' do
-      include_context 'with authenticated user'
+      include_context 'with authenticated user' if check_authenticate(:update)
+      let(:update_params) { body_params(model_params).merge(record_param(:update, model)) }
+      let(:update_attributes) { model_params.merge(record_param(:update, model).except(:id)) }
 
       before do
         model
-        put :update, params: model_params.merge(id: model.id)
+        put :update, params: with_route_params(update_params), format: request_format(:update)
       end
 
-      it 'response with 200 status code' do
-        expect(response).to have_http_status(:ok)
-      end
+      if check_html(:update)
+        it 'redirects to the updated record' do
+          expect(response).to have_http_status(:found)
+        end
+      else
+        it 'response with 200 status code' do
+          expect(response).to have_http_status(:ok)
+        end
 
-      it 'updates an model' do
-        expect(model_class_object.count).to be 1
+        it 'updates an model' do
+          expect(model_class_object.count).to be 1
+        end
+
+        it 'updates an model with valid attributes' do
+          expect(model_class_object.last).to have_attributes(update_attributes)
+        end
       end
 
-      it 'updates an model with valid attributes' do
-        expect(model_class.classify.constantize.last).to have_attributes(model_params)
+      it 'updates a record passing created_record_check', :aggregate_failures do
+        expect(model_class_object.last).to be_present
+        instance_exec(model_class_object.last, &created_record_check) if created_record_check
       end
     end
 
     context 'when updating a model that doesn\'t exist' do
       include_context 'with authenticated user'
 
-      before { put :update, params: { id: 1 } }
+      before do
+        put :update, params: with_route_params(record_param(:update, nil, not_found: true)),
+                     format: request_format(:update)
+      end
 
-      it 'response with 422 status code' do
+      it 'response with 404 status code' do
         expect(response).to have_http_status(:not_found)
       end
 
@@ -57,5 +77,32 @@ shared_examples 'simple crud for update' do
         expect(model_class_object.count).to be 0
       end
     end
+
+    unless check_raise_on_invalid(:update)
+      context 'when updating with invalid attributes' do
+        include_context 'with authenticated user' if check_authenticate(:update)
+
+        before do
+          model
+          put :update, params: with_route_params(body_params({ required_attribute => nil }.merge(owner_params))
+                                                         .merge(record_param(:update, model))),
+                       format: request_format(:update)
+        end
+
+        unless check_block(:update)
+          if check_html(:update)
+            include_examples 'simple crud renders template', :edit, -> { invalid_status }
+          else
+            it 'responds with unprocessable entity' do
+              expect(response).to have_http_status(unprocessable_status)
+            end
+
+            it 'returns the validation errors' do
+              expect(response_body['errors']).to include(required_error)
+            end
+          end
+        end
+      end
+    end
   end
 end

data/lib/spec/shared_examples/simple_crud_for_update_with_finder.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+shared_examples 'simple crud for update with finder' do
+  describe 'PUT #update with a custom finder' do
+    include_context 'with authenticated user' if check_authenticate(:update)
+
+    context 'when successfully updating a model' do
+      let(:update_params) do
+        with_route_params(body_params(required_attribute => 'Updated').merge(owner_params))
+          .merge(finder_key => model.public_send(finder_key))
+      end
+
+      before do
+        model
+        put :update, params: update_params, format: request_format(:update)
+      end
+
+      it 'updates the model' do
+        expect(model.reload.public_send(required_attribute)).to eq('Updated')
+      end
+
+      if check_block(:update)
+        it 'passes the updated model to the block' do
+          expect(rendered_record.id).to eq(model.id)
+        end
+      elsif check_html(:update)
+        it 'redirects to the updated record' do
+          expect(response).to have_http_status(:found)
+        end
+      else
+        it 'responds with ok status' do
+          expect(response).to have_http_status(:ok)
+        end
+      end
+    end
+
+    include_examples 'simple crud not found with finder', :put, :update
+  end
+end

data/lib/spec/shared_examples/simple_crud_not_found_with_finder.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+shared_examples 'simple crud not found with finder' do |http_method, action|
+  context 'when the model does not exist' do
+    before do
+      send(http_method, action, params: with_route_params(finder_key => "nonexistent-#{finder_key}"),
+                                format: request_format(action))
+    end
+
+    it 'responds with not found status' do
+      expect(response).to have_http_status(:not_found)
+    end
+  end
+end

data/lib/spec/shared_examples/simple_crud_renders_template.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+shared_examples 'simple crud renders template' do |template, status = :ok|
+  it "renders the #{template} template", :aggregate_failures do
+    expect(response).to have_http_status(resolve(status))
+    expect(response).to render_template(template) if assert_html_template
+  end
+end

data/lib/spec/shared_examples/simple_crud_when_not_authorized.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+shared_examples 'simple crud when not authorized' do |action, verb, request_params = {}|
+  if check_authorize(action)
+    context 'when not authorized' do
+      include_context 'with authenticated user' if check_authenticate(action)
+
+      before do
+        make_policies_fail(action)
+        send(verb, action, params: with_route_params(request_params))
+      end
+
+      it 'fails with forbidden' do
+        expect(response).to have_http_status(:forbidden)
+      end
+    end
+  end
+end

data/lib/spec/shared_examples/simple_crud_without_authenticated_user.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+shared_examples 'simple crud without authenticated user' do |action|
+  context 'without authenticated user' do
+    subject!(:req) { get action, params: with_route_params({}) }
+
+    include_examples 'unauthorized when not logged in' if check_authenticate(action)
+  end
+end

data/lib/spec/shared_examples/unauthorized_when_not_logged_in.rb

@@ -2,9 +2,9 @@
 
 shared_examples 'unauthorized when not logged in' do
   context 'when the user is not logged in' do
-    it 'returns status code unauthorized' do
+    it 'returns the unauthenticated status' do
       req
-      expect(response).to have_http_status(:unauthorized)
+      expect(response).to have_http_status(unauthenticated_status)
     end
   end
 end

data/spec/block/dummy_models_controller_spec.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Block::DummyModelsController, type: :controller do
+  include_examples 'simple crud for index with block'
+  include_examples 'simple crud for index'
+end

data/spec/block_create/dummy_models_controller_spec.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockCreate::DummyModelsController, type: :controller do
+  include_examples 'simple crud for create with block'
+end

data/spec/block_destroy/dummy_models_controller_spec.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockDestroy::DummyModelsController, type: :controller do
+  include_examples 'simple crud for destroy with block'
+end

data/spec/block_finder/dummy_models_controller_spec.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockFinder::DummyModelsController, type: :controller do
+  include_examples 'simple crud for edit'
+  include_examples 'simple crud for show with finder'
+  include_examples 'simple crud for update with finder'
+  include_examples 'simple crud for destroy with finder'
+end

data/spec/block_json/dummy_models_controller_spec.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockJson::DummyModelsController, type: :controller do
+  include_examples 'simple crud for index with block'
+  include_examples 'simple crud for show with block'
+  include_examples 'simple crud for new with block'
+end

data/spec/block_new/dummy_models_controller_spec.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockNew::DummyModelsController, type: :controller do
+  include_examples 'simple crud for new with block'
+end

data/spec/block_redirect/dummy_models_controller_spec.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockRedirect::DummyModelsController, type: :controller do
+  include_examples 'simple crud for destroy'
+end

data/spec/block_show/dummy_models_controller_spec.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe BlockShow::DummyModelsController, type: :controller do
+  include_examples 'simple crud for show with block'
+end

data/spec/built/dummy_models_controller_spec.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Built::DummyModelsController, type: :controller do
+  include_examples 'simple crud for new with build'
+  include_examples 'simple crud for create with build'
+end

data/spec/dummy/app/controllers/block/dummy_models_controller.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Block
+  class DummyModelsController < ApplicationController
+    include Wor::Paginate
+    extend SimpleCrudController
+
+    before_action :set_params
+    def set_params
+      SimpleCrudController.params = params
+    end
+
+    simple_crud_for :index, paginate: false, authenticate: false, authorize: false do |records|
+      render :index, locals: { models: records }
+    end
+  end
+end

data/spec/dummy/app/controllers/block_create/dummy_models_controller.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module BlockCreate
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :create, html: false, authenticate: false, authorize: false do |record, saved|
+      if saved
+        render json: { saved: true, id: record.id }
+      else
+        render json: { saved: false, errors: record.errors.full_messages }, status: 422
+      end
+    end
+  end
+end

data/spec/dummy/app/controllers/block_destroy/dummy_models_controller.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module BlockDestroy
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :destroy, html: false, authenticate: false, authorize: false do |_record, destroyed|
+      render json: { destroyed: destroyed ? true : false }
+    end
+  end
+end

data/spec/dummy/app/controllers/block_finder/dummy_models_controller.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module BlockFinder
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :show, html: false, authenticate: false, authorize: false,
+                           finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) } do |record|
+      render json: { found_id: record.id }
+    end
+
+    simple_crud_for :update, html: false, authenticate: false, authorize: false,
+                             finder: :find_by_slug do |_record, saved|
+      render json: { updated: saved }
+    end
+
+    simple_crud_for :destroy, html: false, authenticate: false, authorize: false,
+                              finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) } do |_record, destroyed|
+      render json: { destroyed: destroyed }
+    end
+
+    simple_crud_for :edit, html: false, authenticate: false, authorize: false,
+                           finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) } do |record|
+      render json: { edit_id: record.id }
+    end
+  end
+end

data/spec/dummy/app/controllers/block_json/dummy_models_controller.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module BlockJson
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :index, html: false, authenticate: false, authorize: false, paginate: false do |records|
+      render json: { ids: records.map(&:id) }
+    end
+
+    simple_crud_for :show, html: false, authenticate: false, authorize: false do |record|
+      render json: { id: record.id }
+    end
+
+    simple_crud_for :new, html: false, authenticate: false, authorize: false do |record|
+      render json: { id: record.id }
+    end
+  end
+end

data/spec/dummy/app/controllers/block_modes/base_controller.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module BlockModes
+  class BaseController < HtmlModes::BaseController
+    private
+
+    def render_model(record, template)
+      render template, locals: { model: record }
+    end
+  end
+end

data/spec/dummy/app/controllers/block_new/dummy_models_controller.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module BlockNew
+  class DummyModelsController < BlockModes::BaseController
+    simple_crud_for :new, authenticate: false, authorize: false do |record|
+      render_model(record, :new)
+    end
+  end
+end

data/spec/dummy/app/controllers/block_redirect/dummy_models_controller.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module BlockRedirect
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :destroy, authenticate: false, authorize: false do |_record, destroyed|
+      if destroyed
+        redirect_to action: :show, notice: 'removed'
+      else
+        redirect_to action: :show, alert: 'could not remove'
+      end
+    end
+  end
+end

data/spec/dummy/app/controllers/block_show/dummy_models_controller.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module BlockShow
+  class DummyModelsController < BlockModes::BaseController
+    simple_crud_for :show, authenticate: false, authorize: false do |record|
+      render_model(record, :show)
+    end
+  end
+end

data/spec/dummy/app/controllers/built/dummy_models_controller.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Built
+  class DummyModelsController < ApplicationController
+    include Wor::Paginate
+    include Pundit::Authorization
+    extend SimpleCrudController
+
+    before_action :set_params
+    def set_params
+      SimpleCrudController.params = params
+    end
+
+    def dummy_model_params
+      params.permit(:name, :something, :user_id, :slug)
+    end
+
+    simple_crud_for :new, build: -> { current_user.dummy_models.build }
+    simple_crud_for :create, build: -> { current_user.dummy_models.build }
+  end
+end

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

@@ -12,12 +12,13 @@ class DummyModelsController < ApplicationController
   end
 
   def dummy_model_params
-    params.permit(:name, :something, :user_id)
+    params.permit(:name, :something, :user_id, :slug)
   end
 
   simple_crud_for :create
   simple_crud_for :destroy
   simple_crud_for :update
   simple_crud_for :show
+  simple_crud_for :new
   simple_crud_for :index
 end

data/spec/dummy/app/controllers/finder/dummy_models_controller.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Finder
+  class DummyModelsController < ApplicationController
+    include Wor::Paginate
+    include Pundit::Authorization
+    extend SimpleCrudController
+
+    before_action :set_params
+    def set_params
+      SimpleCrudController.params = params
+    end
+
+    def dummy_model_params
+      params.permit(:name, :something, :user_id)
+    end
+
+    simple_crud_for :show, finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+    simple_crud_for :update, finder: :find_by_slug
+    simple_crud_for :destroy, finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+    simple_crud_for :edit, finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+  end
+end

data/spec/dummy/app/controllers/html/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Html
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :index, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_create/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module HtmlCreate
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :create, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_destroy/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module HtmlDestroy
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :destroy, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_finder/dummy_models_controller.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module HtmlFinder
+  class DummyModelsController < HtmlModes::BaseController
+    include Pundit::Authorization
+
+    simple_crud_for :show, html: true, authenticate: false, authorize: false,
+                           finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+    simple_crud_for :update, html: true, authenticate: false, authorize: false,
+                             finder: :find_by_slug
+    simple_crud_for :destroy, html: true, authenticate: false, authorize: false,
+                              finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+    simple_crud_for :new, html: true, build: -> { current_user.dummy_models.build }
+    simple_crud_for :edit, html: true, authenticate: false, authorize: false,
+                           finder: ->(params) { DummyModel.find_by!(slug: params[:slug]) }
+  end
+end

data/spec/dummy/app/controllers/html_modes/base_controller.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module HtmlModes
+  class BaseController < ApplicationController
+    include Wor::Paginate
+    extend SimpleCrudController
+
+    before_action :set_params
+    def set_params
+      SimpleCrudController.params = params
+    end
+
+    def dummy_model_params
+      params.permit(:name, :something, :user_id, :slug)
+    end
+  end
+end

data/spec/dummy/app/controllers/html_new/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module HtmlNew
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :new, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_scoped/dummy_models_controller.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module HtmlScoped
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :index, html: true, scope: ->(user) { DummyModel.where(user: user) },
+                            authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_show/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module HtmlShow
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :show, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/html_update/dummy_models_controller.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module HtmlUpdate
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :update, html: true, authenticate: false, authorize: false
+  end
+end

data/spec/dummy/app/controllers/invalid_status/dummy_models_controller.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module InvalidStatus
+  class DummyModelsController < HtmlModes::BaseController
+    simple_crud_for :create, authenticate: false, authorize: false do |record, saved|
+      if saved
+        redirect_to(record)
+      else
+        render :new, status: 422
+      end
+    end
+  end
+end

data/spec/dummy/app/controllers/nested/dummy_models_controller.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Nested
+  class DummyModelsController < ApplicationController
+    include Wor::Paginate
+    include Pundit::Authorization
+    extend SimpleCrudController
+
+    before_action :set_params
+    def set_params
+      SimpleCrudController.params = params
+    end
+
+    def dummy_model_params
+      params.require(:dummy_model).permit(:name, :something, :user_id, :slug)
+    end
+
+    simple_crud_for :create
+    simple_crud_for :update
+  end
+end