azeroth 0.0.7 → 0.1.0

data/lib/azeroth/version.rb

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

data/spec/controllers/documents_controller_spec.rb

@@ -15,6 +15,10 @@ describe DocumentsController do
       end
     end
 
+    let(:expected_json) do
+      Document::Decorator.new(documents).as_json
+    end
+
     context 'when calling on format json' do
       before do
         get :index, params: { format: :json }
@@ -23,7 +27,7 @@ describe DocumentsController do
       it { expect(response).to be_successful }
 
       it 'returns empty array' do
-        expect(parsed_response).to eq([])
+        expect(parsed_response).to eq(expected_json)
       end
 
       context 'when there is a document' do
@@ -32,7 +36,7 @@ describe DocumentsController do
         it { expect(response).to be_successful }
 
         it 'renders documents json' do
-          expect(parsed_response).to eq(documents.as_json)
+          expect(parsed_response).to eq(expected_json)
         end
       end
     end
@@ -52,6 +56,10 @@ describe DocumentsController do
     let(:document)    { Document.create }
     let(:document_id) { document.id }
 
+    let(:expected_json) do
+      Document::Decorator.new(document).as_json
+    end
+
     context 'when calling on format json' do
       before do
         get :show, params: { id: document_id, format: :json }
@@ -60,7 +68,7 @@ describe DocumentsController do
       it { expect(response).to be_successful }
 
       it 'returns document json' do
-        expect(parsed_response).to eq(document.as_json)
+        expect(parsed_response).to eq(expected_json)
       end
     end
 
@@ -93,25 +101,34 @@ describe DocumentsController do
   describe 'POST create' do
     let(:parameters) do
       {
+        format: format,
         document: {
           name: 'My document'
         }
       }
     end
 
-    it do
-      post :create, params: parameters
-      expect(response).to be_successful
-    end
+    context 'when requesting format json' do
+      let(:format) { :json }
 
-    it 'returns created document json' do
-      post :create, params: parameters
-      expect(parsed_response).to eq(Document.last.as_json)
-    end
+      let(:expected_json) do
+        Document::Decorator.new(Document.last).as_json
+      end
 
-    it do
-      expect { post :create, params: parameters }
-        .to change(Document, :count).by(1)
+      it do
+        post :create, params: parameters
+        expect(response).to be_successful
+      end
+
+      it 'returns created document json' do
+        post :create, params: parameters
+        expect(parsed_response).to eq(expected_json)
+      end
+
+      it do
+        expect { post :create, params: parameters }
+          .to change(Document, :count).by(1)
+      end
     end
   end
 
@@ -119,9 +136,14 @@ describe DocumentsController do
     let(:document)    { Document.create }
     let(:document_id) { document.id }
 
+    let(:expected_json) do
+      Document::Decorator.new(Document.last).as_json
+    end
+
     let(:parameters) do
       {
         id: document_id,
+        format: :json,
         document: {
           name: 'My document'
         }
@@ -135,7 +157,7 @@ describe DocumentsController do
 
     it 'returns updated document json' do
       patch :update, params: parameters
-      expect(parsed_response).to eq(Document.last.as_json)
+      expect(parsed_response).to eq(expected_json)
     end
 
     it do
@@ -161,13 +183,29 @@ describe DocumentsController do
   end
 
   describe 'GET new' do
-    before { get :new }
+    context 'when calling with format json' do
+      before { get :new, params: { format: :json } }
 
-    it do
-      expect(response).to be_successful
+      let(:expected_json) do
+        Document::Decorator.new(Document.new).as_json
+      end
+
+      it do
+        expect(response).to be_successful
+      end
+
+      it { expect(parsed_response).to eq(expected_json) }
     end
 
-    it { expect(response).to render_template('documents/new') }
+    context 'when calling with format html' do
+      before { get :new }
+
+      it do
+        expect(response).to be_successful
+      end
+
+      it { expect(response).to render_template('documents/new') }
+    end
   end
 
   describe 'GET edit' do
@@ -175,6 +213,10 @@ describe DocumentsController do
     let(:document_id) { document.id }
 
     context 'when calling on format json' do
+      let(:expected_json) do
+        Document::Decorator.new(document).as_json
+      end
+
       before do
         get :edit, params: { id: document_id, format: :json }
       end
@@ -182,7 +224,7 @@ describe DocumentsController do
       it { expect(response).to be_successful }
 
       it 'returns document json' do
-        expect(parsed_response).to eq(document.as_json)
+        expect(parsed_response).to eq(expected_json)
       end
     end
 
@@ -216,9 +258,14 @@ describe DocumentsController do
     let!(:document)   { Document.create }
     let(:document_id) { document.id }
 
+    let(:expected_json) do
+      Document::Decorator.new(document).as_json
+    end
+
     let(:parameters) do
       {
-        id: document_id
+        id: document_id,
+        format: :json
       }
     end
 
@@ -229,7 +276,7 @@ describe DocumentsController do
 
     it do
       delete :destroy, params: parameters
-      expect(response.body).to eq('')
+      expect(parsed_response).to eq(expected_json)
     end
 
     it do

data/spec/dummy/db/schema.rb

@@ -4,8 +4,8 @@ ActiveRecord::Schema.define do
   self.verbose = false
 
   create_table :documents, force: true do |t|
-    t.string :name
-    t.string :reference
+    t.string   :name
+    t.string   :reference
   end
 
   create_table :users, force: true do |t|

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

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Create do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let(:extra_params) do
+        {
+          document: {
+            name: 'My Document'
+          }
+        }
+      end
+
+      let(:expected_json) do
+        { 'name' => 'My Document' }
+      end
+
+      it 'creates entry' do
+        expect { handler.process }
+          .to change(Document, :count)
+          .by(1)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Destroy do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let(:expected_resource) { document }
+      let!(:document)         { create(:document) }
+
+      let(:extra_params) { { id: document.id } }
+
+      it 'updates the values' do
+        expect { handler.process }
+          .to change(Document, :count)
+          .by(-1)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Edit do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let!(:document)         { create(:document) }
+      let(:expected_resource) { document }
+      let(:extra_params)      { { 'id' => document.id } }
+    end
+  end
+end

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

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Index do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let(:documents_count)   { 3 }
+      let(:expected_resource) { Document.all }
+    end
+  end
+end

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

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::New do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let(:document)          { create(:document) }
+      let(:expected_resource) { Document.new }
+    end
+  end
+end

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

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Show do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let!(:document)         { create(:document) }
+      let(:expected_resource) { document }
+      let(:extra_params)      { { 'id' => document.id } }
+    end
+  end
+end

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

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Update do
+  describe '#process' do
+    it_behaves_like 'a request handler' do
+      let(:expected_resource) { document }
+      let!(:document)         { create(:document) }
+
+      let(:extra_params) do
+        {
+          id: document.id,
+          document: {
+            name: 'New Name'
+          }
+        }
+      end
+
+      let(:expected_json) do
+        decorator.as_json.merge('name' => 'New Name')
+      end
+
+      it 'updates the values' do
+        expect { handler.process }
+          .to change { document.reload.name }
+          .from(document.name)
+          .to('New Name')
+      end
+    end
+  end
+end

data/spec/lib/azeroth/request_handler_spec.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler do
+  describe '#process' do
+    subject(:handler) { handler_class.new(controller, model) }
+
+    let(:controller) { controller_class.new }
+    let(:params)     { ActionController::Parameters.new(parameters) }
+    let(:model)      { Azeroth::Model.new(:document) }
+
+    let(:decorator)       { Document::Decorator.new(Document.all) }
+    let(:expected_json)   { decorator.as_json }
+    let(:documents_count) { 3 }
+
+    let(:controller_class) { RequestHandlerController }
+
+    let(:parameters) do
+      { format: format }
+    end
+
+    let(:handler_class) do
+      Class.new(described_class) do
+        private
+
+        def resource
+          Document.all
+        end
+      end
+    end
+
+    before do
+      documents_count.times { create(:document) }
+
+      allow(controller).to receive(:params)
+        .and_return(params)
+
+      allow(controller).to receive(:render)
+        .with(json: expected_json)
+        .and_return(expected_json)
+    end
+
+    context 'with format json' do
+      let(:format) { 'json' }
+
+      it 'returns json for resource' do
+        expect(handler.process).to eq(expected_json)
+      end
+
+      it 'renders the json' do
+        handler.process
+
+        expect(controller).to have_received(:render)
+      end
+    end
+
+    context 'with format html' do
+      let(:format) { 'html' }
+
+      it do
+        handler.process
+
+        expect(controller).not_to have_received(:render)
+      end
+    end
+  end
+end

data/spec/lib/azeroth/resourceable/builder_spec.rb

@@ -14,8 +14,6 @@ describe Azeroth::Resourceable::Builder do
   end
 
   describe '#build' do
-    it_behaves_like 'a route resource build' do
-      let(:builder) { subject }
-    end
+    it 'adds spec'
   end
 end

data/spec/lib/azeroth/routes_builder_spec.rb

@@ -7,17 +7,38 @@ describe Azeroth::RoutesBuilder do
     described_class.new(model, builder, options)
   end
 
-  let(:model)        { Azeroth::Model.new(:document) }
-  let(:builder)      { Sinclair.new(klass) }
-  let(:klass)        { Class.new(RoutesBuilderController) }
-  let(:instance)     { klass.new(params) }
-  let(:params)       { {} }
+  let(:controller) { controller_class.new }
+  let(:params)     { ActionController::Parameters.new(parameters) }
+  let(:model)      { Azeroth::Model.new(:document) }
+  let(:builder)    { Sinclair.new(controller_class) }
+  let(:parameters) { { action: :index, format: :json } }
+
   let(:options)      { Azeroth::Options.new(options_hash) }
   let(:options_hash) { {} }
 
+  let(:expected_json) do
+    Document::Decorator.new(Document.all).as_json
+  end
+
+  let(:controller_class) do
+    Class.new(ActionController::Base) do
+      include Azeroth::Resourceable
+
+      def documents
+        Document.all
+      end
+    end
+  end
+
   before do
-    routes_builder.append
     10.times { Document.create }
+
+    allow(controller).to receive(:params)
+      .and_return(params)
+
+    allow(controller).to receive(:render)
+      .with(json: expected_json)
+      .and_return(expected_json)
   end
 
   describe '#append' do
@@ -26,14 +47,21 @@ describe Azeroth::RoutesBuilder do
     it 'adds index route' do
       expect do
         builder.build
-      end.to add_method(:index).to(klass.new)
+      end.to add_method(:index).to(controller)
     end
 
     describe 'when calling index' do
       before { builder.build }
 
       it 'returns the index object' do
-        expect(instance.perform(:index)).to eq(json: 'index_json')
+        expect(controller.index).to eq(expected_json)
+      end
+
+      it 'renders the json' do
+        controller.index
+
+        expect(controller).to have_received(:render)
+          .with(json: expected_json)
       end
     end
   end