bcms_cas 1.2.2 → 1.3.0

data/README.markdown

@@ -17,6 +17,7 @@ Here are the necessary steps to install this module.
 
 Install the module by running the following command in your project.
 
+    $ gem install bcms_cas
     $ rails g cms:install bcms_cas
 	$ rake db:seed
 
@@ -34,17 +35,14 @@ If you are using the same cas server for each environment (development, producti
 
 Go to your CAS domain and test that you can log in.
 
-## 2a. Ensure SITE_DOMAIN is configured.
+## 2a. Ensure site_domain is configured.
 
-Edit the production.rb to make sure your SITE_DOMAIN variable in production is correctly set to the right top level domain. This will be needed to allow redirects between the servers to happen correctly (it requires Absolute URLs).
+Edit the production.rb to make sure your site_domain configuration in production is correctly set to the right top level domain. This will be needed to allow redirects between the servers to happen correctly (it requires Absolute URLs).
 
-    SITE_DOMAIN="www.yourdomainname.com"
+    config.cms.site_domain = "www.your_site_domiain.com"
 
 ## 3. Configure the 'CAS Authenticated User' Group
-When you run rake db:migrate, this module will add a new group to the CMS called 'CAS Authenticated Users'. All users that
-log in successfully will be assigned to members of this group. You will potentially want to rename this group to something
-that more accurately reflects who these users are (i.e. Members, Staff, etc) and then set which sections of the website this
-group can visit.
+When you run rake db:migrate, this module will add a new group to the CMS called 'CAS Authenticated Users'. All users that log in successfully will be assigned to members of this group. You will potentially want to rename this group to something that more accurately reflects who these users are (i.e. Members, Staff, etc) and then set which sections of the website this group can visit.
 
 ## 4. Create a Login Form
 

data/app/assets/javascripts/bcms_cas/application.js

@@ -0,0 +1,15 @@
+// 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.
+//
+// WARNING: THE FIRST BLANK LINE MARKS THE END OF WHAT'S TO BE PROCESSED, ANY BLANK LINE SHOULD
+// GO AFTER THE REQUIRES BELOW.
+//
+//= require jquery
+//= require jquery_ujs
+//= require_tree .

data/app/assets/stylesheets/bcms_cas/application.css

@@ -0,0 +1,13 @@
+/*
+ * 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 .
+ */

data/app/controllers/bcms_cas/application_controller.rb

@@ -0,0 +1,4 @@
+module BcmsCas
+  class ApplicationController < ActionController::Base
+  end
+end

data/app/helpers/bcms_cas/application_helper.rb

@@ -0,0 +1,4 @@
+module BcmsCas
+  module ApplicationHelper
+  end
+end

data/app/models/cas_user.rb

@@ -19,6 +19,6 @@ class CasUser < Cms::TemporaryUser
   # @deprecated
   # This exists only for backwards compatibility for when this used to inherit from GuestUser. It should be removed in 1.2.
   def group
-    Group.find_by_code(GROUP_NAME)
+    Cms::Group.find_by_code(GROUP_NAME)
   end
 end

data/config/routes.rb

@@ -0,0 +1,2 @@
+BcmsCas::Engine.routes.draw do
+end

data/db/bcms_cas.seeds.rb

@@ -0,0 +1,4 @@
+group = Group.create!(:name=>"CAS Authenticated Users", 
+                      :code=>"cas_group", 
+                      :group_type=>GroupType.find_by_name("Registered Public User"))
+group.sections = Section.all

data/lib/bcms_cas.rb

@@ -1,4 +1,4 @@
-require 'bcms_cas/engine'
-require 'bcms_cas/routes'
-
+require "bcms_cas/engine"
 
+module BcmsCas
+end

data/lib/bcms_cas/authentication.rb

@@ -19,7 +19,7 @@ module Cas
       controller_class.before_filter :verify_cas_configured
       controller_class.before_filter CASClient::Frameworks::Rails::GatewayFilter
       controller_class.before_filter :login_from_cas_ticket
-      controller_class.before_filter :try_to_stream_file, :only=>[:show]      
+      controller_class.before_filter :try_to_stream_file      
       controller_class.before_filter :check_access_to_page
       
     end
@@ -42,7 +42,7 @@ module Cas
 
           # Having to set both of these feels very duplicative. Ideally I would like a way
           #   to set only once, but calling current_user= has side effects.
-          @current_user = User.current = user
+          @current_user = Cms::User.current = user
 
           logger.debug "CasUser information found in session. Setting current_user as '#{user.login}" if @current_user
         end
@@ -64,7 +64,7 @@ module Cas
       def destroy_with_cas
         logger.debug "Logging user out of both cms and CAS server."
         logout_user
-        Cas::Utils.logout(self, "http://#{SITE_DOMAIN}/")
+        Cas::Utils.logout(self, "http://#{Rails.application.config.cms.site_domain}/")
       end
     end
   end

data/lib/bcms_cas/utils.rb

@@ -68,7 +68,7 @@ module Cas
     private
 
     def self.to_absolute_url(path)
-      "http://#{SITE_DOMAIN}#{path}"
+      "http://#{Rails.application.config.cms.site_domain}#{path}"
     end
   end
 end

data/lib/bcms_cas/version.rb

@@ -1,3 +1,3 @@
 module BcmsCas
-  VERSION = "1.2.2"
+  VERSION = "1.3.0"
 end

data/lib/tasks/bcms_cas_tasks.rake

@@ -0,0 +1,4 @@
+# desc "Explaining what the task does"
+# task :bcms_cas do
+#   # Task goes here
+# end

data/test/bcms_cas_test.rb

@@ -0,0 +1,7 @@
+require 'test_helper'
+
+class BcmsCasTest < ActiveSupport::TestCase
+  test "truth" do
+    assert_kind_of Module, BcmsCas
+  end
+end

data/test/test_helper.rb

@@ -1,15 +1,15 @@
+# Configure Rails Environment
 ENV["RAILS_ENV"] = "test"
-require File.expand_path('../../config/environment', __FILE__)
-require 'rails/test_help'
 
-require 'mocha'
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+require "rails/test_help"
 
-class ActiveSupport::TestCase
-  # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order.
-  #
-  # Note: You'll currently still have to declare fixtures explicitly in integration tests
-  # -- they do not yet inherit this setting
-  fixtures :all
+Rails.backtrace_cleaner.remove_silencers!
 
-  # Add more helper methods to be used by all tests here...
+# Load support files
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+# Load fixtures from the engine
+if ActiveSupport::TestCase.method_defined?(:fixture_path=)
+  ActiveSupport::TestCase.fixture_path = File.expand_path("../fixtures", __FILE__)
 end

data/test/unit/cas/cas_authentication_test.rb

@@ -24,88 +24,90 @@ class MyController < ActionController::Base
   end
 end
 
-class CasAuthTest < ActiveSupport::TestCase
+module Cms
+  class CasAuthTest < ActiveSupport::TestCase
 
-  def setup
-    MyController.expects(:skip_filter).with(:check_access_to_page)
-    MyController.expects(:skip_filter).with(:try_to_stream_file)
+    def setup
+      MyController.expects(:skip_filter).with(:check_access_to_page)
+      MyController.expects(:skip_filter).with(:try_to_stream_file)
 
-    MyController.expects(:before_filter).with(:verify_cas_configured)
-    MyController.expects(:before_filter).with(CASClient::Frameworks::Rails::GatewayFilter)
-    MyController.expects(:before_filter).with(:login_from_cas_ticket)
-    MyController.expects(:before_filter).with(:try_to_stream_file)
-    MyController.expects(:before_filter).with(:check_access_to_page)
+      MyController.expects(:before_filter).with(:verify_cas_configured)
+      MyController.expects(:before_filter).with(CASClient::Frameworks::Rails::GatewayFilter)
+      MyController.expects(:before_filter).with(:login_from_cas_ticket)
+      MyController.expects(:before_filter).with(:try_to_stream_file)
+      MyController.expects(:before_filter).with(:check_access_to_page)
 
-    MyController.send(:include, Cas::Authentication)
+      MyController.send(:include, Cas::Authentication)
 
-  end
+    end
 
-  def teardown
-    User.current = nil
-  end
+    def teardown
+      User.current = nil
+    end
 
 
-  test "adds current_user and login from session to class" do
-    c = MyController.new
+    test "adds current_user and login from session to class" do
+      c = MyController.new
 
-    assert c.respond_to?(:current_user)
-    assert c.respond_to?(:login_from_cas_ticket)
+      assert c.respond_to?(:current_user)
+      assert c.respond_to?(:login_from_cas_ticket)
 
-  end
+    end
 
 
-  test "login_from_cas_ticket will create and set the current user and User.current if a session attribute was found." do
-    c = MyController.new
-    c.session[:cas_user] = "1234"
+    test "login_from_cas_ticket will create and set the current user and User.current if a session attribute was found." do
+      c = MyController.new
+      c.session[:cas_user] = "1234"
 
-    User.current = Group.find_by_code("guest")
+      User.current = Group.find_by_code("guest")
 
-    c.login_from_cas_ticket
+      c.login_from_cas_ticket
 
-    current_user = c.get_at_current_user
-    assert_equal CasUser, current_user.class
-    assert_equal "1234", current_user.login
-    assert_equal current_user, User.current
-  end
+      current_user = c.get_at_current_user
+      assert_equal CasUser, current_user.class
+      assert_equal "1234", current_user.login
+      assert_equal current_user, User.current
+    end
 
-  test "Cms::ContentController gets augmented" do
-    assert (Cms::ContentController.new.respond_to? :login_from_cas_ticket)
-  end
+    test "Cms::ContentController gets augmented" do
+      assert (Cms::ContentController.new.respond_to? :login_from_cas_ticket)
+    end
 
-end
+  end
 
-class CasSessionControllerTest < ActiveSupport::TestCase
+  class CasSessionControllerTest < ActiveSupport::TestCase
 
 
-  test "alias_method_chain the normal methods" do
-    MyController.send(:include, Cas::SingleLogOut)
+    test "alias_method_chain the normal methods" do
+      MyController.send(:include, Cas::SingleLogOut)
 
-    c = MyController.new
+      c = MyController.new
     
-    assert( c.respond_to? :destroy)
-    assert( c.respond_to? :destroy_with_cas)
-    assert( c.respond_to? :destroy_without_cas)
+      assert( c.respond_to? :destroy)
+      assert( c.respond_to? :destroy_with_cas)
+      assert( c.respond_to? :destroy_without_cas)
 
-  end
+    end
 
-  test "destroy_with_cas redirects to server and calls logout_user" do
-    MyController.send(:include, Cas::SingleLogOut)
-    c = MyController.new
+    test "destroy_with_cas redirects to server and calls logout_user" do
+      MyController.send(:include, Cas::SingleLogOut)
+      c = MyController.new
 
-    c.expects(:logout_user)
+      c.expects(:logout_user)
 
-    Cas::Utils.expects(:logout).with(c, "http://#{SITE_DOMAIN}/")
+      Cas::Utils.expects(:logout).with(c, "http://#{Rails.application.config.cms.site_domain}/")
 
-    c.destroy_with_cas
+      c.destroy_with_cas
 
 
-  end
+    end
 
-  test "that BrowserCMS 3.0.3 installed." do
-    assert Cms::SessionsController.new.respond_to?(:logout_user), "Need to have BrowserCMS 3.0.3 installed for Cas module to work."
-  end
+    test "that BrowserCMS 3.0.3 installed." do
+      assert Cms::SessionsController.new.respond_to?(:logout_user), "Need to have BrowserCMS 3.0.3 installed for Cas module to work."
+    end
   
-  test "Cms::SessionController gets augmented" do
-    assert (Cms::SessionsController.new.respond_to? :destroy_with_cas)    
+    test "Cms::SessionController gets augmented" do
+      assert (Cms::SessionsController.new.respond_to? :destroy_with_cas)    
+    end
   end
-end
+end

data/test/unit/cas_user_test.rb

@@ -1,39 +1,41 @@
 require 'test_helper'
 
-class CasUserTest < ActiveSupport::TestCase
-
-  def setup
-    @viewable_section = Section.create!(:name=>"root", :root=>true, :path=>"/")
-    @viewable_page = Page.create!(:name=>"Home", :section=>@viewable_section, :path=>"/p")
-    @cas_group = Group.create!(:name=>"G", :code=>"cas_group")
-    @cas_group.sections = Section.all
-  end
-
-  test "group returns the cas_group" do
-    user = CasUser.new
-    assert_equal @cas_group, user.group
-  end
-
-  test "cas_user should be able to view all sections (based on group)" do
-    user = CasUser.new
-    assert user.able_to_view?(@viewable_page)
-  end
-
-  test "setting login" do
-    user = CasUser.new(:login=>"bob")
-    assert_equal "bob", user.login
-  end
-
-  test "that CasUsers are not considered guests" do
-    user = CasUser.new
-    assert !user.guest?
-  end
-
-  test "determine if a user has cms_access" do
-    user = CasUser.new
-    user.groups << @cas_group
-
-    @cas_group.expects(:cms_access?).returns(true)
-    assert_equal true, user.cms_access? 
+module Cms
+  class CasUserTest < ActiveSupport::TestCase
+
+    def setup
+      @viewable_section = Section.create!(:name=>"root", :root=>true, :path=>"/")
+      @viewable_page = Page.create!(:name=>"Home", :section=>@viewable_section, :path=>"/p")
+      @cas_group = Group.create!(:name=>"G", :code=>"cas_group")
+      @cas_group.sections = Section.all
+    end
+
+    test "group returns the cas_group" do
+      user = CasUser.new
+      assert_equal @cas_group, user.group
+    end
+
+    test "cas_user should be able to view all sections (based on group)" do
+      user = CasUser.new
+      assert user.able_to_view?(@viewable_page)
+    end
+
+    test "setting login" do
+      user = CasUser.new(:login=>"bob")
+      assert_equal "bob", user.login
+    end
+
+    test "that CasUsers are not considered guests" do
+      user = CasUser.new
+      assert !user.guest?
+    end
+
+    test "determine if a user has cms_access" do
+      user = CasUser.new
+      user.groups << @cas_group
+
+      @cas_group.expects(:cms_access?).returns(true)
+      assert_equal true, user.cms_access? 
+    end
   end
-end
+end

data/test/unit/cas_utils_test.rb

@@ -1,83 +1,85 @@
 require 'test_helper'
 require 'mocha'
 
-class CasUtilsTest < ActiveSupport::TestCase
+module Cms
+  class CasUtilsTest < ActiveSupport::TestCase
 
-  def setup
-    @current_page = Page.new(:path=>"/expected")
-    @portlet = LoginPortlet.new
-  end
+    def setup
+      @current_page = Page.new(:path=>"/expected")
+      @portlet = LoginPortlet.new
+    end
 
-  test "service_url returns absolute success_url if specified" do
-    @portlet.success_url = "/stuff"
-    assert_equal "http://localhost:3000/stuff", Cas::Utils.service_url(@portlet, @current_page), "This really need to be an absolute path for it work w/ CAS."
-  end
+    test "service_url returns absolute success_url if specified" do
+      @portlet.success_url = "/stuff"
+      assert_equal "http://localhost:3000/stuff", Cas::Utils.service_url(@portlet, @current_page), "This really need to be an absolute path for it work w/ CAS."
+    end
 
-  test "service_url returns current page if specified" do
-    p = LoginPortlet.new
-    p.current_page = Page.new(:path=>"/expected")
+    test "service_url returns current page if specified" do
+      p = LoginPortlet.new
+      p.current_page = Page.new(:path=>"/expected")
 
-    s = Cas::Utils.service_url(p, @current_page)
-    assert_equal "http://localhost:3000/expected", s
-  end
+      s = Cas::Utils.service_url(p, @current_page)
+      assert_equal "http://localhost:3000/expected", s
+    end
 
-  test "service_url returns redirect_to if available" do
-    s = Cas::Utils.service_url(@portlet, @current_page, "/redirected")
-    assert_equal "http://localhost:3000/redirected", s
-  end
+    test "service_url returns redirect_to if available" do
+      s = Cas::Utils.service_url(@portlet, @current_page, "/redirected")
+      assert_equal "http://localhost:3000/redirected", s
+    end
 
-  test "Nil page doesn't cause Nil exception" do
-    s = Cas::Utils.service_url(@portlet, nil, nil)
-    assert_equal "http://localhost:3000", s
-  end
+    test "Nil page doesn't cause Nil exception" do
+      s = Cas::Utils.service_url(@portlet, nil, nil)
+      assert_equal "http://localhost:3000", s
+    end
 
-  test "Portlets work as expected (CMS Core check)" do
-    p = LoginPortlet.new
+    test "Portlets work as expected (CMS Core check)" do
+      p = LoginPortlet.new
 
-    assert_equal nil, p.success_url, "Validate that LoginPortlet returns nil if no success_url is set (rather than an empty string."
-  end
+      assert_equal nil, p.success_url, "Validate that LoginPortlet returns nil if no success_url is set (rather than an empty string."
+    end
 
-  test "cas_server_url looks up CAS server from config in environment" do
-    expected = "https://127.0.0.1"
-    CASClient::Frameworks::Rails::Filter.expects(:config).returns({:cas_base_url=>expected})
+    test "cas_server_url looks up CAS server from config in environment" do
+      expected = "https://127.0.0.1"
+      CASClient::Frameworks::Rails::Filter.expects(:config).returns({:cas_base_url=>expected})
 
-    assert_equal expected, Cas::Utils.cas_server_url
-  end
+      assert_equal expected, Cas::Utils.cas_server_url
+    end
 
-  test "get login ticket" do
-    expected_host = "https://random.com"
-    Cas::Utils.expects(:cas_server_url).with.returns(expected_host)
+    test "get login ticket" do
+      expected_host = "https://random.com"
+      Cas::Utils.expects(:cas_server_url).with.returns(expected_host)
 
-    Net::HTTP.any_instance.stubs(:start).returns(mock(:body=>"Ticket 100"))
-    uri = stub(:path=>"#{expected_host}/loginTicket", :host=>"", :port=>80)
-    URI.expects(:parse).with("#{expected_host}/loginTicket").returns(uri)
+      Net::HTTP.any_instance.stubs(:start).returns(mock(:body=>"Ticket 100"))
+      uri = stub(:path=>"#{expected_host}/loginTicket", :host=>"", :port=>80)
+      URI.expects(:parse).with("#{expected_host}/loginTicket").returns(uri)
 
-    ticket = Cas::Utils.fetch_lt_from_cas
-    assert_equal "Ticket 100", ticket
-  end
+      ticket = Cas::Utils.fetch_lt_from_cas
+      assert_equal "Ticket 100", ticket
+    end
 
-  # Verification test of CasClient API behavior.
-  test "[CASClient] Verify expected behavior of CASClient#logout_url" do
-    destination_url = "localhost"
+    # Verification test of CasClient API behavior.
+    test "[CASClient] Verify expected behavior of CASClient#logout_url" do
+      destination_url = "localhost"
 
-    client = CASClient::Client.new({:cas_base_url=>"/", :logout_url=>"/"})
+      client = CASClient::Client.new({:cas_base_url=>"/", :logout_url=>"/"})
 
-    logout_url = client.logout_url(destination_url, destination_url)
-    assert_equal "/?destination=localhost&url=localhost", logout_url, "Verifies that logout_url generates a correct URL."
-    assert_equal String, logout_url.class
-  end
+      logout_url = client.logout_url(destination_url, destination_url)
+      assert_equal "/?destination=localhost&url=localhost", logout_url, "Verifies that logout_url generates a correct URL."
+      assert_equal String, logout_url.class
+    end
 
-  test "Cas::Utils#logout redirects and attaches gateway=true to logout_url" do
-    destination_url = "localhost"
-    logout_url = "/?destination=localhost&url=localhost"
+    test "Cas::Utils#logout redirects and attaches gateway=true to logout_url" do
+      destination_url = "localhost"
+      logout_url = "/?destination=localhost&url=localhost"
 
-    CASClient::Frameworks::Rails::Filter.expects(:client).returns(CASClient::Client.new({:cas_base_url=>""}))
-    CASClient::Client.any_instance.expects(:logout_url).with(destination_url, destination_url).returns(logout_url)
+      CASClient::Frameworks::Rails::Filter.expects(:client).returns(CASClient::Client.new({:cas_base_url=>""}))
+      CASClient::Client.any_instance.expects(:logout_url).with(destination_url, destination_url).returns(logout_url)
 
-    controller = stub()
-    controller.expects(:redirect_to).with("#{logout_url}&gateway=true")
-    Cas::Utils.expects(:reset_session_and_get_referrer).returns(destination_url)
+      controller = stub()
+      controller.expects(:redirect_to).with("#{logout_url}&gateway=true")
+      Cas::Utils.expects(:reset_session_and_get_referrer).returns(destination_url)
     
-    Cas::Utils.logout(controller, "/")
+      Cas::Utils.logout(controller, "/")
+    end
   end
-end
+end

data/test/unit/cms/temporary_user_test.rb

@@ -1,81 +1,83 @@
 require "test_helper"
 
-class TemporaryUserTest < ActiveSupport::TestCase
+module Cms
+  class TemporaryUserTest < ActiveSupport::TestCase
 
-  MINIMUM_VALID_ATTRIBUTES = {:login=>"abc@bm.com", :password=>"123", :password_confirmation=>"123", :email=>"abc@bm.com"}
-  def setup
-    @user = Cms::TemporaryUser.new(MINIMUM_VALID_ATTRIBUTES)
-  end
+    MINIMUM_VALID_ATTRIBUTES = {:login=>"abc@bm.com", :password=>"123", :password_confirmation=>"123", :email=>"abc@bm.com"}
+    def setup
+      @user = Cms::TemporaryUser.new(MINIMUM_VALID_ATTRIBUTES)
+    end
 
-  def teardown
+    def teardown
 
-  end
+    end
 
-  test "Should not be able to save or save!" do
+    test "Should not be able to save or save!" do
     
-    assert_equal false, @user.save
-    assert_equal true, @user.valid?
+      assert_equal false, @user.save
+      assert_equal true, @user.valid?
 
-    assert_raise NotImplementedError do
-      @user.save! 
-    end
+      assert_raise NotImplementedError do
+        @user.save! 
+      end
 
-  end
-  test "Shouldn't be able to update attributes" do
-    assert_equal false, @user.update_attribute(:login, "OTHER")
-    assert_equal false, @user.update_attributes({:login =>"OTHER"})
-  end
+    end
+    test "Shouldn't be able to update attributes" do
+      assert_equal false, @user.update_attribute(:login, "OTHER")
+      assert_equal false, @user.update_attributes({:login =>"OTHER"})
+    end
 
-  test "Should belong to no groups by default" do
-    assert_equal 0, @user.groups.size
-  end
+    test "Should belong to no groups by default" do
+      assert_equal 0, @user.groups.size
+    end
 
-  test "Temp users should be able to view sections their groups have permissions to" do
-    group = Group.create!(:name=>"My Group")
-    section = Section.create!(:name=>"My Section", :path=>"/mysection")
-    section.groups << group
-    section.save!
+    test "Temp users should be able to view sections their groups have permissions to" do
+      group = Group.create!(:name=>"My Group")
+      section = Section.create!(:name=>"My Section", :path=>"/mysection")
+      section.groups << group
+      section.save!
 
-    user = Cms::TemporaryUser.new
-    user.groups << group
+      user = Cms::TemporaryUser.new
+      user.groups << group
 
-    assert user.able_to_view?(section)
+      assert user.able_to_view?(section)
 
 
-  end
+    end
 
-  test "Viewable sections" do
-    group = Group.create!(:name=>"Group 1")
-    first_section = Section.create!(:name=>"Section1", :path=>"/section1")
-    first_section.groups << group
-    first_section.save!
+    test "Viewable sections" do
+      group = Group.create!(:name=>"Group 1")
+      first_section = Section.create!(:name=>"Section1", :path=>"/section1")
+      first_section.groups << group
+      first_section.save!
 
-    group2 = Group.create!(:name=>"Group 2")
-    second_section = Section.create!(:name=>"Section 2", :path=>"/section2")
-    second_section.groups << group2
-    second_section.save!
+      group2 = Group.create!(:name=>"Group 2")
+      second_section = Section.create!(:name=>"Section 2", :path=>"/section2")
+      second_section.groups << group2
+      second_section.save!
 
-    user = Cms::TemporaryUser.new
-    user.groups << group << group2
+      user = Cms::TemporaryUser.new
+      user.groups << group << group2
 
-    assert_equal [first_section, second_section], user.viewable_sections
-    assert user.able_to_view?(first_section)
-    assert user.able_to_view?(second_section)
+      assert_equal [first_section, second_section], user.viewable_sections
+      assert user.able_to_view?(first_section)
+      assert user.able_to_view?(second_section)
 
-  end
+    end
 
-  test "A user with no groups should not have CMS access" do
-    user = Cms::TemporaryUser.new
-    assert_equal false, user.cms_access? 
+    test "A user with no groups should not have CMS access" do
+      user = Cms::TemporaryUser.new
+      assert_equal false, user.cms_access? 
+    end
   end
-end
 
 
-class GuestUserTest <ActiveSupport::TestCase
+  class GuestUserTest <ActiveSupport::TestCase
 
-  test "Verify save behavior of Guest is similar to TemporaryUser" do
-    guest = GuestUser.new(TemporaryUserTest::MINIMUM_VALID_ATTRIBUTES)
-    assert_equal false, guest.save()
-    assert_equal true, guest.valid?
+    test "Verify save behavior of Guest is similar to TemporaryUser" do
+      guest = GuestUser.new(TemporaryUserTest::MINIMUM_VALID_ATTRIBUTES)
+      assert_equal false, guest.save()
+      assert_equal true, guest.valid?
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bcms_cas
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,24 +9,30 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-31 00:00:00.000000000 Z
+date: 2012-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: browsercms
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - <
       - !ruby/object:Gem::Version
-        version: 3.3.4
+        version: 3.6.0
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 3.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - <
+      - !ruby/object:Gem::Version
+        version: 3.6.0
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 3.3.4
+        version: 3.5.0
 - !ruby/object:Gem::Dependency
   name: rubycas-client
   requirement: !ruby/object:Gem::Requirement
@@ -51,12 +57,16 @@ extensions: []
 extra_rdoc_files:
 - README.markdown
 files:
-- app/controllers/application_controller.rb
-- app/helpers/application_helper.rb
+- app/assets/javascripts/bcms_cas/application.js
+- app/assets/stylesheets/bcms_cas/application.css
+- app/controllers/bcms_cas/application_controller.rb
+- app/helpers/bcms_cas/application_helper.rb
 - app/models/cas_user.rb
 - app/models/cms/temporary_user.rb
 - app/portlets/helpers/login_portlet_helper.rb
 - app/views/portlets/login/render.html.erb
+- config/routes.rb
+- db/bcms_cas.seeds.rb
 - lib/bcms_cas/acts_as_content_page_extention.rb
 - lib/bcms_cas/authentication.rb
 - lib/bcms_cas/configuration.rb
@@ -72,12 +82,13 @@ files:
 - lib/generators/bcms_cas/install/templates/login_portlet_helper.rb
 - lib/generators/bcms_cas/install/templates/render.html.erb
 - lib/generators/bcms_cas/install/USAGE
+- lib/tasks/bcms_cas_tasks.rake
 - README.markdown
 - Gemfile
 - LICENSE.txt
 - COPYRIGHT.txt
 - GPL.txt
-- test/performance/browsing_test.rb
+- test/bcms_cas_test.rb
 - test/test_helper.rb
 - test/unit/cas/cas_authentication_test.rb
 - test/unit/cas/configuration_test.rb
@@ -100,7 +111,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3576433894796172697
+      hash: 754055201174339068
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -109,7 +120,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3576433894796172697
+      hash: 754055201174339068
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24
@@ -117,7 +128,7 @@ signing_key:
 specification_version: 3
 summary: A CAS Module for BrowserCMS
 test_files:
-- test/performance/browsing_test.rb
+- test/bcms_cas_test.rb
 - test/test_helper.rb
 - test/unit/cas/cas_authentication_test.rb
 - test/unit/cas/configuration_test.rb

data/app/controllers/application_controller.rb

@@ -1,3 +0,0 @@
-class ApplicationController < ActionController::Base
-  protect_from_forgery
-end

data/app/helpers/application_helper.rb

@@ -1,2 +0,0 @@
-module ApplicationHelper
-end

data/test/performance/browsing_test.rb

@@ -1,9 +0,0 @@
-require 'test_helper'
-require 'rails/performance_test_help'
-
-# Profiling results for each test method are written to tmp/performance.
-class BrowsingTest < ActionDispatch::PerformanceTest
-  def test_homepage
-    get '/'
-  end
-end