refinerycms-core 1.0.11 → 2.0.0

data/spec/lib/refinery/core_spec.rb

@@ -0,0 +1,176 @@
+require 'spec_helper'
+
+describe Refinery do
+  describe "#extensions" do
+    it "should return an array of modules representing registered extensions" do
+      subject.extensions.should be_a(Array)
+      subject.extensions.each do |e|
+        e.should be_a(Module)
+      end
+    end
+  end
+
+  describe "#register_extension" do
+    before(:each) { subject.extensions.clear }
+
+    it "should add the extension's module to the array of registered extensions" do
+      subject.register_extension(Refinery::Core)
+
+      Refinery.extensions.should include(Refinery::Core)
+      Refinery.extensions.should have(1).item
+    end
+
+    it "should not allow same extension to be registered twice" do
+      subject.register_extension(Refinery::Core)
+      subject.register_extension(Refinery::Core)
+
+      Refinery.extensions.should have(1).item
+    end
+  end
+
+  describe "#extension_registered?" do
+    context "with Refinery::Core::Engine registered" do
+      before(:each) { subject.register_extension(Refinery::Core) }
+
+      it "should return true if the extension is registered" do
+        subject.extension_registered?(Refinery::Core).should == true
+      end
+    end
+
+    context "with no extensions registered" do
+      before(:each) { subject.extensions.clear }
+
+      it "should return false if the extension is not registered" do
+        subject.extension_registered?(Refinery::Core).should == false
+      end
+    end
+  end
+
+  describe "#unregister_extension" do
+    before(:each) do
+      subject.extensions.clear
+      subject.register_extension(Refinery::Images)
+    end
+
+    it "should remove the extension's module from the array of registered extensions" do
+      subject.unregister_extension(Refinery::Images)
+
+      subject.extensions.should have(0).item
+    end
+  end
+
+  describe "#validate_extension!" do
+    context "with a valid extension" do
+      it "should return nil" do
+        subject.send(:validate_extension!, Refinery::ValidEngine).should be_nil
+      end
+    end
+
+    context "with an invalid extension" do
+      it "should raise invalid extension exception" do
+        lambda {
+          subject.send(:validate_extension!, Refinery::InvalidEngine)
+        }.should raise_error(Refinery::InvalidEngineError, "Engine must define a root accessor that returns a pathname to its root")
+      end
+    end
+  end
+
+  describe "#roots" do
+    it "should return pathname to extension root when given constant as parameter" do
+      subject.roots(Refinery::Core).should == Refinery::Core.root
+    end
+
+    it "should return pathname to extension root when given symbol as parameter" do
+      subject.roots(:'refinery/core').should == Refinery::Core.root
+    end
+
+    it "should return pathname to extension root when given string as parameter" do
+      subject.roots("refinery/core").should == Refinery::Core.root
+    end
+
+    it "should return an array of all pathnames if no extension_name is specified" do
+      subject.roots.should be_a(Array)
+      subject.roots.each do |root|
+        root.should be_a(Pathname)
+      end
+    end
+  end
+
+  describe "#deprecate" do
+    before(:each) do
+      @errors = StringIO.new
+      @old_err = $stderr
+      $stderr = @errors
+    end
+
+    after(:each) { $stderr = @old_err }
+
+    it "shows a deprecation warning" do
+      Refinery.deprecate("ugis")
+      @errors.rewind
+      @errors.read.should == "\n-- DEPRECATION WARNING --\nThe use of 'ugis' is deprecated.\n"
+    end
+
+    it "takes when option" do
+      Refinery.deprecate("ugis", :when => "10.0")
+      @errors.rewind
+      @errors.read.should == "\n-- DEPRECATION WARNING --\nThe use of 'ugis' is deprecated and will be removed at version 10.0.\n"
+    end
+
+    it "takes replacement option" do
+      Refinery.deprecate("ugis", :when => "10.0", :replacement => "philip")
+      @errors.rewind
+      @errors.read.should == "\n-- DEPRECATION WARNING --\nThe use of 'ugis' is deprecated and will be removed at version 10.0.\nPlease use philip instead.\n"
+    end
+  end
+
+  describe "#i18n_enabled?" do
+    it "returns true when Refinery::I18n.enabled? is true" do
+      Refinery::I18n.stub(:enabled?).and_return(true)
+      subject.i18n_enabled?.should == true
+    end
+
+    it "returns false when Refinery::I18n.enabled? is false" do
+      Refinery::I18n.stub(:enabled?).and_return(false)
+      subject.i18n_enabled?.should == false
+    end
+  end
+
+  describe ".route_for_model" do
+    context "when passed Refinery::Dummy" do
+      it "returns admin_dummy_path" do
+        Refinery.route_for_model("Refinery::Dummy").should == "admin_dummy_path"
+      end
+    end
+
+    context "when passed Refinery::Dummy and true" do
+      it "returns admin_dummies_path" do
+        Refinery.route_for_model("Refinery::Dummy", true).should == "admin_dummies_path"
+      end
+    end
+
+    context "when passed Refinery::DummyName" do
+      it "returns admin_dummy_name_path" do
+        Refinery.route_for_model("Refinery::DummyName").should == "admin_dummy_name_path"
+      end
+    end
+
+    context "when passed Refinery::DummyName and true" do
+      it "returns admin_dummy_names_path" do
+        Refinery.route_for_model("Refinery::DummyName", true).should == "admin_dummy_names_path"
+      end
+    end
+
+    context "when passed Refinery::Dummy::Name" do
+      it "returns dummy_admin_name_path" do
+        Refinery.route_for_model("Refinery::Dummy::Name").should == "dummy_admin_name_path"
+      end
+    end
+
+    context "when passed Refinery::Dummy::Name and true" do
+      it "returns dummy_admin_names_path" do
+        Refinery.route_for_model("Refinery::Dummy::Name", true).should == "dummy_admin_names_path"
+      end
+    end
+  end
+end

data/spec/lib/refinery/plugin_spec.rb

@@ -3,6 +3,7 @@ require 'spec_helper'
 module Refinery
   module RefineryRspec
     class Engine < ::Rails::Engine
+      isolate_namespace ::Refinery
       ::Refinery::Plugin.register do |plugin|
         plugin.name = 'refinery_rspec'
         plugin.hide_from_menu = true
@@ -14,14 +15,15 @@ end
 module Refinery
   describe Plugin do
 
-    let(:plugin) { Refinery::Plugins.registered.detect { |plugin| plugin.name == 'refinery_rspec' } }
+    let(:plugin) { ::Refinery::Plugins.registered.detect { |plugin| plugin.name == 'refinery_rspec' } }
 
-    def setup_i18n
-      ::I18n.backend = ::I18n::Backend::Simple.new
-      ::I18n.backend.store_translations :en, :plugins => {
-        :refinery_rspec => {
-          :title => 'RefineryCMS RSpec',
-          :description => 'RSpec tests for plugin.rb'
+    before do
+      ::I18n.backend.store_translations :en, :refinery => {
+        :plugins => {
+          :refinery_rspec => {
+            :title => "Refinery CMS RSpec",
+            :description => "RSpec tests for plugin.rb"
+          }
         }
       }
     end
@@ -39,29 +41,17 @@ module Refinery
     end
 
     describe '#title' do
-      before { setup_i18n }
-
       it 'returns plugin title defined by I18n' do
-        plugin.title.should == 'RefineryCMS RSpec'
+        plugin.title.should == 'Refinery CMS RSpec'
       end
     end
 
     describe '#description' do
-      before { setup_i18n }
-
       it 'returns plugin description defined by I18n' do
         plugin.description.should == 'RSpec tests for plugin.rb'
       end
     end
 
-    describe '#activity' do
-
-    end
-
-    describe '#activity=' do
-
-    end
-
     describe '#pathname' do
       it 'should be set by default' do
         plugin.pathname.should_not == nil
@@ -85,18 +75,18 @@ module Refinery
       end
     end
 
-    describe '#always_allow_access?' do
+    describe '#always_allow_access' do
       it 'returns false if @always_allow_access is not set or its set to false' do
-        plugin.always_allow_access?.should be_false
+        plugin.always_allow_access.should be_false
       end
 
       it 'returns true if set so' do
-        plugin.stub(:always_allow_access?).and_return(true)
-        plugin.always_allow_access?.should be
+        plugin.stub(:always_allow_access).and_return(true)
+        plugin.always_allow_access.should be
       end
     end
 
-    describe '#dashboard?' do
+    describe '#dashboard' do
       it 'returns false if @dashboard is not set or its set to false' do
         plugin.dashboard.should be_false
       end
@@ -109,17 +99,18 @@ module Refinery
 
     describe '#menu_match' do
       it 'returns regexp based on plugin name' do
-        plugin.menu_match.should == /(admin|refinery)\/refinery_rspec$/
+        plugin.menu_match.should == %r{refinery/refinery_rspec(/.+?)?$}
       end
     end
 
     describe '#highlighted?' do
       it 'returns true if params[:controller] match menu_match regexp' do
-        plugin.highlighted?({:controller => 'refinery/refinery_rspec'}).should be
+        plugin.highlighted?({:controller => '/refinery/admin/refinery_rspec'}).should be
+        plugin.highlighted?({:controller => '/refinery/refinery_rspec'}).should be
       end
 
-      it 'returns true if dashboard? is true and params[:action] == error_404' do
-        plugin.stub(:dashboard?).and_return(true)
+      it 'returns true if dashboard is true and params[:action] == error_404' do
+        plugin.stub(:dashboard).and_return(true)
         plugin.highlighted?({:action => 'error_404'}).should be
       end
     end
@@ -135,15 +126,15 @@ module Refinery
 
       context 'when @url is already defined' do
         it 'returns hash' do
-          plugin.stub(:url).and_return({:controller => '/admin/testa'})
-          plugin.url.should == {:controller => '/admin/testa'}
+          plugin.stub(:url).and_return({:controller => 'refinery/admin/testa'})
+          plugin.url[:controller].should == 'refinery/admin/testa'
         end
       end
 
       context 'when controller is present' do
         it 'returns hash based on it' do
           plugin.stub(:controller).and_return('testb')
-          plugin.url.should == {:controller => '/admin/testb'}
+          plugin.url[:controller].should == 'refinery/admin/testb'
         end
       end
 
@@ -151,13 +142,24 @@ module Refinery
 
         it 'returns hash based on it' do
           plugin.stub(:directory).and_return('first/second/testc')
-          plugin.url.should == {:controller => '/admin/testc'}
+          plugin.url[:controller].should == 'refinery/admin/testc'
         end
       end
 
       context 'when controller and directory not present' do
         it 'returns hash based on plugins name' do
-          plugin.url.should == {:controller => '/admin/refinery_rspec'}
+          plugin.url[:controller].should == 'refinery/admin/refinery_rspec'
+        end
+      end
+    end
+
+    describe '#activity_by_class_name' do
+      before { plugin.activity = [ { :class_name => "X::Y::Z" }, { :class_name => "X::Y::ZZ" }] }
+
+      context 'when the plugin have diferents activities' do
+        it 'returns the correct activity' do
+          plugin.activity_by_class_name("X::Y::Z").first.should == plugin.activity.first
+          plugin.activity_by_class_name("X::Y::ZZ").first.should == plugin.activity.last
         end
       end
     end

data/spec/lib/refinery/plugins_spec.rb

@@ -3,6 +3,7 @@ require 'spec_helper'
 module Refinery
   module MyPlugin
     class Engine < ::Rails::Engine
+      isolate_namespace ::Refinery
       ::Refinery::Plugin.register do |plugin|
         plugin.name = "my_plugin"
         plugin.hide_from_menu = true
@@ -12,42 +13,52 @@ module Refinery
 
   module MyOtherPlugin
     class Engine < ::Rails::Engine
+      isolate_namespace ::Refinery
       ::Refinery::Plugin.register do |plugin|
         plugin.name = "my_other_plugin"
         plugin.hide_from_menu = true
       end
     end
   end
-end
 
-module Refinery
+  ::I18n.backend.store_translations :en, :refinery => {
+    :plugins => {
+      :my_plugin => {
+        :title => "my plugin"
+      },
+      :my_other_plugin => {
+        :title => "my other plugin"
+      }
+    }
+  }
+
   describe Plugins do
 
     before(:each) do
       # First, deactivate all.
-      Plugins.set_active([])
+      subject.class.set_active([])
     end
 
     describe '#activate' do
       it "activates a plugin" do
         subject.class.activate("my_plugin")
 
-        Plugins.active.names.should include("my_plugin")
+        subject.class.active.names.should include("my_plugin")
       end
 
       it "only activates the same plugin once" do
          subject.class.activate("my_other_plugin")
          subject.class.activate("my_other_plugin")
 
-         Plugins.active.names.count("my_other_plugin").should == 1
+         subject.class.active.names.count("my_other_plugin").should == 1
       end
 
       it "doesn't deactivate the first plugin when another is activated" do
         subject.class.activate("my_plugin")
         subject.class.activate("my_other_plugin")
 
-        Plugins.active.names.should include("my_plugin")
-        Plugins.active.names.should include("my_other_plugin")
+        subject.class.active.names.should include("my_plugin")
+        subject.class.active.names.should include("my_other_plugin")
       end
     end
 
@@ -56,33 +67,34 @@ module Refinery
         subject.class.activate("my_plugin")
         subject.class.deactivate("my_plugin")
 
-        Plugins.active.count.should == 0
+        subject.class.active.count.should == 0
       end
     end
 
     describe '#set_active' do
+
       it "activates a single plugin" do
         subject.class.set_active(%w(my_plugin))
 
-        Plugins.active.names.should include("my_plugin")
+        subject.class.active.names.should include("my_plugin")
       end
 
       it "activates a list of plugins" do
         subject.class.set_active(%w(my_plugin my_other_plugin))
 
-        Plugins.active.names.should include("my_plugin")
-        Plugins.active.names.should include("my_plugin")
+        subject.class.active.names.should include("my_plugin")
+        subject.class.active.names.should include("my_plugin")
 
-        Plugins.active.count.should == 2
+        subject.class.active.count.should == 2
       end
 
       it "deactivates the initial plugins when another set is set_active" do
         subject.class.set_active(%w(my_plugin))
         subject.class.set_active(%w(my_other_plugin))
 
-        Plugins.active.names.should_not include("my_plugin")
-        Plugins.active.names.should include("my_other_plugin")
-        Plugins.active.count.should == 1
+        subject.class.active.names.should_not include("my_plugin")
+        subject.class.active.names.should include("my_other_plugin")
+        subject.class.active.count.should == 1
       end
 
     end
@@ -112,7 +124,7 @@ module Refinery
 
       it 'only contains items that are always allowed' do
         subject.class.always_allowed.any?.should be_true
-        subject.class.always_allowed.all? { |p| p.always_allow_access? }.should be_true
+        subject.class.always_allowed.all? { |p| p.always_allow_access }.should be_true
       end
     end
 
@@ -127,5 +139,19 @@ module Refinery
       end
     end
 
+    describe ".find_by_name" do
+      it "finds plugin by given name" do
+        subject.class.set_active(%w(my_plugin))
+        subject.class.active.find_by_name("my_plugin").name.should == "my_plugin"
+      end
+    end
+
+    describe ".find_by_title" do
+      it "finds plugin by given title" do
+        subject.class.set_active(%w(my_plugin))
+        subject.class.active.find_by_title("my plugin").title.should == "my plugin"
+      end
+    end
+
   end
 end

data/spec/requests/refinery/admin/custom_assets_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+
+module Refinery
+  describe "custom" do
+    login_refinery_user
+
+    context "javascripts" do
+      before(:each) do
+        ::Refinery::Core.clear_javascripts!
+      end
+
+      it "should be rendered when specified" do
+        ::Refinery::Core.config.register_javascript('custom_js')
+        visit refinery.admin_dashboard_path
+        page.body.include?('custom_js.js').should be
+      end
+    end
+
+    context "stylesheets" do
+      before(:each) do
+        ::Refinery::Core.clear_stylesheets!
+      end
+
+      it "should be rendered when specified" do
+        ::Refinery::Core.config.register_stylesheet('custom_css')
+        visit refinery.admin_dashboard_path
+        page.body.include?('custom_css.css').should be
+      end
+    end
+
+  end
+end

data/spec/requests/refinery/admin/dialogs_spec.rb

@@ -0,0 +1,28 @@
+require "spec_helper"
+
+module Refinery
+  describe "dialog" do
+    login_refinery_user
+
+    context "links" do
+      it "have iframe src" do
+        visit refinery.admin_dialog_path('Link')
+        page.should have_selector("iframe[src='/refinery/pages_dialogs/link_to']")
+      end
+    end
+
+    context "images" do
+      it "have iframe src" do
+        visit refinery.admin_dialog_path('Image')
+        page.should have_selector("iframe[src='/refinery/images/insert?modal=true']")
+      end
+    end
+
+    context "a" do
+      it "have empty iframe src" do
+        visit refinery.admin_dialog_path('a')
+        page.should have_selector("iframe[src='']")
+      end
+    end
+  end
+end

data/spec/requests/refinery/search_spec.rb

@@ -0,0 +1,71 @@
+require "spec_helper"
+
+module Refinery
+  describe "search" do
+    login_refinery_user
+
+    context "when searched item exists" do
+      describe "image extension" do
+        before(:each) { FactoryGirl.create(:image) }
+
+        it "returns found image" do
+          visit refinery.admin_images_path
+          fill_in "search", :with => "beach"
+          click_button "Search"
+          page.should have_selector("img[src*='beach.jpeg']")
+        end
+      end
+
+      describe "resource extension" do
+        before(:each) { FactoryGirl.create(:resource) }
+
+        it "returns found resource" do
+          visit refinery.admin_resources_path
+          fill_in "search", :with => "refinery"
+          click_button "Search"
+          page.should have_content("Refinery Is Awesome.txt")
+        end
+      end
+
+      describe "page extension" do
+        before(:each) { FactoryGirl.create(:page, :title => "Ugis Ozols") }
+
+        it "returns found page" do
+          visit refinery.admin_pages_path
+          fill_in "search", :with => "ugis"
+          click_button "Search"
+          page.should have_content("Ugis Ozols")
+        end
+      end
+    end
+
+    context "when searched item don't exist" do
+      def shared_stuff
+        fill_in "search", :with => "yada yada"
+        click_button "Search"
+        page.should have_content("Sorry, no results found")
+      end
+
+      describe "image extension" do
+        it "returns no results" do
+          visit refinery.admin_images_path
+          shared_stuff
+        end
+      end
+
+      describe "resource extension" do
+        it "returns no results" do
+          visit refinery.admin_resources_path
+          shared_stuff
+        end
+      end
+
+      describe "page extension" do
+        it "returns no results" do
+          visit refinery.admin_pages_path
+          shared_stuff
+        end
+      end
+    end
+  end
+end

data/spec/requests/refinery/site_bar_spec.rb

@@ -0,0 +1,49 @@
+require "spec_helper"
+
+module Refinery
+  describe "site bar" do
+    login_refinery_user
+
+    it "have logout link" do
+      visit refinery.admin_dashboard_path
+
+      page.should have_content("Log out")
+      page.should have_selector("a[href='/refinery/logout']")
+    end
+
+    context "when in backend" do
+      before { visit refinery.admin_dashboard_path }
+
+      it "have a 'switch to your website button'" do
+        page.should have_content("Switch to your website")
+        page.should have_selector("a[href='/']")
+      end
+
+      it "switches to frontend" do
+        page.current_path.should == refinery.admin_dashboard_path
+        click_link "Switch to your website"
+        page.current_path.should == refinery.root_path
+      end
+    end
+
+    context "when in frontend" do
+      before do
+        # make a page in order to avoid 404
+        FactoryGirl.create(:page, :link_url => "/")
+
+        visit refinery.root_path
+      end
+
+      it "have a 'switch to your website editor' button" do
+        page.should have_content("Switch to your website editor")
+        page.should have_selector("a[href='/refinery']")
+      end
+
+      it "switches to backend" do
+        page.current_path.should == refinery.root_path
+        click_link "Switch to your website editor"
+        page.current_path.should == refinery.admin_root_path
+      end
+    end
+  end
+end

data/spec/support/background_selenium_firefox_driver.rb

@@ -0,0 +1,14 @@
+if Capybara.javascript_driver == :selenium
+  require 'selenium-webdriver'
+  require 'selenium/webdriver/firefox'
+
+  if defined?(Selenium::WebDriver::Firefox::Launcher) && Selenium::WebDriver::Platform.mac?
+    # Run firefox in the background, on OS X
+    Selenium::WebDriver::Firefox::Launcher.class_eval do
+      def start
+        assert_profile
+        @binary.start_with @profile, @profile_dir, ''
+      end
+    end
+  end
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,15 @@
+require 'database_cleaner'
+
+RSpec.configure do |config|
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :truncation
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/fixtures/invalid_engine.rb

@@ -0,0 +1,6 @@
+module Refinery
+  module InvalidEngine
+    class Engine < Rails::Engine
+    end
+  end
+end