qbrick 2.5.0.pre → 2.5.0

data/lib/templates/qbrick/custom_model_generator/inherited_views/base/new.html.haml

@@ -0,0 +1,5 @@
+%h1= t("cms.new_title", singular_name: resource_class.model_name.human)
+
+= render 'form'
+
+= link_to t('cms.back'), collection_path

data/lib/templates/qbrick/custom_model_generator/translations/qbrick_base.yml

@@ -0,0 +1,11 @@
+de:
+  cms:
+    are_you_sure: 'Sind Sie sicher?'
+    destroy: 'Löschen'
+    edit: 'Bearbeiten'
+    back: 'Zurück'
+    index_title: "%{plural_name} Verwalten"
+    edit_title: "%{singular_name} Bearbeiten"
+    new_title: "%{singular_name} Erstellen"
+    default:
+      new: 'Neuer Eintrag'

data/lib/templates/qbrick/custom_model_generator/translations/resource.yml.erb

@@ -0,0 +1,14 @@
+<%= I18n.locale %>:
+  cms:
+    navigation_title:
+      <%= model_name %>: TRANSLATEME
+  activerecord:
+    models:
+      <%= model_name %>:
+        one: TRANSLATEME
+        other: TRNASLATEME
+    attributes:
+      <%= model_name -%>:
+        <%- attributes.each do |attr| -%>
+        <%= "#{attr}: TRANSLATE ME" %>
+        <%- end -%>

data/lib/templates/qbrick/translations/add_translation.erb

@@ -0,0 +1,8 @@
+class Add<%= formatted_locale.camelize %>Translation < ActiveRecord::Migration
+
+  def change
+    <%- translated_columns.each do |column_hash| %>
+      add_column(:qbrick_pages, "<%= get_attribute column_hash %>", :<%= get_type column_hash %>) unless Qbrick::Page.column_names.include?("<%= get_attribute column_hash %>")
+    <%- end %>
+  end
+end

data/spec/command_wrapper_helper.rb

@@ -0,0 +1,21 @@
+require 'active_support/all'
+
+[:rails, :rake].each do |command|
+  define_method command do |task|
+    Dir.chdir test_app_path do
+      # this is commented bc the output is annoying and not useful imho
+      # if you cant provide a reason to keep this i'll delete
+      # dbg_comment "cd #{test_app_path}"
+      reload_application_gemfile
+      `bundle exec "#{command} #{task}"`
+    end
+  end
+end
+
+# def dbg_comment(msg)
+#   puts " # #{ts} - #{msg}"
+# end
+
+# def ts
+#   Time.now.strftime('%H:%M:%S')
+# end

data/spec/controllers/{kuhsaft → qbrick}/api/pages_controller_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Kuhsaft::Api::PagesController do
+describe Qbrick::Api::PagesController, type: :controller do
   describe '#index' do
     before do
       @pages = []
@@ -15,7 +15,7 @@ describe Kuhsaft::Api::PagesController do
 
     it 'gets only published pages' do
       I18n.with_locale :de do
-        get :index, use_route: :kuhsaft
+        get :index, use_route: :qbrick
         expect(JSON.parse(response.body)).to eq([@page1, @page2].as_json)
       end
     end
@@ -23,12 +23,12 @@ describe Kuhsaft::Api::PagesController do
     it 'gets specific translated pages for each locale' do
       I18n.with_locale :de do
         @pages << @only_german = create(:page, published: true, title: 'foobar de', url: 'de/foobar-de')
-        get :index, use_route: :kuhsaft
+        get :index, use_route: :qbrick
         expect(JSON.parse(response.body)).to eq([@page1, @page2, @only_german].as_json)
       end
 
       I18n.with_locale :en do
-        get :index, use_route: :kuhsaft
+        get :index, use_route: :qbrick
         expect(JSON.parse(response.body)).to eq([@page1, @page2].as_json)
       end
     end
@@ -43,7 +43,7 @@ describe Kuhsaft::Api::PagesController do
                                        url_de: 'de/barfoo-de', title_en: 'barfoo en', url_en: 'en/barfoo-en')
 
       I18n.with_locale :de do
-        get :index, use_route: :kuhsaft
+        get :index, use_route: :qbrick
         @json = JSON.parse(response.body)
         @page_hash = @json.first
       end

data/spec/controllers/qbrick/cms/pages_controller_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+# TODO: For some reason rspec does not recognize this as controller
+#       spec even though it is in spec/controllers?
+describe Qbrick::Cms::PagesController, type: :controller do
+  subject { described_class }
+
+  describe 'mirroring' do
+    around(:each) do |example|
+      I18n.with_locale :de do
+        example.run
+      end
+    end
+
+    let(:page) { FactoryGirl.create(:page, url_de: 'de', url_en: 'en') }
+    let!(:brick) do
+      FactoryGirl.create(:text_brick,
+                         brick_list_id: page.id,
+                         brick_list_type: 'Qbrick::Page',
+                         text: 'DEUTSCH')
+    end
+    let!(:en_brick) do
+      FactoryGirl.create(:text_brick,
+                         brick_list_id: page.id,
+                         brick_list_type: 'Qbrick::Page',
+                         locale: 'en',
+                         text: 'ENGLISH')
+    end
+
+    before do
+      admin = double('admin')
+      allow_message_expectations_on_nil
+      allow(request.env['warden']).to receive(:authenticate!) { admin }
+      allow(controller).to receive(:current_admin) { admin }
+    end
+
+    context 'with no bricks on target locale' do
+      it 'clones the existing bricks' do
+        xhr :get, :mirror, use_route: :qbrick, target_locale: :en, page_id: page.id
+        I18n.with_locale :en do
+          expect(page.bricks.count).to eq(1)
+        end
+      end
+    end
+
+    context 'with bricks on target locale' do
+      it 'does not clone anything without the required parameter' do
+        xhr :get, :mirror, use_route: :qbrick, target_locale: :en, page_id: page.id
+        expect(page.bricks.unscoped.where(locale: :en, brick_list_id: page.id).first.text).to eq('ENGLISH')
+      end
+
+      it 'clones the bricks when required parameter is set' do
+        expect(page.bricks).to be_any
+        xhr :get, :mirror, use_route: :qbrick, target_locale: :en, page_id: page.id, rutheless: 'true'
+        I18n.with_locale :en do
+          expect(Qbrick::Page.find(page.id).bricks.first.text).to eq('DEUTSCH')
+        end
+      end
+    end
+  end
+end

data/spec/controllers/{kuhsaft → qbrick}/pages_controller_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Kuhsaft::PagesController do
+describe Qbrick::PagesController, type: :controller do
   subject { described_class }
 
   describe '#index' do
@@ -14,9 +14,9 @@ describe Kuhsaft::PagesController do
     context 'with search parameter' do
       it 'assigns the search results' do
         I18n.with_locale :de do
-          get(:index,  use_route: :kuhsaft, search: 'foobar')
+          get(:index,  use_route: :qbrick, search: 'foobar')
         end
-        assigns(:pages).should eq([@pages.first])
+        expect(assigns(:pages)).to eq([@pages.first])
       end
     end
   end
@@ -31,17 +31,15 @@ describe Kuhsaft::PagesController do
         context 'with matching locale' do
           it 'sets the corresponding page' do
             I18n.with_locale(:de) do
-              get(:show,   use_route: :kuhsaft)
+              get(:show,   use_route: :qbrick)
             end
-            assigns(:page).should eq(@page)
+            expect(assigns(:page)).to eq(@page)
           end
         end
 
         context 'without matching locale' do
           it 'raises a routing error' do
-            I18n.with_locale(:en) do
-              expect { get(:show,   use_route: :kuhsaft) }.to raise_error(ActionController::RoutingError)
-            end
+            expect { get(:show, use_route: :qbrick, locale: :en) }.to raise_error(ActionController::RoutingError)
           end
         end
       end
@@ -58,8 +56,8 @@ describe Kuhsaft::PagesController do
         it 'responds with page' do
           page = FactoryGirl.create(:page, slug: 'dumdidum',
                                            url: 'de/dumdidum')
-          get :show,   url: page.slug, use_route: :kuhsaft
-          assigns(:page).should eq(page)
+          get :show,   url: page.slug, use_route: :qbrick
+          expect(assigns(:page)).to eq(page)
         end
       end
 
@@ -67,21 +65,21 @@ describe Kuhsaft::PagesController do
         it 'redirects to the redirected url' do
           page = FactoryGirl.create(:page, page_type: 'redirect', slug: 'dumdidum',
                                            url: 'de/dumdidum', redirect_url: 'de/redirect_page')
-          get :show,   url: page.slug, use_route: :kuhsaft
+          get :show,   url: page.slug, use_route: :qbrick
           expect(response).to redirect_to('/de/redirect_page')
         end
 
         it 'redirects to invalid redirect urls with too many preceding slashes' do
           page = FactoryGirl.create(:page, page_type: 'redirect', slug: 'dumdidum',
                                            url: 'de/dumdidum', redirect_url: '///de/redirect_page')
-          get :show,   url: page.slug, use_route: :kuhsaft
+          get :show,   url: page.slug, use_route: :qbrick
           expect(response).to redirect_to('/de/redirect_page')
         end
 
         it 'redirects to root' do
           page = FactoryGirl.create(:page, page_type: 'redirect', slug: 'dumdidum',
                                            url: 'de/dumdidum', redirect_url: '/')
-          get :show,   url: page.slug, use_route: :kuhsaft
+          get :show,   url: page.slug, use_route: :qbrick
           expect(response).to redirect_to('/')
         end
       end

data/spec/controllers/{kuhsaft → qbrick}/sitemaps_controller_spec.rb

@@ -1,13 +1,13 @@
 require 'spec_helper'
 
-describe Kuhsaft::SitemapsController do
+describe Qbrick::SitemapsController, type: :controller do
   describe '#index' do
     before do
       @page = FactoryGirl.create(:page)
     end
 
     it 'should be able to send a xml file' do
-      get(:index,  use_route: :kuhsaft, format: 'xml')
+      get(:index,  use_route: :qbrick, format: 'xml')
     end
   end
 end

data/spec/dummy/app/controllers/application_controller.rb

@@ -1,4 +1,4 @@
 class ApplicationController < ActionController::Base
   protect_from_forgery
-  helper Kuhsaft::Engine.helpers
+  helper Qbrick::Engine.helpers
 end

data/spec/dummy/config/application.rb

@@ -6,7 +6,7 @@ require 'rails/all'
 # you've limited to :test, :development, or :production.
 Bundler.require(:default, Rails.env)
 
-require 'kuhsaft'
+require 'qbrick'
 
 module Dummy
   class Application < Rails::Application
@@ -19,8 +19,9 @@ module Dummy
     # config.time_zone = 'Central Time (US & Canada)'
 
     # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
-    # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
+    config.i18n.load_path += Dir[Rails.root.join('config', 'locales', '**', '*.{rb,yml}').to_s]
     config.i18n.available_locales = [:de, :en]
     #config.i18n.default_locale = :de
+    config.active_record.raise_in_transactional_callbacks = true
   end
 end

data/spec/dummy/config/database.yml

@@ -1,6 +1,6 @@
 development:
   adapter: postgresql
-  database: kuhsaft_development
+  database: qbrick_development
   username: screenconcept
   password:
   host: localhost
@@ -10,7 +10,7 @@ development:
 
 test:
   adapter: postgresql
-  database: kuhsaft_test
+  database: qbrick_test
   username: screenconcept
   password:
   host: localhost

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

@@ -20,7 +20,7 @@ Dummy::Application.configure do
   # config.action_dispatch.rack_cache = true
 
   # Disable Rails's static asset server (Apache or nginx will already do this).
-  config.serve_static_assets = false
+  config.serve_static_files = false
 
   # Compress JavaScripts and CSS.
   config.assets.js_compressor = :uglifier
@@ -60,7 +60,7 @@ Dummy::Application.configure do
   # Precompile additional assets.
   # application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
   # config.assets.precompile += %w( search.js )
-  config.assets.precompile += %w( kuhsaft/cms/customizations.css kuhsaft/cms/customizations.js )
+  config.assets.precompile += %w( qbrick/cms/customizations.css qbrick/cms/customizations.js )
 
   # Ignore bad email addresses and do not raise email delivery errors.
   # Set this to true and configure the email server for immediate delivery to raise delivery errors.

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

@@ -13,7 +13,7 @@ Dummy::Application.configure do
   config.eager_load = false
 
   # Configure static asset server for tests with Cache-Control for performance.
-  config.serve_static_assets  = true
+  config.serve_static_files  = true
   config.static_cache_control = "public, max-age=3600"
 
   # Show full error reports and disable caching.

data/spec/dummy/config/initializers/{kuhsaft.rb → qbrick.rb}

@@ -1,5 +1,5 @@
 Rails.application.config.to_prepare do
-  Kuhsaft::Engine.configure do
+  Qbrick::Engine.configure do
     config.image_sizes.build_defaults! # creates 960x540 and 320x180 sizes
   end
 end

data/spec/dummy/config/routes.rb

@@ -1,3 +1,6 @@
 Dummy::Application.routes.draw do
-  mount Kuhsaft::Engine => "/"
+  scope "(:locale)", locale: /#{I18n.available_locales.join('|')}/ do
+    root "qbrick/pages#show"
+  end
+  mount Qbrick::Engine => "/"
 end

data/spec/factories.rb

@@ -1,26 +1,41 @@
 FactoryGirl.define do
-  sequence(:title) { |n| n }
+  sequence(:title) { |n| "English Title #{n}" }
 
-  factory :page, class: 'Kuhsaft::Page' do |p|
+  factory :page, class: 'Qbrick::Page' do |p|
     p.parent nil
     p.position 1
-    p.title { "English Title #{FactoryGirl.generate(:title)}" }
+    p.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'
+  factory :text_brick, class: 'Qbrick::TextBrick' do |tb|
+    tb.type 'Qbrick::TextBrick'
     tb.text 'DummyText'
   end
 
-  factory :placeholder_brick, class: 'Kuhsaft::PlaceholderBrick' do |tb|
-    tb.type 'Kuhsaft::PlaceholderBrick'
+  factory :placeholder_brick, class: 'Qbrick::PlaceholderBrick' do |tb|
+    tb.type 'Qbrick::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")
+  factory :image_brick, class: 'Qbrick::ImageBrick' do |ib|
+    ib.image File.open("#{Qbrick::Engine.root}/spec/dummy/app/assets/images/spec-image.png")
+    ib.image_size 'gallery'
+  end
+
+  factory :asset, class: Qbrick::Asset do |a|
+    a.file File.open("#{Qbrick::Engine.root}/spec/dummy/app/assets/images/spec-image.png")
+  end
+
+  factory :setting, class: Qbrick::Setting do |s|
+    s.key 'code'
+    s.value 'SomeValue'
+  end
+
+  factory :admin, class: Qbrick::Admin do
+    email 'test@test.com'
+    password 'somel33tPW'
   end
 end

data/spec/features/administrator_management_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+
+describe 'Administrator Management', type: :feature do
+  def create_admin
+    @admin ||= FactoryGirl.create(:admin)
+  end
+
+  def create_and_login_admin
+    create_admin
+    visit qbrick.cms_pages_path
+    fill_in 'E-Mail', with: @admin.email
+    fill_in 'Password', with: @admin.password
+    click_on 'Login'
+  end
+
+  before :each do
+    @page = FactoryGirl.create(:page, page_type: 'navigation', published: true, title: 'home')
+    create_and_login_admin
+  end
+
+  describe 'admin' do
+    it 'can log in with his/her credentials' do
+      expect(page).to have_content 'Signed in successfully'
+    end
+
+    it 'can log out' do
+      click_on 'Log out'
+      visit qbrick.cms_pages_path
+      expect(current_url).to eq(qbrick.new_admin_session_url)
+    end
+
+    context 'when logged in' do
+      let(:new_password) { 'newAdminPW!' }
+
+      it 'can change his/her password' do
+        click_on 'Change Password'
+        fill_in 'Current Password', with: @admin.password
+        fill_in 'Password', with: new_password
+        fill_in 'Password Confirmation', with: new_password
+        expect { click_on 'Update Admin' }.to change { Qbrick::Admin.find_by_email(@admin.email).encrypted_password }
+      end
+
+      it 'can create a new admin user' do
+        click_on 'Admins'
+        click_on 'New'
+        fill_in 'E-Mail', with: 'another@admin.com'
+        fill_in 'Password', with: 'fancyPassword33'
+        fill_in 'Password Confirmation', with: 'fancyPassword33'
+        expect { click_on 'Create Admin' }.to change { Qbrick::Admin.count }
+      end
+
+      it 'can edit an admin' do
+        FactoryGirl.create(:admin, email: 'somemail@admin.com')
+        click_on 'Admins'
+        find('tr', text: 'somemail@admin.com').find_link('Edit').click
+        fill_in 'E-Mail', with: 'new_email@admin.com'
+        fill_in 'Password', with: 'fancyPassword33'
+        fill_in 'Password Confirmation', with: 'fancyPassword33'
+        click_on 'Update Admin'
+        expect(page).to have_content('new_email@admin.com')
+      end
+
+      it 'can delete an admin' do
+        FactoryGirl.create(:admin, email: 'somemail@admin.com')
+        click_on 'Admins'
+        expect { find('tr', text: 'somemail@admin.com').find_link('Delete').click }.to change { Qbrick::Admin.count }
+      end
+    end
+  end
+end