locomotivecms 3.0.0.pre.alpha.3 → 3.0.0.pre.beta.1

data/spec/dummy/config/application.rb

@@ -45,5 +45,9 @@ module Dummy
     # config.after_initialize do |c|
     #   c.middleware.delete(Sass::Plugin::Rack)
     # end
+
+    # let Locomotive handle custom error page
+    require 'locomotive/middlewares/custom_public_exceptions'
+    config.exceptions_app = Locomotive::Middlewares::CustomPublicExceptions.new(Rails.public_path)
   end
 end

data/spec/dummy/config/environments/profile.rb

@@ -0,0 +1,48 @@
+Dummy::Application.configure do
+  # Settings specified here will take precedence over those in config/application.rb.
+
+  config.cache_store = :memory_store
+
+  # In the development environment your application's code is reloaded on
+  # every request. This slows down response time but is perfect for development
+  # since you don't have to restart the web server when you make code changes.
+  config.cache_classes = true
+
+  # Do not eager load code on boot.
+  config.eager_load = true
+
+  # Show full error reports and disable caching.
+  config.consider_all_requests_local       = true
+  config.action_controller.perform_caching = true
+
+  # Don't care if the mailer can't send.
+  config.action_mailer.raise_delivery_errors = false
+
+  # Print deprecation notices to the Rails logger.
+  config.active_support.deprecation = :log
+
+  # Debug mode disables concatenation and preprocessing of assets.
+  # This option may cause significant delays in view rendering with a large
+  # number of complex assets.
+  config.assets.debug = false #true
+
+  config.assets.compress = false
+
+  config.log_level = :info
+
+  # Adds additional error checking when serving assets at runtime.
+  # Checks for improperly declared sprockets dependencies.
+  # Raises helpful error messages.
+  config.assets.raise_runtime_errors = true
+
+  # Raises error for missing translations
+  # config.action_view.raise_on_missing_translations = true
+
+  # MockSmtp settings
+  config.action_mailer.delivery_method = :smtp
+  config.action_mailer.smtp_settings = {
+    :address => "localhost",
+    :port => 1025,
+    :domain => "example.com"
+  }
+end

data/spec/dummy/config/mongoid.yml

@@ -5,14 +5,13 @@ development:
     default:
       # Defines the name of the default database that Mongoid can connect to.
       # (required).
-      database: locomotive_engine_coal_dev
-      # database: locomotive_engine_dev
-      # database: locomotive_engine_v3_dev
-      # database: locomotive_mounter_dev
+      # database: locomotive_engine_wagon_dev
+      database: locomotive_engine_dev
       # Provides the hosts the default session can connect to. Must be an array
       # of host:port pairs. (required)
       hosts:
         - localhost:27017
+      # uri: 'mongodb://localhost:27017/locomotive_engine_coal_dev'
       options:
         # Change whether the session persists in safe mode by default.
         # (default: false)
@@ -65,7 +64,7 @@ development:
 profile:
   sessions:
     default:
-      database: locomotive_engine_coal_dev
+      database: locomotive_engine_profile
       hosts:
         - localhost:27017
       options:
@@ -85,13 +84,14 @@ test:
 production:
   sessions:
     default:
-      database: locomotive_engine_coal_dev
+      database: locomotive_engine_prod
       hosts:
         - localhost:27017
       options:
         # consistency: :strong
         # identity_map_enabled: true
 
+
 # defaults: &defaults
 #   host: localhost
 #   port: 27017

data/spec/lib/locomotive/steam/services/api_entry_submission_service_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe Locomotive::Steam::APIEntrySubmissionService do
+
+  let(:site)          { create(:site) }
+  let(:locale)        { :en }
+  let!(:content_type) { create('message content type', site: site).reload }
+  let(:service)       { described_class.new(site, locale) }
+
+  describe '#submit' do
+
+    let(:attributes) { { 'name' => 'John Doe', 'message' => 'Hello' } }
+
+    subject { service.submit(content_type.slug, attributes) }
+
+    it { expect { subject }.to change { content_type.entries.count } }
+
+    it 'returns a content entry' do
+      expect(subject.name).to eq 'John Doe'
+      expect(subject.message).to eq 'Hello'
+    end
+
+  end #
+
+  describe '#to_json' do
+
+    let(:entry) { content_type.entries.build(_id: 42, name: 'John Doe', message: 'Hello') }
+
+    subject { service.to_json(entry) }
+
+    it 'includes _id and fields' do
+      expect(subject).to match('"_id":"42"')
+      expect(subject).to match('"name":"John Doe"')
+      expect(subject).to match('"message":"Hello"')
+    end
+
+  end
+
+end

data/spec/models/locomotive/concerns/content_type/entry_template_spec.rb

@@ -0,0 +1,55 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Locomotive::Concerns::ContentType::EntryTemplate do
+
+  before { allow_any_instance_of(Locomotive::Site).to receive(:create_default_pages!).and_return(true) }
+
+  let(:entry_template)  { nil }
+  let(:content_type)    { build_content_type(entry_template) }
+
+  describe '#valid?' do
+
+    let(:entry_template) { 'Hello world' }
+
+    before { content_type.valid? }
+
+    subject { content_type.errors }
+
+    it { expect(subject.size).to eq 0 }
+
+    context 'wrong template' do
+
+      let(:entry_template) { '{{ foo' }
+
+      it { expect(subject.size).to eq 1 }
+      it { expect(subject[:entry_template]).to eq ["Liquid syntax error: Variable '{{' was not properly terminated with regexp: /\\}\\}/"] }
+
+    end
+
+  end
+
+  describe '#render_entry_template' do
+
+    let(:entry_template) { '{{ entry.title }} world' }
+    let(:entry)     { content_type.entries.build(title: 'Hello') }
+    let(:registers) { { services: Locomotive::Steam::Services.build_instance } }
+    let(:context)   { ::Liquid::Context.new({}, { 'entry' => entry }, registers, true) }
+
+    before { content_type.save! }
+
+    subject { content_type.render_entry_template(context) }
+
+    it { expect(subject).to eq 'Hello world' }
+
+  end
+
+  def build_content_type(entry_template)
+    build(:content_type, entry_template: entry_template).tap do |content_type|
+      content_type.entries_custom_fields.build label: 'Title', type: 'string'
+      content_type.valid?
+    end
+  end
+
+end

data/spec/models/locomotive/concerns/page/layout_spec.rb

@@ -21,7 +21,7 @@ describe Locomotive::Concerns::Page::Layout do
 
     describe 'false for an existing page without the allow_layout attribute' do
 
-      let(:page) { p = Locomotive::Page.new; p.save(validate: false); p.unset(:allow_layout); Locomotive::Page.find(p._id) }
+      let(:page) { p = Locomotive::Page.new(site: site); p.save(validate: false); p.unset(:allow_layout); Locomotive::Page.find(p._id) }
 
       it { is_expected.to eq false }
 
@@ -37,7 +37,7 @@ describe Locomotive::Concerns::Page::Layout do
 
     it { is_expected.to eq '{% extends "awesome-layout" %}' }
 
-    describe 'but changing the layout is not allowed' do
+    context 'but changing the layout is not allowed' do
 
       let(:allow_layout) { false }
 

data/spec/models/locomotive/concerns/site/access_points_spec.rb

@@ -2,8 +2,10 @@ require 'spec_helper'
 
 describe Locomotive::Concerns::Site::AccessPoints do
 
- describe '#domains=' do
-    let(:site) { Locomotive::Site.new }
+  let(:domains) { [] }
+  let(:site) { build(:site, domains: domains) }
+
+  describe '#domains=' do
 
     it 'downcases the domains' do
       site.domains = ['FIRST.com', 'second.com', 'THIRD.com']
@@ -12,4 +14,87 @@ describe Locomotive::Concerns::Site::AccessPoints do
     end
   end
 
+  describe 'domain sync' do
+
+    let!(:listener) { DomainEventListener.new }
+    after { listener.shutdown }
+
+    subject { listener }
+
+    describe 'on saving' do
+
+      before  { site.save }
+
+      it 'does not emit an event' do
+        expect(subject.size).to eq 0
+      end
+
+      context 'new site' do
+
+        let(:domains) { ['www.example.com', 'example.com'] }
+
+        it 'only tracks new domains' do
+          expect(subject.added).to eq(['www.example.com', 'example.com'])
+          expect(subject.removed).to eq([])
+        end
+
+      end
+
+      context 'existing site' do
+
+        let(:domains) { ['www.boring.org', 'www.awesome.io'] }
+
+        before { listener.clear; site.domains = ['www.acme.com', 'www.awesome.io']; site.save; site.reload }
+
+        it 'tracks new domains and removed ones' do
+          expect(subject.added).to eq(['www.acme.com'])
+          expect(subject.removed).to eq(['www.boring.org'])
+        end
+
+      end
+
+    end
+
+    describe 'on destroying' do
+
+      let(:domains) { ['www.example.com', 'example.com'] }
+
+      before { site.save; listener.clear; site.destroy }
+
+      it 'tracks removed domains' do
+        expect(subject.added).to eq([])
+        expect(subject.removed).to eq(['www.example.com', 'example.com'])
+      end
+
+    end
+
+  end
+
+  class DomainEventListener
+    def initialize
+      @stack = []
+      @subscriber = ActiveSupport::Notifications.subscribe('locomotive.site.domain_sync') do |name, start, finish, id, payload|
+        emit(name, payload)
+      end
+    end
+    def emit(name, info = {})
+      @stack << [name, info]
+    end
+    def size
+      @stack.size
+    end
+    def added
+      @stack.map { |(_, info)| info[:added] }.flatten.compact
+    end
+    def removed
+      @stack.map { |(_, info)| info[:removed] }.flatten.compact
+    end
+    def clear
+      @stack.clear
+    end
+    def shutdown
+      ActiveSupport::Notifications.unsubscribe(@subscriber)
+    end
+  end
+
 end

data/spec/models/locomotive/concerns/site/locales_spec.rb

@@ -37,15 +37,14 @@ describe Locomotive::Concerns::Site::Locales do
   end
 
   def build_page(slug = nil, translations = {})
-    FactoryGirl.build(:page).tap do |page|
+    FactoryGirl.build(:page, site: site).tap do |page|
       page.slug = slug if slug
-      page.send(:build_fullpath)
       translations.each do |locale, translation|
         ::Mongoid::Fields::I18n.with_locale(locale) do
           page.slug = translation
-          page.send(:build_fullpath)
         end
       end
+      page.send(:build_fullpath)
     end
   end
 

data/spec/models/locomotive/content_entry_spec.rb

@@ -109,7 +109,7 @@ describe Locomotive::ContentEntry do
       expect(@content_entry.translated?).to eq(true)
     end
 
-    describe '.slug' do
+    describe '#slug' do
 
       it 'is not nil in the default locale' do
         ::Mongoid::Fields::I18n.locale = 'en'
@@ -120,6 +120,14 @@ describe Locomotive::ContentEntry do
         expect(@content_entry._slug).to eq(nil) # French
       end
 
+      it 'is translated in all the locales when being created' do
+        @content_entry.site.locales = %w(en fr)
+        ::Mongoid::Fields::I18n.locale = 'en'
+        @content_entry.send(:localize_slug)
+        ::Mongoid::Fields::I18n.locale = 'fr'
+        expect(@content_entry._slug).to eq('hello-world') # French
+      end
+
     end
 
   end
@@ -360,41 +368,6 @@ describe Locomotive::ContentEntry do
 
   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)
-      allow(site).to receive(:accounts).and_return([@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
-      expect(Locomotive::Notifications).to_not receive(:new_content_entry)
-      @content_entry.save
-    end
-
-    it 'does not send email notifications if no api accounts' do
-      @content_type.public_submission_accounts = nil
-      expect(Locomotive::Notifications).to_not receive(:new_content_entry)
-      @content_entry.save
-    end
-
-    it 'sends email notifications when a new instance is created' do
-      expect(Locomotive::Notifications).to receive(:new_content_entry).with(@account_1, @content_entry).and_return(instance_double('mailer', deliver: true))
-      expect(Locomotive::Notifications).to receive(:new_content_entry).with(@account_2, @content_entry).and_return(instance_double('mailer', deliver: true))
-      @content_entry.save
-    end
-
-  end
-
   describe '#site' do
 
     it 'assigns a site when saving the content entry' do
@@ -405,13 +378,25 @@ describe Locomotive::ContentEntry do
 
   end
 
+  describe '#to_liquid' do
+
+    let(:entry) { build_content_entry }
+
+    subject { entry.to_liquid }
+
+    it { expect(subject._label).to eq 'Locomotive' }
+
+  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', created_at: Time.zone.parse('2013-07-05 00:00:00') }.merge(options))
+    @content_type.entries.build({ title: 'Locomotive', description: 'Lorem ipsum....', _label_field_name: 'title', created_at: Time.zone.parse('2013-07-05 00:00:00') }.merge(options)).tap do |entry|
+      entry.send(:set_site)
+    end
   end
 
   def fake_bson_id(id)

data/spec/models/locomotive/page_spec.rb

@@ -128,6 +128,17 @@ describe Locomotive::Page do
 
   end
 
+  describe 'localizing slugs and fullpaths' do
+
+    let(:site)  { create(:site, locales: %w(en fr)) }
+
+    subject { create(:sub_page, site: site) }
+
+    it { expect(subject.slug_translations).to eq('en' => 'subpage', 'fr' => 'subpage') }
+    it { expect(subject.fullpath_translations).to eq('en' => 'subpage', 'fr' => 'subpage') }
+
+  end
+
   describe '#deleting' do
 
     before(:each) do