releaf-content 0.2.1

data/spec/builders/content/nodes/form_builder_spec.rb

@@ -0,0 +1,218 @@
+require "rails_helper"
+
+describe Releaf::Content::Nodes::FormBuilder, type: :class do
+  class FormBuilderTestHelper < ActionView::Base
+    include Releaf::ApplicationHelper
+    include Releaf::ButtonHelper
+    include FontAwesome::Rails::IconHelper
+    def controller_scope_name; end
+    def generate_url_releaf_content_nodes_path(args); end
+  end
+
+  let(:template){ FormBuilderTestHelper.new }
+  let(:object){ Node.new(content_type: "TextPage", slug: "b", id: 2,
+                         parent: Node.new(content_type: "TextPage", slug: "a", id: 1)) }
+  let(:subject){ described_class.new(:resource, object, template, {}) }
+
+  describe "#field_names" do
+    it "returns hidden, node and content object fields" do
+      expect(subject.field_names).to eq(["node_fields_block", "content_fields_block"])
+    end
+  end
+
+  describe "#node_fields" do
+    it "returns array with renderable node fields" do
+      list = [:parent_id, :name, :content_type, :slug, :item_position, :active, :locale]
+      expect(subject.node_fields).to eq(list)
+    end
+  end
+
+  describe "#render_node_fields_block" do
+    it "renders node fields" do
+      allow(subject).to receive(:node_fields).and_return([1, 2])
+      allow(subject).to receive(:releaf_fields).with([1, 2]).and_return("x")
+      content = '<div class="section node-fields">x</div>'
+      expect(subject.render_node_fields_block).to eq(content)
+    end
+  end
+
+  describe "#render_parent_id" do
+    it "renders hidden parent if field for new object" do
+      allow(subject.object).to receive(:new_record?).and_return(true)
+      allow(subject).to receive(:hidden_field).with(:parent_id).and_return("x")
+      expect(subject.render_parent_id).to eq("x")
+    end
+
+    it "renders nothing for existing object" do
+      allow(subject.object).to receive(:new_record?).and_return(false)
+      expect(subject.render_parent_id).to eq(nil)
+    end
+  end
+
+  describe "#render_content_fields_block?" do
+    before do
+      subject.object.build_content
+    end
+
+    it "returns array of node content object fields" do
+      allow(object.content_class).to receive(:respond_to?).with(:acts_as_node_fields).and_return(true)
+      expect(subject.render_content_fields_block?).to be true
+    end
+
+    context "when object content class do not respond to `acts_as_node_fields`" do
+      it "returns nil" do
+        allow(object.content_class).to receive(:respond_to?).with(:acts_as_node_fields).and_return(false)
+        expect(subject.render_content_fields_block?).to be false
+      end
+    end
+  end
+
+  describe "#render_content_fields_block" do
+    before do
+      subject.object.build_content
+    end
+
+    it "renders content fields block" do
+      allow(subject).to receive(:content_builder_class).and_return("_b_")
+      allow(subject).to receive(:render_content_fields_block?).and_return(true)
+      subform = described_class.new(:resource, object, template, {})
+      allow(subject).to receive(:fields_for).with(:content, subject.object.content, builder: "_b_").and_yield(subform)
+      allow(subform).to receive(:field_names).and_return([1, 2])
+      allow(subform).to receive(:releaf_fields).with([1, 2]).and_return("yy")
+      content = '<div class="section content-fields">yy</div>'
+      expect(subject.render_content_fields_block).to eq(content)
+    end
+
+    it "casts form fields to array before passign to `releaf_fields`" do
+      allow(subject).to receive(:content_builder_class).and_return("_b_")
+      allow(subject).to receive(:render_content_fields_block?).and_return(true)
+      subform = described_class.new(:resource, object, template, {})
+      allow(subject).to receive(:fields_for).with(:content, subject.object.content, builder: "_b_").and_yield(subform)
+      allow(subform).to receive(:field_names).and_return({a: 1, b: 2})
+      expect(subform).to receive(:releaf_fields).with([[:a, 1], [:b, 2]])
+      subject.render_content_fields_block
+    end
+
+    context "when content have no fields" do
+      it "returns nil" do
+        allow(subject).to receive(:render_content_fields_block?).and_return(false)
+        expect(subject.render_content_fields_block).to be nil
+      end
+    end
+  end
+
+  describe "#content_builder_class" do
+    it "returns `Releaf::Content::Nodes::ContentFormBuilder`" do
+      expect(subject.content_builder_class).to eq(Releaf::Content::Nodes::ContentFormBuilder)
+    end
+  end
+
+  describe "#render_locale" do
+    context "when node node has locale select enabled" do
+      it "renders locale with #render_locale_options" do
+        allow(subject.object).to receive(:locale_selection_enabled?).and_return(true)
+        allow(subject).to receive(:render_locale_options).and_return(a: "b")
+        allow(subject).to receive(:releaf_item_field).with(:locale, options: {a: "b"}).and_return("x")
+        expect(subject.render_locale).to eq("x")
+      end
+    end
+
+    context "when node node does not have locale select enabled" do
+      it "renders locale with #render_locale_options" do
+        allow(subject.object).to receive(:locale_selection_enabled?).and_return(false)
+        expect(subject.render_locale).to be nil
+      end
+    end
+  end
+
+  describe "#render_locale_options" do
+    it "returns :select_options and :include_blank values" do
+      expect(subject.render_locale_options.keys).to eq([:select_options, :include_blank])
+    end
+
+    it ":select_options contains all available locales" do
+      allow(I18n).to receive(:available_locales).and_return([:lt, :et])
+      expect(subject.render_locale_options[:select_options]).to eq([:lt, :et])
+    end
+
+    context "when subject have defined locale" do
+      it ":include_blank is false" do
+        subject.object.locale = :lt
+        expect(subject.render_locale_options[:include_blank]).to be false
+      end
+    end
+
+    context "when subject have no locale" do
+      it ":include_blank is true" do
+        subject.object.locale = nil
+        expect(subject.render_locale_options[:include_blank]).to be true
+      end
+    end
+  end
+
+  describe "#render_content_type" do
+    it "renders disabled content type field with localized content type value" do
+      options = {disabled: true, value: "Translated content type"}
+      allow(I18n).to receive(:t).with("text_page", scope: "admin.content_types").and_return("Translated content type")
+      allow(subject).to receive(:releaf_text_field).with(:content_type, input: options).and_return("x")
+      expect(subject.render_content_type).to eq("x")
+    end
+  end
+
+  describe "#render_slug" do
+    it "renders customized field" do
+      controller = Releaf::BaseController.new
+      allow(subject).to receive(:controller).and_return(controller)
+      allow(subject).to receive(:slug_base_url).and_return("http://localhost/parent")
+      allow(subject).to receive(:url_for).with(controller: "/releaf/content/nodes", action: "generate_url", parent_id: 1, id: 2)
+        .and_return("http://localhost/slug-generation-url")
+
+      content = '
+          <div class="field type-text" data-name="slug">
+              <div class="label-wrap">
+                  <label for="resource_slug">Slug</label>
+              </div>
+              <div class="value">
+                  <input value="b" class="text" data-generator-url="http://localhost/slug-generation-url" type="text" name="resource[slug]" id="resource_slug" />
+                  <button class="button only-icon secondary generate" title="Suggest slug" type="button" autocomplete="off">
+                      <i class="fa fa-keyboard-o"></i>
+                  </button>
+                  <div class="link">
+                      <a href="/a/b">http://localhost/parent<span>b</span>/</a>
+                  </div>
+             </div>
+         </div>
+      '
+
+      expect(subject.render_slug).to match_html(content)
+    end
+  end
+
+  describe "#render_item_position" do
+    it "renders locale with #item_position_options" do
+      allow(subject).to receive(:item_position_options).and_return(a: "b")
+      allow(subject).to receive(:releaf_item_field).with(:item_position, options: {a: "b"}).and_return("x")
+      expect(subject.render_item_position).to eq("x")
+    end
+  end
+
+  describe "#item_position_options" do
+    before do
+      object.item_position = 2
+      allow(subject).to receive(:item_position_select_options).and_return([["a", 1], ["b", 2], ["c", 3]])
+    end
+
+    it "returns :select_options and :include_blank values" do
+      expect(subject.item_position_options.keys).to eq([:include_blank, :select_options])
+    end
+
+    it ":select_options correct select options" do
+      options = '<option value="1">a</option>'
+      options << "\n"
+      options << '<option selected="selected" value="2">b</option>'
+      options << "\n"
+      options << '<option value="3">c</option>'
+      expect(subject.item_position_options[:select_options]).to eq(options)
+    end
+  end
+end

data/spec/builders/content/nodes/toolbox_builder_spec.rb

@@ -0,0 +1,108 @@
+require "spec_helper"
+
+describe Releaf::Content::Nodes::ToolboxBuilder, type: :class do
+
+  class NodeToolboxBuilderTestHelper < ActionView::Base
+    include Releaf::ButtonHelper
+    include Releaf::ApplicationHelper
+  end
+
+  let(:template){ NodeToolboxBuilderTestHelper.new }
+  subject { described_class.new(template) }
+
+  let(:node){ Node.new(content_type: "TextPage", slug: "a", id: 99) }
+
+  before do
+    allow(subject).to receive(:resource).and_return(node)
+    allow(subject).to receive(:destroy_confirmation_link).and_return(:super_item)
+    allow(subject).to receive(:feature_available?).with(:destroy).and_return true
+  end
+
+  it "extends Releaf::Builders::ToolboxBuilder" do
+    expect(described_class.ancestors).to include Releaf::Builders::ToolboxBuilder
+  end
+
+  describe "#items" do
+
+    before do
+      allow(subject).to receive(:params).and_return({})
+      allow(subject).to receive(:add_child_button).and_return( :add_child_item )
+      allow(subject).to receive(:go_to_button).and_return( :go_to_item )
+      allow(subject).to receive(:copy_button).and_return( :copy_item )
+      allow(subject).to receive(:move_button).and_return( :move_item )
+    end
+
+    context "when applied to a new record" do
+
+      it "returns only items returned by parent class" do
+        allow(node).to receive(:new_record?).and_return true
+        expect(subject.items).to eq([ :super_item ])
+      end
+
+    end
+
+    context "when applied to an existing record" do
+
+      before do
+        allow(node).to receive(:new_record?).and_return false
+      end
+
+      it "prepends add child, go to, copy and move items to the list returned by parent class" do
+        expect(subject.items).to eq([ :add_child_item, :go_to_item, :copy_item, :move_item, :super_item ])
+      end
+
+      context "when in index context" do
+
+        it "does not include go_to_item" do
+          allow(subject).to receive(:params).and_return({ context: "index" })
+          expect(subject.items).to eq([ :add_child_item, :copy_item, :move_item, :super_item ])
+        end
+      end
+
+    end
+
+  end
+
+  describe "item methods" do
+
+    describe "#add_child_button" do
+      it "returns an ajaxbox link to content type dialog" do
+        allow(subject).to receive(:t).with('Add child').and_return('addchildxx')
+        allow(subject).to receive(:url_for).with(action: 'content_type_dialog', parent_id: 99).and_return('dialogurl')
+        html = '<a class="button ajaxbox" title="addchildxx" href="dialogurl">addchildxx</a>'
+        expect(subject.add_child_button).to eq(html)
+      end
+    end
+
+    describe "#go_to_button" do
+      it "returns an ajaxbox link to go to dialog" do
+        allow(subject).to receive(:t).with('Go to').and_return('gotoxx')
+        allow(subject).to receive(:url_for).with(action: 'go_to_dialog').and_return('dialogurl')
+        html = '<a class="button ajaxbox" title="gotoxx" href="dialogurl">gotoxx</a>'
+        expect(subject.go_to_button).to eq(html)
+      end
+    end
+
+
+    describe "#copy_button" do
+      it "returns an ajaxbox link to copy dialog" do
+        allow(subject).to receive(:t).with('Copy').and_return('copyxx')
+        allow(subject).to receive(:url_for).with(action: 'copy_dialog', id: 99).and_return('dialogurl')
+        html = '<a class="button ajaxbox" title="copyxx" href="dialogurl">copyxx</a>'
+        expect(subject.copy_button).to eq(html)
+      end
+    end
+
+
+    describe "#move_button" do
+      it "returns an ajaxbox link to move dialog" do
+        allow(subject).to receive(:t).with('Move').and_return('movexx')
+        allow(subject).to receive(:url_for).with(action: 'move_dialog', id: 99).and_return('dialogurl')
+        html = '<a class="button ajaxbox" title="movexx" href="dialogurl">movexx</a>'
+        expect(subject.move_button).to eq(html)
+      end
+    end
+
+  end
+
+end

data/spec/controllers/releaf/content/nodes_controller_spec.rb

@@ -0,0 +1,21 @@
+require 'rails_helper'
+
+describe Releaf::Content::NodesController do
+  describe "#builder_scopes" do
+    it "adds node builder scope as first scope before default builder scopes" do
+      allow(subject).to receive(:application_scope).and_return("Admin")
+      allow(subject).to receive(:node_builder_scope).and_return("xx")
+      expect(subject.builder_scopes).to eq(["xx", "Releaf::Content::Nodes", "Admin::Builders"])
+    end
+  end
+
+  describe "#node_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.node_builder_scope).to eq("Admin::Nodes")
+
+      allow(subject).to receive(:application_scope).and_return(nil)
+      expect(subject.node_builder_scope).to eq("Nodes")
+    end
+  end
+end

data/spec/features/nodes_spec.rb

@@ -0,0 +1,239 @@
+require 'rails_helper'
+describe "Nodes", js: true, with_tree: true, with_root: true do
+  before do
+    Rails.cache.clear
+    # preload ActsAsNode classes
+    Rails.application.eager_load!
+    @user = auth_as_user
+  end
+
+  before with_root: true do
+    @lv_root = create(:home_page_node, name: "lv", locale: "lv")
+  end
+
+  before with_tree: true do
+    @how_to = create(:text_page_node, parent_id: @lv_root.id)
+    @about_us = create(:text_page_node, parent_id: @lv_root.id, name: "about us")
+    @history_node = create(:text_page_node, parent_id: @about_us.id, name: "history")
+
+    @en_root = create(:home_page_node, name: "en", locale: "en")
+
+    visit releaf_content_nodes_path
+  end
+
+  describe "new node" do
+
+    context "when creating node under root" do
+      it "creates new node in content tree" do
+        @en_root.destroy
+        click_link "Create new resource"
+        click_link "Home page"
+        expect(page).to have_css('.button',    text: 'Save')
+        expect(page).to have_no_css('.button', text: 'Save and create another')
+        create_resource do
+          fill_in "resource_name", with: "en"
+          select "en", from: "Locale"
+        end
+
+        expect(page).to have_breadcrumbs("Releaf/content/nodes", "en")
+      end
+    end
+
+    context "when creating node under another node" do
+      it "creates new node" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .toolbox-cell button').click
+        click_link "Add child"
+        click_link "Contacts controller"
+        expect(page).to have_css('.button',    text: 'Save')
+        expect(page).to have_no_css('.button', text: 'Save and create another')
+        create_resource do
+          fill_in "resource_name", with: "Contacts"
+        end
+
+        expect(page).to have_breadcrumbs("Releaf/content/nodes", "lv", "Contacts")
+      end
+    end
+  end
+
+  describe "tree collapsing" do
+    context "when not opened before" do
+      it "does not show node's children" do
+        expect(page).to have_css('li[data-id="' + @lv_root.id.to_s + '"].collapsed')
+      end
+    end
+
+    context "when clicked to uncollapse node" do
+      it "shows node children" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+
+        expect(page).to have_css('li[data-id="' + @lv_root.id.to_s + '"]:not(.collapsed)')
+      end
+
+      it "keeps opened node children visibility permanent" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        wait_for_settings_update("content.tree.expanded.#{@lv_root.id}")
+        visit releaf_content_nodes_path
+
+        expect(page).to have_css('li[data-id="' + @lv_root.id.to_s + '"]:not(.collapsed)')
+      end
+
+      it "keeps closed node children visibility permanent" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        wait_for_settings_update("content.tree.expanded.#{@lv_root.id}", false)
+        visit releaf_content_nodes_path
+
+        expect(page).to have_css('li[data-id="' + @lv_root.id.to_s + '"].collapsed')
+      end
+    end
+  end
+
+  describe "go_to node" do
+    before do
+      visit edit_releaf_content_node_path @en_root
+    end
+
+    context "when going to node from toolbox list" do
+      it "navigates to targeted node's edit view" do
+        expect(page).to have_no_header(text: 'lv')
+        open_toolbox_dialog "Go to"
+
+        click_link "lv"
+        expect(page).to have_header(text: 'lv')
+      end
+    end
+  end
+
+  describe "copy node to" do
+    context "when copying node" do
+      it "shows copied node in tree" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        find('li[data-id="' + @about_us.id.to_s + '"] > .toolbox-cell button').click
+        click_link("Copy")
+
+        within_dialog do
+          find('label > span', text: "en").click
+          click_button "Copy"
+        end
+
+        expect(page).to have_notification("Copy succeeded")
+
+        find('li[data-id="' + @en_root.id.to_s + '"] > .collapser-cell button').click
+        expect(page).to have_css('li > .node-cell a', text: "about us", count: 2)
+      end
+    end
+
+    context "when copying node under itself" do
+      it "displays flash error message" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        find('li[data-id="' + @about_us.id.to_s + '"] > .toolbox-cell button').click
+        click_link("Copy")
+
+        within_dialog do
+          find('label > span', text: "about us").click
+          click_button "Copy"
+        end
+
+        error_text = "Node with id #{@about_us.id} has error \"source or descendant node can't be parent of new node\""
+        expect(page).to have_css('.dialog .form-error-box', text: error_text)
+      end
+    end
+  end
+
+  describe "move node to", js: true do
+    context "when moving node to another parent" do
+      it "moves selected node to new position" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        find('li[data-id="' + @about_us.id.to_s + '"] > .toolbox-cell button').click
+        click_link("Move")
+
+        within_dialog do
+          find('label > span', text: "en").click
+          click_button "Move"
+        end
+
+        expect(page).to have_css('.notifications .notification .message', text: "Move succeeded")
+        expect(page).to have_css('li > .node-cell a', text: "about us", count: 0)
+        find('li[data-id="' + @en_root.id.to_s + '"] > .collapser-cell button').click
+        expect(page).to have_css('li > .node-cell a', text: "about us", count: 1)
+      end
+    end
+
+    context "when moving node under itself" do
+      it "displays flash error message" do
+        find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+        find('li[data-id="' + @about_us.id.to_s + '"] > .toolbox-cell button').click
+        click_link("Move")
+
+        within_dialog do
+          find('label > span', text: "about us").click
+          click_button "Move"
+        end
+
+        error_text = "Node with id #{@about_us.id} has error \"can't be parent to itself\" on attribute \"parent_id\""
+        expect(page).to have_css('.dialog .form-error-box', text: error_text)
+      end
+    end
+
+  end
+
+  describe "node order", with_tree: false do
+    def create_child parent, child_text, position=nil
+
+      visit releaf_content_nodes_path
+      open_toolbox_dialog 'Add child', parent, ".view-index .collection li"
+      within_dialog do
+        click_link("Text page")
+      end
+
+      create_resource do
+        fill_in 'Name', with: child_text
+        fill_in "Slug", with: child_text
+        fill_in_richtext 'Text', with: child_text
+        if position
+          select position, from: 'Item position'
+        end
+      end
+
+    end
+
+    it "creates nodes in correct order" do
+      create_child @lv_root, 'a'
+      create_child @lv_root, 'b', 'After a'
+      create_child @lv_root, 'c', 'After b'
+      create_child @lv_root, 'd', 'After b'
+      create_child @lv_root, 'e', 'First'
+
+      visit releaf_content_nodes_path
+      find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+
+      within(".collection li[data-level='1'][data-id='#{@lv_root.id}'] ul") do
+        expect( page ).to have_content 'e a b d c'
+      end
+    end
+
+    it "by default adds new nodes as last" do
+      create_child @lv_root, 'a'
+      create_child @lv_root, 'b'
+      create_child @lv_root, 'c'
+
+      visit releaf_content_nodes_path
+      find('li[data-id="' + @lv_root.id.to_s + '"] > .collapser-cell button').click
+
+      within(".collection li[data-level='1'][data-id='#{@lv_root.id}'] ul") do
+        expect( page ).to have_content 'a b c'
+      end
+    end
+  end
+
+  describe "creating node for placeholder model", with_tree: false, with_root: false, js: false do
+    it "create record in association table" do
+      allow_any_instance_of(Releaf::Content::Node::RootValidator).to receive(:validate)
+      visit new_releaf_content_node_path(content_type: 'Bundle')
+      fill_in("resource_name", with: "placeholder model node")
+      expect do
+        click_button 'Save'
+      end.to change { [Node.count, Bundle.count] }.from([0, 0]).to([1, 1])
+    end
+  end
+end