locomotivecms_steam 1.4.1 → 1.5.0.beta1

data/spec/integration/services/content_entry_service_spec.rb

@@ -45,7 +45,7 @@ describe Locomotive::Steam::ContentEntryService do
 
       let(:site_id)   { mongodb_site_id }
       let(:adapter)   { Locomotive::Steam::MongoDBAdapter.new(database: mongodb_database, hosts: ['127.0.0.1:27017']) }
-      let(:entry_id)  { BSON::ObjectId.from_string('5ae73458e051bb4f5525c92f') }
+      let(:entry_id)  { BSON::ObjectId.from_string('5baf7d38a953300567956448') }
 
       describe '#create' do
         subject { service.create('messages', { name: 'John', email: 'john@doe.net', message: 'Hello world!' }) }

data/spec/support/helpers.rb

@@ -4,11 +4,11 @@ module Spec
   module Helpers
 
     def mongodb_database
-      'steam_test_1_4_x'
+      'steam_test_1_5_x'
     end
 
     def mongodb_site_id
-      BSON::ObjectId.from_string('5ae73303e051bb4f2bcbeefa')
+      BSON::ObjectId.from_string('5baf76f4a9533004e4ae9840')
     end
 
     def reset!
@@ -49,7 +49,7 @@ module Spec
       File.expand_path('../../fixtures/default/', __FILE__)
     end
 
-    def env_for(url, opts={})
+    def env_for(url, opts = {})
       Rack::MockRequest.env_for(url, opts)
     end
 

data/spec/unit/adapters/filesystem/sanitizers/section_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/sanitizer.rb'
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/sanitizers/section.rb'
+require_relative '../../../../../lib/locomotive/steam/errors.rb'
+
+describe Locomotive::Steam::Adapters::Filesystem::Sanitizers::Section do
+
+  let(:template_path) { 'spec/fixtures/default/app/views/sections/header.liquid' }
+  let(:entity)    { instance_double('SectionEntity', template_path: template_path) }
+  let(:site)    { instance_double('Site', _id: 1) }
+  let(:scope)   { instance_double('Scope', site: site) }
+  let(:sanitizer) { described_class.new }
+
+  before(:each) do
+    sanitizer.setup(scope);
+  end
+
+  describe '#apply_to_entity' do
+    subject { sanitizer.apply_to_entity(entity) }
+
+    describe 'with correct json' do
+
+      it 'sanitize entity' do
+        expect(entity).to receive(:definition=).with(hash_including({ 'name' => 'header' }))
+        expect(entity).to receive(:template=).with((<<-LIQUID
+<h1> {{ section.settings.brand }} </h1>
+<ul>
+  {% for block in section.blocks %}
+    <li>
+      <a href="{{ block.settings.url }}" target="{% if block.settings.new_tab %}_blank{% endif %}">
+        {{ block.settings.label }}
+      </a>
+    </li>
+  {% endfor %}
+</ul>
+LIQUID
+        ).gsub /^$\n/, '')
+        expect(entity).to receive(:[]=).with(:site_id, 1)
+        subject
+      end
+    end
+    describe 'errors' do
+      before(:each) do
+        allow(entity).to receive(:[]=)
+      end
+
+      describe 'in json header' do
+        let(:template_path) { 'spec/fixtures/errors/section_bad_json_header.liquid' }
+
+        it 'should throw an error' do
+          expect { subject }.to raise_error(Locomotive::Steam::ParsingRenderingError)
+        end
+      end
+
+      describe 'json content' do
+        let(:template_path) { 'spec/fixtures/errors/section_bad_json_content.liquid' }
+
+        it 'should throw an error' do
+          expect { subject }.to raise_error(JSON::ParserError)
+        end
+      end
+    end
+  end
+end

data/spec/unit/adapters/filesystem/sanitizers/site_spec.rb

@@ -6,14 +6,40 @@ require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/saniti
 describe Locomotive::Steam::Adapters::Filesystem::Sanitizers::Site do
 
   let(:schema)    { nil }
-  let(:entity)    { instance_double('SiteEntity', metafields_schema: schema) }
+  let(:routes)    { nil }
+  let(:entity)    { instance_double('SiteEntity', metafields_schema: schema, routes: routes) }
   let(:sanitizer) { described_class.new }
 
   describe '#apply_to_entity' do
 
     subject { sanitizer.apply_to_entity(entity) }
 
-    it { expect(entity).to receive(:metafields_schema=).with(nil); subject }
+    it 'modifies the entity' do
+      expect(entity).to receive(:metafields_schema=).with(nil)
+      expect(entity).to receive(:routes=).with([])
+      subject
+    end
+
+  end
+
+  describe '#build_routes' do
+
+    let(:routes) { nil }
+
+    subject { sanitizer.send(:build_routes, routes) }
+
+    it { is_expected.to eq [] }
+
+    describe 'various formats of the routes' do
+
+      let(:routes) { [{ '/blog/:year/:month' => 'blog' }, { 'route' => '/products/:category/:slug', 'page_handle' => 'product' }] }
+
+      it { is_expected.to eq([
+        { 'route' => '/blog/:year/:month', 'page_handle' => 'blog' },
+        { 'route' => '/products/:category/:slug', 'page_handle' => 'product' }
+      ]) }
+
+    end
 
   end
 

data/spec/unit/adapters/filesystem/yaml_loaders/content_entry_spec.rb

@@ -35,11 +35,14 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::ContentEntry do
 
     context 'a content type with a select field' do
 
-      let(:field)         { instance_double('Field', name: 'kind', type: :select) }
+      let(:options)       { instance_double('Options') }
+      let(:field)         { instance_double('Field', name: 'kind', type: :select, select_options: options) }
       let(:content_type)  { instance_double('Bands', slug: 'bands', select_fields: [field], association_fields: [], file_fields: [], password_fields: []) }
 
       it 'adds a new attribute for the foreign key' do
-        expect(subject.first[:kind_id]).to eq 'grunge'
+        expect(options).to receive(:by_name).twice.with('grunge').and_return(instance_double('GrungeOption', _id: 0))
+        expect(options).to receive(:by_name).with('rock').and_return(instance_double('RockOption', _id: 1))
+        expect(subject.first[:kind_id]).to eq 0
         expect(subject.first[:kind]).to eq nil
       end
 
@@ -65,10 +68,11 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::ContentEntry do
       let(:content_type)  { instance_double('Updates', slug: 'updates', select_fields: [field], association_fields: [], file_fields: [], password_fields: []) }
 
       it 'adds a new localized attribute for the foreign key' do
-        option = instance_double('Option', _id: 'General')
+        option = instance_double('Option', _id: 0)
+        allow(options_scope).to receive(:with_locale) { |_, &block| block.call }
         allow(options).to receive(:by_name).with('General').and_return(option)
         allow(options).to receive(:by_name).with('Général').and_return(option)
-        expect(subject.last[:category_id]).to eq({ en: 'General', fr: 'General' })
+        expect(subject.last[:category_id]).to eq({ en: 0, fr: 0 })
         expect(subject.last[:category]).to eq nil
       end
 

data/spec/unit/adapters/filesystem/yaml_loaders/content_type_spec.rb

@@ -29,7 +29,7 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::ContentType do
 
     subject { loader.send(:build_select_options_from_hash, options) }
 
-    it { is_expected.to eq [{ _id: 'General', name: { en: 'General', fr: 'Général' }, position: 0 }, { _id: 'Gigs', name: { en: 'Gigs', fr: 'Concerts' }, position: 1 }, { _id: 'Bands', name: { en: 'Bands', fr: 'Groupes' }, position: 2 }] }
+    it { is_expected.to eq [{ _id: 0, name: { en: 'General', fr: 'Général' }, position: 0 }, { _id: 1, name: { en: 'Gigs', fr: 'Concerts' }, position: 1 }, { _id: 2, name: { en: 'Bands', fr: 'Groupes' }, position: 2 }] }
 
   end
 
@@ -40,7 +40,7 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::ContentType do
 
     subject { loader.send(:build_select_options_from_array, options) }
 
-    it { is_expected.to eq [{ _id: 'General', name: { en: 'General', fr: 'Général' }, position: 0 }, { _id: 'Gigs', name: { en: 'Gigs', fr: 'Concerts' }, position: 1 }, { _id: 'Bands', name: { en: 'Bands', fr: 'Groupes' }, position: 2 }] }
+    it { is_expected.to eq [{ _id: 0, name: { en: 'General', fr: 'Général' }, position: 0 }, { _id: 1, name: { en: 'Gigs', fr: 'Concerts' }, position: 1 }, { _id: 2, name: { en: 'Bands', fr: 'Groupes' }, position: 2 }] }
 
   end
 

data/spec/unit/adapters/filesystem/yaml_loaders/page_spec.rb

@@ -15,7 +15,7 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::Page do
     subject { loader.load(scope).sort { |a, b| a[:_fullpath] <=> b[:_fullpath] } }
 
     it 'tests various stuff' do
-      expect(subject.size).to eq 34
+      expect(subject.size).to eq 35
       expect(subject.first[:title]).to eq(en: 'Page not found', fr: 'Page non trouvée')
       expect(subject[23][:is_layout]).to eq true
       expect(subject[23][:listed]).to eq false

data/spec/unit/adapters/filesystem/yaml_loaders/section_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/yaml_loader.rb'
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/yaml_loaders/section.rb'
+
+describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::Section do
+
+  let(:site_path) { default_fixture_site_path }
+  let(:loader)    { described_class.new(site_path) }
+
+  describe '#load' do
+
+    let(:scope) { instance_double('Scope', default_locale: :en) }
+
+    subject { loader.load(scope).sort { |a, b| a[:name] <=> b[:name] } }
+
+    it 'tests various stuff' do
+      expect(subject.size).to eq 3
+      expect(subject.first[:slug]).to eq('carousel')
+      expect(subject[1][:name]).to eq('Footer')
+      expect(subject[2][:name]).to eq('Header')
+      expect(subject[2][:slug]).to eq('header')
+    end
+
+  end
+
+end

data/spec/unit/entities/section_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::Section do
+
+  let(:attributes)  { {} }
+  let(:section)     { described_class.new(attributes) }
+
+  describe '#source' do
+
+    let(:attributes) { { template:  "Hello world"} }
+
+    subject { section.source }
+    it { is_expected.to eq 'Hello world' }
+
+  end
+
+  describe '#definition' do
+    let(:attributes) { { definition: { name: 'aName' } } }
+    subject { section.definition }
+    it { is_expected.to eq({ 'name' => 'aName' }) }
+  end
+
+  describe '#type' do
+    let(:attributes) { { slug: 'hero' } }
+    subject { section.type }
+    it { is_expected.to eq 'hero' }
+  end
+
+end

data/spec/unit/liquid/drops/params_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::Liquid::Drops::Params do
+
+  let(:params)    { { 'foo' => '42' } }
+  let(:drop)      { described_class.new(params) }
+
+  it { expect(drop.before_method('bar').to_s).to eq '' }
+
+  it { expect(drop.before_method('foo').to_s).to eq '42' }
+
+  describe 'prevent XSS attack' do
+
+    context 'passing data from Liquid to HTML' do
+
+      let(:params) { { 'foo' => 'Hello<script>alert(document.cookie)</script>' } }
+
+      it { expect(drop.before_method('foo').to_s).to eq 'Hello&lt;script&gt;alert(document.cookie)&lt;/script&gt;' }
+
+      context 'security is disabled' do
+
+        it { expect(drop.before_method('foo').html_safe).to eq 'Hello<script>alert(document.cookie)</script>' }
+
+      end
+
+    end
+
+    context 'passing data from Liquid to Javascript' do
+
+      let(:params) { { 'foo' => "'+alert(document.cookie)+'" } }
+
+      it { expect(drop.before_method('foo').to_s).to eq '&#39;+alert(document.cookie)+&#39;' }
+
+    end
+
+  end
+
+end

data/spec/unit/liquid/drops/section_content_proxy_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::Liquid::Drops::SectionContentProxy do
+
+  let(:url_finder_service) { instance_double('UrlFinderService') }
+  let(:services)  { instance_double('Services', url_finder: url_finder_service) }
+  let(:site)      { instance_double('Site', default_locale: 'en') }
+  let(:context)   { ::Liquid::Context.new({}, {}, { locale: 'en', services: services, site: site }) }
+  let(:drop)      { described_class.new(content, settings).tap { |d| d.context = context } }
+
+  describe 'text type setting' do
+
+    let(:settings)  { [{ 'id' => 'title', 'type' => 'text' }] }
+    let(:content)   { { 'title' => %(Click <a href="//locomotive/_locomotive-link/aaaa">here</a>) } }
+
+    subject { drop.before_method(:title) }
+
+    it 'calls the url_finder_service to transform encoded links to existing urls' do
+      expect(url_finder_service).to receive(:decode_urls_for).with(%(Click <a href="//locomotive/_locomotive-link/aaaa">here</a>)).and_return('done')
+      is_expected.to eq 'done'
+    end
+
+  end
+
+  describe 'url type setting' do
+
+    let(:settings)  { [{ 'id' => 'link', 'type' => 'url' }] }
+    let(:content)   { { 'link' => { 'type' => 'page', 'value' => 42, 'new_window' => true } } }
+    let(:page)      { instance_double('Page') }
+
+    before do
+      expect(url_finder_service).to receive(:url_for).with({ 'type' => 'page', 'value' => 42, 'new_window' => true }).and_return(['/foo/bar', true])
+    end
+
+    subject { drop.before_method(:link).to_s }
+
+    it 'returns the url to the page' do
+      is_expected.to eq '/foo/bar'
+    end
+
+    context 'it knows if the link has to be opened in a new window or not' do
+
+      subject { drop.before_method(:link).new_window }
+
+      it { is_expected.to eq true }
+
+    end
+
+  end
+
+  describe 'image picker type setting' do
+
+    let(:settings)    { [{ 'id' => 'image', 'type' => 'image_picker' }] }
+    let(:value)       { nil }
+    let(:content)     { { 'image' => value } }
+    let(:page)        { instance_double('Page') }
+    let(:image_drop)  { drop.before_method(:image) }
+
+    subject { image_drop.to_s }
+
+    it { is_expected.to eq('') }
+
+    context 'the content is a string' do
+
+      let(:value) { 'banner.jpg' }
+
+      it { is_expected.to eq('banner.jpg') }
+
+    end
+
+    context 'the content is a hash' do
+
+      let(:value) { { source: 'awesome_banner.jpg', cropped: 'cropped_awesome_banner.jpg', width: 42, height: 30 } }
+
+      it { is_expected.to eq('cropped_awesome_banner.jpg') }
+
+      it 'has access to the width and height of the image' do
+        expect(image_drop.source).to eq('awesome_banner.jpg')
+        expect(image_drop.cropped).to eq('cropped_awesome_banner.jpg')
+        expect(image_drop.width).to eq(42)
+        expect(image_drop.height).to eq(30)
+      end
+
+    end
+
+  end
+
+end

data/spec/unit/liquid/drops/section_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::Liquid::Drops::Section do
+
+  let(:context)     { ::Liquid::Context.new({}, {}, { locale: 'en' }) }
+  let(:settings)    { [] }
+  let(:definition)  { instance_double('SectionDefinition', definition: { 'settings' => settings }) }
+  let(:drop)        { described_class.new(definition, content).tap { |d| d.context = context } }
+
+  describe 'text type setting' do
+
+    let(:url_finder) { instance_double('UrlFinder', decode_urls_for: 'Hello world') }
+    let(:settings) { [{ 'id' => 'title', 'type' => 'text' }] }
+    let(:content) { { 'settings' => { 'title' => 'Hello world' } } }
+
+    before { expect(drop.settings).to receive(:url_finder).and_return(url_finder) }
+
+    subject { drop.settings.before_method(:title) }
+
+    it 'returns the value of the text setting' do
+      is_expected.to eq 'Hello world'
+    end
+
+  end
+
+end

data/spec/unit/liquid/filters/html_spec.rb

@@ -10,7 +10,7 @@ describe Locomotive::Steam::Liquid::Filters::Html do
   let(:services)      { Locomotive::Steam::Services.build_instance }
   let(:context)       { instance_double('Context', registers: { services: services }) }
 
-  let(:theme_asset_url)         { services.theme_asset_url }
+  let(:_theme_asset_url)        { services.theme_asset_url }
   let(:theme_asset_repository)  { services.repositories.theme_asset }
 
   before { services.repositories.theme_asset = EngineThemeAsset.new(nil, site) }
@@ -19,239 +19,267 @@ describe Locomotive::Steam::Liquid::Filters::Html do
 
   before { @context = context }
 
-  it 'writes the tag to display a rss/atom feed' do
-    expect(auto_discovery_link_tag('/foo/bar')).to eq %(
-      <link rel="alternate" type="application/rss+xml" title="RSS" href="/foo/bar" />
-    ).strip
+  describe '#auto_discovery_link_tag' do
 
-    expect(auto_discovery_link_tag('/foo/bar', 'rel:alternate2', 'type:atom', 'title:Hello world')).to eq %(
-      <link rel="alternate2" type="atom" title="Hello world" href="/foo/bar" />
-    ).strip
+    it 'writes the tag to display a rss/atom feed' do
+      expect(auto_discovery_link_tag('/foo/bar')).to eq %(
+        <link rel="alternate" type="application/rss+xml" title="RSS" href="/foo/bar" />
+      ).strip
+
+      expect(auto_discovery_link_tag('/foo/bar', 'rel:alternate2', 'type:atom', 'title:Hello world')).to eq %(
+        <link rel="alternate2" type="atom" title="Hello world" href="/foo/bar" />
+      ).strip
+    end
   end
 
-  it 'returns an url for a stylesheet file' do
-    result = "/sites/42/theme/stylesheets/main.css"
-    expect(stylesheet_url('main.css')).to eq(result)
-    expect(stylesheet_url('main')).to eq(result)
-    expect(stylesheet_url(nil)).to eq('')
+  describe '#theme_asset_url' do
+
+    it 'returns the url to a JSON file' do
+      expect(theme_asset_url('others/manifest.json')).to eq('/sites/42/theme/others/manifest.json')
+    end
+
   end
 
-  describe 'with checksum' do
+  describe '#stylesheet_url' do
 
-    before do
-      Locomotive::Steam.configure { |c| c.theme_assets_checksum = true }
-      allow(theme_asset_repository).to receive(:checksums).and_return('stylesheets/main.css' => 42)
+    it 'returns an url for a stylesheet file' do
+      result = "/sites/42/theme/stylesheets/main.css"
+      expect(stylesheet_url('main.css')).to eq(result)
+      expect(stylesheet_url('main')).to eq(result)
+      expect(stylesheet_url(nil)).to eq('')
     end
 
-    after do
-      Locomotive::Steam.reset
+    it 'returns an url for a stylesheet file with folder' do
+      result = "/sites/42/theme/stylesheets/trash/main.css"
+      expect(stylesheet_url('trash/main.css')).to eq(result)
     end
 
-    it 'returns an url with the checksum' do
-      result = "/sites/42/theme/stylesheets/main.css?42"
-      expect(stylesheet_url('main.css')).to eq(result)
+    it 'returns an url for a stylesheet file without touching the url that starts with "/"' do
+      result = "/trash/main.css"
+      expect(stylesheet_url('/trash/main.css')).to eq(result)
+      expect(stylesheet_url('/trash/main')).to eq(result)
     end
 
-  end
+    it 'returns an url for a stylesheet file without touching the url that starts with "http:"' do
+      expect(stylesheet_url('http://cdn.example.com/trash/main.css')).to eq "http://cdn.example.com/trash/main.css"
+    end
 
-  it 'returns an url for a stylesheet file with folder' do
-    result = "/sites/42/theme/stylesheets/trash/main.css"
-    expect(stylesheet_url('trash/main.css')).to eq(result)
-  end
+    it 'returns an url for a stylesheet file without touching the url that starts with "https:"' do
+      expect(stylesheet_url('https://cdn.example.com/trash/main.css')).to eq "https://cdn.example.com/trash/main.css"
+    end
 
-  it 'returns an url for a stylesheet file without touching the url that starts with "/"' do
-    result = "/trash/main.css"
-    expect(stylesheet_url('/trash/main.css')).to eq(result)
-    expect(stylesheet_url('/trash/main')).to eq(result)
-  end
+    it 'returns an url for a stylesheet file with respect to URL-parameters' do
+      result = "/sites/42/theme/stylesheets/main.css?v=42"
+      expect(stylesheet_url('main.css?v=42')).to eq(result)
+      expect(stylesheet_url('main?v=42')).to eq(result)
+    end
 
-  it 'returns an url for a stylesheet file without touching the url that starts with "http:"' do
-    expect(stylesheet_url('http://cdn.example.com/trash/main.css')).to eq "http://cdn.example.com/trash/main.css"
-  end
+    it 'returns a link tag for a stylesheet file' do
+      result = "<link href=\"/sites/42/theme/stylesheets/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('main.css')).to eq(result)
+      expect(stylesheet_tag('main')).to eq(result)
+      expect(stylesheet_tag(nil)).to eq('')
+    end
 
-  it 'returns an url for a stylesheet file without touching the url that starts with "https:"' do
-    expect(stylesheet_url('https://cdn.example.com/trash/main.css')).to eq "https://cdn.example.com/trash/main.css"
-  end
+    it 'returns a link tag for a stylesheet file with folder' do
+      result = "<link href=\"/sites/42/theme/stylesheets/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('trash/main.css')).to eq(result)
+    end
 
-  it 'returns an url for a stylesheet file with respect to URL-parameters' do
-    result = "/sites/42/theme/stylesheets/main.css?v=42"
-    expect(stylesheet_url('main.css?v=42')).to eq(result)
-    expect(stylesheet_url('main?v=42')).to eq(result)
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "/"' do
+      expect(stylesheet_tag('/trash/main.css')).to eq "<link href=\"/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('/trash/main')).to eq "<link href=\"/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+    end
 
-  it 'returns a link tag for a stylesheet file' do
-    result = "<link href=\"/sites/42/theme/stylesheets/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('main.css')).to eq(result)
-    expect(stylesheet_tag('main')).to eq(result)
-    expect(stylesheet_tag(nil)).to eq('')
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "http:"' do
+      expect(stylesheet_tag('http://cdn.example.com/trash/main.css')).to eq "<link href=\"http://cdn.example.com/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+    end
 
-  it 'returns a link tag for a stylesheet file with folder' do
-    result = "<link href=\"/sites/42/theme/stylesheets/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('trash/main.css')).to eq(result)
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "https:"' do
+      expect(stylesheet_tag('https://cdn.example.com/trash/main.css')).to eq "<link href=\"https://cdn.example.com/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+    end
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "/"' do
-    expect(stylesheet_tag('/trash/main.css')).to eq "<link href=\"/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('/trash/main')).to eq "<link href=\"/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-  end
+    it 'returns a link tag for a stylesheet stored in Amazon S3' do
+      url = 'https://com.citrrus.locomotive.s3.amazonaws.com/sites/42/theme/stylesheets/bootstrap2.css'
+      allow(_theme_asset_url).to receive(:build).and_return(url)
+      result = "<link href=\"#{url}\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('bootstrap2.css')).to eq(result)
+    end
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "http:"' do
-    expect(stylesheet_tag('http://cdn.example.com/trash/main.css')).to eq "<link href=\"http://cdn.example.com/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-  end
+    it 'returns a link tag for a stylesheet file and media attribute set to print' do
+      result = "<link href=\"/sites/42/theme/stylesheets/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('main.css','print')).to eq(result)
+      expect(stylesheet_tag('main','print')).to eq(result)
+      expect(stylesheet_tag(nil)).to eq('')
+    end
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "https:"' do
-    expect(stylesheet_tag('https://cdn.example.com/trash/main.css')).to eq "<link href=\"https://cdn.example.com/trash/main.css\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-  end
+    it 'returns a link tag for a stylesheet file with folder and media attribute set to print' do
+      result = "<link href=\"/sites/42/theme/stylesheets/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('trash/main.css','print')).to eq(result)
+    end
 
-  it 'returns a link tag for a stylesheet stored in Amazon S3' do
-    url = 'https://com.citrrus.locomotive.s3.amazonaws.com/sites/42/theme/stylesheets/bootstrap2.css'
-    allow(theme_asset_url).to receive(:build).and_return(url)
-    result = "<link href=\"#{url}\" media=\"screen\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('bootstrap2.css')).to eq(result)
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "/" and media attribute set to print' do
+      result = "<link href=\"/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('/trash/main.css','print')).to eq(result)
+      expect(stylesheet_tag('/trash/main','print')).to eq(result)
+    end
 
-  it 'returns a link tag for a stylesheet file and media attribute set to print' do
-    result = "<link href=\"/sites/42/theme/stylesheets/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('main.css','print')).to eq(result)
-    expect(stylesheet_tag('main','print')).to eq(result)
-    expect(stylesheet_tag(nil)).to eq('')
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "http:" and media attribute set to print' do
+      expect(stylesheet_tag('http://cdn.example.com/trash/main.css', 'print')).to eq "<link href=\"http://cdn.example.com/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('http://cdn.example.com/trash/main', 'print')).to eq "<link href=\"http://cdn.example.com/trash/main\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+    end
 
-  it 'returns a link tag for a stylesheet file with folder and media attribute set to print' do
-    result = "<link href=\"/sites/42/theme/stylesheets/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('trash/main.css','print')).to eq(result)
-  end
+    it 'returns a link tag for a stylesheet file without touching the url that starts with "https:" and media attribute set to print' do
+      expect(stylesheet_tag('https://cdn.example.com/trash/main.css', 'print')).to eq "<link href=\"https://cdn.example.com/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+      expect(stylesheet_tag('https://cdn.example.com/trash/main', 'print')).to eq "<link href=\"https://cdn.example.com/trash/main\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
+    end
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "/" and media attribute set to print' do
-    result = "<link href=\"/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('/trash/main.css','print')).to eq(result)
-    expect(stylesheet_tag('/trash/main','print')).to eq(result)
-  end
+    describe 'with checksum' do
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "http:" and media attribute set to print' do
-    expect(stylesheet_tag('http://cdn.example.com/trash/main.css', 'print')).to eq "<link href=\"http://cdn.example.com/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('http://cdn.example.com/trash/main', 'print')).to eq "<link href=\"http://cdn.example.com/trash/main\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-  end
+      before do
+        Locomotive::Steam.configure { |c| c.theme_assets_checksum = true }
+        allow(theme_asset_repository).to receive(:checksums).and_return('stylesheets/main.css' => 42)
+      end
 
-  it 'returns a link tag for a stylesheet file without touching the url that starts with "https:" and media attribute set to print' do
-    expect(stylesheet_tag('https://cdn.example.com/trash/main.css', 'print')).to eq "<link href=\"https://cdn.example.com/trash/main.css\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-    expect(stylesheet_tag('https://cdn.example.com/trash/main', 'print')).to eq "<link href=\"https://cdn.example.com/trash/main\" media=\"print\" rel=\"stylesheet\" type=\"text/css\" />"
-  end
+      after do
+        Locomotive::Steam.reset
+      end
 
-  it 'returns an url for a javascript file' do
-    result = "/sites/42/theme/javascripts/main.js"
-    expect(javascript_url('main.js')).to eq(result)
-    expect(javascript_url('main')).to eq(result)
-    expect(javascript_url(nil)).to eq('')
-  end
+      it 'returns an url with the checksum' do
+        result = "/sites/42/theme/stylesheets/main.css?42"
+        expect(stylesheet_url('main.css')).to eq(result)
+      end
 
-  it 'returns an url for a javascript file with folder' do
-    result = "/sites/42/theme/javascripts/trash/main.js"
-    expect(javascript_url('trash/main.js')).to eq(result)
-    expect(javascript_url('trash/main')).to eq(result)
-  end
+    end
 
-  it 'returns an url for a javascript file without touching the url that starts with "/"' do
-    result = "/trash/main.js"
-    expect(javascript_url('/trash/main.js')).to eq(result)
-    expect(javascript_url('/trash/main')).to eq(result)
   end
 
-  it 'returns an url for a javascript file without touching the url that starts with "http:"' do
-    expect(javascript_url('http://cdn.example.com/trash/main.js')).to eq "http://cdn.example.com/trash/main.js"
-    expect(javascript_url('http://cdn.example.com/trash/main')).to eq "http://cdn.example.com/trash/main"
-  end
+  describe '#javascript_url' do
 
-  it 'returns an url for a javascript file without touching the url that starts with "https:"' do
-    expect(javascript_url('https://cdn.example.com/trash/main.js')).to eq "https://cdn.example.com/trash/main.js"
-    expect(javascript_url('https://cdn.example.com/trash/main')).to eq "https://cdn.example.com/trash/main"
-  end
+    it 'returns an url for a javascript file' do
+      result = "/sites/42/theme/javascripts/main.js"
+      expect(javascript_url('main.js')).to eq(result)
+      expect(javascript_url('main')).to eq(result)
+      expect(javascript_url(nil)).to eq('')
+    end
 
-  it 'returns an url for a javascript file with respect to URL-parameters' do
-    expect(javascript_url('main.js?v=42')).to eq "/sites/42/theme/javascripts/main.js?v=42"
-  end
+    it 'returns an url for a javascript file with folder' do
+      result = "/sites/42/theme/javascripts/trash/main.js"
+      expect(javascript_url('trash/main.js')).to eq(result)
+      expect(javascript_url('trash/main')).to eq(result)
+    end
 
-  it 'returns a script tag for a javascript file' do
-    result = %{<script src="/sites/42/theme/javascripts/main.js" type="text/javascript" ></script>}
-    expect(javascript_tag('main.js')).to eq(result)
-    expect(javascript_tag('main')).to eq(result)
-    expect(javascript_tag(nil)).to eq('')
-  end
+    it 'returns an url for a javascript file without touching the url that starts with "/"' do
+      result = "/trash/main.js"
+      expect(javascript_url('/trash/main.js')).to eq(result)
+      expect(javascript_url('/trash/main')).to eq(result)
+    end
 
-  it 'returns a script tag for a javascript file with folder' do
-    result = %{<script src="/sites/42/theme/javascripts/trash/main.js" type="text/javascript" ></script>}
-    expect(javascript_tag('trash/main.js')).to eq(result)
-    expect(javascript_tag('trash/main')).to eq(result)
-  end
+    it 'returns an url for a javascript file without touching the url that starts with "http:"' do
+      expect(javascript_url('http://cdn.example.com/trash/main.js')).to eq "http://cdn.example.com/trash/main.js"
+      expect(javascript_url('http://cdn.example.com/trash/main')).to eq "http://cdn.example.com/trash/main"
+    end
 
-  it 'returns a script tag for a javascript file without touching the url that starts with "/"' do
-    result = %{<script src="/trash/main.js" type="text/javascript" ></script>}
-    expect(javascript_tag('/trash/main.js')).to eq(result)
-    expect(javascript_tag('/trash/main')).to eq(result)
-  end
+    it 'returns an url for a javascript file without touching the url that starts with "https:"' do
+      expect(javascript_url('https://cdn.example.com/trash/main.js')).to eq "https://cdn.example.com/trash/main.js"
+      expect(javascript_url('https://cdn.example.com/trash/main')).to eq "https://cdn.example.com/trash/main"
+    end
 
-  it 'returns a script tag for a javascript file without touching the url that starts with "http:"' do
-    expect(javascript_tag('http://cdn.example.com/trash/main.js')).to eq %{<script src="http://cdn.example.com/trash/main.js" type="text/javascript" ></script>}
-    expect(javascript_tag('http://cdn.example.com/trash/main')).to eq %{<script src="http://cdn.example.com/trash/main" type="text/javascript" ></script>}
-  end
+    it 'returns an url for a javascript file with respect to URL-parameters' do
+      expect(javascript_url('main.js?v=42')).to eq "/sites/42/theme/javascripts/main.js?v=42"
+    end
 
-  it 'returns a script tag for a javascript file without touching the url that starts with "https:"' do
-    expect(javascript_tag('https://cdn.example.com/trash/main.js')).to eq %{<script src="https://cdn.example.com/trash/main.js" type="text/javascript" ></script>}
-    expect(javascript_tag('https://cdn.example.com/trash/main')).to eq %{<script src="https://cdn.example.com/trash/main" type="text/javascript" ></script>}
-  end
+    it 'returns a script tag for a javascript file' do
+      result = %{<script src="/sites/42/theme/javascripts/main.js" type="text/javascript" ></script>}
+      expect(javascript_tag('main.js')).to eq(result)
+      expect(javascript_tag('main')).to eq(result)
+      expect(javascript_tag(nil)).to eq('')
+    end
 
-  it 'returns a script tag for a javascript file with "defer" option' do
-    result = %{<script src="https://cdn.example.com/trash/main.js" type="text/javascript" defer="defer" ></script>}
-    expect(javascript_tag('https://cdn.example.com/trash/main.js', ['defer:defer'])).to eq(result)
-  end
+    it 'returns a script tag for a javascript file with folder' do
+      result = %{<script src="/sites/42/theme/javascripts/trash/main.js" type="text/javascript" ></script>}
+      expect(javascript_tag('trash/main.js')).to eq(result)
+      expect(javascript_tag('trash/main')).to eq(result)
+    end
 
-  it 'returns an image url for a given theme file without parameters' do
-    expect(theme_image_url('foo.jpg')).to eq "/sites/42/theme/images/foo.jpg"
-  end
+    it 'returns a script tag for a javascript file without touching the url that starts with "/"' do
+      result = %{<script src="/trash/main.js" type="text/javascript" ></script>}
+      expect(javascript_tag('/trash/main.js')).to eq(result)
+      expect(javascript_tag('/trash/main')).to eq(result)
+    end
 
-  it 'returns an image tag for a given theme file without parameters' do
-    expect(theme_image_tag('foo.jpg')).to eq "<img src=\"/sites/42/theme/images/foo.jpg\" >"
-  end
+    it 'returns a script tag for a javascript file without touching the url that starts with "http:"' do
+      expect(javascript_tag('http://cdn.example.com/trash/main.js')).to eq %{<script src="http://cdn.example.com/trash/main.js" type="text/javascript" ></script>}
+      expect(javascript_tag('http://cdn.example.com/trash/main')).to eq %{<script src="http://cdn.example.com/trash/main" type="text/javascript" ></script>}
+    end
 
-  it 'returns an image tag for a given theme file with size' do
-    expect(theme_image_tag('foo.jpg', 'width:100', 'height:100')).to eq("<img src=\"/sites/42/theme/images/foo.jpg\" height=\"100\" width=\"100\" >")
-  end
+    it 'returns a script tag for a javascript file without touching the url that starts with "https:"' do
+      expect(javascript_tag('https://cdn.example.com/trash/main.js')).to eq %{<script src="https://cdn.example.com/trash/main.js" type="text/javascript" ></script>}
+      expect(javascript_tag('https://cdn.example.com/trash/main')).to eq %{<script src="https://cdn.example.com/trash/main" type="text/javascript" ></script>}
+    end
 
-  it 'returns an image tag without parameters' do
-    expect(image_tag('foo.jpg')).to eq("<img src=\"foo.jpg\" >")
-  end
+    it 'returns a script tag for a javascript file with "defer" option' do
+      result = %{<script src="https://cdn.example.com/trash/main.js" type="text/javascript" defer="defer" ></script>}
+      expect(javascript_tag('https://cdn.example.com/trash/main.js', ['defer:defer'])).to eq(result)
+    end
 
-  it 'returns an image tag for a file with a leading slash' do
-    expect(image_tag('/foo.jpg')).to eq "<img src=\"/foo.jpg\" >"
   end
 
-  it 'returns an image tag with size' do
-    expect(image_tag('foo.jpg', 'width:100', 'height:50')).to eq("<img src=\"foo.jpg\" height=\"50\" width=\"100\" >")
+  describe '#theme_image_url' do
+
+    it 'returns an image url for a given theme file without parameters' do
+      expect(theme_image_url('foo.jpg')).to eq "/sites/42/theme/images/foo.jpg"
+    end
+
+    it 'returns an image tag for a given theme file without parameters' do
+      expect(theme_image_tag('foo.jpg')).to eq "<img src=\"/sites/42/theme/images/foo.jpg\" >"
+    end
+
+    it 'returns an image tag for a given theme file with size' do
+      expect(theme_image_tag('foo.jpg', 'width:100', 'height:100')).to eq("<img src=\"/sites/42/theme/images/foo.jpg\" height=\"100\" width=\"100\" >")
+    end
+
+    it 'returns an image tag without parameters' do
+      expect(image_tag('foo.jpg')).to eq("<img src=\"foo.jpg\" >")
+    end
+
+    it 'returns an image tag for a file with a leading slash' do
+      expect(image_tag('/foo.jpg')).to eq "<img src=\"/foo.jpg\" >"
+    end
+
+    it 'returns an image tag with size' do
+      expect(image_tag('foo.jpg', 'width:100', 'height:50')).to eq("<img src=\"foo.jpg\" height=\"50\" width=\"100\" >")
+    end
+
   end
 
-  it 'returns a flash tag without parameters' do
-    expect(flash_tag('foo.flv')).to eq(%{
+  describe '#flash_tag' do
+
+    it 'returns a flash tag without parameters' do
+      expect(flash_tag('foo.flv')).to eq(%{
 <object>
   <param name="movie" value="foo.flv">
   <embed src="foo.flv">
   </embed>
 </object>
-    }.strip)
-  end
+      }.strip)
+    end
 
-  it 'returns a flash tag with size' do
-    expect(flash_tag('foo.flv', 'width:100', 'height:50')).to eq(%{
+    it 'returns a flash tag with size' do
+      expect(flash_tag('foo.flv', 'width:100', 'height:50')).to eq(%{
 <object height=\"50\" width=\"100\">
   <param name="movie" value="foo.flv">
   <embed src="foo.flv" height=\"50\" width=\"100\">
   </embed>
 </object>
-    }.strip)
+      }.strip)
+    end
+
   end
 
   context 'asset_host' do
-    let(:asset_host)    { 'http://asset.dev' }
+
+    let(:asset_host) { 'http://asset.dev' }
 
     it 'returns an url for a stylesheet file with respect to URL-parameters' do
       result = "http://asset.dev/sites/42/theme/stylesheets/main.css?v=42"