locomotive_cms 2.0.0.rc12 → 2.0.0

data/spec/mailers/locomotive/notifications_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe Locomotive::Notifications do
+
+  describe 'new_content_entry' do
+
+    let(:site) { FactoryGirl.build(:site, :domains => %w{www.acme.com}) }
+
+    let(:account) { FactoryGirl.build(:account) }
+
+    let(:content_type) { FactoryGirl.build(:content_type, :site => site) }
+
+    let(:content_entry) { FactoryGirl.build(:content_entry, :content_type => content_type, :site => site) }
+
+    let(:mail) { Locomotive::Notifications.new_content_entry(account, content_entry) }
+
+    it 'renders the subject' do
+      mail.subject.should == '[www.acme.com][My project] new entry'
+    end
+
+    it 'renders the receiver email' do
+      mail.to.should == ['bart@simpson.net']
+    end
+
+    it 'renders the sender email' do
+      mail.from.should == ['support@dummy.com']
+    end
+
+    it 'outputs the domain in the email body' do
+      mail.body.encoded.should match('<b>www.acme.com</b>')
+    end
+
+    it 'outputs the description of the content type in the email body' do
+      mail.body.encoded.should match('The list of my projects')
+    end
+
+    context 'multi-site not enabled' do
+
+      before(:each) do
+        Locomotive.config.stubs(:multi_sites_or_manage_domains?).returns(false)
+      end
+
+      it 'renders the subject with the domain name coming from the ActionMailer settings' do
+        ActionMailer::Base.default_url_options = { host: 'www.acme.fr' }
+        mail.subject.should == '[www.acme.fr][My project] new entry'
+      end
+
+      it 'renders the subject without setting ActionMailer' do
+        ActionMailer::Base.default_url_options[:host] = nil
+        mail.subject.should == '[localhost][My project] new entry'
+      end
+
+    end
+
+  end
+
+end

data/spec/models/locomotive/ability_spec.rb

@@ -0,0 +1,132 @@
+require 'spec_helper'
+
+describe Locomotive::Ability do
+
+  before :each do
+    @site = FactoryGirl.create(:site)
+    @account = FactoryGirl.create(:account)
+
+    @admin    = FactoryGirl.create(:membership, :account => FactoryGirl.build(:account), :site => FactoryGirl.build(:site))
+    @designer = FactoryGirl.create(:membership, :account => FactoryGirl.build(:account), :site => @site, :role => %(designer))
+    @author   = FactoryGirl.create(:membership, :account => FactoryGirl.build(:account), :site => @site, :role => %(author))
+  end
+
+  context 'pages' do
+
+    subject { Locomotive::Page.new }
+
+    context 'management' do
+      it 'should allow management of pages from (admin, designer, author)' do
+        should     allow_permission_from :manage, @admin
+        should     allow_permission_from :manage, @designer
+        should_not allow_permission_from :manage, @author
+      end
+    end
+
+    context 'touching' do
+      it 'should allow touching of pages from (author)' do
+        should allow_permission_from :touch, @author
+      end
+    end
+
+  end
+
+  context 'content instance' do
+
+    subject { Locomotive::ContentEntry.new }
+
+    context 'management' do
+      it 'should allow management of pages from (admin, designer, author)' do
+        should allow_permission_from :manage, @admin
+        should allow_permission_from :manage, @designer
+        should allow_permission_from :manage, @author
+      end
+    end
+
+  end
+
+  context 'content type' do
+
+    subject { Locomotive::ContentType.new }
+
+    context 'management' do
+      it 'should allow management of pages from (admin, designer)' do
+        should     allow_permission_from :manage, @admin
+        should     allow_permission_from :manage, @designer
+        should_not allow_permission_from :manage, @author
+      end
+    end
+
+    # context 'touching' do
+    #   it 'should allow touching of pages from (author)' do
+    #     should_not allow_permission_from :touch, @author
+    #   end
+    # end
+
+  end
+
+  context 'theme assets' do
+
+    subject { Locomotive::ThemeAsset.new }
+
+    context 'management' do
+      it 'should allow management of pages from (admin, designer)' do
+        should     allow_permission_from :manage, @admin
+        should     allow_permission_from :manage, @designer
+        should_not allow_permission_from :manage, @author
+      end
+    end
+
+    context 'touching' do
+      it 'should allow touching of pages from (author)' do
+        should allow_permission_from :touch, @author
+      end
+    end
+
+  end
+
+  context 'site' do
+
+    subject { Locomotive::Site.new }
+
+    context 'management' do
+      it 'should allow management of pages from (admin)' do
+        should     allow_permission_from :manage, @admin
+        should_not allow_permission_from :manage, @designer
+        should_not allow_permission_from :manage, @author
+      end
+    end
+
+    context 'pointing' do
+      it 'should allow importing of sites from (designer)' do
+        should     allow_permission_from :point, @designer
+        should_not allow_permission_from :point, @author
+      end
+    end
+
+  end
+
+  context 'membership' do
+
+    subject { Locomotive::Membership.new }
+
+    context 'management' do
+      it 'should allow management of memberships from (admin, designer)' do
+        should     allow_permission_from :manage, @admin
+        should     allow_permission_from :manage, @designer
+        should_not allow_permission_from :manage, @author
+      end
+    end
+
+    context 'granting admin' do
+      it 'should allow only admins to grant admin role' do
+        should     allow_permission_from :grant_admin, @admin
+        should_not allow_permission_from :grant_admin, @designer
+        should_not allow_permission_from :grant_admin, @author
+      end
+
+    end
+
+  end
+
+end

data/spec/models/locomotive/account_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+describe Locomotive::Account do
+  let!(:existing_account) { FactoryGirl.create(:account, :email => 'another@email.com') }
+
+  it 'has a valid factory' do
+    FactoryGirl.build(:account).should be_valid
+  end
+
+  ## Validations ##
+  it { should validate_presence_of :name }
+  it { should validate_presence_of :email }
+  it { should validate_presence_of :password }
+  it { should validate_uniqueness_of(:email).with_message(/is already taken/) }
+  it { should allow_value('valid@email.com').for(:email) }
+  it { should allow_value('prefix+suffix@email.com').for(:email) }
+  it { should_not allow_value('not-an-email').for(:email) }
+
+  it "has a default locale" do
+    account = Locomotive::Account.new
+    account.locale.should == 'en'
+  end
+
+  it "validates the uniqueness of email" do
+    FactoryGirl.create(:account)
+    (account = FactoryGirl.build(:account)).should_not be_valid
+    account.errors[:email].should == ["is already taken"]
+  end
+
+  ## Associations ##
+
+  it 'owns many sites' do
+    account = FactoryGirl.create(:account)
+    site_1  = FactoryGirl.create(:site, :memberships => [Locomotive::Membership.new(:account => account)])
+    site_2  = FactoryGirl.create(:site, :subdomain => 'another_one', :memberships => [Locomotive::Membership.new(:account => account)])
+    sites   = [site_1, site_2].map(&:_id)
+    account.reload.sites.all? { |s| sites.include?(s._id) }.should be_true
+  end
+
+  describe 'deleting' do
+
+    before(:each) do
+      @account = FactoryGirl.build(:account)
+      @site_1 = FactoryGirl.build(:site,:memberships => [FactoryGirl.build(:membership, :account => @account)])
+      @site_2 = FactoryGirl.build(:site,:memberships => [FactoryGirl.build(:membership, :account => @account)])
+      @account.stubs(:sites).returns([@site_1, @site_2])
+      Locomotive::Site.any_instance.stubs(:save).returns(true)
+    end
+
+    it 'also deletes memberships' do
+      Locomotive::Site.any_instance.stubs(:admin_memberships).returns(['junk', 'dirt'])
+      @site_1.memberships.first.expects(:destroy)
+      @site_2.memberships.first.expects(:destroy)
+      @account.destroy
+    end
+
+    it 'raises an exception if account is the only remaining admin' do
+      @site_1.memberships.first.stubs(:admin?).returns(true)
+      @site_1.stubs(:admin_memberships).returns(['junk'])
+      lambda {
+        @account.destroy
+      }.should raise_error(Exception, "One admin account is required at least")
+    end
+
+  end
+
+  describe '#admin?' do
+
+    it 'is considered as an admin if she/he has a membership with an admin role' do
+      create_site_and_account
+      @account.admin?.should be_true
+    end
+
+    it 'is not considered as an admin if she/he does not have a membership with an admin role' do
+      create_site_and_account('author')
+      @account.admin?.should be_false
+    end
+
+    def create_site_and_account(role = 'admin')
+      @account    = FactoryGirl.create(:account)
+      @membership = Locomotive::Membership.new(:account => @account, :role => role)
+      @site       = FactoryGirl.create(:site, :memberships => [@membership])
+    end
+
+  end
+
+end

data/spec/models/locomotive/content_asset_spec.rb

@@ -0,0 +1,46 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe Locomotive::ContentAsset do
+
+  describe 'attaching a file' do
+
+    before(:each) do
+      Locomotive::ContentAsset.any_instance.stubs(:site_id).returns('test')
+      @asset = FactoryGirl.build(:asset)
+    end
+
+    it 'should process picture' do
+      @asset.source = FixturedAsset.open('5k.png')
+      @asset.source.file.content_type.should_not be_nil
+      @asset.image?.should be_true
+    end
+
+    it 'should get width and height from the image' do
+      @asset.source = FixturedAsset.open('5k.png')
+      @asset.width.should == 32
+      @asset.height.should == 32
+    end
+
+  end
+
+  describe 'vignette' do
+
+    before(:each) do
+      @asset = FactoryGirl.build(:asset, :source => FixturedAsset.open('5k.png'))
+    end
+
+    it 'does not resize image smaller than 50x50' do
+      @asset.vignette_url.should =~ /^\/spec\/.*\/5k.png/
+    end
+
+    it 'has any possible resized versions' do
+      @asset.stubs(:with).returns(90)
+      @asset.stubs(:height).returns(90)
+      @asset.vignette_url.should =~ /^\/images\/dynamic\/.*\/5k.png/
+    end
+
+  end
+
+end

data/spec/models/locomotive/content_entry_spec.rb

@@ -0,0 +1,269 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Locomotive::ContentEntry do
+
+  before(:each) do
+    Locomotive::Site.any_instance.stubs(:create_default_pages!).returns(true)
+    @content_type = FactoryGirl.build(:content_type)
+    @content_type.entries_custom_fields.build :label => 'Title', :type => 'string'
+    @content_type.entries_custom_fields.build :label => 'Description', :type => 'text'
+    @content_type.entries_custom_fields.build :label => 'Visible ?', :type => 'boolean', :name => 'visible'
+    @content_type.entries_custom_fields.build :label => 'File', :type => 'file'
+    @content_type.valid?
+    @content_type.send(:set_label_field)
+  end
+
+  describe '#validation' do
+
+    it 'is valid' do
+      build_content_entry.should be_valid
+    end
+
+    ## Validations ##
+
+    it 'requires the presence of title' do
+      content_entry = build_content_entry :title => nil
+      content_entry.should_not be_valid
+      content_entry.errors[:title].should == ["can't be blank"]
+    end
+
+    it 'requires the presence of the permalink (_slug)' do
+      content_entry = build_content_entry :title => nil
+      content_entry.should_not be_valid
+      content_entry.errors[:_slug].should == ["can't be blank"]
+    end
+
+  end
+
+  context 'setting the slug' do
+    before :each do
+      build_content_entry(:_slug => 'dogs').tap(&:save!)._slug.should == 'dogs'
+    end
+
+    it 'uses the given slug if it is unique' do
+      build_content_entry(:_slug => 'monkeys').tap(&:save!)._slug.should == 'monkeys'
+      build_content_entry(:_slug => 'cats-2').tap(&:save!)._slug.should == 'cats-2'
+    end
+
+    it 'appends a number to the end of the slug if it is not unique' do
+      build_content_entry(:_slug => 'dogs').tap(&:save!)._slug.should == 'dogs-1'
+      build_content_entry(:_slug => 'dogs').tap(&:save!)._slug.should == 'dogs-2'
+      build_content_entry(:_slug => 'dogs-2').tap(&:save!)._slug.should == 'dogs-3'
+      build_content_entry(:_slug => 'dogs-2').tap(&:save!)._slug.should == 'dogs-4'
+    end
+
+    it 'ignores the case of a slug' do
+      build_content_entry(:_slug => 'dogs').tap(&:save!)._slug.should == 'dogs-1'
+      build_content_entry(:_slug => 'DOGS').tap(&:save!)._slug.should == 'dogs-2'
+    end
+
+    it 'correctly handles slugs with multiple numbers' do
+      build_content_entry(:_slug => 'fish-1-2').tap(&:save!)._slug.should == 'fish-1-2'
+      build_content_entry(:_slug => 'fish-1-2').tap(&:save!)._slug.should == 'fish-1-3'
+
+      build_content_entry(:_slug => 'fish-1-hi').tap(&:save!)._slug.should == 'fish-1-hi'
+      build_content_entry(:_slug => 'fish-1-hi').tap(&:save!)._slug.should == 'fish-1-hi-1'
+    end
+  end
+
+  context '#i18n' do
+
+    before(:each) do
+      localize_content_type @content_type
+      ::Mongoid::Fields::I18n.locale = 'en'
+      @content_entry = build_content_entry(:title => 'Hello world')
+      ::Mongoid::Fields::I18n.locale = 'fr'
+    end
+
+    after(:all) do
+      ::Mongoid::Fields::I18n.locale = 'en'
+    end
+
+    it 'tells if an entry has been translated or not' do
+      @content_entry.translated?.should be_false
+      @content_entry.title = 'Bonjour'
+      @content_entry.translated?.should be_true
+    end
+
+  end
+
+  describe "#navigation" do
+    before(:each) do
+      @content_type.order_by = '_position'
+      @content_type.save
+
+      %w(first second third).each_with_index do |item, index|
+        content = build_content_entry(:title => item.to_s, :_position => index)
+        content.save
+        instance_variable_set "@#{item}", content
+      end
+    end
+
+    it 'should find previous item when available' do
+      @second.previous.title.should == 'first'
+      @second.previous._position.should == 1
+    end
+
+    it 'should find next item when available' do
+      @second.next.title.should == 'third'
+      @second.next._position.should == 3
+    end
+
+    it 'should return nil when fetching previous item on first in list' do
+      @first.previous.should == nil
+    end
+
+    it 'should return nil when fetching next item on last in list' do
+      @third.next.should == nil
+    end
+  end
+
+  describe '#permalink' do
+
+    before(:each) do
+      @content_entry = build_content_entry
+    end
+
+    it 'has a default value based on the highlighted field' do
+      @content_entry.send(:set_slug)
+      @content_entry._permalink.should == 'locomotive'
+    end
+
+    it 'is empty if no value for the highlighted field is provided' do
+      @content_entry.title = nil; @content_entry.send(:set_slug)
+      @content_entry._permalink.should be_nil
+    end
+
+    it 'includes dashes instead of white spaces' do
+      @content_entry.title = 'my content instance'; @content_entry.send(:set_slug)
+      @content_entry._permalink.should == 'my-content-instance'
+    end
+
+    it 'removes accentued characters' do
+      @content_entry.title = "une chèvre dans le pré"; @content_entry.send(:set_slug)
+      @content_entry._permalink.should == 'une-chevre-dans-le-pre'
+    end
+
+    it 'removes dots' do
+      @content_entry.title = "my.test"; @content_entry.send(:set_slug)
+      @content_entry._permalink.should == 'my-dot-test'
+    end
+
+    it 'accepts non-latin chars' do
+      @content_entry.title = "абракадабра"; @content_entry.send(:set_slug)
+      @content_entry._permalink.should == 'abrakadabra'
+    end
+
+    it 'also accepts a file field as the highlighted field' do
+      @content_entry.stubs(:_label_field_name).returns('file')
+      @content_entry.file = FixturedAsset.open('5k.png'); @content_entry.send(:set_slug)
+      @content_entry._permalink.should == '5k'
+    end
+
+  end
+
+  describe '#visibility' do
+
+    before(:each) do
+      @content_entry = build_content_entry
+    end
+
+    it 'is not visible by default' do
+      @content_entry.send(:set_visibility)
+      @content_entry.visible?.should be_false
+    end
+
+    it 'can be visible even if it is nil' do
+      @content_entry.visible = nil
+      @content_entry.send(:set_visibility)
+      @content_entry.visible?.should be_true
+    end
+
+    it 'can not be visible' do
+      @content_entry.visible = false
+      @content_entry.send(:set_visibility)
+      @content_entry.visible?.should be_false
+    end
+
+  end
+
+  describe '#label' do
+
+    it 'has a label based on the value of the first field' do
+      build_content_entry._label.should == 'Locomotive'
+    end
+
+    it 'uses the to_label method if the value of the label field defined it' do
+      entry = build_content_entry(:_label_field_name => 'with_to_label')
+      entry.stubs(:with_to_label).returns(mock('with_to_label', :to_label => 'acme'))
+      entry._label.should == 'acme'
+    end
+
+    it 'uses the to_s method at last if the label field did not define the to_label method' do
+      entry = build_content_entry(:_label_field_name => 'not_a_string')
+      entry.stubs(:not_a_string).returns(mock('not_a_string', :to_s => 'not_a_string'))
+      entry._label.should == 'not_a_string'
+    end
+
+  end
+
+  describe '#public_submission' do
+
+    before(:each) do
+      @account_1 = FactoryGirl.build('admin user', :id => fake_bson_id('1'))
+      @account_2 = FactoryGirl.build('frenchy user', :id => fake_bson_id('2'))
+
+      @content_type.public_submission_enabled = true
+      @content_type.public_submission_accounts = ['', @account_1._id, @account_2._id.to_s]
+
+      site = FactoryGirl.build(:site)
+      site.stubs(:accounts).returns([@account_1, @account_2])
+
+      @content_entry = build_content_entry(:site => site)
+    end
+
+    it 'does not send email notifications if the api is disabled' do
+      @content_type.public_submission_enabled = false
+      Locomotive::Notifications.expects(:new_content_entry).never
+      @content_entry.save
+    end
+
+    it 'does not send email notifications if no api accounts' do
+      @content_type.public_submission_accounts = nil
+      Locomotive::Notifications.expects(:new_content_entry).never
+      @content_entry.save
+    end
+
+    it 'sends email notifications when a new instance is created' do
+      Locomotive::Notifications.expects(:new_content_entry).with(@account_1, @content_entry).returns(mock('mailer', :deliver => true))
+      Locomotive::Notifications.expects(:new_content_entry).with(@account_2, @content_entry).returns(mock('mailer', :deliver => true))
+      @content_entry.save
+    end
+
+  end
+
+  describe '#site' do
+
+    it 'assigns a site when saving the content entry' do
+      content_entry = build_content_entry
+      content_entry.save
+      content_entry.site.should_not be_nil
+    end
+
+  end
+
+  def localize_content_type(content_type)
+    content_type.entries_custom_fields.first.localized = true
+    content_type.save
+  end
+
+  def build_content_entry(options = {})
+    @content_type.entries.build({ :title => 'Locomotive', :description => 'Lorem ipsum....', :_label_field_name => 'title' }.merge(options))
+  end
+
+  def fake_bson_id(id)
+    BSON::ObjectId(id.to_s.rjust(24, '0'))
+  end
+end