locomotive_cms 2.0.0.rc12 → 2.0.0

data/spec/models/locomotive/site_spec.rb

@@ -0,0 +1,176 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe Locomotive::Site do
+
+  it 'should have a valid factory' do
+    FactoryGirl.build(:site).should be_valid
+  end
+
+  ## Validations ##
+
+  it 'should validate presence of name' do
+    site = FactoryGirl.build(:site, :name => nil)
+    site.should_not be_valid
+    site.errors[:name].should == ["can't be blank"]
+  end
+
+  it 'should validate presence of subdomain' do
+    site = FactoryGirl.build(:site, :subdomain => nil)
+    site.should_not be_valid
+    site.errors[:subdomain].should == ["can't be blank"]
+  end
+
+  %w{test test42 foo_bar}.each do |subdomain|
+    it "should accept subdomain like '#{subdomain}'" do
+      FactoryGirl.build(:site, :subdomain => subdomain).should be_valid
+    end
+  end
+
+  ['-', '_test', 'test_', 't est', '42', '42test'].each do |subdomain|
+    it "should not accept subdomain like '#{subdomain}'" do
+      (site = FactoryGirl.build(:site, :subdomain => subdomain)).should_not be_valid
+      site.errors[:subdomain].should == ['is invalid']
+    end
+  end
+
+  it "should not use reserved keywords as subdomain" do
+    %w{www admin email blog webmail mail support help site sites}.each do |subdomain|
+      (site = FactoryGirl.build(:site, :subdomain => subdomain)).should_not be_valid
+      site.errors[:subdomain].should == ['is reserved']
+    end
+  end
+
+  it 'should validate uniqueness of subdomain' do
+    FactoryGirl.create(:site)
+    (site = FactoryGirl.build(:site)).should_not be_valid
+    site.errors[:subdomain].should == ["is already taken"]
+  end
+
+  it 'should validate uniqueness of domains' do
+    FactoryGirl.create(:site, :domains => %w{www.acme.net www.acme.com})
+
+    (site = FactoryGirl.build(:site, :domains => %w{www.acme.com})).should_not be_valid
+    site.errors[:domains].should == ["www.acme.com is already taken"]
+
+    (site = FactoryGirl.build(:site, :subdomain => 'foo', :domains => %w{acme.example.com})).should_not be_valid
+    site.errors[:domains].should == ["acme.example.com is already taken"]
+  end
+
+  it 'should validate format of domains' do
+    site = FactoryGirl.build(:site, :domains => ['barformat.superlongextension', '-foo.net'])
+    site.should_not be_valid
+    site.errors[:domains].should == ['barformat.superlongextension is invalid', '-foo.net is invalid']
+  end
+
+  ## Named scopes ##
+
+  it 'should retrieve sites by domain' do
+    site_1 = FactoryGirl.create(:site, :domains => %w{www.acme.net})
+    site_2 = FactoryGirl.create(:site, :subdomain => 'test', :domains => %w{www.example.com})
+
+    sites = Locomotive::Site.match_domain('www.acme.net')
+    sites.size.should == 1
+    sites.first.should == site_1
+
+    sites = Locomotive::Site.match_domain('www.example.com')
+    sites.size.should == 1
+    sites.first.should == site_2
+
+    sites = Locomotive::Site.match_domain('test.example.com')
+    sites.size.should == 1
+    sites.first.should == site_2
+
+    sites = Locomotive::Site.match_domain('www.unknown.com')
+    sites.should be_empty
+  end
+
+  ## Associations ##
+
+  it 'should have many accounts' do
+    site = FactoryGirl.build(:site)
+    account_1, account_2 = FactoryGirl.create(:account), FactoryGirl.create(:account, :name => 'homer', :email => 'homer@simpson.net')
+    site.memberships.build(:account => account_1, :admin => true)
+    site.memberships.build(:account => account_2)
+    site.memberships.size.should == 2
+    site.accounts.should == [account_1, account_2]
+  end
+
+  ## Methods ##
+
+  it 'should return domains without subdomain' do
+    site = FactoryGirl.create(:site, :domains => %w{www.acme.net www.acme.com})
+    site.domains.should == %w{www.acme.net www.acme.com acme.example.com}
+    site.domains_without_subdomain.should == %w{www.acme.net www.acme.com}
+  end
+
+  describe 'once created' do
+
+    before(:each) do
+      @site = FactoryGirl.create(:site)
+    end
+
+    it 'should create index and 404 pages' do
+      @site.pages.size.should == 2
+      @site.pages.map(&:fullpath).sort.should == %w{404 index}
+    end
+
+    it 'translates the index/404 pages if a new locale is added' do
+      @site.update_attributes :locales => %w(en fr)
+
+      @site.errors.should be_empty
+
+      ::Mongoid::Fields::I18n.with_locale('fr') do
+        @site.pages.root.first.tap do |page|
+          page.title.should == "Page d'accueil"
+          page.slug.should == 'index'
+        end
+
+        @site.pages.not_found.first.tap do |page|
+          page.title.should == 'Page non trouvée'
+          page.slug.should == '404'
+        end
+      end
+    end
+
+    it 'translates the index/404 pages if the default locale changes' do
+      @site.update_attributes :locales => %w(fr en)
+
+      @site.errors.should be_empty
+
+      ::Mongoid::Fields::I18n.with_locale('fr') do
+        @site.pages.root.first.tap do |page|
+          page.title.should == "Page d'accueil"
+          page.slug.should == 'index'
+        end
+
+        @site.pages.not_found.first.tap do |page|
+          page.title.should == 'Page non trouvée'
+          page.slug.should == '404'
+        end
+      end
+    end
+
+    it 'does not allow to remove the default locale' do
+      @site.update_attributes :locales => %w(fr)
+      @site.errors[:locales].should == ['The previous default locale can not be removed right away.']
+    end
+
+  end
+
+  describe 'deleting in cascade' do
+
+    before(:each) do
+      @site = FactoryGirl.create(:site)
+    end
+
+    it 'should also destroy pages' do
+      lambda {
+        @site.destroy
+      }.should change(Locomotive::Page, :count).by(-2)
+    end
+
+  end
+
+end

data/spec/models/locomotive/snippet_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+
+describe Locomotive::Snippet do
+
+  it 'should have a valid factory' do
+    FactoryGirl.build(:snippet).should be_valid
+  end
+
+  # Validations ##
+
+  %w{site name template}.each do |field|
+    it "should validate presence of #{field}" do
+      template = FactoryGirl.build(:snippet, field.to_sym => nil)
+      template.should_not be_valid
+      template.errors[field.to_sym].should == ["can't be blank"]
+    end
+  end
+
+  describe '#update_templates' do
+
+    before :each do
+      @site    = FactoryGirl.create(:site, :subdomain => 'omg')
+      @snippet = FactoryGirl.create(:snippet, :site => @site, :slug => 'my_test_snippet', :template => 'a testing template')
+    end
+
+    context 'with a normal top level snippet' do
+
+      before :each do
+        @page = FactoryGirl.create(:page, :site => @site, :slug => 'my_page_here', :raw_template => "{% include 'my_test_snippet'  %}")
+      end
+
+      it 'updates templates with the new snippet template' do
+        @snippet.update_attributes(:template => 'a new template')
+        Locomotive::Page.find(@page.id).render({}).should == 'a new template'
+      end
+
+    end
+
+    context 'for snippets inside of a block' do
+
+      before :each do
+        @page = FactoryGirl.create(:page, :site => @site, :slug => 'my_page_here', :raw_template => "{% block main %}{% include 'my_test_snippet'  %}{% endblock %}")
+      end
+
+      it 'updates templates with the new snippet template' do
+        @snippet.update_attributes(:template => 'a new template')
+        Locomotive::Page.find(@page.id).render({}).should == 'a new template'
+      end
+
+    end
+
+    context '#i18n' do
+
+      before :each do
+        Mongoid::Fields::I18n.with_locale(:fr) do
+          @snippet = FactoryGirl.create(:snippet, :site => @site, :slug => 'my_localized_test_snippet', :template => 'a testing template')
+          @page = FactoryGirl.create(:page, :site => @site, :slug => 'my_localized_test_snippet', :raw_template => "{% block main %}{% include 'my_localized_test_snippet' %}{% endblock %}")
+        end
+      end
+
+      it 'returns the snippet dependencies depending on the UI locale' do
+        Mongoid::Fields::I18n.with_locale(:fr) { @page.snippet_dependencies.should_not be_empty }
+        Mongoid::Fields::I18n.with_locale(:en) { @page.snippet_dependencies.should be_nil }
+      end
+
+      it 'updates the templates with the new snippet' do
+        Mongoid::Fields::I18n.with_locale(:fr) do
+          @snippet.update_attributes(:template => 'a new template')
+          Locomotive::Page.find(@page.id).render({}).should == 'a new template'
+        end
+      end
+
+    end
+
+  end
+
+end

data/spec/models/locomotive/theme_asset_spec.rb

@@ -0,0 +1,128 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe Locomotive::ThemeAsset do
+
+  describe 'attaching a file' do
+
+    before(:each) do
+      Locomotive::ThemeAsset.any_instance.stubs(:site_id).returns('test')
+      @asset = FactoryGirl.build(:theme_asset)
+    end
+
+    describe 'file is a picture' do
+
+      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 'local path and folder' do
+
+      it 'should set the local path based on the content type' do
+        @asset.source = FixturedAsset.open('5k.png')
+        @asset.save
+        @asset.local_path.should == 'images/5k.png'
+      end
+
+      it 'should set the local path based on the folder' do
+        @asset.folder = 'trash'
+        @asset.source = FixturedAsset.open('5k.png')
+        @asset.save
+        @asset.local_path.should == 'images/trash/5k.png'
+      end
+
+      it 'should not only use the folder to build the local path' do
+        @asset.folder = 'images42'
+        @asset.source = FixturedAsset.open('5k.png')
+        @asset.save
+        @asset.local_path.should == 'images/images42/5k.png'
+      end
+
+      it 'should set sanitize the local path' do
+        @asset.folder = '/images/à la poubelle'
+        @asset.source = FixturedAsset.open('5k.png')
+        @asset.save
+        @asset.local_path.should == 'images/a_la_poubelle/5k.png'
+      end
+
+    end
+
+    describe '#validation' do
+
+      it 'does not accept text file' do
+        @asset.source = FixturedAsset.open('wrong.txt')
+        @asset.valid?.should be_false
+        @asset.errors[:source].should_not be_blank
+      end
+
+      it 'is not valid if another file with the same path exists' do
+        @asset.source = FixturedAsset.open('5k.png')
+        @asset.save!
+
+        another_asset = FactoryGirl.build(:theme_asset, :site => @asset.site)
+        another_asset.source = FixturedAsset.open('5k.png')
+        another_asset.valid?.should be_false
+        another_asset.errors[:local_path].should_not be_blank
+      end
+
+    end
+
+    it 'should process stylesheet' do
+      @asset.source = FixturedAsset.open('main.css')
+      @asset.source.file.content_type.should_not be_nil
+      @asset.stylesheet?.should be_true
+    end
+
+    it 'should process javascript' do
+      @asset.source = FixturedAsset.open('application.js')
+      @asset.source.file.content_type.should_not be_nil
+      @asset.javascript?.should be_true
+    end
+
+    it 'should get size' do
+      @asset.source = FixturedAsset.open('main.css')
+      @asset.size.should == 25
+    end
+
+  end
+
+  describe 'creating from plain text' do
+
+    before(:each) do
+      Locomotive::ThemeAsset.any_instance.stubs(:site_id).returns('test')
+      @asset = FactoryGirl.build(:theme_asset, {
+        :site => FactoryGirl.build(:site),
+        :plain_text_name => 'test',
+        :plain_text => 'Lorem ipsum',
+        :performing_plain_text => true
+      })
+    end
+
+    it 'should handle stylesheet' do
+      @asset.plain_text_type = 'stylesheet'
+      @asset.valid?.should be_true
+      @asset.stylesheet?.should be_true
+      @asset.source.should_not be_nil
+    end
+
+    it 'should handle javascript' do
+      @asset.plain_text_type = 'javascript'
+      @asset.valid?.should be_true
+      @asset.javascript?.should be_true
+      @asset.source.should_not be_nil
+    end
+
+  end
+
+end

data/spec/requests/seo_trailing_slash_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe 'Locomotive::Middlewares::SeoTrailingSlash' do
+
+  it 'does not process the "/" url' do
+    get '/'
+    response.status.should_not be(301)
+  end
+
+  it 'does not process the "/locomotive/" url' do
+    get '/locomotive/'
+    response.status.should_not be(301)
+  end
+
+  it 'does not process the "/locomotive/*" urls' do
+    get '/locomotive/login'
+    response.status.should_not be(301)
+  end
+
+  it 'redirects to the url without the trailing slash' do
+    get '/hello_world/'
+    response.status.should be(301)
+  end
+
+end

data/spec/support/carrierwave.rb

@@ -0,0 +1,50 @@
+require 'carrierwave/test/matchers'
+
+CarrierWave.configure do |config|
+  config.storage = :file
+  config.cache_dir = "spec/tmp/cache"
+  config.root = File.join(Rails.root, 'spec', 'tmp')
+end
+
+module FixturedAsset
+  def self.open(filename)
+    File.new(self.path(filename))
+  end
+
+  def self.path(filename)
+    File.join(File.dirname(__FILE__), '..', 'fixtures', 'assets', filename)
+  end
+
+  def self.duplicate(filename)
+    dst = File.join(File.dirname(__FILE__), '..', 'tmp', filename)
+    FileUtils.cp self.path(filename), dst
+    dst
+  end
+
+  def self.reset!
+    FileUtils.rm_rf(File.join(File.dirname(__FILE__), '..', 'tmp'))
+    FileUtils.mkdir(File.join(File.dirname(__FILE__), '..', 'tmp'))
+  end
+end
+
+module FixturedTheme
+  def self.open(filename)
+    File.new(self.path(filename))
+  end
+
+  def self.path(filename)
+    File.join(File.dirname(__FILE__), '..', 'fixtures', 'themes', filename)
+  end
+
+  def self.duplicate(filename)
+    dst = File.join(File.dirname(__FILE__), '..', 'tmp', filename)
+    FileUtils.cp self.path(filename), dst
+    dst
+  end
+
+  def self.duplicate_and_open(filename)
+    File.open(self.duplicate(filename))
+  end
+end
+
+FixturedAsset.reset!