alchemy_cms 2.9.1 → 3.0.0.rc5

data/spec/controllers/pictures_controller_spec.rb

@@ -8,33 +8,129 @@ end
 module Alchemy
   describe PicturesController do
 
-    let(:public_page)        { FactoryGirl.create(:public_page, :restricted => false) }
-    let(:restricted_page)    { FactoryGirl.create(:public_page, :restricted => true) }
-    let(:element)            { FactoryGirl.create(:element, :page => public_page, :name => 'bild', :create_contents_after_create => true) }
-    let(:restricted_element) { FactoryGirl.create(:element, :page => restricted_page, :name => 'bild', :create_contents_after_create => true) }
-    let(:picture)            { Picture.create(:image_file => fixture_file_upload(File.expand_path('../../fixtures/image.png', __FILE__), 'image/png')) }
+    let(:public_page)        { FactoryGirl.create(:public_page, restricted: false) }
+    let(:restricted_page)    { FactoryGirl.create(:public_page, restricted: true) }
+    let(:element)            { FactoryGirl.create(:element, page: public_page, name: 'bild', create_contents_after_create: true) }
+    let(:restricted_element) { FactoryGirl.create(:element, page: restricted_page, name: 'bild', create_contents_after_create: true) }
+    let(:picture)            { Picture.create(image_file: fixture_file_upload(File.expand_path('../../fixtures/image.png', __FILE__), 'image/png')) }
+
+    describe '#zoom' do
+      let(:picture) { Picture.create(image_file: fixture_file_upload(File.expand_path('../../fixtures/80x60.png', __FILE__), 'image/png')) }
+
+      before { sign_in(editor_user) }
+
+      it "renders the original image without any resizing" do
+        get :zoom, id: picture.id, format: :png, sh: picture.security_token
+        response.body[0x10..0x18].unpack('NN').should == [80, 60]
+      end
+    end
+
+    context "Requesting a picture with tempared security token" do
+      it "should render status 400" do
+        get :show, id: picture.id, format: :png, sh: '14m4b4dh4ck3r'
+        response.status.should == 400
+      end
+    end
+
+    context "Requesting a picture with another format then the original image" do
+      it "should convert the picture format" do
+        get :show, id: picture.id, format: :jpeg, sh: picture.security_token
+        response.content_type.should == 'image/jpeg'
+      end
+    end
+
+    context "Requesting a picture with not allowed format" do
+      it "should raise error" do
+        expect {
+          get :show, id: picture.id, format: :wim, sh: picture.security_token
+        }.to raise_error(ActionController::UnknownFormat)
+      end
+    end
+
+    describe '#thumbnail' do
+      let(:picture) { Picture.create(image_file: fixture_file_upload(File.expand_path('../../fixtures/500x500.png', __FILE__), 'image/png')) }
+
+      before { sign_in(author_user) }
+
+      context 'with size param set to small' do
+        it "resizes the image to 80x60 while maintaining aspect ratio" do
+          get :thumbnail, id: picture.id, size: 'small', format: :png, sh: picture.security_token(size: 'small')
+          response.body[0x10..0x18].unpack('NN').should == [60, 60]
+        end
+      end
+
+      context 'with size param set to medium' do
+        it "resizes the image to 160x120 while maintaining aspect ratio" do
+          get :thumbnail, id: picture.id, size: 'medium', format: :png, sh: picture.security_token(size: 'medium')
+          response.body[0x10..0x18].unpack('NN').should == [120, 120]
+        end
+      end
+
+      context 'with size param set to large' do
+        it "resizes the image to 240x180 while maintaining aspect ratio" do
+          get :thumbnail, id: picture.id, size: 'large', format: :png, sh: picture.security_token(size: 'large')
+          response.body[0x10..0x18].unpack('NN').should == [180, 180]
+        end
+      end
+
+      context 'with size param set to nil' do
+        it "resizes the image to 111x93 while maintaining aspect ratio" do
+          get :thumbnail, id: picture.id, format: :png, sh: picture.security_token
+          response.body[0x10..0x18].unpack('NN').should == [93, 93]
+        end
+      end
+
+      context 'with size param set to another value' do
+        it "resizes the image to the given size while maintaining aspect ratio" do
+          get :thumbnail, id: picture.id, size: '33x33', format: :png, sh: picture.security_token(size: '33x33')
+          response.body[0x10..0x18].unpack('NN').should == [33, 33]
+        end
+      end
+    end
+
+    context "Requesting a picture that has no image file attached" do
+      before do
+        picture.stub(image_file: nil)
+        Picture.stub(find: picture)
+      end
+
+      it "raises missing file error" do
+        expect {
+          get :show, id: picture.id, format: :png, sh: picture.security_token
+        }.to raise_error(Alchemy::MissingImageFileError)
+      end
+    end
+
+    context "Requesting a picture with crop_from and crop_size parameters" do
+      let(:picture) { Picture.create(image_file: fixture_file_upload(File.expand_path('../../fixtures/500x500.png', __FILE__), 'image/png')) }
+
+      it "renders the cropped picture" do
+        get :show, id: picture.id, crop: 'crop', size: '123x44', crop_size: '123x44', crop_from: '0x0', format: :png, sh: picture.security_token(crop_size: '123x44', crop_from: '0x0', crop: true, size: '123x44')
+        response.body[0x10..0x18].unpack('NN').should == [123, 44]
+      end
+    end
 
     context "Requesting a picture that is not assigned with any page" do
       it "should render the picture" do
-        get :show, :id => picture.id, :format => :png, :sh => picture.security_token
+        get :show, id: picture.id, format: :png, sh: picture.security_token
         response.status.should == 200
       end
     end
 
     context "Requesting a picture that is assigned on restricted and non-restricted pages" do
       before do
-        essence = element.contents.where(:name => 'image').first.essence
+        essence = element.contents.where(name: 'image').first.essence
         essence.picture_id = picture.id
         essence.save
 
-        essence = restricted_element.contents.where(:name => 'image').first.essence
+        essence = restricted_element.contents.where(name: 'image').first.essence
         essence.picture_id = picture.id
         essence.save
       end
 
       context "as guest user" do
         it "should render the picture" do
-          get :show, :id => picture.id, :format => :png, :sh => picture.security_token
+          get :show, id: picture.id, format: :png, sh: picture.security_token
           response.status.should == 200
         end
       end
@@ -42,66 +138,54 @@ module Alchemy
 
     context "Requesting a picture that is assigned with restricted pages only" do
       before do
-        essence = restricted_element.contents.where(:name => 'image').first.essence
+        essence = restricted_element.contents.where(name: 'image').first.essence
         essence.picture_id = picture.id
         essence.save
       end
 
       context "as guest user" do
         it "should not render the picture, but redirect to login path" do
-          get :show, :id => picture.id, :sh => picture.security_token
+          get :show, id: picture.id, sh: picture.security_token
           response.status.should == 302
           response.should redirect_to(Alchemy.login_path)
         end
       end
 
-      context "as registered user" do
+      context "as member user" do
         before do
-          sign_in(registered_user)
+          sign_in(member_user)
         end
 
         it "should render the picture" do
-          get :show, :id => picture.id, :format => :png, :sh => picture.security_token
+          get :show, id: picture.id, format: :png, sh: picture.security_token
           response.status.should == 200
         end
       end
     end
 
     describe 'Picture processing' do
-      let(:big_picture) { Picture.create(:image_file => fixture_file_upload(File.expand_path('../../fixtures/80x60.png', __FILE__), 'image/png')) }
-
       context "with crop and size parameters" do
+
+        let(:big_picture) { Picture.create(image_file: fixture_file_upload(File.expand_path('../../fixtures/80x60.png', __FILE__), 'image/png')) }
+
         it "should return a cropped image." do
           options = {
-            :crop => 'crop',
-            :size => '10x10',
-            :format => 'png'
+            crop: 'crop',
+            size: '10x10',
+            format: 'png'
           }
-          get :show, options.merge(:id => big_picture.id, :sh => big_picture.security_token(options))
+          get :show, options.merge(id: big_picture.id, sh: big_picture.security_token(options))
           response.body[0x10..0x18].unpack('NN').should == [10,10]
         end
 
-        context "without a full size specification" do
-          it "should raise an error" do
-            options = {
-              :crop => 'crop',
-              :size => '10',
-              :format => 'png'
-            }
-            expect do
-              get :show, options.merge(:id => big_picture.id, :sh => big_picture.security_token(options))
-            end.to raise_error ArgumentError
-          end
-        end
-
         context "without upsample parameter" do
           it "should not upsample the image." do
             options = {
-              :crop => 'crop',
-              :size => '10x10',
-              :format => 'png'
+              crop: 'crop',
+              size: '10x10',
+              format: 'png'
             }
-            get :show, options.merge(:id => picture.id, :sh => picture.security_token(options))
+            get :show, options.merge(id: picture.id, sh: picture.security_token(options))
             response.body[0x10..0x18].unpack('NN').should == [1,1]
           end
         end
@@ -109,46 +193,16 @@ module Alchemy
         context "and with upsample true" do
           it "should return an upsampled image." do
             options = {
-              :crop => 'crop',
-              :size => '10x10',
-              :upsample => 'true',
-              :format => 'png'
+              crop: 'crop',
+              size: '10x10',
+              upsample: 'true',
+              format: 'png'
             }
-            get :show, options.merge(:id => picture.id, :sh => picture.security_token(options))
+            get :show, options.merge(id: picture.id, sh: picture.security_token(options))
             response.body[0x10..0x18].unpack('NN').should == [10,10]
           end
         end
       end
-
-      context "without crop but with size parameter" do
-        it "should resize the image preserving aspect ratio" do
-          options = {
-            :size => '40x40',
-            :format => 'png'
-          }
-          get :show, options.merge(:id => big_picture.id, :sh => big_picture.security_token(options))
-          response.body[0x10..0x18].unpack('NN').should == [40,30]
-        end
-
-        it "should resize the image inferring the height if not given" do
-          options = {
-            :size => '40',
-            :format => 'png'
-          }
-          get :show, options.merge(:id => big_picture.id, :sh => big_picture.security_token(options))
-          response.body[0x10..0x18].unpack('NN').should == [40,30]
-        end
-
-        it "should resize the image inferring the width if not given" do
-          options = {
-            :size => 'x30',
-            :format => 'png'
-          }
-          get :show, options.merge(:id => big_picture.id, :sh => big_picture.security_token(options))
-          response.body[0x10..0x18].unpack('NN').should == [40,30]
-        end
-      end
-
     end
 
   end

data/spec/dummy/Rakefile

@@ -1,4 +1,3 @@
-#!/usr/bin/env rake
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 

data/spec/dummy/app/assets/javascripts/application.js

@@ -1,9 +1,13 @@
-// This is a manifest file that'll be compiled into including all the files listed below.
-// Add new JavaScript/Coffee code in separate files in this directory and they'll automatically
-// be included in the compiled file accessible from http://example.com/assets/application.js
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or vendor/assets/javascripts of plugins, if any, can be referenced here using a relative path.
+//
 // It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
-// the compiled file.
+// compiled file.
+//
+// Read Sprockets README (https://github.com/sstephenson/sprockets#sprockets-directives) for details
+// about supported directives.
 //
-//= require jquery
-//= require jquery_ujs
-//= require_tree .
+//= require alchemy/alchemy

data/spec/dummy/app/assets/stylesheets/application.css

@@ -1,7 +1,13 @@
 /*
- * This is a manifest file that'll automatically include all the stylesheets available in this directory
- * and any sub-directories. You're free to add application-wide styles to this file and they'll appear at
- * the top of the compiled file, but it's generally better to create a new file per style scope.
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or vendor/assets/stylesheets of plugins, if any, can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the top of the
+ * compiled file, but it's generally better to create a new file per style scope.
+ *
  *= require_self
- *= require_tree . 
-*/
+ *= require_tree .
+ */

data/spec/dummy/app/models/ability.rb

@@ -0,0 +1,9 @@
+class Ability
+  include CanCan::Ability
+
+  def initialize(user)
+    can :manage, Event
+    can :index, :admin_events
+  end
+
+end

data/spec/dummy/app/models/dummy_user.rb

@@ -0,0 +1,7 @@
+class DummyUser < ActiveRecord::Base
+  attr_accessor :alchemy_roles
+
+  def self.logged_in
+    []
+  end
+end

data/spec/dummy/app/models/event.rb

@@ -1,5 +1,10 @@
 class Event < ActiveRecord::Base
-  attr_accessible :name, :hidden_name, :starts_at, :ends_at, :description, :published, :entrance_fee, :location_id
   validates_presence_of :name
   belongs_to :location
-end
+
+  def self.alchemy_resource_relations
+    {
+      location: {attr_method: 'name', attr_type: 'string'}
+    }
+  end
+end

data/spec/dummy/app/models/location.rb

@@ -1,4 +1,3 @@
 class Location < ActiveRecord::Base
-  attr_accessible :name
   has_many :events
 end

data/spec/dummy/app/views/alchemy/elements/_article_editor.html.erb

@@ -2,7 +2,6 @@
 <%= render_essence_editor_by_name(element, 'image', {
   :image_size => "450x300",
   :crop => true,
-  :image_float_selector => true,
   :fixed_ratio => false
 }) %>
 <% element.contents.where(:name => ['headline', 'text']).each do |content| %>

data/spec/dummy/app/views/alchemy/elements/_headline_editor.html.erb

@@ -0,0 +1 @@
+<%= render_essence_editor_by_name(element, 'headline') %>

data/spec/dummy/app/views/layouts/application.html.erb

@@ -3,6 +3,8 @@
 <head>
   <%= render_meta_data %>
   <%= stylesheet_link_tag 'application' %>
+  <%= javascript_include_tag "application" %>
+  <%= csrf_meta_tags %>
 </head>
 <body>
   <%= yield %>
@@ -10,6 +12,6 @@
     <%= render_navigation(:all_sub_menues => true) %>
   </div>
   <%= alchemy_preview_mode_code %>
-  <%= alchemy_menu_bar %>
+  <%= render "alchemy/menubar" %>
 </body>
 </html>

data/spec/dummy/bin/bundle

@@ -0,0 +1,3 @@
+#!/usr/bin/env ruby
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+load Gem.bin_path('bundler', 'bundle')

data/spec/dummy/bin/rails

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require_relative '../config/boot'
+require 'rails/commands'

data/spec/dummy/bin/rake

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+require_relative '../config/boot'
+require 'rake'
+Rake.application.run

data/spec/dummy/config.ru

@@ -1,4 +1,4 @@
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path('../config/environment', __FILE__)
-run Dummy::Application
+require ::File.expand_path('../config/environment',  __FILE__)
+run Rails.application

data/spec/dummy/config/alchemy/elements.yml

@@ -81,6 +81,3 @@
     type: EssenceSelect
     validate:
       - presence
-
-- name: searchresult
-  unique: true

data/spec/dummy/config/alchemy/page_layouts.yml

@@ -19,12 +19,6 @@
   elements: [headline, text, contactform]
   autogenerate: [headline, text, contactform]
 
-- name: search
-  unique: true
-  elements: [headline, text, searchresult]
-  autogenerate: [searchresult]
-  searchresults: true
-
 - name: external
   unique: false
   redirects_to_external: true

data/spec/dummy/config/application.rb

@@ -1,30 +1,9 @@
 require File.expand_path('../boot', __FILE__)
 
-# Pick the frameworks you want:
-if ENV['FAST_SPECS']
-  require "active_record/railtie"
-  require "action_controller/railtie"
-  require "action_mailer/railtie"
-  require "active_resource/railtie"
-  require "alchemy_cms"
-else
-  require "active_record/railtie"
-  require "action_controller/railtie"
-  require "action_mailer/railtie"
-  require "active_resource/railtie"
-  require "sprockets/railtie"
-end
+require 'rails/all'
 
-if defined?(Bundler)
-  # If you precompile assets before deploying to production, use this line
-  # Bundler.require(*Rails.groups(:assets => %w(development test)))
-  # If you want your assets lazily compiled in production, use this line
-  if ENV['FAST_SPECS']
-    Bundler.require(:test)
-  else
-    Bundler.require(:default, :assets, Rails.env)
-  end
-end
+Bundler.require(*Rails.groups)
+require "alchemy_cms"
 
 module Dummy
   class Application < Rails::Application
@@ -32,45 +11,14 @@ module Dummy
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
 
-    # Custom directories with classes and modules you want to be autoloadable.
-    # config.autoload_paths += %W(#{config.root}/extras)
-
-    # Only load the plugins named here, in the order given (default is alphabetical).
-    # :all can be used as a placeholder for all plugins not explicitly named.
-    # config.plugins = [ :exception_notification, :ssl_requirement, :all ]
-
-    # Activate observers that should always be running.
-    # config.active_record.observers = :cacher, :garbage_collector, :forum_observer
-
     # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone.
     # Run "rake -D time" for a list of tasks for finding time zone names. Default is UTC.
     # 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.default_locale = :en
-
-    # Configure the default encoding used in templates for Ruby 1.9.
-    config.encoding = "utf-8"
-
-    # Configure sensitive parameters which will be filtered from the log file.
-    config.filter_parameters += [:password, :password_confirmation]
-
-    # Use SQL instead of Active Record's schema dumper when creating the database.
-    # This is necessary if your schema can't be completely dumped by the schema dumper,
-    # like if you have constraints or database-specific column types
-    # config.active_record.schema_format = :sql
-
-    # Enforce whitelist mode for mass assignment.
-    # This will create an empty whitelist of attributes available for mass-assignment for all models
-    # in your app. As such, your models will need to explicitly whitelist or blacklist accessible
-    # parameters by using an attr_accessible or attr_protected declaration.
-    config.active_record.whitelist_attributes = true
-
-    # Enable the asset pipeline
-    config.assets.enabled = true
+    # config.i18n.default_locale = :de
 
-    # Version of your assets, change this if you want to expire all your assets
-    config.assets.version = '1.0'
+    I18n.enforce_available_locales = true
   end
 end