radiant-vhost-extension 2.1.0

data/spec/controllers/admin/pages_controller_spec.rb

@@ -0,0 +1,173 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe Admin::PagesController do
+  dataset :sites_site_users_and_site_pages
+  
+  before :each do
+    VhostExtension.HOST = sites(:site_a).hostname # Pretend we're connected to site_a so the SiteScope works right
+    rescue_action_in_public!  # ActionController::TestCase no longer considers this request a local request
+
+    # don't bork results with stale cache items
+    Radiant::Cache.clear
+    login_as :usera
+  end
+
+  describe "creating pages" do
+    
+    it "should be associated with the site corresponding to the current hostname" do
+      # Need to call get :new or else the :before_filter in the ApplicationController
+      # that sets up the SiteScope doesn't run.
+      slug = "should-be-associated-with-site"
+      post :create, :page => page_params(:slug => slug)
+      Page.find_by_slug(slug).site.id.should == site_id(:site_a)
+    end
+    
+  end
+
+  describe "permissions" do
+
+    # @todo it "should allow <various> actions for users that belong to multiple sites" 
+    # (or something like that. probably just hook into the block below.)
+  
+    [:admina, :developera, :usera].each do |user|
+      {
+        :post => :create,
+        :put => :update,
+        :delete => :destroy
+      }.each do |method, action|
+        it "should allow the #{action} action to a page belonging to a site #{user.to_s.humanize} has access to" do
+          login_as user
+          send method, action, :id => page_id(:page_a)
+          response.should redirect_to('admin/pages')
+        end
+      end
+    end
+
+    [:developera, :usera].each do |user|
+      {
+        :post => :create,
+        :put => :update,
+        :delete => :destroy
+      }.each do |method, action|
+        it "should show a missing page (404) for the #{action} action on a page NOT belonging to a site #{user.to_s.humanize} has access to" do
+          login_as user
+          send method, action, :id => page_id(:page_b)
+          response.should be_missing
+        end
+      end
+    end
+
+  end
+
+
+=begin
+  
+  describe "permissions" do
+    
+    [:admin, :developer, :non_admin, :existing].each do |user|
+      {
+        :post => :create,
+        :put => :update,
+        :delete => :destroy
+      }.each do |method, action|
+        it "should require login to access the #{action} action" do
+          logout
+          send method, action, :id => Page.first.id
+          response.should redirect_to('/admin/login')
+        end
+        
+        it "should allow access to #{user.to_s.humanize}s for the #{action} action" do
+          login_as user
+          send method, action, :id => Page.first.id
+          response.should redirect_to('/admin/pages')
+        end
+      end
+    end
+    
+    [:index, :show, :new, :edit, :remove].each do |action|
+      before :each do
+        @parameters = lambda do 
+          case action
+          when :index
+            {}
+          when :new
+            {:page_id => page_id(:home)}
+          else
+            {:id => Page.first.id} 
+          end
+        end
+      end
+      
+      it "should require login to access the #{action} action" do
+        logout
+        lambda { send(:get, action, @parameters.call) }.should require_login
+      end
+
+      it "should allow access to admins for the #{action} action" do
+        lambda { 
+          send(:get, action, @parameters.call) 
+        }.should restrict_access(:allow => [users(:admin)], 
+                                 :url => '/admin/pages')
+      end
+
+      it "should allow access to developers for the #{action} action" do
+        lambda { 
+          send(:get, action, @parameters.call) 
+        }.should restrict_access(:allow => [users(:developer)], 
+                                 :url => '/admin/pages')
+      end
+    
+      it "should allow non-developers and non-admins for the #{action} action" do
+        lambda { 
+          send(:get, action, @parameters.call) 
+        }.should restrict_access(:allow => [users(:non_admin), users(:existing)],
+                                 :url => '/admin/pages')
+      end
+    end
+  end
+  
+  
+  describe "prompting page removal" do
+    integrate_views
+    
+    # TODO: This should be in a view or integration spec
+    it "should render the expanded descendants of the page being removed" do
+      get :remove, :id => page_id(:parent), :format => 'html' # shouldn't need this!
+      rendered_pages = [:parent, :child, :grandchild, :great_grandchild, :child_2, :child_3].map {|p| pages(p) }
+      rendered_pages.each do |page|
+        response.should have_tag("tr#page-#{page.id}")
+      end
+    end
+  end
+  
+  it "should initialize meta and buttons_partials in new action" do
+    get :new, :page_id => page_id(:home)
+    response.should be_success
+    assigns(:meta).should be_kind_of(Array)
+    assigns(:buttons_partials).should be_kind_of(Array)
+  end
+
+  it "should initialize meta and buttons_partials in edit action" do
+    get :edit, :id => page_id(:home)
+    response.should be_success
+    assigns(:meta).should be_kind_of(Array)
+    assigns(:buttons_partials).should be_kind_of(Array)
+  end
+  
+  protected
+
+    def assert_rendered_nodes_where(&block)
+      wanted, unwanted = Page.find(:all).partition(&block)
+      wanted.each do |page|
+        response.should have_tag("tr#page-#{page.id}")
+      end
+      unwanted.each do |page|
+        response.should_not have_tag("tr#page-#{page.id}")
+      end
+    end
+
+    def write_cookie(name, value)
+      request.cookies[name] = CGI::Cookie.new(name, value)
+    end
+=end
+end

data/spec/controllers/admin/sites_controller_spec.rb

@@ -0,0 +1,33 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe Admin::SitesController do
+  dataset :sites_site_users_and_site_pages
+
+  { :get => [:index, :new, :edit, :remove],
+    :post => [:create],
+    :put => [:update],
+    :delete => [:destroy] }.each do |method, actions|
+    actions.each do |action|
+      it "should require login to access the #{action} action" do
+        logout
+        lambda { send(method, action, :id => site_id(:site_a)).should require_login }
+      end
+
+      it "should allow you to access to #{action} action if you are a site_admin" do
+        lambda { 
+          send(method, action, :id => site_id(:site_a)) 
+        }.should restrict_access(:allow => users(:admina),
+                                 :url => '/admin/page')
+      end
+      
+      it "should deny you access to #{action} action if you are not a site_admin" do
+        lambda { 
+          send(method, action, :id => site_id(:site_a)) 
+        }.should restrict_access(:deny => [users(:developera), users(:developerb), users(:usera), users(:userb)],
+                                 :url => '/admin/page')
+      end
+    end
+  end
+
+end
+

data/spec/controllers/site_controller_spec.rb

@@ -0,0 +1,33 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe SiteController do
+  dataset :sites_site_users_and_site_pages, :site_home_pages
+ 
+  before(:each) do
+    logout
+    VhostExtension.HOST = sites(:site_a).hostname # Pretend we're connected to site_a so the SiteScope works right
+    rescue_action_in_public!  # ActionController::TestCase no longer considers this request a local request
+
+    # don't bork results with stale cache items
+    Radiant::Cache.clear
+  end
+
+  it "should find and render the home page for the #{VhostExtension.HOST} site" do
+    get :show_page, :url => ''
+    response.should be_success
+    response.body.should == 'Hello A'
+  end
+
+  it "should find and render a child page for the #{VhostExtension.HOST} site" do
+    get :show_page, :url => 'page-a/'
+    response.should be_success
+    response.body.should == 'PageA Body'
+  end
+
+  it "should NOT find and render a child page for a site other than the #{VhostExtension.HOST} site" do
+    get :show_page, :url => 'page-b/'
+    response.should be_missing
+  end
+
+end
+

data/spec/datasets/site_home_pages_dataset.rb

@@ -0,0 +1,76 @@
+class SiteHomePagesDataset < Dataset::Base
+  uses :sites
+
+  def load
+    create_page "Home A", :site_id => site_id(:site_a),
+                        :slug => "/", :parent_id => nil,
+                        :description => "The homepage A", 
+                        :keywords => "Home, Page" do
+      create_page_part "body", :content => "Hello A"
+      create_page_part "sidebar", :content => "<r:title /> sidebar."
+      create_page_part "extended", :content => "Just a test."
+      create_page_part "titles", :content => "<r:title /> <r:page:title />"
+    end
+    create_page "Home B", :site_id => site_id(:site_b),
+                        :slug => "/", :parent_id => nil,
+                        :description => "The homepage B", 
+                        :keywords => "Home, Page" do
+      create_page_part "body", :content => "Hello B"
+      create_page_part "sidebar", :content => "<r:title /> sidebar."
+      create_page_part "extended", :content => "Just a test."
+      create_page_part "titles", :content => "<r:title /> <r:page:title />"
+    end
+  end
+  
+  helpers do
+    def create_page(name, attributes={})
+      attributes = page_params(attributes.reverse_merge(:title => name))
+      body = attributes.delete(:body) || name
+      symbol = name.symbolize
+      create_record :page, symbol, attributes
+      if block_given?
+        old_page_id = @current_page_id
+        @current_page_id = page_id(symbol)
+        yield
+        @current_page_id = old_page_id
+      end
+      if pages(symbol).parts.empty?
+        create_page_part "#{name}_body".symbolize, :name => "body", :content => body + ' body.', :page_id => page_id(symbol)
+      end
+    end
+    def page_params(attributes={})
+      title = attributes[:title] || unique_page_title
+      attributes = {
+        :title => title,
+        :breadcrumb => title,
+        :slug => attributes[:slug] || title.symbolize.to_s.gsub("_", "-"),
+        :class_name => nil,
+        :status_id => Status[:published].id,
+        :published_at => Time.now.to_s(:db)
+      }.update(attributes)
+      attributes[:parent_id] = @current_page_id unless attributes.has_key?(:parent_id)
+      attributes
+    end
+    
+    def create_page_part(name, attributes={})
+      attributes = page_part_params(attributes.reverse_merge(:name => name))
+      # Need to include the page_id here so we're creating new records for all sub-parts
+      create_record :page_part, name.symbolize.to_s+@current_page_id.to_s, attributes
+    end
+    def page_part_params(attributes={})
+      name = attributes[:name] || "unnamed"
+      attributes = {
+        :name => name,
+        :content => name,
+        :page_id => @current_page_id
+      }.update(attributes)
+    end
+    
+    private
+      @@unique_page_title_call_count = 0
+      def unique_page_title
+        @@unique_page_title_call_count += 1
+        "Page #{@@unique_page_title_call_count}"
+      end
+  end
+end

data/spec/datasets/site_pages_dataset.rb

@@ -0,0 +1,31 @@
+class VirtualPage < Page
+  def virtual?
+    true
+  end
+end
+
+class SitePagesDataset < Dataset::Base
+  uses :site_home_pages, :sites
+  
+  def load
+    create_page "Page A", :parent_id => page_id(:home_a), :site_id => site_id(:site_a) do
+      create_page_part "body", :content => "PageA Body", :id => 1
+    end
+
+    create_page "Page B", :parent_id => page_id(:home_b), :site_id => site_id(:site_b) do
+      create_page_part "body", :content => "PageB Body", :id => 2
+    end
+
+    create_page "Parent", :parent_id => page_id(:home_a), :site_id => site_id(:site_a) do
+      create_page "Child", :site_id => site_id(:site_a) do
+        create_page "Grandchild", :site_id => site_id(:site_a) do
+          create_page "Great Grandchild", :site_id => site_id(:site_a)
+        end
+      end
+      create_page "Child 2", :site_id => site_id(:site_a)
+      create_page "Child 3", :site_id => site_id(:site_a)
+    end
+
+    end
+  
+end

data/spec/datasets/site_users_dataset.rb

@@ -0,0 +1,50 @@
+class SiteUsersDataset < Dataset::Base
+  uses :users
+  
+  def load
+    create_user "UserA"
+    create_user "UserB"
+    create_user "AdminA", :admin => true, :site_admin => true
+    create_user "AdminB", :admin => true
+    create_user "DeveloperA", :designer => true
+    create_user "DeveloperB", :designer => true
+  end
+  
+  helpers do
+    def create_user(name, attributes={})
+      create_model :user, name.symbolize, user_attributes(attributes.update(:name => name))
+    end
+    def user_attributes(attributes={})
+      name = attributes[:name] || "John Doe"
+      symbol = name.symbolize
+      attributes = { 
+        :name => name,
+        :email => "test@example.com", 
+        :login => symbol.to_s,
+        :password => "password"
+      }.merge(attributes)
+      attributes[:password_confirmation] = attributes[:password]
+      attributes
+    end
+    def user_params(attributes={})
+      password = attributes[:password] || "password"
+      user_attributes(attributes).update(:password => password, :password_confirmation => password)
+    end
+    
+    def login_as(user)
+      login_user = user.is_a?(User) ? user : users(user)
+      # Set the Vhost HOST to the first hostname in the sites list for the user.
+      # It works for these tests although it may be problematic as we add more
+      # rigorous tests that include multi-site users.
+      VhostExtension.HOST = login_user.sites[0].hostname
+      flunk "Can't login as non-existing user #{user.to_s}." unless login_user
+      request.session['user'] = login_user # Added this because it was in the old PagesController tests
+      request.session['user_id'] = login_user.id
+      login_user
+    end
+    
+    def logout
+      request.session['user_id'] = nil
+    end
+  end
+end

data/spec/datasets/sites_dataset.rb

@@ -0,0 +1,10 @@
+class SitesDataset < Dataset::Base
+  uses :site_users
+  
+  def load
+    # Needs to be test.host so the SiteScope works right
+    create_record :site, :site_a, {:hostname => "test.host"}
+    create_record :site, :site_b, {:hostname => "siteB.host"}
+  end
+
+end

data/spec/datasets/sites_site_users_and_site_pages_dataset.rb

@@ -0,0 +1,8 @@
+class SitesSiteUsersAndSitePagesDataset < Dataset::Base
+  uses :site_pages, :sites_site_users
+  
+  def load
+    Page.update_all "created_by_id = #{user_id(:usera)}, updated_by_id = #{user_id(:usera)}", "id = '#{page_id(:page_a)}'"
+    Page.update_all "created_by_id = #{user_id(:userb)}, updated_by_id = #{user_id(:userb)}", "id = '#{page_id(:page_b)}'"
+  end
+end

data/spec/datasets/sites_site_users_dataset.rb

@@ -0,0 +1,13 @@
+class SitesSiteUsersDataset < Dataset::Base
+  uses :site_users, :sites
+  
+  def load
+    sites(:site_a).users << users(:usera)
+    sites(:site_b).users << users(:userb)
+    sites(:site_a).users << users(:developera)
+    sites(:site_b).users << users(:developerb)
+    sites(:site_a).users << users(:admina)
+    sites(:site_b).users << users(:adminb)
+  end
+
+end

data/spec/fixtures/page_parts.yml

@@ -0,0 +1,11 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+ruby_body:
+  id: 1
+  name: body
+  content: This is the body portion of the Ruby home page.
+  page_id: 1
+rails_body:
+  id: 2
+  name: body
+  content: This is the Rails home page.
+  page_id: 2

data/spec/fixtures/pages.yml

@@ -0,0 +1,19 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+homepage:
+  id: 1
+  title: Ruby Home Page
+  breadcrumb: Home
+  slug: /
+  status_id: 100
+  parent_id:
+  published_at: 2006-01-30 08:41:07 
+  site_id: 1
+documentation:
+  id: 2
+  title: Rails Home Page
+  breadcrumb: Home
+  slug: /
+  status_id: 100
+  parent_id:
+  published_at: 2006-01-30 08:41:07 
+  site_id: 2

data/spec/fixtures/sites.yml

@@ -0,0 +1,7 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+one:
+  id: 1
+  hostname: test.host
+two:
+  id: 2
+  hostname: test2.host

data/spec/fixtures/sites_users.yml

@@ -0,0 +1,6 @@
+user_1_site_1:
+  user_id: 1
+  site_id: 1
+user_2_site_2:
+  user_id: 2
+  site_id: 2

data/spec/fixtures/users.yml

@@ -0,0 +1,35 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+existing:
+  id: 1
+  name: Existing User
+  login: existing
+  salt: 42ddb5b6479872565981adc78e86ddc50fa21637
+  password: 4c5f9d5a0d4cd65d61f6b1505890576d1272986c # password
+  email: existing.user@gmail.com
+another:
+  id: 2
+  name: Another User
+  login: another
+  salt: 42ddb5b6479872565981adc78e86ddc50fa21637
+  password: 4c5f9d5a0d4cd65d61f6b1505890576d1272986c # password
+admin:
+  id: 3
+  name: Admin User
+  login: admin
+  salt: 42ddb5b6479872565981adc78e86ddc50fa21637
+  password: 4c5f9d5a0d4cd65d61f6b1505890576d1272986c # password
+  admin: true
+developer:
+  id: 4
+  name: Developer User
+  login: developer
+  salt: 42ddb5b6479872565981adc78e86ddc50fa21637
+  password: 4c5f9d5a0d4cd65d61f6b1505890576d1272986c # password
+  developer: true
+non_admin:
+  id: 5
+  name: Non-Admin User
+  login: non-admin
+  salt: 42ddb5b6479872565981adc78e86ddc50fa21637
+  password: 4c5f9d5a0d4cd65d61f6b1505890576d1272986c # password
+  admin: false

data/spec/models/page_spec.rb

@@ -0,0 +1,22 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Page, "site scope" do
+  dataset :site_pages, :sites, :site_users
+  test_helper :validations
+
+  before :each do
+    @page = @model = Page.new(page_params)
+  end
+
+  it 'should validate uniqueness of' do
+    @page.parent = pages(:parent)
+    # Need to manually set the site_id since we're not going through the controller stack
+    @page.site_id = site_id(:site_a)
+    @page.valid?
+    @page.save
+    puts "ERRORS: "+@page.errors.length.to_s
+    assert_invalid :slug, 'slug already in use for child of parent', 'child', 'child-2', 'child-3'
+    assert_valid :slug, 'child-4'
+  end
+
+end

data/spec/models/site_spec.rb

@@ -0,0 +1,19 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Site do
+  dataset :site_users
+  
+  before(:each) do
+    @site = Site.new
+  end
+
+  it "should be valid" do
+    @site.should be_valid
+  end
+
+  it "should allow users to be associated" do
+    user = users(:usera)
+    @site.users << user
+    @site.users.should have(1).items
+  end
+end

data/spec/models/user_spec.rb

@@ -0,0 +1,16 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe User do
+  dataset :sites_site_users
+  test_helper :validations
+  
+  before :each do
+    @model = @user = User.new(user_params)
+    @user.confirm_password = false
+  end
+  
+  it 'should have at least one associated site' do
+    users(:usera).should have_at_least(1).sites
+  end
+
+end

data/spec/spec.opts

@@ -0,0 +1,6 @@
+--colour
+--format
+progress
+--loadby
+mtime
+--reverse

data/spec/spec_helper.rb

@@ -0,0 +1,42 @@
+unless defined? RADIANT_ROOT
+  ENV["RAILS_ENV"] = "test"
+  case
+  when ENV["RADIANT_ENV_FILE"]
+    require ENV["RADIANT_ENV_FILE"]
+  when File.dirname(__FILE__) =~ %r{vendor/radiant/vendor/extensions}
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../../../")}/config/environment"
+  else
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../")}/config/environment"
+  end
+end
+require "#{RADIANT_ROOT}/spec/spec_helper"
+
+if File.directory?(File.dirname(__FILE__) + "/scenarios")
+  Scenario.load_paths.unshift File.dirname(__FILE__) + "/scenarios"
+end
+if File.directory?(File.dirname(__FILE__) + "/matchers")
+  Dir[File.dirname(__FILE__) + "/matchers/*.rb"].each {|file| require file }
+end
+
+Dataset::Resolver.default << (File.dirname(__FILE__) + "/datasets")
+
+# Needs to be test.host so the SiteScope works right
+VhostExtension.HOST = "test.host"
+
+Spec::Runner.configure do |config|
+  # config.use_transactional_fixtures = true
+  # config.use_instantiated_fixtures  = false
+  # config.fixture_path = RAILS_ROOT + '/spec/fixtures'
+
+  # You can declare fixtures for each behaviour like this:
+  #   describe "...." do
+  #     fixtures :table_a, :table_b
+  #
+  # Alternatively, if you prefer to declare them only once, you can
+  # do so here, like so ...
+  #
+  # config.global_fixtures = :page_parts, :pages, :sites_users, :sites, :users
+  #
+  # If you declare global fixtures, be aware that they will be declared
+  # for all of your examples, even those that don't use them.
+end

data/test/fixtures/page_parts.yml

@@ -0,0 +1,11 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+ruby_body:
+  id: 1
+  name: body
+  content: This is the body portion of the Ruby home page.
+  page_id: 1
+rails_body:
+  id: 2
+  name: body
+  content: This is the Rails home page.
+  page_id: 2

data/test/fixtures/pages.yml

@@ -0,0 +1,19 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+homepage:
+  id: 1
+  title: Ruby Home Page
+  breadcrumb: Home
+  slug: /
+  status_id: 100
+  parent_id:
+  published_at: 2006-01-30 08:41:07 
+  site_id: 1
+documentation:
+  id: 2
+  title: Rails Home Page
+  breadcrumb: Home
+  slug: /
+  status_id: 100
+  parent_id:
+  published_at: 2006-01-30 08:41:07 
+  site_id: 2

data/test/fixtures/sites.yml

@@ -0,0 +1,7 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+one:
+  id: 1
+  hostname: test.host
+two:
+  id: 2
+  hostname: test2.host

data/test/fixtures/sites_users.yml

@@ -0,0 +1,6 @@
+user_1_site_1:
+  user_id: 1
+  site_id: 1
+user_2_site_2:
+  user_id: 2
+  site_id: 2