releaf-core 0.2.1

data/spec/builders/core/settings/form_builder_spec.rb

@@ -0,0 +1,69 @@
+require "rails_helper"
+
+describe Releaf::Core::Settings::FormBuilder, type: :class do
+  class FormBuilderTestHelper < ActionView::Base; end
+  before do
+    Releaf::Settings.register([
+      { key: "myapp.rating", default: 5.65, description: "x", type: :decimal },
+      { key: "myapp.confirmed", default: true, type: :boolean },
+      { key: "myapp.intro" }
+    ])
+  end
+
+  let(:template){ FormBuilderTestHelper.new }
+
+  describe "#field_names" do
+    let(:subject){ described_class.new(:resource, Releaf::Settings.first, template, {}) }
+
+    it "returns :value as only editable field within array" do
+      expect(subject.field_names).to eq([:value])
+    end
+  end
+
+  describe "#render_value" do
+    let(:subject){ described_class.new(:resource, Releaf::Settings.first, template, {}) }
+
+    it "renders text field for value" do
+      allow(subject).to receive(:settings_field_label_text).and_return("x")
+      allow(subject).to receive(:settings_field_type).and_return("text")
+      allow(subject).to receive(:releaf_text_field).with(:value, { options: { label: { label_text: "x" }}}).and_return("y")
+      expect(subject.render_value).to eq("y")
+    end
+  end
+
+  describe "#settings_field_label_text" do
+    context "when description is available" do
+      let(:subject){ described_class.new(:resource, Releaf::Settings.first, template, {}) }
+
+      it "returns translated description text" do
+        allow(subject).to receive(:t).with("x", { scope: "settings"}).and_return("y")
+        expect(subject.settings_field_label_text).to eq("y")
+      end
+    end
+
+    context "when description is not available" do
+      let(:subject){ described_class.new(:resource, Releaf::Settings.last, template, {}) }
+
+      it "returns 'Value'" do
+        expect(subject.settings_field_label_text).to eq("Value")
+      end
+    end
+  end
+
+  describe "#settings_field_type" do
+    it "returns correct field type for the first setting" do
+      subject = described_class.new(:resource, Releaf::Settings.first, template, {})
+      expect(subject.settings_field_type).to eq(:decimal)
+    end
+
+    it "returns correct field type for the second setting" do
+      subject = described_class.new(:resource, Releaf::Settings.second, template, {})
+      expect(subject.settings_field_type).to eq(:boolean)
+    end
+
+    it "returns correct default field type" do
+      subject = described_class.new(:resource, Releaf::Settings.last, template, {})
+      expect(subject.settings_field_type).to eq(:text)
+    end
+  end
+end

data/spec/builders/core/settings/table_builder_spec.rb

@@ -0,0 +1,21 @@
+require "rails_helper"
+
+describe Releaf::Core::Settings::TableBuilder, type: :class do
+  class TableBuilderTestHelper < ActionView::Base; end
+  let(:template){ TableBuilderTestHelper.new }
+  let(:resource_class){ Releaf::Settings }
+  let(:subject){ described_class.new([], resource_class, template, {}) }
+
+  describe "#column_names" do
+    it "returns var, value and updated_at as column names array" do
+      expect(subject.column_names).to eq([:var, :value, :updated_at])
+    end
+  end
+
+  describe "#value_content" do
+    it "return resource value casted to string" do
+      resource = Releaf::Settings.new(value: Date.parse("2012-01-01"))
+      expect(subject.value_content(resource)).to eq("2012-01-01")
+    end
+  end
+end

data/spec/controllers/concerns/releaf/richtext_attachments_spec.rb

@@ -0,0 +1,51 @@
+require 'rails_helper'
+
+describe Releaf::Content::NodesController, type: :controller do
+  before do
+    sign_in create(:user)
+  end
+
+  describe "#create_releaf_richtext_attachment" do
+    let(:file) { Rack::Test::UploadedFile.new(File.expand_path('../../../fixtures/cs.png', __dir__), "image/png") }
+
+    context "when file is uploaded" do
+      it "renders 'create_releaf_richtext_attachment'" do
+        post :create_releaf_richtext_attachment, upload: file
+        expect( response ).to be_successful
+        expect( response ).to render_template('create_releaf_richtext_attachment')
+      end
+
+      it "creates attachment" do
+        expect do
+          post :create_releaf_richtext_attachment, upload: file
+        end.to change { Releaf::RichtextAttachment.count }.by(1)
+      end
+    end
+
+    context "when no file is uploaded" do
+      it "responds with success" do
+        post :create_releaf_richtext_attachment
+        expect( response ).to be_successful
+      end
+
+      it "doesn't create attachment" do
+        expect do
+          post :create_releaf_richtext_attachment
+        end.to_not change { Releaf::RichtextAttachment.count }
+      end
+    end
+  end
+
+  describe "#releaf_richtext_attachment_upload_url" do
+    it "returns upload url" do
+      allow(subject).to receive(:url_for).with(action: :create_releaf_richtext_attachment).and_return("a")
+      expect(subject.releaf_richtext_attachment_upload_url).to eq("a")
+    end
+
+    context "when no route exists for controller" do
+      it "returns nil" do
+        expect(subject.releaf_richtext_attachment_upload_url).to be nil
+      end
+    end
+  end
+end

data/spec/controllers/releaf/base_controller_spec.rb

@@ -0,0 +1,447 @@
+require 'rails_helper'
+
+describe Releaf::BaseController do
+  let(:new_resource){ Author.new }
+  let(:resource){ create(:author) }
+  let(:subject){ DummyController.new }
+  class DummyController < Releaf::BaseController
+    def resource_class
+      Author
+    end
+  end
+  class FooFormBuilder; end
+
+  describe "#action_views"  do
+    it "returns action > view translation hash" do
+      hash = {
+        new: :edit,
+        update: :edit,
+        create: :edit,
+      }
+      expect(subject.action_views).to eq(hash)
+    end
+  end
+
+  describe "#action_features" do
+    before do
+      subject.setup
+    end
+
+    it "returns action > feature mapped hash" do
+      expect(subject.action_features).to eq({
+        index: :index,
+        new: :create,
+        create: :create,
+        show: :edit,
+        edit: :edit,
+        update: :edit,
+        confirm_destroy: :destroy,
+        destroy: :destroy
+      }.with_indifferent_access)
+    end
+
+    context "when `show` feature is available" do
+      it "returns show > show feature mapping" do
+        allow(subject).to receive(:feature_available?).and_call_original
+        allow(subject).to receive(:feature_available?).with(:show).and_return(true)
+        expect(subject.action_features[:show]).to eq(:show)
+      end
+    end
+  end
+
+  describe "#action_view" do
+    context "when given view does not exists within action views hash" do
+      it "returns given action" do
+        expect(subject.action_view("a")).to eq("a")
+      end
+    end
+
+    context "when given view does not exists within action views hash" do
+      it "returns given action" do
+        expect(subject.action_view(:update)).to eq(:edit)
+      end
+    end
+
+    it "lookup given action as symbol to within action views hash" do
+      expect(subject.action_view("new")).to eq(:edit)
+    end
+  end
+
+  describe "#active_view" do
+    it "returns generic view name for current action" do
+      allow(subject).to receive(:action_name).and_return("a")
+      allow(subject).to receive(:action_view).with("a").and_return("b")
+      expect(subject.active_view).to eq("b")
+    end
+  end
+
+  describe "#form_url" do
+    context "when given resource is new record" do
+      it "returns url for create method" do
+        allow(subject).to receive(:url_for).with(action: 'create', id: nil).and_return("/res/new")
+        expect(subject.form_url(:edit, new_resource)).to eq("/res/new")
+      end
+    end
+
+    context "when given resource is existing record" do
+      it "returns url for update method" do
+        allow(subject).to receive(:url_for).with(action: 'update', id: resource.id).and_return("/res/edit/")
+        expect(subject.form_url(:edit, resource)).to eq("/res/edit/")
+      end
+    end
+  end
+
+  describe "#form_attributes" do
+    it "returns basic releaf form attributes" do
+      attributes = {
+         multipart: true,
+         novalidate: "",
+         class: ["new-user"],
+         id: "new-user",
+         data: {
+           "remote"=>true,
+           "remote-validation"=>true,
+           "type"=>:json
+         }
+      }
+      expect(subject.form_attributes(:edit, new_resource, :user)).to eq(attributes)
+    end
+
+    it "changes class/id depending whether given object is persisted" do
+      expect(subject.form_attributes(:edit, new_resource, :user)[:id]).to eq("new-user")
+      expect(subject.form_attributes(:edit, new_resource, :user)[:class]).to eq(["new-user"])
+
+      expect(subject.form_attributes(:edit, resource, :user)[:id]).to eq("edit-user")
+      expect(subject.form_attributes(:edit, resource, :user)[:class]).to eq(["edit-user"])
+    end
+
+    it "adds has-error class if object has any errors" do
+      resource.name = nil
+      expect(resource.valid?).to be false
+      expect(subject.form_attributes(:edit, resource, :user)[:class]).to eq(["edit-user", "has-error"])
+    end
+
+  end
+
+  describe "#builder_class" do
+    it "returns controller class scoped builder for given builder type" do
+      allow(subject).to receive(:builder_scopes).and_return(["a", "b"])
+      allow(Releaf::Builders).to receive(:builder_class).with(["a", "b"], :form).and_return("x")
+      expect(subject.builder_class(:form)).to eq("x")
+    end
+  end
+
+  describe "#builder_scopes" do
+    it "returns array with normalized controller class name scope and project builder scope" do
+      allow(subject).to receive(:application_builder_scope).and_return("xxx")
+      expect(subject.builder_scopes).to eq(["Dummy", "xxx"])
+    end
+  end
+
+  describe "#application_builder_scope" do
+    it "returns node builder scope within releaf mount location scope" do
+      allow(subject).to receive(:application_scope).and_return("Admin")
+      expect(subject.application_builder_scope).to eq("Admin::Builders")
+
+      allow(subject).to receive(:application_scope).and_return("")
+      expect(subject.application_builder_scope).to eq("Builders")
+
+      allow(subject).to receive(:application_scope).and_return(nil)
+      expect(subject.application_builder_scope).to eq("Builders")
+    end
+  end
+
+  describe "#application_scope" do
+    it "returns node builder scope within releaf mount location scope" do
+      allow(Releaf::Builders).to receive(:constant_defined_at_scope?).and_call_original
+      allow(Releaf.application.config).to receive(:mount_location).and_return("admin")
+
+      allow(Releaf::Builders).to receive(:constant_defined_at_scope?)
+        .with("Admin", Object).and_return(true)
+      expect(subject.application_scope).to eq("Admin")
+
+      allow(Releaf::Builders).to receive(:constant_defined_at_scope?)
+        .with("Admin", Object).and_return(false)
+      expect(subject.application_scope).to eq(nil)
+
+      allow(Releaf.application.config).to receive(:mount_location).and_return("")
+      expect(subject.application_scope).to eq(nil)
+    end
+  end
+
+  describe "#table_options" do
+    it "returns table options" do
+      allow(subject).to receive(:builder_class).with(:table).and_return("CustomTableBuilderClassHere")
+      allow(subject).to receive(:feature_available?).with(:toolbox).and_return("boolean_value_here")
+
+      options = {
+        builder: "CustomTableBuilderClassHere",
+        toolbox: "boolean_value_here"
+      }
+      expect(subject.table_options).to eq(options)
+    end
+  end
+
+  describe "#form_options" do
+    it "returns form options" do
+      allow(subject).to receive(:builder_class).with(:form).and_return("CustomFormBuilderClassHere")
+      allow(subject).to receive(:form_url).with(:delete, resource).and_return("/some-url-here")
+      allow(subject).to receive(:form_attributes).with(:delete, resource, :author).and_return(some: "options_here")
+
+      options = {
+        builder: "CustomFormBuilderClassHere",
+        as: :author,
+        url: "/some-url-here",
+        html: {some: "options_here"}
+      }
+      expect(subject.form_options(:delete, resource, :author)).to eq(options)
+    end
+  end
+end
+
+# use Admin::BooksController / Admin::AuthorsController as it inherit Releaf::BaseController and
+# have no extra methods or overrides
+describe Admin::AuthorsController do
+  before do
+    sign_in FactoryGirl.create(:user)
+  end
+
+  describe "#index_url" do
+    context "when action is other than :index" do
+      context "when params have 'index_url' defined" do
+        it "returns params 'index_url'" do
+          url = "/admin/something?a=1&b=2"
+          get :new, index_url: url
+          expect(subject.index_url).to eq(url)
+        end
+      end
+
+      context "when does not have 'index_url' defined" do
+        it "returns index action url" do
+          get :new
+          expect(subject.index_url).to eq("http://test.host/admin/authors")
+        end
+      end
+    end
+
+    context "when action is :index" do
+      it "returns #current_url value" do
+        get :index
+        allow(subject).to receive(:current_url).and_return("random_string")
+        expect(subject.index_url).to eq("random_string")
+      end
+    end
+  end
+
+  describe "#current_url" do
+    it "returns current url without `ajax` param" do
+      get :index, ajax: 1, search: "something", page: 1
+      expect(subject.current_url).to eq("/admin/authors?page=1&search=something")
+    end
+
+    context "when no query parameters exists" do
+      it "returns only request path" do
+        get :index
+        expect(subject.current_url).to eq("/admin/authors")
+      end
+    end
+
+    it "caches current url value" do
+      get :index
+      expect(subject).to receive(:request).twice.and_call_original
+      subject.current_url
+      subject.current_url
+      subject.current_url
+      subject.current_url
+    end
+  end
+
+  describe "#ajax?" do
+    it "returns @_ajax instance variable value" do
+      subject.instance_variable_set("@_ajax", "ll")
+      expect(subject.ajax?).to eq("ll")
+    end
+
+    context "when @_ajax instance variable has not been set" do
+      it "returns false" do
+        expect(subject.ajax?).to be false
+      end
+    end
+  end
+
+  describe "#manage_ajax" do
+    context "when `ajax` params does not exists within params" do
+      it "assigns `false` to @_ajax instance variable" do
+        expect{ get :index }.to change{ subject.instance_variable_get("@_ajax") }.from(nil).to(false)
+      end
+    end
+
+    context "when `ajax` params exists within params" do
+      it "assigns `true` to @_ajax instance variable" do
+        expect{ get :index, ajax: 1 }.to change{ subject.instance_variable_get("@_ajax") }.from(nil).to(true)
+      end
+
+      it "removes ajax from `params`" do
+        expect{ get :index, ajax: 1 }.to_not change{ subject.params[:ajax] }.from(nil)
+      end
+
+      it "removes ajax from `request.query_parameters`" do
+        expect{ get :index, ajax: 1 }.to_not change{ subject.request.query_parameters[:ajax] }.from(nil)
+      end
+    end
+  end
+
+  describe "GET show" do
+    let(:author){ create(:author) }
+
+    context "when show feature is available" do
+      it "assigns all resources to @collection" do
+        allow(subject).to receive(:feature_available?).with(:show).and_return(true)
+        get :show, id: author
+        expect(assigns(:resource)).to eq(author)
+      end
+    end
+
+    context "when show feature is not available" do
+      it "does assign resource" do
+        allow(subject).to receive(:feature_available?).and_call_original
+        allow(subject).to receive(:feature_available?).with(:show).and_return(false)
+        get :show, id: author
+        expect(assigns(:resource)).to be nil
+      end
+    end
+  end
+
+  describe "GET index" do
+    before do
+      21.times do |i|
+        FactoryGirl.create(:author)
+      end
+    end
+
+    context "when @resources_per_page is nil" do
+      it "assigns all resources to @collection" do
+        get :index, show_all: 1
+        expect(assigns(:collection).is_a?(ActiveRecord::Relation)).to be true
+        expect(assigns(:collection).size).to eq(21)
+      end
+    end
+
+    context "when @resources_per_page is not nil" do
+      it "assigns maximum 20 resources to @collection" do
+        get :index
+        expect(assigns(:collection).is_a?(ActiveRecord::Relation)).to be true
+        expect(assigns(:collection).size).to eq(20)
+      end
+    end
+  end
+
+  describe "DELETE #destroy" do
+    before do
+       @author = FactoryGirl.create(:author)
+       FactoryGirl.create(:book, title: "The book", author: @author)
+       FactoryGirl.create(:book, title: "Almost the book", author: @author)
+    end
+
+    it "creates flash error with message" do
+      delete :destroy, id: @author
+      expect(flash["error"]).to eq({"id" => "resource_status", "message" => "Cant destroy, because relations exists"})
+    end
+  end
+end
+
+
+describe Admin::BooksController do
+  before do
+    sign_in FactoryGirl.create(:user)
+    @breadcrumbs_base = [
+      {name: I18n.t('home', scope: 'admin.breadcrumbs'), url: releaf_root_path},
+      {name: I18n.t('admin/books', scope: "admin.controllers"), url: admin_books_path},
+    ]
+  end
+
+  describe "GET #index" do
+    before do
+      FactoryGirl.create(:book, title: "great one")
+      FactoryGirl.create(:book, title: "bad one")
+      FactoryGirl.create(:book, title: "average third")
+    end
+
+    context "when empty search string given" do
+      it "shows all records" do
+        get :index, search: ""
+        expect(assigns(:collection).count).to eq(3)
+      end
+    end
+
+    context "when search string with multiple words given" do
+      it "searches by given string" do
+        get :index, search: "one grea"
+        expect(assigns(:collection).count).to eq(1)
+      end
+    end
+
+    context "when search string given" do
+      it "searches by given string" do
+        get :index, search: "great"
+        expect(assigns(:collection).count).to eq(1)
+      end
+    end
+
+    context "when no search given" do
+      it "shows all records" do
+        get :index
+        expect(assigns(:collection).count).to eq(3)
+      end
+    end
+  end
+
+  describe "GET #new" do
+    it "assigns the requested record to @resource" do
+      get :new
+
+      expect(assigns(:resource).new_record?).to be true
+    end
+
+    context "when the requested record responds to #to_text" do
+      it "uses the result of #to_text for resource's breadcrumb name" do
+        get :new
+        breadcrumbs = @breadcrumbs_base + [{name: "New record", url: new_admin_book_path}]
+
+        expect(assigns(:breadcrumbs)).to eq(breadcrumbs)
+      end
+    end
+  end
+
+  describe "GET #edit" do
+    before do
+      @resource = create(:book)
+    end
+
+    it "assigns the requested record to @resource" do
+      get :edit, id: @resource
+
+      expect(assigns(:resource)).to eq(@resource)
+    end
+
+    context "when the requested record responds to #to_text" do
+      it "uses the result of #to_text for resource's breadcrumb name" do
+        get :edit, id: @resource
+        breadcrumbs = @breadcrumbs_base + [{name: @resource.to_text, url: edit_admin_book_path(@resource.id)}]
+
+        expect(assigns(:breadcrumbs)).to eq(breadcrumbs)
+      end
+    end
+
+    context "when the requested record does not respond to #to_text" do
+      it "uses default translation for resource's breadcrumb name" do
+        skip "Find out way how to stub loaded resource #respond_to?(:to_text)"
+        allow_any_instance_of(Book).to receive(:respond_to?).with(:to_text).and_return(false)
+        get :edit, id: @resource
+        breadcrumbs = @breadcrumbs_base + [{name: "Edit resource", url: edit_admin_book_path(@resource.id)}]
+
+        expect(assigns(:breadcrumbs)).to eq(breadcrumbs)
+      end
+    end
+  end
+end