tenon 1.1.8 → 2.0.0

data/spec/lib/tenon/asset_style_generator_spec.rb

@@ -1,84 +0,0 @@
-require 'spec_helper'
-
-describe Tenon::AssetStyleGenerator do
-  let(:asg) { Tenon::AssetStyleGenerator.new(asset) }
-
-  describe '#styles' do
-    context 'with a Page that has a banner with a :thumbnail style and a :medium style' do
-      let(:asset) do
-        double(
-          item_assets: item_assets
-        )
-      end
-
-      let(:page) do
-        double(banner_styles: { medium: '800x800>', thumbnail: '300x300#' })
-      end
-
-      let(:item_assets) do
-        [double(item: page, asset_name: 'banner', item_type: 'Tenon::Page')]
-      end
-
-      before do
-        Tenon::Page.stub(:new) { page }
-      end
-
-      it 'should return a combination of the default styles and the banner styles' do
-        expect(asg.styles).to eq asg.send(:default_styles).merge(
-          tenon_page_banner_medium: '800x800>',
-          tenon_page_banner_thumbnail: '300x300#'
-        )
-      end
-
-      context 'with post with a header with a :wide style as well' do
-        let(:post) do
-          double(header_styles: { wide: '800x300#' })
-        end
-
-        let(:item_assets) do
-          [
-            double(item: page, asset_name: 'banner', item_type: 'Tenon::Page'),
-            double(item: post, asset_name: 'header', item_type: 'Tenon::Post')
-          ]
-        end
-
-        before do
-          Tenon::Post.stub(:new) { post }
-        end
-
-        it 'should return a combination of the default styles, the page banner styles, and the post header styles' do
-          expect(asg.styles).to eq asg.send(:default_styles).merge(
-            tenon_page_banner_medium: '800x800>',
-            tenon_page_banner_thumbnail: '300x300#',
-            tenon_post_header_wide: '800x300#'
-          )
-        end
-      end
-
-      context 'with a second page' do
-        let(:other_page) do
-          double(banner_styles: { medium: '800x800>', thumbnail: '300x300#' })
-        end
-
-        let(:item_assets) do
-          [
-            double(item: page, asset_name: 'banner', item_type: 'Tenon::Page'),
-            double(item: other_page, asset_name: 'banner', item_type: 'Tenon::Page')
-          ]
-        end
-
-        before do
-          Tenon::Page.stub(:new) { page }
-        end
-
-        it 'should return a combination of the default styles and the banner styles without duplicating' do
-          expect(asg.styles).to eq asg.send(:default_styles).merge(
-            tenon_page_banner_medium: '800x800>',
-            tenon_page_banner_thumbnail: '300x300#'
-          )
-        end
-      end
-    end
-
-  end
-end

data/spec/lib/tenon/can_be_foreign_spec.rb

@@ -1,20 +0,0 @@
-require 'spec_helper'
-
-connection = ActiveRecord::Base.connection
-connection.drop_table :dogs rescue nil
-connection.create_table :dogs do |t|
-  # Nada
-end
-
-class Dog < ActiveRecord::Base
-end
-
-describe Tenon::CanBeForeign do
-  before do
-    stub_const('Tenon::I18nFields', dog: 'dog_fields')
-  end
-
-  it 'should lookup the fields from Tenon::I18nFields' do
-    expect(Dog.foreign_fields).to eq 'dog_fields'
-  end
-end

data/spec/lib/tenon/can_have_comments_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-
-connection = ActiveRecord::Base.connection
-connection.drop_table :dogs rescue nil
-connection.create_table :dogs do |t|
-  # Nada
-end
-
-class Dog < ActiveRecord::Base
-  can_have_comments
-end
-
-describe Tenon::CanHaveComments do
-  let(:dog) { Dog.new }
-  it 'should respond_to :comments' do
-    expect(dog).to respond_to (:comments)
-  end
-
-  it 'should respond_to :subscribers' do
-    expect(dog).to respond_to(:subscribers)
-  end
-end

data/spec/lib/tenon/has_asset_spec.rb

@@ -1,46 +0,0 @@
-require 'spec_helper'
-include ActionDispatch::TestProcess
-
-connection = ActiveRecord::Base.connection
-connection.drop_table :dogs rescue nil
-connection.create_table :dogs do |t|
-  # Nada
-end
-
-class Dog < ActiveRecord::Base
-  has_asset :avatar, styles: { dog_size: '250x250#' }
-end
-
-describe Tenon::HasAsset do
-  let(:dog) { Dog.new }
-  context '.has_asset' do
-    %i(avatar_join avatar avatar= avatar_id=).each do |meth|
-      it "should define ##{meth}" do
-        expect(dog).to respond_to(meth)
-      end
-    end
-  end
-
-  context '#avatar' do
-    context 'when no avatar has been set' do
-      it 'should be nil' do
-        expect(dog.avatar).to be_nil
-      end
-    end
-
-    context 'when an avatar has been set on an unsaved dog' do
-      before do
-        dog.avatar = fixture_file_upload('files/test.png')
-      end
-
-      it 'should be a Tenon::ProxyAttachment' do
-        expect(dog.avatar).to be_a(Tenon::ProxyAttachment)
-      end
-
-      it 'should include a dog_size style on the original attachment' do
-        styles = dog.avatar.attachment.styles.keys
-        expect(styles).to include(:dog_avatar_dog_size)
-      end
-    end
-  end
-end

data/spec/lib/tenon/has_history/attr_serializer_spec.rb

@@ -1,63 +0,0 @@
-require 'spec_helper'
-
-describe Tenon::HasHistory::AttrSerializer do
-  let(:serializer) { Tenon::HasHistory::AttrSerializer.new(attrs, item_version) }
-  let(:item_version) { Tenon::ItemVersion.new(item: item) }
-  let(:item) { create(:page) }
-
-  context '#serialize' do
-    context 'with basic attrs' do
-      let(:attrs) { predefined_attrs[:basic] }
-
-      it 'should clear out the ID' do
-        expect(serializer.serialize).to eq(Marshal.dump(title: 'Title'))
-      end
-    end
-
-    context 'when the attrs have excepted fields' do
-      let(:attrs) { predefined_attrs[:with_excepted] }
-
-      it 'should clear out the ID' do
-        expect(serializer.serialize).to eq(Marshal.dump(title: 'Title'))
-      end
-    end
-
-    context 'when the attrs have children that are not included' do
-      let(:attrs) { predefined_attrs[:unincluded_children] }
-
-      it 'should clear them out' do
-        expect(serializer.serialize).to eq(Marshal.dump(title: 'Title'))
-      end
-    end
-
-    context 'when the attrs have children that are included' do
-      let(:attrs) { predefined_attrs[:included_children] }
-
-      it 'should keep them' do
-        expected = Marshal.dump(attrs.deep_reject_key!(:id))
-        expect(serializer.serialize).to eq(expected)
-      end
-    end
-  end
-end
-
-def predefined_attrs
-  {
-    basic: { title: 'Title', id: 14 },
-    with_excepted: { title: 'Title', lft: 3, rgt: 5, depth: 6 },
-    unincluded_children: {
-      title: 'Title',
-      fake_children_attributes: {
-        first: { test: 'item' },
-        second: { test: 'item' }
-      }
-    },
-    included_children: {
-      title: 'Title',
-      content_tenon_content_rows_attributes: {
-        first: { foo: 'bar', id: 1 },
-        second: { foo: 'bar', id: 2 }
-      }
-    }
-  }
-end

data/spec/lib/tenon/i18n_lookup_spec.rb

@@ -1,88 +0,0 @@
-require 'spec_helper'
-
-describe Tenon::I18nLookup do
-  describe '.new' do
-    context "when the given class is a Decorator" do
-      it 'assigns the decorated class.to_s to @klass' do
-        i = Tenon::I18nLookup.new(Tenon::Post.new.decorate.class)
-        expect(i.instance_variable_get('@klass')).to eq(Tenon::Post.new.class.to_s)
-      end
-    end
-
-    context "when the given class is not a Decorator" do
-      it 'assigns class.to_s to @klass' do
-        i = Tenon::I18nLookup.new(String)
-        expect(i.instance_variable_get('@klass')).to eq(String.to_s)
-      end
-    end
-  end
-
-  describe '#fields' do
-    context "when there's no match for the class" do
-      before do
-        Tenon::I18nLookup.stub(:fields) do {
-          tables: {}
-        }
-        end
-      end
-
-      it 'should return an empty array' do
-        expect(Tenon::I18nLookup.new(String).fields).to eq([])
-      end
-    end
-
-    context "when there's a match for the class" do
-      before do
-        Tenon::I18nLookup.stub(:fields) do {
-          tables: { strings: %w(a b c) }
-        }
-        end
-      end
-
-      it 'should return the field names' do
-        expect(Tenon::I18nLookup.new(String).fields).to eq(%w(a b c))
-      end
-    end
-  end
-
-  describe '.fields' do
-    before do
-      # Reset the class variable before each spec
-      Tenon::I18nLookup.class_variable_set :@@fields, nil
-    end
-
-    context 'when the file exists' do
-      before do
-        File.stub(:exist?) { true }
-      end
-
-      it 'should load the YAML' do
-        # for the love of Christ, make sure that the loaded YAML is a real
-        # representative of the file when loaded!
-        # alternatively, reload this with real data when done!
-        # MLK spent a shit-ton of time trying to fix testing errors caused
-        # by this:
-        # expect(YAML).to receive(:load) { {} }
-        expect(YAML).to receive(:load) { { tables: {} } }
-        Tenon::I18nLookup.fields
-      end
-
-      it 'should symbolize the keys' do
-        yaml = double
-        YAML.stub(:load) { yaml }
-        expect(yaml).to receive(:recursive_symbolize_keys!)
-        Tenon::I18nLookup.fields
-      end
-    end
-
-    context "when the file doens't exist" do
-      before do
-        File.stub(:exist?) { false }
-      end
-
-      it 'should return an empty set of data' do
-        expect(Tenon::I18nLookup.fields).to eq(tables: {})
-      end
-    end
-  end
-end

data/spec/lib/tenon/proxy_attachment_spec.rb

@@ -1,96 +0,0 @@
-require 'spec_helper'
-
-class Item; end
-
-module Tenon
-  class Item; end
-end
-
-describe Tenon::ProxyAttachment do
-  let(:pa) { Tenon::ProxyAttachment.new(attachment, klass, asset_name) }
-  let(:asset_name) { 'banner' }
-  let(:klass) { Item }
-
-  describe '.new' do
-    context 'when instantiating with an Asset' do
-      let(:attachment) { double(attachment: 'hello') }
-      it "should set the attachment as the Asset's attachment" do
-        expect(pa.attachment).to eq('hello')
-      end
-    end
-
-    context 'when instantiating with an attachment' do
-      let(:attachment) { double }
-      it 'should set the attachment as the attachment' do
-        expect(pa.attachment).to eq(attachment)
-      end
-    end
-  end
-
-  describe '#original_filename' do
-    let(:attachment) { double(original_filename: 'filename') }
-
-    it 'should get the original filename from the asset attachment' do
-      expect(pa.original_filename).to eq('filename')
-    end
-  end
-
-  describe '#display_name' do
-    let(:ad) { double(display_name: 'display') }
-    let(:attachment) { double.as_null_object }
-    before do
-      Tenon::AssetDecorator.stub(:new) { ad }
-    end
-
-    it 'should get the display name from the original asset' do
-      expect(pa.display_name).to eq(ad.display_name)
-    end
-  end
-
-  describe '#url' do
-    let(:attachment) { double(styles: styles) }
-    let(:styles) { nil }
-    context 'with a preceding underscore' do
-      it 'should get the url from the attachment' do
-        expect(attachment).to receive(:url).with('original')
-        pa.url(:_original)
-      end
-    end
-
-    context 'without a preceding underscore' do
-      it 'should get the url from the attachment including the prefix' do
-        expect(attachment).to receive(:url).with('item_banner_medium')
-        pa.url(:medium)
-      end
-
-      context 'when the model is namespaced' do
-        let(:klass) { Tenon::Item }
-
-        it 'should get the url from the attachment including the prefix' do
-          expect(attachment).to receive(:url).with('tenon_item_banner_medium')
-          pa.url(:medium)
-        end
-      end
-    end
-
-    context 'when asking for the orginal' do
-      context 'when an original style has been set on the model' do
-        let(:styles) { { item_banner_original: 'foo' } }
-
-        it 'should get the url from the attachment including the prefix' do
-          expect(attachment).to receive(:url).with(:item_banner_original)
-          pa.url(:original)
-        end
-      end
-
-      context 'when no original style has been set on the model' do
-        let(:styles) { {} }
-
-        it 'should get the url from the attachment without the prefix' do
-          expect(attachment).to receive(:url).with(:original)
-          pa.url(:original)
-        end
-      end
-    end
-  end
-end

data/spec/lib/tenon/reorderable_spec.rb

@@ -1,25 +0,0 @@
-require 'spec_helper'
-
-class DummyClass
-  include Tenon::Reorderable
-  attr_accessor :id, :list_order
-
-  def initialize(id)
-    @id = id
-  end
-
-  def save
-    true
-  end
-end
-
-describe 'Tenon::Reorderable' do
-  it 'should set the list order on the correct item' do
-    one, two, three = DummyClass.new(1), DummyClass.new(2), DummyClass.new(3)
-    DummyClass.stub(:find_by_id).with(1).and_return(one)
-    DummyClass.stub(:find_by_id).with(2).and_return(two)
-    DummyClass.stub(:find_by_id).with(3).and_return(three)
-    DummyClass.reorder!([2, 1, 3])
-    [one.list_order, two.list_order, three.list_order].should == [1, 0, 2]
-  end
-end

data/spec/lib/tenon/tenon_content_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-
-connection = ActiveRecord::Base.connection
-connection.drop_table :dogs rescue nil
-connection.create_table :dogs do |t|
-  # Nada
-end
-
-class Dog < ActiveRecord::Base
-  tenon_content :content
-end
-
-describe Tenon::TenonContent do
-  let(:dog) { Dog.new }
-  it 'should respond_to :content_tenon_content_rows' do
-    expect(dog).to respond_to(:content_tenon_content_rows)
-  end
-
-  it 'should respond_to :content_tenon_content_rows_attributes=' do
-    expect(dog).to respond_to(:content_tenon_content_rows_attributes=)
-  end
-end

data/spec/lib/tenon/warning_generator_spec.rb

@@ -1,153 +0,0 @@
-require 'spec_helper'
-
-describe Tenon::WarningGenerator do
-  let(:settings) { double.as_null_object }
-  let(:wg) { Tenon::WarningGenerator.new }
-  root_path = '/tenon/'
-  before do
-    stub_const('Tenon::MySettings', settings)
-  end
-
-  context 'when there is no site title' do
-    before do
-      settings.stub(:site_title) { nil }
-    end
-
-    it 'should include a warning about no site title' do
-      warning = "Your site needs a title.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>OK</a>"
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a site title' do
-    before do
-      settings.stub(:site_title) { 'test' }
-    end
-
-    it 'should not include a warning about no site title' do
-      warning = "Your site needs a title.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>OK</a>"
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-
-  context 'when there is no site domain' do
-    before do
-      settings.stub(:site_url) { nil }
-    end
-
-    it 'should include a warning about no site domain' do
-      warning = "Your site needs an address.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>OK</a>"
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a site domain' do
-    before do
-      settings.stub(:site_url) { 'test' }
-    end
-
-    it 'should not include a warning about no site title' do
-      warning = "Your site needs an address.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>OK</a>"
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-
-  context 'when there is no contact email' do
-    before do
-      settings.stub(:contact_email) { nil }
-    end
-
-    it 'should include a warning about no contact email' do
-      warning = "You haven't entered an email address to receive contact messages.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a contact email' do
-    before do
-      settings.stub(:contact_email) { 'test' }
-    end
-
-    it 'should include a warning about no contact email' do
-      warning = "You haven't entered an email address to receive contact messages.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-
-  context 'when there is no from email' do
-    before do
-      settings.stub(:from_email) { nil }
-    end
-
-    it 'should include a warning about no from email' do
-      warning = "You haven't entered an email address from which to send contact messages and other site notices.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a from email' do
-    before do
-      settings.stub(:from_email) { 'test' }
-    end
-
-    it 'should not include a warning about no from email' do
-      warning = "You haven't entered an email address from which to send contact messages and other site notices.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-
-  context 'when there is no Google Analytics' do
-    before do
-      settings.stub(:google_analytics) { nil }
-    end
-
-    it 'should include a warning about no Google Analytics' do
-      warning = "You haven't entered the Google Analytics code.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a Google Analytics' do
-    before do
-      settings.stub(:google_analytics) { 'test' }
-    end
-
-    it 'should not include a warning about no Google Analytics' do
-      warning = "You haven't entered the Google Analytics code.<br /><br /><a class='btn btn-comp btn-block' href='#{root_path}settings'>Correct This</a>"
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-
-  context 'when there is no favicon' do
-    before do
-      wg.stub(:favicon) { nil }
-    end
-
-    it 'should include a warning about no favicon' do
-      warning = 'You have not included a custom favicon.'
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a favicon but its size is 0' do
-    before do
-      wg.stub(:favicon) { double(size: 0) }
-    end
-
-    it 'should include a warning about no favicon' do
-      warning = 'You have not included a custom favicon.'
-      expect(wg.warnings(root_path)).to include(warning)
-    end
-  end
-
-  context 'when there is a good favicon' do
-    before do
-      wg.stub(:favicon) { double(size: 0.1) }
-    end
-
-    it 'should not include a warning about no favicon' do
-      warning = 'You have not included a custom favicon.'
-      expect(wg.warnings(root_path)).not_to include(warning)
-    end
-  end
-end