locomotivecms_steam 1.0.0 → 1.0.1

data/spec/integration/server/metafields_spec.rb

@@ -0,0 +1,26 @@
+require File.dirname(__FILE__) + '/../integration_helper'
+
+describe 'Site metafields' do
+
+  include Rack::Test::Methods
+
+  def app
+    run_server
+  end
+
+  it 'returns all the values of the site metafields' do
+    get '/basic'
+    expect(last_response.body).to include 'Color scheme=white'
+    expect(last_response.body).to include 'Facebook ID=FB42'
+    expect(last_response.body).to include 'Google ID=G42'
+    expect(last_response.body).to include 'API URL=https://api.github.com/repos/vmg/redcarpet/issues?state=closed'
+    expect(last_response.body).to include 'Expires In=42'
+  end
+
+  it 'iterates over the metafields of a namespace' do
+    get '/basic'
+    expect(last_response.body).to include "<li class='property'>Facebook(facebook_id)=FB42</li>"
+    expect(last_response.body).to include "<li class='property'>Google(google_id)=G42</li>"
+  end
+
+end

data/spec/integration/server/nav_spec.rb

@@ -37,6 +37,14 @@ describe Locomotive::Steam::Server do
 
     end
 
+    describe 'with snippet to render the label' do
+
+      subject { get '/tags/nav'; last_response.body }
+
+      it { is_expected.to include('<li id="store-link" class="link"><a href="/store">-Store-</a></li>') }
+
+    end
+
     describe 'very deep' do
 
       subject { get '/tags/nav-in-deep'; last_response.body }

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

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/sanitizer.rb'
+require_relative '../../../../../lib/locomotive/steam/adapters/filesystem/sanitizers/site.rb'
+
+describe Locomotive::Steam::Adapters::Filesystem::Sanitizers::Site do
+
+  let(:schema)    { nil }
+  let(:entity)    { instance_double('SiteEntity', metafields_schema: schema) }
+  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 }
+
+  end
+
+  describe '#clean_metafields_schema' do
+
+    subject { sanitizer.send(:clean_metafields_schema, schema) }
+
+    it { is_expected.to eq nil }
+
+    describe 'with a schema' do
+
+      # see the metafields_schema.yml in the fixtures folder
+      let(:schema) { {:social=>{:label=>{:fr=>"Social (FR)"}, :position=>1, :fields=>["facebook_id", "google_id"]}, :github=>{:position=>0, :fields=>{:api_url=>{:label=>"API Url", :type=>"string", :hint=>"API endpoint"}, :expires_in=>{:label=>{:en=>"Expires in", :fr=>"Expire dans"}, :hint=>{:en=>"Cache - In milliseconds", :fr=>"Cache - En millisecondes"}, :type=>"integer", :min=>0, :max=>3600}}}} }
+
+      it 'loads the full schema' do
+        # First namespace
+        expect(subject[0]['name']).to eq 'social'
+        expect(subject[0]['label']).to eq('fr' => 'Social (FR)')
+        expect(subject[0]['position']).to eq 1
+        expect(subject[0]['fields']).to eq([{ 'name' => 'facebook_id', 'position' => 0 }, { 'name' => 'google_id', 'position' => 1 }])
+
+        # Second namespace
+        expect(subject[1]['label']).to eq('default' => 'github')
+        expect(subject[1]['position']).to eq 0
+        expect(subject[1]['fields'].count).to eq 2
+        expect(subject[1]['fields'][0]).to eq('name' => 'api_url', 'position' => 0, 'label' => { 'default' => 'API Url' }, 'type' => 'string', 'hint' => { 'default' => 'API endpoint' })
+      end
+
+      context 'label is a string instead of a hash' do
+
+        let(:schema) { {:social=>{:label=>"Social", :position=>1, :fields=>["facebook_id", "google_id"]}, :github=>{:position=>0, :fields=>{:api_url=>{:label=>"API Url", :type=>"string", :hint=>"API endpoint"}, :expires_in=>{:label=>{:en=>"Expires in", :fr=>"Expire dans"}, :hint=>{:en=>"Cache - In milliseconds", :fr=>"Cache - En millisecondes"}, :type=>"integer", :min=>0, :max=>3600}}}} }
+
+        it 'loads the full schema' do
+          expect(subject[0]['label']).to eq('default' => 'Social')
+        end
+
+      end
+
+    end
+
+  end
+
+end

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

@@ -10,9 +10,19 @@ describe Locomotive::Steam::Adapters::Filesystem::YAMLLoaders::Site do
 
   describe '#load' do
 
-    subject { loader.load(nil) }
+    subject { loader.load(nil).first }
 
-    it { expect(subject.first[:name]).to eq 'Sample site' }
+    it { expect(subject[:name]).to eq 'Sample site' }
+
+    describe '#metafields_schema' do
+
+      subject { loader.load(nil).first[:metafields_schema] }
+
+      it 'loads the full schema' do
+        expect(subject.count).to eq 3
+      end
+
+    end
 
   end
 

data/spec/unit/adapters/filesystem_adapter_spec.rb

@@ -50,4 +50,22 @@ describe Locomotive::Steam::FilesystemAdapter do
 
   end
 
+  describe '#inc' do
+
+    let(:entity) { OpenStruct.new(name: 'My post', views: 41) }
+
+    subject { adapter.inc(mapper, entity, :views) }
+
+    it { expect(subject.views).to eq 42 }
+
+    describe 'by an amount different from 1' do
+
+      subject { adapter.inc(mapper, entity, :views, 3) }
+
+      it { expect(subject.views).to eq 44 }
+
+    end
+
+  end
+
 end

data/spec/unit/adapters/mongodb/query_spec.rb

@@ -40,7 +40,31 @@ describe Locomotive::Steam::Adapters::MongoDB::Query do
 
   describe '#order_by' do
 
-    it { expect(query.order_by(title: :asc, published: :desc).sort).to eq [{title: :asc, published: :desc}] }
+    subject { query.order_by(order_by).sort }
+
+    context 'passing a hash' do
+
+      let(:order_by) { { title: :asc, published: :desc } }
+
+      it { is_expected.to eq [{title: :asc, published: :desc}] }
+
+    end
+
+    context 'passing an array of strings' do
+
+      let(:order_by) { ['title.asc', 'published'] }
+
+      it { is_expected.to eq [[['title', 'asc'], ['published']]] }
+
+    end
+
+    context 'passing a string' do
+
+      let(:order_by) { 'title.asc, published' }
+
+      it { is_expected.to eq [[['title', 'asc'], ['published']]] }
+
+    end
 
   end
 

data/spec/unit/entities/content_type_spec.rb

@@ -30,6 +30,16 @@ describe Locomotive::Steam::ContentType do
 
   end
 
+  describe '#default' do
+
+    subject { content_type.fields_with_default }
+
+    before { expect(repository).to receive(:default).and_return([true]) }
+
+    it { expect(subject).to eq([true])  }
+
+  end
+
   describe '#order_by' do
 
     subject { content_type.order_by }

data/spec/unit/initializers/sprockets_spec.rb

@@ -22,4 +22,40 @@ describe Locomotive::Steam::SprocketsEnvironment do
 
   end
 
+  describe '#install_autoprefixer' do
+
+    let(:options) { { minify: false } }
+
+    subject { env }
+
+    context "config/autoprefixer.yml doesn't exist" do
+
+      before { allow(File).to receive(:exists?).and_return false }
+
+      it { expect(AutoprefixerRails).not_to receive(:install); subject }
+
+    end
+
+    context "config/autoprefixer.yml exists" do
+
+      before {
+        allow(File).to receive(:exists?).and_return(true)
+        allow(YAML).to receive(:load_file).and_return({})
+      }
+
+      it { expect(AutoprefixerRails).to receive(:install).and_return(true); subject }
+
+      it 'warns developers if they notice bad performance when using autoprefixer' do
+        curent_execjs_runtime = ENV['EXECJS_RUNTIME']
+        ENV['EXECJS_RUNTIME'] = 'NodeJS'
+        expect(Locomotive::Common::Logger).not_to receive(:warn)
+        expect(AutoprefixerRails).to receive(:install).and_return(true)
+        subject
+        ENV['EXECJS_RUNTIME'] = curent_execjs_runtime
+      end
+
+    end
+
+  end
+
 end

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

@@ -4,7 +4,7 @@ describe Locomotive::Steam::Liquid::Drops::ContentEntry do
 
   let(:site)      { instance_double('Site', default_locale: 'en') }
   let(:type)      { instance_double('Type', fields_by_name: { title: instance_double('Field', type: :string ) }) }
-  let(:entry)     { instance_double('Article', _id: 42, localized_attributes: {}, content_type: type, title: 'Hello world', _label: 'Hello world', _slug: 'hello-world', _translated: false, seo_title: 'seo title', meta_keywords: 'keywords', meta_description: 'description') }
+  let(:entry)     { instance_double('Article', _id: 42, localized_attributes: {}, content_type: type, title: 'Hello world', _label: 'Hello world', _slug: 'hello-world', _translated: false, seo_title: 'seo title', meta_keywords: 'keywords', meta_description: 'description', created_at: 0, updated_at: 1) }
   let(:assigns)   { {} }
   let(:services)  { Locomotive::Steam::Services.build_instance }
   let(:context)   { ::Liquid::Context.new(assigns, {}, { services: services, site: site, locale: 'en' }) }
@@ -20,6 +20,8 @@ describe Locomotive::Steam::Liquid::Drops::ContentEntry do
     expect(subject.seo_title).to eq 'seo title'
     expect(subject.meta_keywords).to eq 'keywords'
     expect(subject.meta_description).to eq 'description'
+    expect(subject.created_at).to eq 0
+    expect(subject.updated_at).to eq 1
   end
 
   describe '#before_method (dynamic attributes)' do

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

@@ -0,0 +1,81 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::Liquid::Drops::Metafields do
+
+  let(:metafields)  { { 'my_namespace' => { 'analytics_id' => { 'default' => '42' }, 'street' => { 'en' => '7 Albert Camus Alley', 'fr' => '7 allée Albert Camus' } } } }
+  let(:schema)      { [ { 'name' => 'my_namespace', fields: [{ name: 'analytics_id' }, { name: 'street', localized: true }, { name: 'country' }] }].as_json }
+  let(:site)        { instance_double('Site', metafields: metafields, metafields_schema: schema) }
+  let(:context)     { ::Liquid::Context.new({}, {}, { locale: 'en' }) }
+  let(:drop)        { described_class.new(site).tap { |d| d.context = context } }
+
+  describe 'calling a metafield' do
+
+    context 'unknown namespace' do
+
+      subject { drop.before_method(:unknown_namespace) }
+
+      it { is_expected.to eq nil }
+
+    end
+
+    context 'existing namespace' do
+
+      let(:namespace) { drop.before_method(:my_namespace).tap { |d| d.context = context } }
+
+      context 'unknown field' do
+
+        subject { namespace.before_method(:unknown_field) }
+
+        it { is_expected.to eq nil }
+
+      end
+
+      context 'not a localized field' do
+
+        context 'the value exists' do
+
+          subject { namespace.before_method(:analytics_id) }
+
+          it { is_expected.to eq '42' }
+
+        end
+
+        context "the value doesn't exist" do
+
+          subject { namespace.before_method(:country) }
+
+          it { is_expected.to eq nil }
+
+        end
+
+      end
+
+      context 'localized field' do
+
+        subject { namespace.before_method(:street) }
+
+        it { is_expected.to eq '7 Albert Camus Alley' }
+
+        context 'in another locale' do
+
+          let(:context) { ::Liquid::Context.new({}, {}, { locale: 'fr' }) }
+
+          it { is_expected.to eq '7 allée Albert Camus' }
+
+        end
+
+        context 'in a locale with no translation' do
+
+          let(:context) { ::Liquid::Context.new({}, {}, { locale: 'de' }) }
+
+          it { is_expected.to eq nil }
+
+        end
+
+      end
+
+    end
+
+  end
+
+end

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

@@ -119,4 +119,17 @@ describe Locomotive::Steam::Liquid::Filters::Misc do
 
   end
 
+  describe '#shuffle' do
+
+    let(:array) { [1, 2, 3, 4] }
+
+    subject { shuffle(array) }
+
+    it 'returns an array in a random order' do
+      expect(subject.size).to eq 4
+      expect(subject).not_to eq([1, 2, 3, 4])
+    end
+
+  end
+
 end

data/spec/unit/liquid/tags/consume_spec.rb

@@ -71,6 +71,13 @@ describe Locomotive::Steam::Liquid::Tags::Consume do
 
     end
 
+    describe "don't render it if the url is blank" do
+
+      let(:source) { "{% consume blog from \"\" %}{{ blog.title }}{% endconsume %}" }
+      it { is_expected.to eq '' }
+
+    end
+
   end
 
   describe 'timeout' do

data/spec/unit/liquid/tags/nav_spec.rb

@@ -174,7 +174,7 @@ describe 'Locomotive::Steam::Liquid::Tags::Nav' do
       describe 'from a registered snippet' do
 
         let(:source)  { %({% nav site, snippet: nav_title %}) }
-        let(:snippet) { instance_double('Snippet', source: '{{ page.title }}!') }
+        let(:snippet) { instance_double('Snippet', liquid_source: '{{ page.title }}!') }
 
         before do
           allow(services.snippet_finder).to receive(:find).with('nav_title').and_return(snippet)
@@ -182,6 +182,14 @@ describe 'Locomotive::Steam::Liquid::Tags::Nav' do
 
         it { is_expected.to include %{<a href="/child-1">Child #1!</a>} }
 
+        context "the snippet doesn't exist" do
+
+          let(:snippet) { nil }
+
+          it { is_expected.to include %{<a href="/child-1">Child #1</a>} }
+
+        end
+
       end
 
     end

data/spec/unit/liquid/tags/path_to_spec.rb

@@ -57,7 +57,7 @@ describe Locomotive::Steam::Liquid::Tags::PathTo do
 
     let(:assigns)   { { 'about_us' => drop } }
     let(:drop)      { Locomotive::Steam::Liquid::Drops::Page.new(page) }
-    let(:page)      { liquid_instance_double('AboutUs', title: 'About us', handle: 'index', localized_attributes: { fullpath: true }, fullpath: fullpath, localized_attributes: [:fullpath], templatized?: false) }
+    let(:page)      { liquid_instance_double('AboutUs', title: 'About us', handle: 'index', localized_attributes: { fullpath: true }, fullpath: fullpath, templatized?: false) }
     let(:fullpath)  { { en: 'about-us', fr: 'a-notre-sujet' } }
     let(:source)    { '{% path_to about_us %}' }
 

data/spec/unit/middlewares/helpers_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+require_relative '../../../lib/locomotive/steam/middlewares/helpers'
+
+describe Locomotive::Steam::Middlewares::Helpers do
+
+  let(:middleware)  { Class.new { include Locomotive::Steam::Middlewares::Helpers } }
+  let(:instance)    { middleware.new }
+
+  describe '#redirect_to' do
+
+    subject { instance.redirect_to(location)[1]['Location'] }
+
+    context 'mounted_on is not blank' do
+
+      before { allow(instance).to receive(:mounted_on).and_return('/my_app') }
+
+      let(:location) { '/foo/bar' }
+      it { is_expected.to eq '/my_app/foo/bar' }
+
+      describe 'the location already includes mounted_on' do
+
+        let(:location) { '/my_app/foo' }
+        it { is_expected.to eq '/my_app/foo' }
+
+      end
+
+    end
+
+  end
+
+end