qbrick 2.5.0.pre

data/spec/dummy/script/rails

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/spec/factories.rb

@@ -0,0 +1,26 @@
+FactoryGirl.define do
+  sequence(:title) { |n| n }
+
+  factory :page, class: 'Kuhsaft::Page' do |p|
+    p.parent nil
+    p.position 1
+    p.title { "English Title #{FactoryGirl.generate(:title)}" }
+    p.published 1
+    p.body 'lorem ipsum'
+    p.url ''
+  end
+
+  factory :text_brick, class: 'Kuhsaft::TextBrick' do |tb|
+    tb.type 'Kuhsaft::TextBrick'
+    tb.text 'DummyText'
+  end
+
+  factory :placeholder_brick, class: 'Kuhsaft::PlaceholderBrick' do |tb|
+    tb.type 'Kuhsaft::PlaceholderBrick'
+    tb.template_name 'foo'
+  end
+
+  factory :asset, class: Kuhsaft::Asset do |a|
+    a.file File.open("#{Kuhsaft::Engine.root}/spec/dummy/app/assets/images/spec-image.png")
+  end
+end

data/spec/features/cms_pages_spec.rb

@@ -0,0 +1,76 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'Cms/Pages' do
+
+  context '#new' do
+    before do
+      visit kuhsaft.new_cms_page_path
+      fill_in 'Title', with: 'The Title of the page'
+      fill_in 'Keywords', with: 'My keywords'
+      fill_in 'Description', with: 'My Description'
+    end
+
+    describe '#create' do
+      context 'when page is valid' do
+        it 'creates a new page' do
+          expect { click_on 'Create Page' }.to change(Kuhsaft::Page, :count).by(1)
+        end
+
+        it 'is not possible to change the value in url' do
+          page.find('#page_url')['disabled'].should be_true
+        end
+      end
+
+      context 'when page is invalid' do
+        it 'does not create a routing error by switching the locale' do
+          @page = FactoryGirl.create(:page, title: 'DummyPage', title_en: 'DummyEN', slug: 'dummy_page')
+          visit kuhsaft.edit_cms_page_path(@page)
+          fill_in 'page_title', with: ''
+          click_on 'Update Page'
+          within '.nav-pills' do
+            click_on 'EN'
+          end
+          page.should have_content(@page.title_en)
+        end
+      end
+    end
+
+    describe '#update' do
+      context 'when creating a redirect page' do
+        before do
+          @page = FactoryGirl.create(:page, url: 'de/dumdidum')
+          visit kuhsaft.edit_cms_page_path(@page)
+          select 'redirect', from: 'Pagetyp'
+        end
+
+        it 'has a value in redirect_page' do
+          fill_in 'Redirect URL', with: 'target_page'
+          expect { click_on 'Update Page' }.to change { @page.reload.redirect_url }.to('target_page')
+        end
+
+        it 'is invalid when no value is in redirect_page' do
+          click_on 'Update Page'
+          page.should have_css('.error', count: 1)
+        end
+
+        it 'does not change the value in url' do
+          fill_in 'Redirect URL', with: 'target_page'
+          expect { click_on 'Update Page' }.to_not change { @page.reload.url }
+        end
+      end
+    end
+  end
+
+  describe '#edit' do
+    it 'shows error messages on invalid bricks' do
+      @page = FactoryGirl.create(:page)
+      invalid_brick = FactoryGirl.build(:text_brick, text: nil, brick_list: @page)
+      invalid_brick.save(validate: false)
+
+      visit kuhsaft.edit_cms_page_path(@page)
+      page.should have_css('.error', count: 1)
+    end
+  end
+end

data/spec/features/search_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe 'pages#index' do
+  context 'with search parameter' do
+    let! :page1 do
+      p = create :page,
+                 published: true,
+                 title: 'Chromodorididae Ardeadoris'
+      p.bricks << Kuhsaft::TextBrick.new(locale: I18n.locale,
+                                         text: "#{'foo bar' * 300} Chromodorididae #{'foo bar' * 300}")
+      p.save!
+      p
+    end
+
+    let! :page2 do
+      create :page,
+             published: true,
+             title: 'Chromodorididae Berlanguella'
+    end
+
+    let! :page3 do
+      create :page,
+             published: true,
+             title: 'Gastropoda'
+    end
+
+    context 'with fulltext' do
+      before do
+        visit kuhsaft.pages_path(locale: :en, search: 'Chromodorididae')
+      end
+
+      it 'highlights search term in preview' do
+        within('ul.search-results.success') do
+          page.should have_content('Chromodorididae')
+        end
+      end
+
+      it 'truncates the text' do
+        find('.summary .excerpt').text.length.should == 110
+      end
+    end
+
+    context 'with multiple matches' do
+      before do
+        visit kuhsaft.pages_path(locale: :en, search: 'Chromodorididae')
+      end
+
+      it 'renders match count' do
+        page.should have_content('2 results')
+      end
+
+      it 'renders the search results list' do
+        within('ul.search-results.success') do
+          page.should have_content('Chromodorididae Ardeadoris')
+          page.should have_content('Chromodorididae Berlanguella')
+          page.should_not have_content('Gastropoda')
+        end
+      end
+
+      it 'renders links to the pages' do
+        within('ul.search-results.success') do
+          page.should have_link('Chromodorididae Ardeadoris', href: page1.link)
+          page.should have_link('Chromodorididae Berlanguella', href: page2.link)
+        end
+      end
+    end
+
+    context 'without matches' do
+      before do
+        visit kuhsaft.pages_path(locale: :en, search: 'foobar')
+      end
+
+      it 'renders match count' do
+        page.should have_content('No results')
+      end
+    end
+  end
+end

data/spec/helpers/kuhsaft/cms/admin_helper_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe Kuhsaft::Cms::AdminHelper do
+  describe '#render_language_switch?' do
+    context 'when there is one language' do
+      before do
+        I18n.stub(:available_locales).and_return([:de])
+      end
+
+      it 'returns false' do
+        helper.render_language_switch?.should be_false
+      end
+    end
+
+    context 'when there are multiple languages' do
+      before do
+        I18n.stub(:available_locales).and_return([:de, :en])
+      end
+
+      it 'returns true' do
+        helper.render_language_switch?.should be_true
+      end
+    end
+  end
+end

data/spec/helpers/kuhsaft/cms/pages_helper_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe Kuhsaft::Cms::PagesHelper do
+  describe '#content_tab_active' do
+    it 'returns active when page has a title and no errors' do
+      @page = create(:page, title: 'Page 1', slug: 'page1')
+      expect(helper.content_tab_active(@page)).to be(:active)
+    end
+
+    it 'returns nil when page has no translation' do
+      @page = create(:page, title: 'Page 1', slug: 'page1')
+      I18n.with_locale :de do
+        expect(helper.content_tab_active(@page)).to be_nil
+      end
+    end
+  end
+
+  describe '#metadata_tab_active' do
+    it 'returns active when page is not translated' do
+      @page = create(:page, title: 'Page 1', slug: 'page1')
+      I18n.with_locale :de do
+        expect(helper.metadata_tab_active(@page)).to be(:active)
+      end
+    end
+  end
+
+  describe '#hide_content_tab?' do
+    it 'has a page without translations' do
+      @page = create(:page, title: 'Page 1', slug: 'page1')
+      I18n.with_locale :de do
+        expect(helper.hide_content_tab?(@page)).to be_true
+      end
+    end
+
+    it 'has a redirect page' do
+      @page = create(:page, title: 'Page 1', slug: 'page1',
+                            page_type: Kuhsaft::PageType::REDIRECT, redirect_url: 'en/references')
+      expect(helper.hide_content_tab?(@page)).to be_true
+    end
+
+    it 'has a not saved page' do
+      @page = Kuhsaft::Page.new
+      expect(helper.hide_content_tab?(@page)).to be_true
+    end
+  end
+end

data/spec/helpers/kuhsaft/pages_helper_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe PagesHelper do
+  describe '#search_page_form' do
+
+    context 'without block' do
+      it 'renders the default search form' do
+        form = search_page_form
+        form.should have_css('form.form-inline')
+        form.should have_css('input[type=text]')
+        form.should have_css('input[type=submit]')
+      end
+    end
+
+    context 'with block' do
+      it 'calls the given block' do
+        expect { |b| search_page_form(&b) }.to yield_with_no_args
+      end
+    end
+  end
+end

data/spec/kuhsaft_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Kuhsaft do
+  it 'should be valid' do
+    Kuhsaft.should be_a(Module)
+  end
+end

data/spec/lib/brick_list_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Kuhsaft::BrickList do
+
+  class TestBrick
+    include Kuhsaft::BrickList
+  end
+
+  let :brick do
+    TestBrick.new
+  end
+
+  describe '#collect_fulltext' do
+    context 'with bricks' do
+      it 'collects its childs fulltext' do
+        brick.stub_chain(:bricks, :localized).and_return([mock_model(Kuhsaft::Brick, collect_fulltext: 'hallo')])
+        brick.collect_fulltext.should == 'hallo'
+      end
+    end
+
+    context 'with bricks without content' do
+      it 'returns a string' do
+        brick.stub_chain(:bricks, :localized).and_return([])
+        brick.collect_fulltext.should == ''
+      end
+    end
+
+    context 'without bricks' do
+      it 'returns a string' do
+        brick.collect_fulltext.should == ''
+      end
+
+      it 'does not fail' do
+        expect { brick.collect_fulltext }.to_not raise_error
+      end
+    end
+  end
+
+  describe '#allowed_brick_types' do
+    it 'returns an array of possible classes as strings' do
+      brick.allowed_brick_types.should be_a(Array)
+    end
+  end
+
+  describe '#brick_types' do
+    it 'returns a Kuhsaft::BrickTypeFilter' do
+      brick.brick_types.should be_a(Kuhsaft::BrickTypeFilter)
+    end
+  end
+end

data/spec/lib/engine_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe Kuhsaft::Engine do
+
+  describe '.configure' do
+
+    describe '.image_sizes' do
+      it 'delegates .clear to ImageSize' do
+        Kuhsaft::ImageSize.should_receive(:clear!)
+        Kuhsaft::Engine.configure do
+          config.image_sizes.clear!
+        end
+      end
+
+      it 'delegates .add to ImageSize' do
+        Kuhsaft::ImageSize.should_receive(:add).with(:something, 100, 200)
+        Kuhsaft::Engine.configure do
+          config.image_sizes.add(:something, 100, 200)
+        end
+      end
+    end
+
+  end
+
+end

data/spec/lib/gridded_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Kuhsaft::Gridded do
+
+  context 'included gridded module' do
+    class GridClass
+      include Kuhsaft::Gridded
+    end
+
+    it 'returns an array for the gridded class' do
+      expect(GridClass.available_grid_sizes).to be_kind_of(Array)
+    end
+
+    it 'returns false on gridded? if no col count is set' do
+      expect_any_instance_of(GridClass).to receive(:col_count).at_least(:once).and_return(0)
+      expect(GridClass.new.gridded?).to be_false
+    end
+
+    it 'returns true on gridded? if a col count is set' do
+      expect_any_instance_of(GridClass).to receive(:col_count).at_least(:once).and_return(10)
+      expect(GridClass.new.gridded?).to be_true
+    end
+  end
+end

data/spec/lib/image_uploader_mounting_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe Kuhsaft::ImageUploaderMounting do
+
+  let :brick do
+    Kuhsaft::ImageBrick.new
+  end
+
+  describe '#uploader_mounting' do
+    it 'has a uploader mounted' do
+      brick.class.ancestors.include?(CarrierWave::Mount::Extension).should be_true
+    end
+  end
+end

data/spec/lib/page_tree_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+require_relative '../../lib/kuhsaft/page_tree'
+require_relative '../../app/models/kuhsaft/page'
+
+# TODO: THESE SPECS ONLY WORK WHEN THE FULL SUITE IS RUN. FIX THAT!
+
+module Kuhsaft
+  describe PageTree do
+    let(:page_tree) do
+      {
+        '0' => { 'id' => '1', 'children' => { '0' => { 'id' => '2' } } },
+        '1' => { 'id' => '3' }
+      }
+    end
+
+    before :each do
+      @page1 = FactoryGirl.create(:page, id: 1, position: 10)
+      @page2 = FactoryGirl.create(:page, id: 2, position: 10)
+      @page3 = FactoryGirl.create(:page, id: 3, position: 10)
+    end
+
+    describe 'update' do
+      it 'sets the correct position of the root nodes' do
+        PageTree.update(page_tree)
+        expect(@page1.reload.position).to eq(0)
+        expect(@page2.reload.position).to eq(0)
+        expect(@page3.reload.position).to eq(1)
+      end
+
+      it 'sets the correct parent attribute for the nodes' do
+        PageTree.update(page_tree)
+        @page1.reload.parent_id.should be_nil
+        expect(@page2.reload.parent_id).to eq(1)
+        @page3.reload.parent_id.should be nil
+      end
+    end
+  end
+end