releaf-content 0.2.1 → 1.0.3

data/spec/services/releaf/content/node/copy_spec.rb

@@ -0,0 +1,115 @@
+require "rails_helper"
+
+describe Releaf::Content::Node::Copy do
+  class DummyNodeServiceIncluder
+    include Releaf::Content::Node::Service
+  end
+
+  let(:node){ Node.new }
+  subject{ described_class.new(node: node, parent_id: 12) }
+
+  describe "#duplicate_content" do
+    let(:node) { create(:node) }
+
+    context "when content_id is blank" do
+      it "returns nil" do
+        expect( subject.duplicate_content ).to be nil
+      end
+    end
+
+    context "when content_id is not blank" do
+      let(:node) { create(:home_page_node) }
+
+      it "returns saved duplicated content" do
+        content = HomePage.new
+        expect( content ).to receive(:save!)
+        expect( node.content.class ).to receive(:new).and_return(content)
+        expect( subject.duplicate_content ).to eq content
+      end
+
+      it "reassigns dragonfly accessors" do
+        content = HomePage.new
+        expect( content ).to receive(:save!)
+        allow( node.content.class ).to receive(:new).and_return(content)
+        expect( subject ).to receive(:duplicate_content_dragonfly_attributes).with(content)
+        expect( subject.duplicate_content ).to eq content
+      end
+
+      it "doesn't return same as original content" do
+        result = subject.duplicate_content
+        expect( result ).to be_an_instance_of HomePage
+        expect( result.id ).to_not eq node.content_id
+      end
+    end
+  end
+
+  describe "#cloned_content_attributes" do
+    it "returns attributes without id and dragonfly attributes" do
+      node.content = HomePage.new(id: 42, banner_uid: "re", intro_text_html: "some text")
+
+      expect(subject.cloned_content_attributes).to have_key("intro_text_html")
+      expect(subject.cloned_content_attributes).not_to include(:id, :banner_uid)
+    end
+  end
+
+  describe "#duplicate_content_dragonfly_attributes" do
+    context "when dragonfly file is present" do
+      it "reassigns dragonfly accessors to given content instance from node content" do
+        old_content = HomePage.create(banner: File.new("releaf-core/spec/fixtures/cs.png"))
+        new_content = HomePage.new()
+        node.content = old_content
+
+        expect(new_content).to receive(:banner=).with(old_content.banner)
+        expect(new_content).to receive(:banner_uid=).with(nil)
+        subject.duplicate_content_dragonfly_attributes(new_content)
+      end
+    end
+
+    context "when dragonfly file is not present" do
+      it "doesn't reassigns dragonfly accessors to given content instance from node content" do
+        old_content = HomePage.new(banner_uid: "yy")
+        new_content = HomePage.new()
+        node.content = old_content
+
+        expect(new_content).to receive(:banner=).with(nil)
+        expect(new_content).to receive(:banner_uid=).with(nil)
+        subject.duplicate_content_dragonfly_attributes(new_content)
+      end
+    end
+  end
+
+  describe "#content_dragonfly_attributes" do
+    it "returns array of node content object dragonfly attributes" do
+      node.content = HomePage.new
+      expect(subject.content_dragonfly_attributes).to eq(["banner_uid"])
+    end
+  end
+
+  describe "#duplicate_under" do
+    let!(:source_node) { create(:node, locale: "lv") }
+    let!(:target_node) { create(:node, locale: "en") }
+
+    before do
+      allow_any_instance_of(Releaf::Content::Node::RootValidator).to receive(:validate)
+      subject.node = source_node
+      subject.parent_id = target_node.id
+    end
+
+    it "creates duplicated node under target node" do
+      new_node = Node.new
+      duplicated_content = double('content', id: 1234)
+      expect( Node ).to receive(:new).ordered.and_return(new_node)
+      expect( new_node ).to receive(:assign_attributes_from).with(source_node).ordered.and_call_original
+      expect( subject ).to receive(:duplicate_content).ordered.and_return(duplicated_content)
+      expect( new_node ).to receive(:content_id=).with(1234).ordered
+      expect( Releaf::Content::Node::SaveUnderParent ).to receive(:call).with(node: new_node, parent_id: target_node.id)
+        .ordered.and_call_original
+      expect(subject.duplicate_under).to eq(new_node)
+    end
+
+    it "doesn't update settings timestamp" do
+      expect( Node ).to_not receive(:updated)
+      subject.duplicate_under
+    end
+  end
+end

data/spec/services/releaf/content/node/move_spec.rb

@@ -0,0 +1,20 @@
+require "rails_helper"
+
+describe Releaf::Content::Node::Move do
+  class DummyNodeServiceIncluder
+    include Releaf::Content::Node::Service
+  end
+
+  let(:node){ Node.new }
+  subject{ described_class.new(node: node, parent_id: 12) }
+
+  describe "#call" do
+    context "when parent is same" do
+      it "does nothing" do
+        node.parent_id = 12
+        expect(node.class).to_not receive(:transaction)
+        subject.call
+      end
+    end
+  end
+end

data/spec/services/releaf/content/node/save_under_parent_spec.rb

@@ -0,0 +1,49 @@
+require "rails_helper"
+
+describe Releaf::Content::Node::SaveUnderParent do
+  class DummyNodeServiceIncluder
+    include Releaf::Content::Node::Service
+  end
+
+  let(:root_node){ create(:node, locale: "lv") }
+  let(:node){ build(:text_page_node) }
+  subject{ described_class.new(node: node, parent_id: root_node.id) }
+
+  describe "#call" do
+    it "saves node nuder node with given node_id" do
+      node2 = create(:text_page_node, parent: root_node)
+
+      subject.parent_id = node2.id
+      subject.call
+      expect( node ).to_not be_new_record
+      expect( node.parent ).to eq node2
+    end
+
+    it "maintains node name, updates slug and then saves record" do
+      expect( node ).to receive(:maintain_name).ordered.and_call_original
+      expect( node ).to receive(:reasign_slug).ordered.and_call_original
+      expect( node ).to receive(:save!).ordered.and_call_original
+      subject.call
+    end
+
+    context "when #validate_root_locale_uniqueness? returns true" do
+      let(:node){ root_node }
+
+      it "sets locale to nil" do
+        allow(node).to receive(:validate_root_locale_uniqueness?).and_return(true)
+        subject.parent_id = nil
+        expect { subject.call }.to change{ node.locale }.from("lv").to(nil)
+      end
+    end
+
+    context "when #validate_root_locale_uniqueness? returns false" do
+      let(:node){ root_node }
+
+      it "doesn't set locale to nil" do
+        allow(node).to receive(:validate_root_locale_uniqueness?).and_return(false)
+        subject.parent_id = nil
+        expect { subject.call }.to_not change{ node.locale }
+      end
+    end
+  end
+end

data/spec/services/releaf/content/node/service_spec.rb

@@ -0,0 +1,19 @@
+require "rails_helper"
+
+describe Releaf::Content::Node::Service do
+  class DummyNodeServiceIncluder
+    include Releaf::Content::Node::Service
+  end
+
+  let(:node){ Node.new }
+  subject{ DummyNodeServiceIncluder.new(node: node) }
+
+  describe "#add_error_and_raise" do
+    it "adds errors to base and raise validation error" do
+      expect{ subject.add_error_and_raise("shit happen") }.to raise_error do |exception|
+        expect(exception).to be_instance_of ActiveRecord::RecordInvalid
+        expect(exception.record.errors[:base]).to eq(["shit happen"])
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: releaf-content
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 1.0.3
 platform: ruby
 authors:
 - CubeSystems
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-26 00:00:00.000000000 Z
+date: 2016-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: releaf-core
@@ -16,42 +16,42 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: 1.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: 1.0.3
 - !ruby/object:Gem::Dependency
   name: stringex
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.6'
 - !ruby/object:Gem::Dependency
   name: awesome_nested_set
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.1'
 description: Content subsystem for releaf
 email: info@cubesystems.lv
 executables: []
@@ -59,14 +59,13 @@ extensions: []
 extra_rdoc_files: []
 files:
 - LICENSE
-- app/assets/javascripts/releaf/controllers/releaf/content/nodes.js
-- app/assets/stylesheets/releaf/controllers/releaf/content/nodes.scss
+- app/assets/javascripts/controllers/releaf/content/nodes.js
+- app/assets/stylesheets/controllers/releaf/content/nodes.scss
 - app/builders/releaf/content/builders/action_dialog.rb
 - app/builders/releaf/content/builders/dialog.rb
 - app/builders/releaf/content/builders/tree.rb
 - app/builders/releaf/content/content_type_dialog_builder.rb
 - app/builders/releaf/content/copy_dialog_builder.rb
-- app/builders/releaf/content/go_to_dialog_builder.rb
 - app/builders/releaf/content/move_dialog_builder.rb
 - app/builders/releaf/content/nodes/content_form_builder.rb
 - app/builders/releaf/content/nodes/form_builder.rb
@@ -74,39 +73,50 @@ files:
 - app/builders/releaf/content/nodes/toolbox_builder.rb
 - app/controllers/releaf/content/nodes_controller.rb
 - app/middleware/releaf/content/routes_reloader.rb
+- app/services/releaf/content/node/copy.rb
+- app/services/releaf/content/node/move.rb
+- app/services/releaf/content/node/save_under_parent.rb
+- app/services/releaf/content/node/service.rb
 - app/validators/releaf/content/node/parent_validator.rb
 - app/validators/releaf/content/node/root_validator.rb
 - app/validators/releaf/content/node/singleness_validator.rb
 - app/views/releaf/content/nodes/content_type_dialog.ruby
 - app/views/releaf/content/nodes/copy_dialog.ruby
-- app/views/releaf/content/nodes/go_to_dialog.ruby
 - app/views/releaf/content/nodes/move_dialog.ruby
 - lib/releaf-content.rb
 - lib/releaf/content/acts_as_node.rb
 - lib/releaf/content/acts_as_node/action_controller/acts/node.rb
 - lib/releaf/content/acts_as_node/active_record/acts/node.rb
-- lib/releaf/content/builders_autoload.rb
+- lib/releaf/content/configuration.rb
 - lib/releaf/content/engine.rb
 - lib/releaf/content/node.rb
 - lib/releaf/content/node_mapper.rb
 - lib/releaf/content/route.rb
 - lib/releaf/content/router_proxy.rb
-- releaf-content.gemspec
+- spec/builders/content/nodes/action_dialog_spec.rb
 - spec/builders/content/nodes/content_form_builder_spec.rb
 - spec/builders/content/nodes/form_builder_spec.rb
 - spec/builders/content/nodes/toolbox_builder_spec.rb
 - spec/controllers/releaf/content/nodes_controller_spec.rb
+- spec/features/nodes_services_spec.rb
 - spec/features/nodes_spec.rb
 - spec/lib/releaf/content/acts_as_node_spec.rb
+- spec/lib/releaf/content/configuration_spec.rb
+- spec/lib/releaf/content/engine_spec.rb
 - spec/lib/releaf/content/node_spec.rb
 - spec/lib/releaf/content/route_spec.rb
 - spec/middleware/routes_reloader_spec.rb
 - spec/routing/node_mapper_spec.rb
+- spec/services/releaf/content/node/copy_spec.rb
+- spec/services/releaf/content/node/move_spec.rb
+- spec/services/releaf/content/node/save_under_parent_spec.rb
+- spec/services/releaf/content/node/service_spec.rb
 - spec/validators/content/node/parent_validator_spec.rb
 - spec/validators/content/node/root_validator_spec.rb
 - spec/validators/content/node/singleness_validator_spec.rb
 homepage: https://github.com/cubesystems/releaf
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -124,22 +134,29 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Node and content routes support for releaf
 test_files:
+- spec/builders/content/nodes/action_dialog_spec.rb
 - spec/builders/content/nodes/content_form_builder_spec.rb
 - spec/builders/content/nodes/form_builder_spec.rb
 - spec/builders/content/nodes/toolbox_builder_spec.rb
 - spec/controllers/releaf/content/nodes_controller_spec.rb
+- spec/features/nodes_services_spec.rb
 - spec/features/nodes_spec.rb
 - spec/lib/releaf/content/acts_as_node_spec.rb
+- spec/lib/releaf/content/configuration_spec.rb
+- spec/lib/releaf/content/engine_spec.rb
 - spec/lib/releaf/content/node_spec.rb
 - spec/lib/releaf/content/route_spec.rb
 - spec/middleware/routes_reloader_spec.rb
 - spec/routing/node_mapper_spec.rb
+- spec/services/releaf/content/node/copy_spec.rb
+- spec/services/releaf/content/node/move_spec.rb
+- spec/services/releaf/content/node/save_under_parent_spec.rb
+- spec/services/releaf/content/node/service_spec.rb
 - spec/validators/content/node/parent_validator_spec.rb
 - spec/validators/content/node/root_validator_spec.rb
 - spec/validators/content/node/singleness_validator_spec.rb
-has_rdoc: 

data/app/builders/releaf/content/go_to_dialog_builder.rb

@@ -1,9 +0,0 @@
-module Releaf::Content
-  class GoToDialogBuilder
-    include Releaf::Content::Builders::Dialog
-
-    def section_header_text
-      t("Go to node")
-    end
-  end
-end

data/app/views/releaf/content/nodes/go_to_dialog.ruby

@@ -1 +0,0 @@
-Releaf::Content::GoToDialogBuilder.new(self).output

data/lib/releaf/content/builders_autoload.rb

@@ -1,18 +0,0 @@
-root_path = File.expand_path('../..', File.dirname(__dir__))
-files = %w(
-  builders/tree
-  builders/dialog
-  builders/action_dialog
-  content_type_dialog_builder
-  copy_dialog_builder
-  copy_dialog_builder
-  go_to_dialog_builder
-  move_dialog_builder
-  nodes/content_form_builder
-  nodes/form_builder
-  nodes/index_builder
-  nodes/toolbox_builder
-)
-files.each do|file|
- require "#{root_path}/app/builders/releaf/content/#{file}"
-end

data/releaf-content.gemspec

@@ -1,20 +0,0 @@
-require File.expand_path("../../releaf-core/lib/releaf/version.rb", __FILE__)
-
-Gem::Specification.new do |s|
-  s.name        = "releaf-content"
-  s.version     = Releaf::VERSION
-
-  s.summary     = "Node and content routes support for releaf"
-  s.description = "Content subsystem for releaf"
-  s.authors     = ["CubeSystems"]
-  s.email       = 'info@cubesystems.lv'
-  s.homepage    = 'https://github.com/cubesystems/releaf'
-
-  s.files             = `git ls-files`.split("\n")
-  s.test_files = Dir["spec/**/*"]
-
-  s.add_dependency 'releaf-core', Releaf::VERSION
-  s.add_dependency 'stringex'
-  s.add_dependency 'awesome_nested_set'
-
-end