qbrick 2.5.0.pre → 2.5.0

data/spec/models/placeholder_brick_spec.rb

@@ -1,46 +1,45 @@
 require 'spec_helper'
 require 'fileutils'
 
-describe Kuhsaft::PlaceholderBrick do
-
+describe Qbrick::PlaceholderBrick, type: :model do
   let :placeholder_brick do
-    Kuhsaft::PlaceholderBrick.new
+    Qbrick::PlaceholderBrick.new
   end
 
   before do
-    FileUtils.mkdir_p("#{Rails.root}/app/views/kuhsaft/placeholder_bricks/partials")
-    FileUtils.touch("#{Rails.root}/app/views/kuhsaft/placeholder_bricks/partials/_valid_partial.html.haml")
-    FileUtils.touch("#{Rails.root}/app/views/kuhsaft/placeholder_bricks/partials/not_a_partial.html.haml")
-    FileUtils.touch("#{Rails.root}/app/views/kuhsaft/placeholder_bricks/partials/_not_a_haml_file.txt")
+    FileUtils.mkdir_p("#{Rails.root}/app/views/qbrick/placeholder_bricks/partials")
+    FileUtils.touch("#{Rails.root}/app/views/qbrick/placeholder_bricks/partials/_valid_partial.html.haml")
+    FileUtils.touch("#{Rails.root}/app/views/qbrick/placeholder_bricks/partials/not_a_partial.html.haml")
+    FileUtils.touch("#{Rails.root}/app/views/qbrick/placeholder_bricks/partials/_not_a_haml_file.txt")
   end
 
   after do
-    FileUtils.rm_rf(Dir.glob("#{Rails.root}/app/views/kuhsaft"))
+    FileUtils.rm_rf(Dir.glob("#{Rails.root}/app/views/qbrick"))
   end
 
   describe 'available partials' do
     it 'returns haml files' do
-      Kuhsaft::PlaceholderBrick.available_partials.flatten.should include('valid_partial')
+      expect(Qbrick::PlaceholderBrick.available_partials.flatten).to include('valid_partial')
     end
 
     it 'returns only partials' do
-      Kuhsaft::PlaceholderBrick.available_partials.flatten.should_not include('not_a_partial')
+      expect(Qbrick::PlaceholderBrick.available_partials.flatten).not_to include('not_a_partial')
     end
 
     it 'does not return other files' do
-      Kuhsaft::PlaceholderBrick.available_partials.flatten.should_not include('not_a_haml_file')
+      expect(Qbrick::PlaceholderBrick.available_partials.flatten).not_to include('not_a_haml_file')
     end
   end
 
   describe '#bricks' do
     it 'can not have childs' do
-      placeholder_brick.should_not respond_to(:bricks)
+      expect(placeholder_brick).not_to respond_to(:bricks)
     end
   end
 
   describe '#user_can_add_childs?' do
     it 'returns false' do
-      placeholder_brick.user_can_add_childs?.should be_false
+      expect(placeholder_brick.user_can_add_childs?).to be_falsey
     end
   end
 end

data/spec/models/publish_state_spec.rb

@@ -1,33 +1,33 @@
 require 'spec_helper'
 
-describe Kuhsaft::PublishState do
+describe Qbrick::PublishState, type: :model do
   context 'unpublished' do
     before do
-      @publish_state = Kuhsaft::PublishState.new(name: 'unpublished', value: Kuhsaft::PublishState::UNPUBLISHED)
+      @publish_state = Qbrick::PublishState.new(name: 'unpublished', value: Qbrick::PublishState::UNPUBLISHED)
     end
 
     it 'should be UNPUBLISHED' do
-      @publish_state.value.should be(Kuhsaft::PublishState::UNPUBLISHED)
+      expect(@publish_state.value).to be(Qbrick::PublishState::UNPUBLISHED)
     end
   end
 
   context 'published' do
     before do
-      @publish_state = Kuhsaft::PublishState.new(name: 'published', value: Kuhsaft::PublishState::PUBLISHED)
+      @publish_state = Qbrick::PublishState.new(name: 'published', value: Qbrick::PublishState::PUBLISHED)
     end
 
     it 'should be PUBLISHED' do
-      @publish_state.value.should be(Kuhsaft::PublishState::PUBLISHED)
+      expect(@publish_state.value).to be(Qbrick::PublishState::PUBLISHED)
     end
   end
 
   context 'published_at' do
     before do
-      @publish_state = Kuhsaft::PublishState.new(name: 'published_at', value: Kuhsaft::PublishState::PUBLISHED_AT)
+      @publish_state = Qbrick::PublishState.new(name: 'published_at', value: Qbrick::PublishState::PUBLISHED_AT)
     end
 
     it 'should be PUBLISHED_AT' do
-      @publish_state.value.should be(Kuhsaft::PublishState::PUBLISHED_AT)
+      expect(@publish_state.value).to be(Qbrick::PublishState::PUBLISHED_AT)
     end
   end
 end

data/spec/models/setting_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+module Qbrick
+  describe Setting, type: :model do
+    before do
+      create(:setting)
+      create(:setting, key: 'stuff', value: nil)
+    end
+
+    describe '[](key)' do
+      context 'setting exists' do
+        it 'returns the set value' do
+          expect(Setting[:code]).to eq('SomeValue')
+        end
+
+        it 'returns empty string for empty setting' do
+          expect(Setting[:stuff]).to eq('')
+        end
+      end
+
+      context "setting doesn't exist" do
+        it 'returns empty string for a call on a non-present setting' do
+          expect(Setting[:does_not_exist]).to eq('')
+        end
+      end
+    end
+  end
+end

data/spec/models/slider_brick_spec.rb

@@ -1,26 +1,25 @@
 require 'spec_helper'
 
-describe Kuhsaft::SliderBrick do
-
+describe Qbrick::SliderBrick, type: :model do
   let :slider_brick do
-    Kuhsaft::SliderBrick.new
+    Qbrick::SliderBrick.new
   end
 
   describe '#bricks' do
     it 'can have childs' do
-      slider_brick.should respond_to(:bricks)
+      expect(slider_brick).to respond_to(:bricks)
     end
   end
 
   describe '#to_style_class' do
     it 'includes the bootstrap styles' do
-      slider_brick.to_style_class.should == 'kuhsaft-slider-brick carousel slide'
+      expect(slider_brick.to_style_class).to eq('qbrick-slider-brick carousel slide')
     end
   end
 
   describe '#allowed_brick_types' do
     it 'only allows ImageBricks and VideoBricks' do
-      slider_brick.allowed_brick_types.should == %w(Kuhsaft::ImageBrick Kuhsaft::VideoBrick)
+      expect(slider_brick.allowed_brick_types).to eq(%w(Qbrick::ImageBrick Qbrick::VideoBrick))
     end
   end
 end

data/spec/models/text_brick_spec.rb

@@ -1,19 +1,19 @@
 require 'spec_helper'
 
-describe Kuhsaft::TextBrick do
+describe Qbrick::TextBrick, type: :model do
   let :text_brick do
-    Kuhsaft::TextBrick.new
+    Qbrick::TextBrick.new
   end
 
   describe '#bricks' do
     it 'can not have childs' do
-      text_brick.should_not respond_to(:bricks)
+      expect(text_brick).not_to respond_to(:bricks)
     end
   end
 
   describe '#user_can_add_childs?' do
     it 'returns false' do
-      text_brick.user_can_add_childs?.should be_false
+      expect(text_brick.user_can_add_childs?).to be_falsey
     end
   end
 
@@ -24,7 +24,7 @@ describe Kuhsaft::TextBrick do
     end
 
     it 'sanitizes text and read_more_text' do
-      text_brick.collect_fulltext.should == 'foo bar foobar'
+      expect(text_brick.collect_fulltext).to eq('foo bar foobar')
     end
   end
 end

data/spec/models/two_column_brick_spec.rb

@@ -1,82 +1,81 @@
 require 'spec_helper'
 
-describe Kuhsaft::TwoColumnBrick do
-
+describe Qbrick::TwoColumnBrick, type: :model do
   let :two_column_brick do
-    Kuhsaft::TwoColumnBrick.new
+    Qbrick::TwoColumnBrick.new
   end
 
   describe '#user_can_add_childs?' do
     it 'returns false' do
-      two_column_brick.user_can_add_childs?.should be_false
+      expect(two_column_brick.user_can_add_childs?).to be_falsey
     end
   end
 
   describe '#user_can_delete?' do
     it 'returns true' do
-      two_column_brick.user_can_delete?.should be_true
+      expect(two_column_brick.user_can_delete?).to be_truthy
     end
   end
 
   describe '#user_can_save?' do
     it 'returns true' do
-      two_column_brick.user_can_save?.should be_true
+      expect(two_column_brick.user_can_save?).to be_truthy
     end
   end
 
   describe '#renders_own_childs?' do
     it 'returns true' do
-      two_column_brick.renders_own_childs?.should be_true
+      expect(two_column_brick.renders_own_childs?).to be_truthy
     end
   end
 
   describe '#partitioning' do
     context 'when no partition is set' do
       it 'returns 0 (50/50)' do
-        two_column_brick.partitioning.should be(0)
+        expect(two_column_brick.partitioning).to be(0)
       end
     end
 
     context 'when the partition is set' do
       it 'returns the value' do
         two_column_brick.partitioning = 1
-        two_column_brick.partitioning.should be(1)
+        expect(two_column_brick.partitioning).to be(1)
       end
     end
   end
 
   describe '.partitionings' do
     it 'returns the 3 default partitions' do
-      Kuhsaft::TwoColumnBrick.partitionings.should have(3).items
+      expect(Qbrick::TwoColumnBrick.partitionings.size).to eq(3)
     end
   end
 
   describe '#create' do
     it 'creates two single columns as childs' do
       two_column_brick.save
-      two_column_brick.bricks.should be_all { |brick| brick.should be_a(Kuhsaft::ColumnBrick) }
+      expect(two_column_brick.bricks).to be_all { |brick| expect(brick).to be_a(Qbrick::ColumnBrick) }
     end
   end
 
   describe '#bricks' do
     it 'can have childs' do
-      two_column_brick.should respond_to(:bricks)
+      expect(two_column_brick).to respond_to(:bricks)
     end
   end
 
   describe '#to_style_class' do
     it 'adds the row class to the default styles' do
-      Kuhsaft::TwoColumnBrick.new.to_style_class.should == 'kuhsaft-two-column-brick row-fluid'
+      expect(Qbrick::TwoColumnBrick.new.to_style_class).to eq('qbrick-two-column-brick row-fluid')
     end
   end
 
   describe '#add_columns' do
     it 'sets the position of the first column brick to 1' do
-      Kuhsaft::TwoColumnBrick.new.send(:add_columns).to_a.first.position.should == 1
+      expect(Qbrick::TwoColumnBrick.new.send(:add_columns).to_a.first.position).to eq(1)
     end
 
     it 'sets the position of the second column brick to 2' do
-      Kuhsaft::TwoColumnBrick.new.send(:add_columns).to_a.second.position.should == 2
+      expect(Qbrick::TwoColumnBrick.new.send(:add_columns).to_a.second.position).to eq(2)
     end
   end
 end

data/spec/models/video_brick_spec.rb

@@ -1,9 +1,8 @@
 require 'spec_helper'
 
-describe Kuhsaft::VideoBrick do
-
+describe Qbrick::VideoBrick, type: :model do
   let :video_brick do
-    Kuhsaft::VideoBrick.new
+    Qbrick::VideoBrick.new
   end
 
   describe '#valid' do
@@ -13,20 +12,20 @@ describe Kuhsaft::VideoBrick do
 
     context 'without any video source' do
       it 'has en error' do
-        video_brick.should have(1).error_on(:any_source)
+        expect(video_brick.errors[:any_source].count).to eq(1)
       end
     end
   end
 
   describe '#bricks' do
     it 'can not have childs' do
-      video_brick.should_not respond_to(:bricks)
+      expect(video_brick).not_to respond_to(:bricks)
     end
   end
 
   describe '#user_can_add_childs?' do
     it 'returns false' do
-      video_brick.user_can_add_childs?.should be_false
+      expect(video_brick.user_can_add_childs?).to be_falsey
     end
   end
 end

data/spec/postgres_helper.rb

@@ -0,0 +1,66 @@
+module Postgres
+  class << self
+    def exec(query, opts = {})
+      command = ["psql -tAc \"#{query}\""]
+
+      if opts.is_a?(Hash)
+        opts.each_pair { |k, v| command << "--#{k}=#{v}" }
+      else
+        command << opts
+      end
+
+      command = command.join(' ')
+      result = `#{command}`
+      if result.match(/psql: FATAL:  database .+ does not exist/)
+        if result.include?("psql: FATAL:  database \"#{`whoami`.strip}\" does not exist")
+          `createdb`
+          dbg_comment 'user database was missing. I created it for you and recall the postgres command again. :)'
+          `#{command}`
+        end
+      else
+        result
+      end
+    end
+
+    def user_exists?(username)
+      exec("SELECT 1 FROM pg_user WHERE usename='#{username}'", dbname: 'postgres').strip == '1'
+    end
+
+    def database_exists?(name)
+      exec("SELECT 1 from pg_database WHERE datname='#{name}'", dbname: 'postgres').strip == '1'
+    end
+
+    def drop_user(username)
+      `dropuser #{username}`
+    end
+
+    def drop_table(user, database, name)
+      exec("SET client_min_messages TO WARNING; DROP TABLE IF EXISTS #{name};", username: user, dbname: database)
+    end
+
+    def select_users_like(username)
+      exec("SELECT usename FROM pg_user where usename LIKE '#{username}'", dbname: 'postgres').split("\n").map(&:strip)
+    end
+
+    def select_databases_like(name)
+      exec("SELECT datname FROM pg_database WHERE datistemplate = false AND datname LIKE '#{name}';", dbname: 'postgres').split("\n").map(&:strip)
+    end
+
+    def drop_databases_like(name)
+      select_databases_like(name).each { |dbname| `dropdb #{dbname}` }
+    end
+
+    def drop_users_like(username)
+      select_users_like(username).each { |u| drop_user u if user_exists?(u) }
+    end
+
+    def create_user(username)
+      if user_exists?(username)
+        false
+      else
+        puts "#{ts} $ createuser -s #{username}" if debug_shoestrap?
+        `createuser -s #{username}`
+      end
+    end
+  end
+end

data/spec/{kuhsaft_spec.rb → qbrick_spec.rb}

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

data/spec/spec_helper.rb

@@ -2,12 +2,11 @@
 ENV['RAILS_ENV'] = 'test'
 require_relative File.expand_path('../dummy/config/environment.rb', __FILE__)
 require 'rspec/rails'
-require 'rspec/autorun'
 
 [
   'factory_girl',
   'capybara/rails',
-  'pry-debugger',
+  'pry-byebug',
   'database_cleaner'
 ].each { |lib| require lib }
 
@@ -16,6 +15,7 @@ FactoryGirl.find_definitions
 ActionMailer::Base.delivery_method = :test
 ActionMailer::Base.perform_deliveries = true
 ActionMailer::Base.default_url_options[:host] = 'test.com'
+ActiveRecord::Migrator.migrations_paths = 'spec/dummy/db/migrate'
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.
@@ -23,10 +23,15 @@ Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require_relative f }
 
 # Checks for pending migrations before tests are run.
 # If you are not using ActiveRecord, you can remove this line.
-ActiveRecord::Migration.check_pending! if defined?(ActiveRecord::Migration)
+ActiveRecord::Migration.maintain_test_schema! if ActiveRecord::Migration.respond_to? :maintain_test_schema!
 
-RSpec.configure do |config|
+include Warden::Test::Helpers
+Warden.test_mode!
+
+require 'capybara/poltergeist'
+Capybara.javascript_driver = :poltergeist
 
+RSpec.configure do |config|
   # Remove this line if you don't want RSpec's should and should_not
   # methods or matchers
   require 'rspec/expectations'
@@ -35,16 +40,22 @@ RSpec.configure do |config|
 
   config.include RSpec::Matchers
   config.include CarrierWave::Test::Matchers
-  config.include KuhsaftSpecHelper
+  config.include QbrickSpecHelper
   config.include FactoryGirl::Syntax::Methods
-
-  config.before :suite do
-    DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner.clean_with :truncation
+  config.include Rails.application.routes.url_helpers
+
+  config.before :each do
+    if Capybara.current_driver == :rack_test
+      DatabaseCleaner.strategy = :transaction
+    else
+      DatabaseCleaner.strategy = :truncation
+    end
+    DatabaseCleaner.start
   end
 
   config.before do
     DatabaseCleaner.start
+    Qbrick::ImageSize.build_defaults!
     I18n.available_locales = [:en, :de]
   end
 
@@ -66,7 +77,7 @@ RSpec.configure do |config|
   # If you're not using ActiveRecord, or you'd prefer not to run each of your
   # examples within a transaction, remove the following line or assign false
   # instead of true.
-  config.use_transactional_fixtures = true
+  config.use_transactional_fixtures = false
 
   # If true, the base class of anonymous controllers will be inferred
   # automatically. This will be the default behavior in future versions of
@@ -78,4 +89,6 @@ RSpec.configure do |config|
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = 'random'
+
+  config.filter_run_excluding generator: true
 end