hydra-head 4.1.3 → 5.0.0.pre1

data/test_support/spec/controllers/hydra-assets_controller_spec.rb

@@ -1,151 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-
-# See cucumber tests (ie. /features/edit_document.feature) for more tests, including ones that test the edit method & view
-# You can run the cucumber tests with 
-#
-# cucumber --tags @edit
-# or
-# rake cucumber
-
-describe Hydra::AssetsController do
-  before :all do
-    @behavior = Hydra::AssetsController.deprecation_behavior
-    Hydra::AssetsController.deprecation_behavior = :silence
-  end
-
-  after :all do
-    Hydra::AssetsController.deprecation_behavior = @behavior 
-  end
-  
-  before do
-      @user = FactoryGirl.create(:user)
-      sign_in @user
-      controller.stubs(:enforce_access_controls).returns(true)
-  end
-  
-  it "should use DocumentController" do
-    controller.should be_an_instance_of(Hydra::AssetsController)
-  end
-
-  describe "new" do
-    before do
-      @user = FactoryGirl.create(:user)
-      sign_in @user
-      @asset = ModsAsset.new
-      ModsAsset.stubs(:new).returns(@asset)
-    end
-    it "should create and redirect with a flash message" do
-      get :new, :content_type=>'mods_asset'
-      response.should redirect_to edit_catalog_path(@asset, :new_asset=>true)
-      flash[:notice].should == "Created a Mods Asset with pid #{@asset.pid}. Now it's ready to be edited."
-    end
-  end
-  
-  describe "update" do
-    it "should update the object with the attributes provided" do
-      mock_document = ModsAsset.new
-      mock_document.stubs(:pid => '_PID_')
-      mock_document.stubs(:update_from_computing_id).returns(nil)
-      controller.expects(:check_embargo_date_format).returns(nil)
-
-      ModsAsset.expects(:find).with("_PID_").returns(mock_document)
-      
-      simple_request_params = {"asset"=>{
-          "descMetadata"=>{
-            "subject"=>{"0"=>"subject1", "1"=>"subject2", "2"=>"subject3"}
-          }
-        }
-      }
-      
-      mock_document.expects(:update_datastream_attributes).with("descMetadata"=>{"subject"=>{"0"=>"subject1", "1"=>"subject2", "2"=>"subject3"}}).returns({"subject"=>{"2"=>"My Topic"}})
-      mock_document.expects(:save)
-      controller.stubs(:display_release_status_notice)
-      put :update, {:id=>"_PID_"}.merge(simple_request_params)
-
-      response.should redirect_to catalog_path(mock_document.pid, :viewing_context=>"browse")
-
-    end
-    
-    it "should support updating OM::XML datastreams" do
-      mock_document = ModsAsset.new
-      mock_document.stubs(:pid => '_PID_')
-      mock_document.stubs(:update_from_computing_id).returns(nil)
-      ModsAsset.expects(:find).with("_PID_").returns(mock_document)
-      
-      update_method_args = [ "descMetadata" => { [{:person=>0}, :role] => {"0"=>"role1","1"=>"role2","2"=>"role3"} } ]
-      mock_document.expects(:update_datastream_attributes).with( *update_method_args ).returns({"person_0_role"=>{"0"=>"role1","1"=>"role2","2"=>"role3"}})
-      mock_document.expects(:save)
-      
-      
-      nokogiri_request_params = {
-        "id"=>"_PID_", 
-        "content_type"=>"mods_asset",
-        "field_selectors"=>{
-          "descMetadata"=>{
-            "person_0_role"=>[{":person"=>"0"}, "role"]
-          }
-        }, 
-        "asset"=>{
-          "descMetadata"=>{
-            "person_0_role"=>{"0"=>"role1", "1"=>"role2", "2"=>"role3"}
-          }
-        }
-      }
-      controller.stubs(:display_release_status_notice)
-      put :update, nokogiri_request_params
-      # put :update, :id=>"_PID_", "content_type"=>"mods_asset", "datastream"=>"descMetadata", "field_name"=>"person_0_last_name","parent_select"=>[{":person"=>"0"}, ":last_name"], "child_index"=>"0", "value"=>"Sample New Value"
-      response.should redirect_to edit_catalog_path(mock_document.pid, :wf_step=>"publication")
-    end
-    
-    it "should handle complete updates of many fields in many datastreams" do
-      pending("This is failing intermittently. See https://jira.duraspace.org/browse/HYDRUS-166 for more info")
-      request_params = {"id"=>"hydrangea:fixture_mods_article3", "content_type"=>"mods_asset", "action"=>"update", "_method"=>"put"}
-      request_params["field_selectors"] = {"descMetadata"=>{"person_0_computing_id"=>[{"person"=>"0"}, "computing_id"], "journal_0_issue_start_page"=>[{"journal"=>"0"}, "issue", "start_page"], "person_1_description"=>[{"person"=>"1"}, "description"], "person_1_institution"=>[{"person"=>"1"}, "institution"], "journal_0_origin_info_publisher"=>[{"journal"=>"0"}, "origin_info", "publisher"], "abstract"=>["abstract"], "person_0_last_name"=>[{"person"=>"0"}, "last_name"], "person_0_description"=>[{"person"=>"0"}, "description"], "journal_0_issue_volume"=>[{"journal"=>"0"}, "issue", "volume"], "title_info_main_title"=>["title_info", "main_title"], "location_url"=>["location", "url"], "note"=>["note"], "person_1_last_name"=>[{"person"=>"1"}, "last_name"], "subject_topic"=>["subject", "topic"], "person_0_institution"=>[{"person"=>"0"}, "institution"], "person_1_first_name"=>[{"person"=>"1"}, "first_name"], "person_1"=>[{"person"=>"1"}], "journal_0_title_info_main_title"=>[{"journal"=>"0"}, "title_info", "main_title"], "journal_0_issue_level"=>[{"journal"=>"0"}, "issue", "level"], "journal_0_issue_end_page"=>[{"journal"=>"0"}, "issue", "end_page"], "peer_reviewed"=>["peer_reviewed"], "person_0_first_name"=>[{"person"=>"0"}, "first_name"], "person_1_computing_id"=>[{"person"=>"1"}, "computing_id"], "journal_0_issn"=>[{"journal"=>"0"}, "issn"], "journal_0_issue_publication_date"=>[{"journal"=>"0"}, "issue", "publication_date"]}, "rightsMetadata"=>{"embargo_embargo_release_date"=>["embargo", "embargo_release_date"]}, "properties"=>{"release_to"=>["release_to"]}} 
-      request_params["asset"] = {"descMetadata"=>{"person_0_computing_id"=>{"0"=>""}, "journal_0_issue_start_page"=>{"0"=>"195"}, "person_1_description"=>{"0"=>""}, "person_1_institution"=>{"0"=>"Baltimore"}, "journal_0_origin_info_publisher"=>{"0"=>"PUBLISHER"}, "abstract"=>{"0"=>"ABSTRACT"}, "person_0_last_name"=>{"0"=>"Smith"}, "person_0_description"=>{"0"=>""}, "journal_0_issue_volume"=>{"0"=>"2               "}, "title_info_main_title"=>{"0"=>"Test Article"}, "location_url"=>{"0"=>"http://example.com/foo"}, "note"=>{"0"=>""}, "person_1_last_name"=>{"0"=>"Lacks"}, "subject_topic"=>{"0"=>"TOPIC 1", "1"=>"TOPIC 2", "2"=>"CONTROLLED TERM"}, "person_0_institution"=>{"0"=>"FACULTY, UNIVERSITY"}, "person_1_first_name"=>{"0"=>"Henrietta"}, "journal_0_title_info_main_title"=>{"0"=>"The Journal of Mock Object"}, "journal_0_issue_level"=>{"0"=>""}, "journal_0_issue_end_page"=>{"0"=>"230"}, "person_0_first_name"=>{"0"=>"John"}, "person_1_computing_id"=>{"0"=>""}, "journal_0_issn"=>{"0"=>"1234-5678"}, "journal_0_issue_publication_date"=>{"0"=>"FEB. 2007"}}, "rightsMetadata"=>{"embargo_embargo_release_date"=>{"0"=>""}}, "properties"=>{"released"=>{"0"=>"true"}, "release_to"=>{"0"=>"public"}}}
-      expected_response = {"descMetadata"=>{"journal_0_issue_start_page"=>{"0"=>"195"}, "person_0_computing_id"=>{"-1"=>""}, "abstract"=>{"0"=>"ABSTRACT"}, "journal_0_origin_info_publisher"=>{"0"=>"PUBLISHER"}, "person_1_description"=>{"-1"=>""}, "person_1_institution"=>{"0"=>"Baltimore"}, "journal_0_issue_volume"=>{"0"=>"2               "}, "person_0_description"=>{"-1"=>""}, "person_0_last_name"=>{"0"=>"Smith"}, "title_info_main_title"=>{"0"=>"Test Article"}, "note"=>{"-1"=>""}, "location_url"=>{"0"=>"http://example.com/foo"}, "person_1_last_name"=>{"0"=>"Lacks"}, "subject_topic"=>{"0"=>"TOPIC 1", "1"=>"TOPIC 2", "2"=>"CONTROLLED TERM"}, "journal_0_issue_end_page"=>{"0"=>"230"}, "journal_0_title_info_main_title"=>{"0"=>"The Journal of Mock Object"}, "journal_0_issue_level"=>{"-1"=>""}, "person_1_first_name"=>{"0"=>"Henrietta"}, "person_0_institution"=>{"0"=>"FACULTY, UNIVERSITY"}, "journal_0_issn"=>{"0"=>"1234-5678"}, "person_0_first_name"=>{"0"=>"John"}, "person_1_computing_id"=>{"-1"=>""}, "journal_0_issue_publication_date"=>{"0"=>"FEB. 2007"}}, "rightsMetadata"=>{"embargo_embargo_release_date"=>{"-1"=>""}}, "properties"=>{:released=>{"0"=>"true"}, [:release_to]=>{}, :release_to=>{"0"=>"public"}}}
-      
-      post :update, request_params
-      expected_response.each_pair do |datastream_name, fields|
-        fields.each_pair do |field_pointer, value|
-          assigns[:response][datastream_name][field_pointer].should == value
-        end
-      end  
-    end
-  end
-  
-  describe "destroy" do
-    it "should delete the asset identified by pid" do
-      mock_document = mock("asset", :delete)
-      mock_document.expects(:destroy_child_assets).returns([])
-      ActiveFedora::Base.expects(:find).with("__PID__", :cast=>true).returns(mock_document)
-      # stub out authorize!
-      controller.expects(:authorize!).with(:destroy, mock_document)      
-      delete(:destroy, :id => "__PID__")
-      response.should redirect_to catalog_index_path
-    end
-  end
-  
-  # withdraw is a conditional destroy, with the conditions dependant on the project requirements.
-  # Currently, the widthdraw method is an alias for destroy, should behave as such
-  describe "withdraw" do
-    it "should withdraw the asset identified by pid" do
-      mock_document = mock("asset", :delete)
-      mock_document.stubs(:pid => '_PID_')
-      mock_document.expects(:destroy_child_assets).returns([])
-      ActiveFedora::Base.expects(:find).with("_PID_", :cast => true).returns(mock_document)
-      # stub out authorize!
-      controller.expects(:authorize!).with(:destroy, mock_document)
-      delete :withdraw, :id => "_PID_"
-      response.should redirect_to catalog_index_path
-    end
-    it "should restrict withdrawing to authorized users" do
-      mock_obj = mock("asset")
-      ActiveFedora::Base.expects(:find).with("_PID_", :cast=>true).returns(mock_obj)
-      lambda{get :withdraw, :id => "_PID_"}.should raise_error(CanCan::AccessDenied)
-    end
-  end
-  
-  
-   
-end

data/test_support/spec/controllers/hydra_controller_spec.rb

@@ -1,43 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-class HydraControllerTest < ActionController::Base
-end
-describe HydraControllerTest do
-  
-  after :all do
-    Object.send(:remove_const, :HydraControllerTest)
-  end
-  
-  it "should add the necessary helpers to classes that include it" do
-    HydraControllerTest.expects(:helper).with(:hydra)
-    HydraControllerTest.expects(:helper).with(:hydra_assets)
-    HydraControllerTest.stubs(:before_filter)
-    HydraControllerTest.send(:include, Hydra::Controller::ControllerBehavior)
-  end
-  
-  describe "load_fedora_document" do
-    before :all do
-      @behavior = Hydra::Controller::ControllerBehavior.deprecation_behavior
-      Hydra::Controller::ControllerBehavior.deprecation_behavior = :silence
-    end
-
-    after :all do
-      Hydra::Controller::ControllerBehavior.deprecation_behavior = @behavior
-    end
-
-    before do
-      HydraControllerTest.send(:include, Hydra::Controller::ControllerBehavior)
-    end
-
-
-    it "should load @document_fedora and @file_assets" do
-      controller.params[:id] = 'hydrangea:fixture_mods_article1'
-      controller.load_fedora_document
-      
-      assigns[:document_fedora].class.should == ModsAsset 
-      assigns[:document_fedora].pid.should == 'hydrangea:fixture_mods_article1' 
-      assigns[:file_assets].size.should == 1
-      assigns[:file_assets].first["id"].should == "hydrangea:fixture_uploaded_svg1"
-    end
-  end
-end

data/test_support/spec/controllers/hydra_ui_controller_spec.rb

@@ -1,21 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-
-describe Hydra::Controller do
-  before(:all) do
-    class HydraUiControllerTest
-    end
-  end
-  after :all do
-    Object.send(:remove_const, :HydraUiControllerTest)
-  end
-
-  it "should add the necessary helpers to classes that include it" do
-    HydraUiControllerTest.expects(:helper).with(:hydra_fedora_metadata)
-    HydraUiControllerTest.expects(:helper).with(:generic_content_objects)
-    HydraUiControllerTest.expects(:helper).with(:hydra_uploader)
-    HydraUiControllerTest.expects(:helper).with(:article_metadata)
-    HydraUiControllerTest.stubs(:before_filter)
-    HydraUiControllerTest.send(:include, Hydra::UI::Controller)
-  end
-end

data/test_support/spec/controllers/permissions_controller_spec.rb

@@ -1,39 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-describe Hydra::PermissionsController do
-  before :all do
-    @behavior = Hydra::PermissionsController.deprecation_behavior
-    Hydra::PermissionsController.deprecation_behavior = :silence
-  end
-
-  after :all do
-    Hydra::PermissionsController.deprecation_behavior = @behavior 
-  end
-  describe "create" do
-    it "should create a new permissions entry" do
-      # stub out permissions check
-      controller.expects(:enforce_access_controls).returns(true)
-      @asset = ModsAsset.create
-      post :create, :asset_id=>@asset.pid, :permission => {"actor_id"=>"_person_id_","actor_type"=>"person","access_level"=>"read"}      
-      ModsAsset.find(@asset.pid).rightsMetadata.individuals.should == {"_person_id_" => "read"}
-    end
-  end
-  describe "update" do
-    it "should call Hydra::RightsMetadata properties setter" do
-      # stub out permissions check
-      controller.expects(:enforce_access_controls).returns(true)
-      @asset = ModsAsset.new
-      @asset.rightsMetadata.permissions({:group=>"students"})
-      @asset.save
-      post :update, :asset_id=>@asset.pid, :permission => {"group"=>{"_group_id_"=>"discover"}}
-      ModsAsset.find(@asset.pid).rightsMetadata.groups.should == {"_group_id_" => "discover"}
-    end
-    it "should restrict permissions setting to authenticated users" do
-      ActiveFedora::Base.expects(:find).never
-      post :update, :id => "hydrangea:fixture_mods_dataset1"
-      flash[:alert].should == "You do not have sufficient privileges to edit this document. You have been redirected to the read-only view."
-      flash[:notice].should be_nil
-      response.should be_redirect
-    end
-  end
-end

data/test_support/spec/controllers/user_sessions_controller_spec.rb

@@ -1,35 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-describe UserSessionsController do
-
-  before(:each) do
-    @user = mock("User")
-    @user.stubs(:can_be_superuser?).returns true
-    @user2 = mock("User")
-    @user2.stubs(:can_be_superuser?).returns false
-  end
-  
-  it "should allow for toggling on and off session[:superuser_mode]" do
-    controller.stubs(:current_user).returns(@user)
-    request.env["HTTP_REFERER"] = ""
-    get :superuser
-    session[:superuser_mode].should be_true
-    get :superuser
-    session[:superuser_mode].should be_nil
-  end
-  
-  it "should not allow superuser_mode to be set in session if current_user is not a superuser" do
-    controller.stubs(:current_user).returns(@user2)
-    request.env["HTTP_REFERER"] = ""
-    get :superuser
-    session[:superuser_mode].should be_nil
-  end
-  
-  it "should redirect to the referer" do
-    controller.stubs(:current_user).returns(@user)
-    request.env["HTTP_REFERER"] = hydra_file_assets_path
-    get :superuser
-    response.should redirect_to(hydra_file_assets_path)
-  end
-  
-end

data/test_support/spec/factories.rb

@@ -1,11 +0,0 @@
-FactoryGirl.define do
-  sequence :email do |n|
-    "person#{n}@example.com"
-  end
-
-  factory :user do
-    email
-    password 'password'
-  end
-
-end

data/test_support/spec/helpers/access_controls_enforcement_spec.rb

@@ -1,119 +0,0 @@
-# Need way to find way to stub current_user and RoleMapper in order to run these tests
-require File.expand_path( File.join( File.dirname(__FILE__),'..','spec_helper') )
-
-describe Hydra::AccessControlsEnforcement do
-  describe "enforce_access_controls" do
-    describe "when the method exists" do
-      it "should call the method" do
-        params[:action] = :index
-        helper.enforce_access_controls.should be_true
-      end
-    end
-    describe "when the method doesn't exist" do
-      it "should not call the method, but should return true" do
-        params[:action] = :facet
-        helper.enforce_access_controls.should be_true
-      end
-    end
-  end
-  describe "enforce_show_permissions" do
-    it "should allow a user w/ edit permissions to view an embargoed object" do
-      user = User.new :email=>'testuser@example.com'
-      user.stubs(:is_being_superuser?).returns false
-      RoleMapper.stubs(:roles).with(user.email).returns(["archivist"])
-      helper.stubs(:current_user).returns(user)
-      helper.stubs(:load_permissions_from_solr).returns("")
-      helper.stubs(:can?).with(:edit, nil).returns(true)
-      helper.stubs(:can?).with(:read, nil).returns(true)
-      @permissions_solr_document = SolrDocument.new({"edit_access_person_t"=>["testuser@example.com"], "embargo_release_date_dt"=>(Date.parse(Time.now.to_s)+2).to_s})
-      helper.send(:enforce_show_permissions, {})
-      flash[:alert].should be_nil
-    end
-    it "should prevent a user w/o edit permissions from viewing an embargoed object" do
-      user = User.new :email=>'testuser@example.com'
-      user.stubs(:is_being_superuser?).returns false
-      RoleMapper.stubs(:roles).with(user.email).returns([])
-      helper.stubs(:current_user).returns(user)
-      helper.stubs(:load_permissions_from_solr).returns("")
-      helper.stubs(:can?).with(:edit, nil).returns(false)
-      helper.stubs(:can?).with(:read, nil).returns(true)
-      helper.stubs(:redirect_to)
-      @permissions_solr_document = SolrDocument.new({"edit_access_person_t"=>["testuser@example.com"], "embargo_release_date_dt"=>(Date.parse(Time.now.to_s)+2).to_s})
-      lambda {helper.send(:enforce_show_permissions, {})}.should raise_error Hydra::AccessDenied, "This item is under embargo.  You do not have sufficient access privileges to read this document."
-    end
-  end
-  describe "apply_gated_discovery" do
-    before(:each) do
-      @stub_user = User.new :email=>'archivist1@example.com'
-      @stub_user.stubs(:is_being_superuser?).returns false
-      RoleMapper.stubs(:roles).with(@stub_user.email).returns(["archivist","researcher"])
-      helper.stubs(:current_user).returns(@stub_user)
-      @solr_parameters = {}
-      @user_parameters = {}
-    end
-    it "should set query fields for the user id checking against the discover, access, read fields" do
-      helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-      ["discover","edit","read"].each do |type|
-        @solr_parameters[:fq].first.should match(/#{type}_access_person_t\:#{@stub_user.email}/)      
-      end
-    end
-    it "should set query fields for all roles the user is a member of checking against the discover, access, read fields" do
-      helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-      ["discover","edit","read"].each do |type|
-        @solr_parameters[:fq].first.should match(/#{type}_access_group_t\:archivist/)        
-        @solr_parameters[:fq].first.should match(/#{type}_access_group_t\:researcher/)        
-      end
-    end
-    # it "should filter out any content whose embargo date is in the future" do
-    #   helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-    #   @solr_parameters[:fq].should include("-embargo_release_date_dt:[NOW TO *]")
-    # end
-    it "should allow content owners access to their embargoed content" do
-      pending
-      helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-      @solr_parameters[:fq].should include("(NOT embargo_release_date_dt:[NOW TO *]) OR depositor_t:#{@stub_user.email}")
-      
-      # @solr_parameters[:fq].should include("embargo_release_date_dt:[NOW TO *] AND  depositor_t:#{current_user.email}) AND NOT (NOT depositor_t:#{current_user.email} AND embargo_release_date_dt:[NOW TO *]")
-    end
-    
-    describe "for superusers" do
-      it "should return superuser access level" do
-        stub_user = User.new(:email=>'suzie@example.com')
-        stub_user.stubs(:is_being_superuser?).returns true
-        RoleMapper.stubs(:roles).with(stub_user.email).returns(["archivist","researcher"])
-        helper.stubs(:current_user).returns(stub_user)
-        helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-        ["discover","edit","read"].each do |type|    
-          @solr_parameters[:fq].first.should match(/#{type}_access_person_t\:\[\* TO \*\]/)          
-        end
-      end
-      it "should not return superuser access to non-superusers" do
-        stub_user = User.new(:email=>'suzie@example.com')
-        stub_user.stubs(:is_being_superuser?).returns false
-        RoleMapper.stubs(:roles).with(stub_user.email).returns(["archivist","researcher"])
-        helper.stubs(:current_user).returns(stub_user)
-        helper.send(:apply_gated_discovery, @solr_parameters, @user_parameters)
-        ["discover","edit","read"].each do |type|
-          @solr_parameters[:fq].should_not include("#{type}_access_person_t\:\[\* TO \*\]")              
-        end
-      end
-    end
-
-  end
-  
-  describe "exclude_unwanted_models" do
-    before(:each) do
-      stub_user = User.new :email=>'archivist1@example.com'
-      stub_user.stubs(:is_being_superuser?).returns false
-      helper.stubs(:current_user).returns(stub_user)
-      @solr_parameters = {}
-      @user_parameters = {}
-    end
-    it "should set solr query parameters to filter out FileAssets" do
-      helper.send(:exclude_unwanted_models, @solr_parameters, @user_parameters)
-      @solr_parameters[:fq].should include("-has_model_s:\"info:fedora/afmodel:FileAsset\"")  
-    end
-  end
-end
-
-

data/test_support/spec/helpers/access_controls_evaluation_spec.rb

@@ -1,26 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-describe Hydra::AccessControlsEvaluation do
-  
-  describe "editor?" do
-    it "should return true if current_user.is_being_superuser? is true" do
-      mock_user = FactoryGirl.build(:user, :email => "BigWig@example.com")
-      mock_user.stubs(:is_being_superuser?).returns true
-      controller.stubs(:current_user).returns mock_user
-      helper.editor?.should be_true
-    end
-    it "should return false if the session[:user] is not logged in" do
-      controller.stubs(:current_user).returns(nil)
-      helper.editor?.should be_false
-    end    
-    it "should return false if the session[:user] does not have an editor role" do
-      mock_user = mock("User")
-      mock_user.stubs(:email).returns "nobody_special@example.com"
-      mock_user.stubs(:is_being_superuser?).returns(false)
-      mock_user.stubs(:new_record?).returns(false)
-      controller.stubs(:current_user).returns(mock_user)
-      helper.editor?.should be_false
-    end
-  end  
-  
-end