locomotive_cms 2.0.0.rc12 → 2.0.0

data/spec/lib/locomotive/liquid/tags/paginate_spec.rb

@@ -0,0 +1,139 @@
+require 'spec_helper'
+
+describe Locomotive::Liquid::Tags::Paginate do
+
+  it 'has a valid syntax' do
+    markup = "contents.projects by 5"
+    lambda do
+      Locomotive::Liquid::Tags::Paginate.new('paginate', markup, ["{% endpaginate %}"], {})
+    end.should_not raise_error
+  end
+
+  it 'raises an error if the syntax is incorrect' do
+    ["contents.projects by a", "contents.projects", "contents.projects 5"].each do |markup|
+      lambda do
+        Locomotive::Liquid::Tags::Paginate.new('paginate', markup, ["{% endpaginate %}"], {})
+      end.should raise_error
+    end
+  end
+
+  it 'paginates the collection' do
+    template  = Liquid::Template.parse(default_template)
+    text      = template.render!(liquid_context)
+
+    text.should match /!Ruby on Rails!/
+    text.should match /!jQuery!/
+    text.should_not match /!mongodb!/
+
+    text = template.render!(liquid_context(:page => 2))
+
+    text.should_not match /!jQuery!/
+    text.should match /!mongodb!/
+    text.should match /!Liquid!/
+    text.should_not match /!sqlite3!/
+  end
+
+  it 'does not paginate if collection is nil or empty' do
+    template = Liquid::Template.parse(default_template)
+
+    lambda do
+      template.render!(liquid_context(:collection => nil))
+    end.should raise_error
+
+    lambda do
+      template.render!(liquid_context(:collection => PaginatedCollection.new))
+    end.should raise_error
+  end
+
+  it 'keeps the original GET parameters' do
+    context   = liquid_context(:fullpath => '/products?foo=1&bar=1&baz=1')
+    template  = Liquid::Template.parse(default_template)
+    text      = template.render!(context)
+    text.should match /\/products\?foo=1&bar=1&baz=1&page=2/
+  end
+
+  it 'does not include twice the page parameter' do
+    context   = liquid_context(:fullpath => '/products?page=1')
+    template  = Liquid::Template.parse(default_template)
+    text      = template.render!(context)
+    text.should match /\/products\?page=2/
+  end
+
+  it 'respects the window_size option' do
+    template  = Liquid::Template.parse(default_pagination_template('window_size: 10'))
+    text      = template.render!(liquid_context(collection: PaginatedCollection.new((1..100).to_a)))
+
+    text.should match /\/\?page=4/
+    text.should match /\/\?page=10/
+    text.should_not match /\/\?page=11/
+  end
+
+  # ___ helpers methods ___ #
+
+  def liquid_context(options = {})
+    ::Liquid::Context.new(
+      {},
+      {
+      'projects'      => options.has_key?(:collection) ? options[:collection] : PaginatedCollection.new(['Ruby on Rails', 'jQuery', 'mongodb', 'Liquid', 'sqlite3']),
+      'current_page'  => options[:page] || 1,
+      'path'          => '/',
+      'fullpath'      => options[:fullpath] || '/'
+    }, {
+      :page           => FactoryGirl.build(:page)
+    }, true)
+  end
+
+  def default_template
+    "{% paginate projects by 2 %}
+      {% for project in paginate.collection %}
+        !{{ project }}!
+      {% endfor %}
+      {{ paginate.next.url }}
+    {% endpaginate %}"
+  end
+
+  def default_pagination_template(options='')
+    "{% paginate projects by 2 #{options} %}
+      {% for project in paginate.collection %}
+        !{{ project }}!
+      {% endfor %}
+      {{ paginate.next.url }}
+      {{ paginate | default_pagination }}
+    {% endpaginate %}"
+  end
+
+  class PaginatedCollection
+
+    def initialize(collection)
+      @collection = collection || []
+    end
+
+    def paginate(options = {})
+      total_pages = (@collection.size.to_f / options[:per_page].to_f).to_f.ceil + 1
+      offset = (options[:page] - 1) * options[:per_page]
+
+      {
+        :collection     => @collection[offset..(offset + options[:per_page]) - 1],
+        :current_page   => options[:page],
+        :previous_page  => options[:page] == 1 ? 1 : options[:page] - 1,
+        :next_page      => options[:page] == total_pages ? total_pages : options[:page] + 1,
+        :total_entries  => @collection.size,
+        :total_pages    => total_pages,
+        :per_page       => options[:per_page]
+      }
+    end
+
+    def each(&block)
+      @collection.each(&block)
+    end
+
+    def method_missing(method, *args)
+      @collection.send(method, *args)
+    end
+
+    def to_liquid
+      self
+    end
+  end
+
+end

data/spec/lib/locomotive/liquid/tags/seo_spec.rb

@@ -0,0 +1,135 @@
+require 'spec_helper'
+
+describe Locomotive::Liquid::Tags::SEO do
+
+  let(:site) do
+    FactoryGirl.build(:site, :seo_title => 'Site title (SEO)', :meta_description => 'A short site description', :meta_keywords => 'test only cat dog')
+  end
+
+  describe 'rendering' do
+
+    it 'renders everything' do
+      html = render_seo
+      html.should include '<title>Site title (SEO)</title>'
+      html.should include '<meta name="description" content="A short site description">'
+      html.should include '<meta name="keywords" content="test only cat dog">'
+    end
+
+    it 'renders a seo title' do
+      render_seo_title.should include '<title>Site title (SEO)</title>'
+    end
+
+    it 'renders the site title if no seo title is provided' do
+      site.seo_title = nil
+      render_seo_title.should include '<title>Acme Website</title>'
+    end
+
+    it 'renders a meta description tag' do
+      render_seo_metadata.should include '<meta name="description" content="A short site description">'
+    end
+
+    it 'strips and removes quote characters from the description' do
+      site.meta_description = ' String with " " quotes '
+      render_seo_metadata.should include '<meta name="description" content="String with   quotes">'
+    end
+
+    it 'renders a meta keywords tag' do
+      render_seo_metadata.should include '<meta name="keywords" content="test only cat dog">'
+    end
+
+    it 'strips and removes quote characters from the keywords' do
+      site.meta_keywords = ' one " two " three '
+      render_seo_metadata.should include '<meta name="keywords" content="one  two  three">'
+    end
+
+    it 'renders an empty string if no meta' do
+      site.meta_keywords = nil
+      render_seo_metadata.should include '<meta name="keywords" content="">'
+    end
+
+    context "when page" do
+
+      context "has seo title" do
+        let(:page) { site.pages.build(:seo_title => 'Page title (SEO)', :meta_keywords => 'hulk,gamma', :meta_description => "Bruce Banner") }
+        subject { render_seo_title('page' => page) }
+        it { should include(%Q[<title>Page title (SEO)</title>]) }
+      end
+
+      context "does not have seo title" do
+        let(:page) { site.pages.build }
+        subject { render_seo_title('page' => page) }
+        it { should include(%Q[<title>Site title (SEO)</title>]) }
+      end
+
+      context "has metadata" do
+        let(:page) { site.pages.build(:meta_keywords => 'hulk,gamma', :meta_description => "Bruce Banner") }
+        subject { render_seo_metadata('page' => page) }
+        it { should include(%Q[<meta name="keywords" content="#{page.meta_keywords}">]) }
+        it { should include(%Q[<meta name="description" content="#{page.meta_description}">]) }
+      end
+
+      context "does not have metadata" do
+        let(:page) { site.pages.build }
+        subject { render_seo_metadata('page' => page) }
+        it { should include(%Q[<meta name="keywords" content="#{site.meta_keywords}">]) }
+        it { should include(%Q[<meta name="description" content="#{site.meta_description}">]) }
+      end
+
+    end
+
+    context "when content instance" do
+
+      let(:content_type) do
+        FactoryGirl.build(:content_type, :site => site).tap do |ct|
+          ct.entries_custom_fields.build :label => 'anything', :type => 'string'
+        end.tap { |_ct| _ct.valid? }
+      end
+
+      context "has seo title" do
+        let(:content_entry) { content_type.entries.build(:seo_title => 'Content title (SEO)', :meta_keywords => 'Libidinous, Angsty', :meta_description => "Quite the combination.") }
+        subject { render_seo_title('content_entry' => content_entry) }
+        it { should include(%Q[<title>Content title (SEO)</title>]) }
+      end
+
+      context "does not have seo title" do
+        let(:content_entry) { content_type.entries.build }
+        subject { render_seo_title('content_entry' => content_entry) }
+        it { should include(%Q[<title>Site title (SEO)</title>]) }
+      end
+
+      context "has metadata" do
+        let(:content_entry) { content_type.entries.build(:meta_keywords => 'Libidinous, Angsty', :meta_description => "Quite the combination.") }
+        subject { render_seo_metadata('content_entry' => content_entry) }
+        it { should include(%Q[<meta name="keywords" content="#{content_entry.meta_keywords}">]) }
+        it { should include(%Q[<meta name="description" content="#{content_entry.meta_description}">]) }
+      end
+
+      context "does not have metadata" do
+        let(:content_entry) { content_type.entries.build }
+        subject { render_seo_metadata('content_entry' => content_entry) }
+        it { should include(%Q[<meta name="keywords" content="#{site.meta_keywords}">]) }
+        it { should include(%Q[<meta name="description" content="#{site.meta_description}">]) }
+      end
+
+    end
+
+  end
+
+  def render_seo(assigns = {})
+    render_seo_tag('seo', assigns)
+  end
+
+  def render_seo_title(assigns = {})
+    render_seo_tag('seo_title', assigns)
+  end
+
+  def render_seo_metadata(assigns = {})
+    render_seo_tag('seo_metadata', assigns)
+  end
+
+  def render_seo_tag(tag_name, assigns = {})
+    registers = { :site => site }
+    liquid_context = ::Liquid::Context.new({}, assigns, registers)
+    output = Liquid::Template.parse("{% #{tag_name} %}").render(liquid_context)
+  end
+end

data/spec/lib/locomotive/liquid/tags/with_scope_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Locomotive::Liquid::Tags::WithScope do
+
+  it 'decodes basic options (boolean, integer, ...)' do
+    scope = Locomotive::Liquid::Tags::WithScope.new('with_scope', 'active:true price:42 title:\'foo\' hidden:false', ["{% endwith_scope %}"], {})
+    attributes = scope.send(:decode, scope.instance_variable_get(:@attributes), ::Liquid::Context.new)
+    attributes['active'].should == true
+    attributes['price'].should == 42
+    attributes['title'].should == 'foo'
+    attributes['hidden'].should == false
+  end
+
+  it 'decodes context variable' do
+    scope = Locomotive::Liquid::Tags::WithScope.new('with_scope', 'category: params.type', ["{% endwith_scope %}"], {})
+    attributes = scope.send(:decode, scope.instance_variable_get(:@attributes), ::Liquid::Context.new({ 'params' => { 'type' => 'posts' } }))
+    attributes['category'].should == 'posts'
+  end
+
+  it 'allows order_by option' do
+    scope = Locomotive::Liquid::Tags::WithScope.new('with_scope', 'order_by:\'name DESC\'', ["{% endwith_scope %}"], {})
+    attributes = scope.send(:decode, scope.instance_variable_get(:@attributes), ::Liquid::Context.new)
+    attributes['order_by'].should == 'name DESC'
+  end
+
+  it 'stores attributes in the context' do
+    template = ::Liquid::Template.parse("{% with_scope active:true title:'foo' %}{{ with_scope.active }}-{{ with_scope.title }}{% endwith_scope %}")
+    text = template.render
+    text.should == "true-foo"
+  end
+
+  it 'allows a variable condition inside a loop' do
+    template = ::Liquid::Template.parse("{%for i in (1..3)%}{% with_scope number: i %}{{ with_scope.number}}{% endwith_scope %}{%endfor%}")
+    text = template.render
+    text.should == "123"
+  end
+
+  describe "advanced queries thanks to h4s" do
+
+    it 'decodes criteria with gt and lt' do
+      scope = Locomotive::Liquid::Tags::WithScope.new('with_scope', 'price.gt:42.0 price.lt:50', ["{% endwith_scope %}"], {})
+      attributes = scope.send(:decode, scope.instance_variable_get(:@attributes), ::Liquid::Context.new)
+      attributes[:price.gt].should == 42.0
+      attributes[:price.lt].should == 50
+    end
+
+  end
+
+end

data/spec/lib/locomotive/presentable_spec.rb

@@ -0,0 +1,184 @@
+require 'spec_helper'
+require 'ostruct'
+
+describe Locomotive::Presentable do
+
+  before(:each) do
+    @person           = OpenStruct.new(:id => 42, :name => 'John', :age => 23)
+    @project          = OpenStruct.new(:id => 1, :title => 'LocomotiveCMS')
+
+    @presenter        = PersonPresenter.new(@person)
+    @other_presenter  = ProjectPresenter.new(@project)
+  end
+
+  it 'delegates a property to the source' do
+    @presenter.name.should == 'John'
+  end
+
+  it 'has properties from the parent' do
+    @presenter.id.should == 42
+  end
+
+  it 'does not have properties from another sibling class' do
+    @presenter.respond_to?(:title).should be_false
+    @other_presenter.respond_to?(:name).should be_false
+    @other_presenter.respond_to?(:id).should be_true
+  end
+
+  it 'uses the overridden getter' do
+    @presenter.age.should == '23 years old'
+  end
+
+  it 'can set a value and retrieve it' do
+    @presenter.age = 33
+    @presenter.age.should == '33 years old'
+  end
+
+  it 'does not add a getter if specified' do
+    lambda { @presenter.address }.should raise_exception
+  end
+
+  it 'responds to aliases' do
+    @presenter.full_address = '700 S Laflin, Chicago'
+    @person.address.should == '700 S Laflin, Chicago'
+  end
+
+  it 'returns the attributes' do
+    @presenter.as_json.should == { 'id' => 42, 'name' => 'John', 'age' => '23 years old', 'projects' => [] }
+  end
+
+  it 'sets directly the attributes' do
+    @presenter.attributes = { :name => 'henry', :age => 33 }
+    @presenter.name.should == 'Henry'
+    @presenter.age.should == '33 years old'
+  end
+
+  it 'does not call the setter if not registered' do
+    @presenter.expects(:unknown=).never
+    @presenter.attributes = { :name => 'henry', :unknown => true }
+  end
+
+  it 'handles with_options' do
+    @presenter.respond_to?(:notes).should be_true
+    @presenter.respond_to?(:notes=).should be_false
+  end
+
+  it 'stores the list of getters' do
+    @presenter.getters.should == %w(id name age projects notes retirement_place)
+  end
+
+  it 'executes custom code after setting attributes' do
+    @presenter.expects(:hello_world).once
+    @presenter.attributes = {}
+  end
+
+  describe '#if' do
+
+    before(:each) do
+      @person.retirement_place = 'Mansion'
+    end
+
+    context 'false' do
+
+      it 'does not return the property' do
+        @presenter.as_json.keys.should_not include('retirement_place')
+      end
+
+      it 'does not allow to change the property' do
+        @presenter.attributes = { :retirement_place => 'Mansion [UPDATE]' }
+        @person.retirement_place.should == 'Mansion'
+      end
+
+    end
+
+    context 'true' do
+
+      before(:each) do
+        @person.age = 61
+      end
+
+      it 'returns the property if the block returns true' do
+        @presenter.as_json.keys.should include('retirement_place')
+      end
+
+      it 'changes the property' do
+        @presenter.attributes = { :retirement_place => 'Mansion [UPDATE]' }
+        @person.retirement_place.should == 'Mansion [UPDATE]'
+      end
+
+    end
+
+  end
+
+  describe '#collection' do
+
+    before(:each) do
+      @another_project  = OpenStruct.new(:id => 2, :title => 'SocialTennis')
+      @person.projects  = [@project, @another_project]
+
+      @project.to_presenter = @other_presenter
+      @another_project.to_presenter = ProjectPresenter.new(@another_project)
+    end
+
+    it 'has a getter for the collection' do
+      @presenter.respond_to?(:projects)
+    end
+
+    it 'returns a hash as the collection' do
+      @project.expects(:as_json).returns({ 'id' => 1 }).once
+      @another_project.expects(:as_json).returns({ 'id' => 2 }).once
+      @presenter.projects.should == [{ 'id' => 1 }, { 'id' => 2 }]
+    end
+
+  end
+
+  ## classes for tests ##
+
+  class BasePresenter
+
+    include Locomotive::Presentable
+
+    property :id
+
+  end
+
+  class PersonPresenter < BasePresenter
+
+    properties  :name, :age
+    property    :address, :only_setter => true, :alias => :full_address
+
+    collection  :projects
+
+    with_options :only_getter => true, :allow_nil => false do |presenter|
+      presenter.property :notes
+    end
+
+    property :retirement_place, :if => Proc.new { __source.age > 60 }
+
+    set_callback :set_attributes, :after, :hello_world
+
+    def name=(value)
+      self.__source.name = value.capitalize
+    end
+
+    def unknown=(value)
+      raise 'it should never happen'
+    end
+
+    def age
+      "#{self.__source.age} years old"
+    end
+
+    def hello_world
+      # do nothing
+    end
+
+  end
+
+  class ProjectPresenter < BasePresenter
+
+    property  :title
+
+  end
+
+end