alchemy_cms 2.6.3 → 2.7.0

data/spec/helpers/pages_helper_spec.rb

@@ -32,6 +32,23 @@ module Alchemy
       end
     end
 
+    describe '#render_site_layout' do
+      let(:default_site) { Alchemy::Site.default }
+
+      it "renders the partial for current site" do
+        helper.should_receive(:current_site).and_return(default_site)
+        helper.should_receive(:render).with(default_site)
+        helper.render_site_layout
+      end
+
+      context "with missing partial" do
+        it "returns empty string and logges warning" do
+          helper.should_receive(:current_site).twice.and_return(default_site)
+          expect(helper.render_site_layout).to eq("")
+        end
+      end
+    end
+
     describe "#render_navigation" do
       before { visible_page }
 
@@ -280,6 +297,12 @@ module Alchemy
 
     describe "#render_meta_data" do
 
+      context "@page is not set" do
+        it "should reutrn nil" do
+          expect(helper.render_meta_data.should).to eq(nil)
+        end
+      end
+
       it "should render meta keywords of current page" do
         @page = mock_model('Page', :language => language, :title => 'A Public Page', :meta_description => '', :meta_keywords => 'keyword1, keyword2', :robot_index? => false, :robot_follow? => false, :contains_feed? => false, :updated_at => '2011-11-29-23:00:00')
         helper.render_meta_data.should have_selector('meta[name="keywords"][content="keyword1, keyword2"]')
@@ -388,5 +411,30 @@ module Alchemy
 
     end
 
+    describe "#picture_essence_caption" do
+      let(:essence) { mock_model('EssencePicture', caption: 'my caption') }
+      let(:content) { mock_model('Content', essence: essence) }
+
+      it "should return the caption of the contents essence" do
+        expect(helper.picture_essence_caption(content)).to eq "my caption"
+      end
+    end
+
+    describe "#language_switches" do
+      it "should return a deprecation warning" do
+        ActiveSupport::Deprecation.should_receive(:warn)
+        helper.should_receive(:language_links).and_return(nil)
+        helper.language_switches
+      end
+    end
+
+    describe "#language_switcher" do
+      it "should return a deprecation warning" do
+        ActiveSupport::Deprecation.should_receive(:warn)
+        helper.should_receive(:language_links).and_return(nil)
+        helper.language_switcher
+      end
+    end
+
   end
 end

data/spec/helpers/url_helper_spec.rb

@@ -87,6 +87,13 @@ module Alchemy
 
       end
 
+      describe "#page_path_for" do
+        it "is deprecated" do
+          ActiveSupport::Deprecation.should_receive(:warn)
+          helper.page_path_for
+        end
+      end
+
       describe "#show_alchemy_page_url" do
 
         context "when multi_language" do

data/spec/libraries/config_spec.rb

@@ -1,7 +1,114 @@
 require 'spec_helper'
 
-describe Alchemy::Config do
-  it "should have an array of fields for mailer" do
-    Alchemy::Config.get(:mailer)['fields'].should be_a(Array)
+module Alchemy
+  describe Config do
+
+    describe ".get" do
+      it "should call #show" do
+        Config.should_receive(:show).and_return({})
+        Config.get(:mailer)
+      end
+
+      it "should return the requested part of the config" do
+        Config.should_receive(:show).and_return({'mailer' => {'setting' => 'true'}})
+        expect(Config.get(:mailer)).to eq({'setting' => 'true'})
+      end
+    end
+
+    describe '.main_app_config' do
+      let(:main_app_config_path) { "#{Rails.root}/config/alchemy/config.yml" }
+
+      it "should call and return .read_file with the correct config path" do
+        Config.should_receive(:read_file).with(main_app_config_path).once.and_return({setting: 'true'})
+        expect(Config.send(:main_app_config)).to eq({setting: 'true'})
+      end
+    end
+
+    describe '.env_specific_config' do
+      let(:env_specific_config_path) { "#{Rails.root}/config/alchemy/#{Rails.env}.config.yml" }
+
+      it "should call and return .read_file with the correct config path" do
+        Config.should_receive(:read_file).with(env_specific_config_path).once.and_return({setting: 'true'})
+        expect(Config.send(:env_specific_config)).to eq({setting: 'true'})
+      end
+    end
+
+    describe ".show" do
+      context "when ivar @config was not set before" do
+        before { Config.instance_variable_set("@config", nil) }
+
+        it "should call and return .merge_configs!" do
+          Config.should_receive(:merge_configs!).once.and_return({setting: 'true'})
+          expect(Config.show).to eq({setting: 'true'})
+        end
+      end
+
+      context "when ivar @config was already set" do
+        before { Config.instance_variable_set("@config", {setting: 'true'}) }
+        after { Config.instance_variable_set("@config", nil) }
+
+        it "should have memoized the return value of .merge_configs!" do
+          expect(Config.send(:show)).to eq({setting: 'true'})
+        end
+      end
+    end
+
+    describe '.read_file' do
+      context 'when given path to yml file exists' do
+        before { File.stub!(:exists?).and_return(true) }
+
+        it 'should call YAML.load_file with the given config path' do
+          YAML.should_receive(:load_file).once.with('path/to/config.yml').and_return({})
+          Config.send(:read_file, 'path/to/config.yml')
+        end
+
+        context 'but its empty' do
+          before do
+            File.stub!(:exists?).with('empty_file.yml').and_return(true)
+            YAML.stub!(:load_file).and_return(false) # YAML.load_file returns false if file is empty.
+          end
+
+          it "should return an empty Hash" do
+            expect(Config.send(:read_file, 'empty_file.yml')).to eq({})
+          end
+        end
+      end
+
+      context 'when given path to yml file does not exist' do
+        it 'should return an empty Hash' do
+          expect(Config.send(:read_file, 'does/not/exist.yml')).to eq({})
+        end
+      end
+    end
+
+    describe '.merge_configs!' do
+      let(:config_1) do
+        {setting_1: 'same', other_setting: 'something'}
+      end
+
+      let(:config_2) do
+        {setting_1: 'same', setting_2: 'anything'}
+      end
+
+      it "should stringify the keys" do
+        expect(Config.send(:merge_configs!, config_1)).to eq(config_1.stringify_keys!)
+      end
+
+      context 'when all passed configs are empty' do
+        it "should raise an error" do
+          expect { Config.send(:merge_configs!, {}) }.to raise_error
+        end
+      end
+
+      context 'when configs containing same keys' do
+        it "should merge them together" do
+          expect(Config.send(:merge_configs!, config_1, config_2)).to eq(
+            {'setting_1' => 'same', 'other_setting' => 'something', 'setting_2' => 'anything'}
+          )
+        end
+      end
+    end
+
   end
+
 end

data/spec/libraries/essence_spec.rb

@@ -1,18 +1,38 @@
 require 'spec_helper'
 
-describe "ActsAsEssence" do
+module Alchemy
+  describe "ActsAsEssence" do
+    #let(:element) { FactoryGirl.create(:element, :name => 'headline', :create_contents_after_create => true) }
+    let(:essence) { Alchemy::EssenceText.new }
 
-  let(:element) { FactoryGirl.create(:element, :name => 'headline', :create_contents_after_create => true) }
+    describe '#ingredient=' do
+      it 'should set the value to ingredient column' do
+        essence.ingredient = 'Hallo'
+        expect(essence.ingredient).to eq('Hallo')
+      end
+    end
+
+    describe '#open_link_in_new_window?' do
+
+      subject { essence.open_link_in_new_window? }
 
-  describe '#ingredient=' do
+      context 'essence responds to link_taget' do
+        context 'if link_target attribute is set to "blank"' do
 
-    it "should set the value to ingredient column" do
-      content = element.content_by_name('headline')
-      content.essence.ingredient = 'Hallo'
-      content.essence.save
-      content.essence.ingredient.should == 'Hallo'
+          before { essence.link_target = 'blank' }
+
+          it "should return true" do
+            expect(subject).to eq(true)
+          end
+        end
+
+        context 'if link_target attribute is not "blank"' do
+          it "should return false" do
+            expect(subject).to eq(false)
+          end
+        end
+      end
     end
 
   end
-
 end

data/spec/libraries/page_layout_spec.rb

@@ -0,0 +1,134 @@
+require 'spec_helper'
+
+module Alchemy
+  describe PageLayout do
+
+    describe ".all" do
+      # skip memoization
+      before { PageLayout.instance_variable_set("@definitions", nil) }
+
+      subject { PageLayout.all }
+
+      it "should return all page_layouts" do
+        should be_instance_of(Array)
+        subject.collect { |l| l['name'] }.should include('standard')
+      end
+
+      context "with empty layouts file" do
+        before { YAML.should_receive(:load_file).and_return(false) }
+
+        it "returns empty array" do
+          should == []
+        end
+      end
+
+      context "with missing layouts file" do
+        before { File.should_receive(:exists?).and_return(false) }
+
+        it "raises error empty array" do
+          expect { subject }.to raise_error(LoadError)
+        end
+      end
+    end
+
+    describe '.add' do
+      it "adds a definition to all definitions" do
+        PageLayout.add({'name' => 'foo'})
+        PageLayout.all.should include({'name' => 'foo'})
+      end
+
+      it "adds a array of definitions to all definitions" do
+        PageLayout.add([{'name' => 'foo'}, {'name' => 'bar'}])
+        PageLayout.all.should include({'name' => 'foo'})
+        PageLayout.all.should include({'name' => 'bar'})
+      end
+    end
+
+    describe ".get" do
+      it "should return the page_layout description found by given name" do
+        PageLayout.stub(:all).and_return([{'name' => 'default'}, {'name' => 'contact'}])
+        expect(PageLayout.get('default')).to eq({'name' => 'default'})
+      end
+    end
+
+    describe '.layouts_with_own_for_select' do
+      it "should not hold a layout twice" do
+        layouts = PageLayout.layouts_with_own_for_select('standard', 1, false)
+        layouts = layouts.collect(&:last)
+        layouts.select { |l| l == "standard" }.length.should == 1
+      end
+    end
+
+    describe '.selectable_layouts' do
+      let(:language) { Language.get_default }
+      before { language }
+      subject { PageLayout.selectable_layouts(language.id) }
+
+      it "should not display hidden page layouts" do
+        subject.each { |l| l['hide'].should_not == true }
+      end
+
+      context "with already taken layouts" do
+        before {
+          PageLayout.stub(:all).and_return([{'unique' => true}])
+          Page.stub_chain(:where, :pluck).and_return([1])
+        }
+
+        it "should not include unique layouts" do
+          subject.each { |l| l['unique'].should_not == true }
+        end
+      end
+
+      context "with sites layouts present" do
+        let(:site) { Site.new }
+        let(:definitions) { [{'name' => 'default_site', 'page_layouts' => %w(intro)}] }
+        before { Site.stub(:layout_definitions).and_return(definitions) }
+
+        it "should only return layouts for site" do
+          subject.length.should == 1
+          subject.first['name'].should == 'intro'
+        end
+      end
+    end
+
+    describe ".element_names_for" do
+      it "should return all element names for the given pagelayout" do
+        PageLayout.stub(:get).with('default').and_return({'name' => 'default', 'elements' => ['element_1', 'element_2']})
+        expect(PageLayout.element_names_for('default')).to eq(['element_1', 'element_2'])
+      end
+
+      context "when given page_layout name does not exist" do
+        it "should return an empty array" do
+          expect(PageLayout.element_names_for('layout_does_not_exist!')).to eq([])
+        end
+      end
+
+      context "when page_layout description does not contain the elements key" do
+        it "should return an empty array" do
+          PageLayout.stub(:get).with('layout_without_elements_key').and_return({'name' => 'layout_without_elements_key'})
+          expect(PageLayout.element_names_for('layout_without_elements_key')).to eq([])
+        end
+      end
+    end
+
+    describe '.human_layout_name' do
+      let(:layout) { {'name' => 'contact'} }
+      subject { PageLayout.human_layout_name(layout['name']) }
+
+      context "with no translation present" do
+        it "returns the name capitalized" do
+          should == 'Contact'
+        end
+      end
+
+      context "with translation present" do
+        before { I18n.should_receive(:t).and_return('Kontakt') }
+
+        it "returns the translated name" do
+          should == 'Kontakt'
+        end
+      end
+    end
+
+  end
+end

data/spec/libraries/resource_spec.rb

@@ -1,4 +1,6 @@
-require 'spec_helper'
+require 'rspec'
+require File.dirname(__FILE__) + '/../../lib/alchemy/resource'
+require File.dirname(__FILE__) + '/../../lib/alchemy/errors'
 
 class Event
 end
@@ -152,13 +154,6 @@ module Alchemy
       end
     end
 
-    describe "#model_name" do
-      it "is deprecated" do
-        ActiveSupport::Deprecation.should_receive(:warn)
-        Resource.new("admin/events").model_name
-      end
-    end
-
     describe "#namespaced_resource_name" do
 
       it "returns resource_name with namespace (namespace_event for Namespace::Event), i.e. for use in forms" do
@@ -266,14 +261,6 @@ module Alchemy
         end
       end
 
-      describe "#namespaced_model_name" do
-        it "is deprecated" do
-          ActiveSupport::Deprecation.should_receive(:warn)
-          resource = Resource.new("admin/events")
-          resource.namespaced_model_name
-        end
-      end
-
     end
   end
 end

data/spec/libraries/resources_helper_spec.rb

@@ -1,4 +1,7 @@
-require 'spec_helper'
+require 'rspec'
+require 'ostruct'
+require File.dirname(__FILE__) + "/../../lib/alchemy/resource"
+require File.dirname(__FILE__) + "/../../lib/alchemy/resources_helper"
 
 module Namespace
   class MyResource
@@ -135,13 +138,6 @@ describe Alchemy::ResourcesHelper do
     end
   end
 
-  describe "#resource_model_name" do
-    it "is deprecated" do
-      ActiveSupport::Deprecation.should_receive(:warn)
-      @controller.resource_model_name
-    end
-  end
-
   describe "#resource_name" do
     it "returns resource_handler.resource_name" do
       @controller.resource_name.should == "my_resource"

data/spec/libraries/shell_spec.rb

@@ -1,4 +1,5 @@
 require 'spec_helper'
+require 'alchemy/shell'
 
 module Alchemy