hydra-head 3.0.0pre1

data/test_support/spec/controllers/assets_controller_spec.rb

@@ -0,0 +1,113 @@
+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 AssetsController do
+  
+  before do
+    request.env['WEBAUTH_USER']='bob'
+  end
+  
+  it "should use DocumentController" do
+    controller.should be_an_instance_of(AssetsController)
+  end
+  
+  describe "update" do
+    it "should update the object with the attributes provided" do
+      mock_document = mock("document")
+      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)
+    end
+    
+    it "should support updating OM::XML datastreams" do
+      mock_document = mock("document")
+      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"
+    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_obj = mock("asset", :delete)
+      mock_obj.expects(:destroy_child_assets).returns([])
+      ActiveFedora::Base.expects(:load_instance_from_solr).with("__PID__").returns(mock_obj)
+      ActiveFedora::ContentModel.expects(:known_models_for).with(mock_obj).returns([ModsAsset])
+      ModsAsset.expects(:load_instance_from_solr).with("__PID__").returns(mock_obj)
+      delete(:destroy, :id => "__PID__")
+    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_obj = mock("asset", :delete)
+      mock_obj.expects(:destroy_child_assets).returns([])
+      ActiveFedora::Base.expects(:load_instance_from_solr).with("__PID__").returns(mock_obj)
+      ActiveFedora::ContentModel.expects(:known_models_for).with(mock_obj).returns([ModsAsset])
+      ModsAsset.expects(:load_instance_from_solr).with("__PID__").returns(mock_obj)
+      delete(:withdraw, :id => "__PID__")
+    end
+  end
+  
+  
+   
+end

data/test_support/spec/controllers/catalog_controller_spec.rb

@@ -0,0 +1,148 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require 'mocha'
+
+
+# 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 CatalogController do
+  
+  before do
+    #controller.stubs(:protect_from_forgery).returns("meh")
+    session[:user]='bob'
+  end
+  
+  it "should use CatalogController" do
+    controller.should be_an_instance_of(CatalogController)
+  end
+  
+  
+  describe "Paths Generated by Custom Routes:" do
+    # paths generated by custom routes
+    it "should map {:controller=>'catalog', :action=>'index'} to GET /catalog" do
+      { :get => "/catalog" }.should route_to(:controller => 'catalog', :action => 'index')
+    end
+    it "should map {:controller=>'catalog', :action=>'show', :id=>'test:3'} to GET /catalog/test:3" do
+      { :get => "/catalog/test:3" }.should route_to(:controller => 'catalog', :action => 'show', :id=>'test:3')
+    end
+    it "should map {:controller=>'catalog', :action=>'edit', :id=>'test:3'} to GET /catalog/test:3" do
+      { :get => "/catalog/test:3/edit" }.should route_to(:controller => 'catalog', :action => 'edit', :id=>'test:3')
+    end
+
+    it "should map catalog_path" do
+      # catalog_path.should == '/catalog'
+      catalog_path("test:3").should == '/catalog/test:3'
+    end
+  end
+  
+  it "should not choke on objects with periods in ids (ie Fedora system objects)" do    
+     pending "Need to override blacklight routes"	
+
+	## We could do something like this to remove the catalog/show route and replace it with a route that allows dots (e.g. resources :catalog, :id=> /.+/)
+	  # def add_route
+	  #   new_route = ActionController::Routing::Routes.builder.build(name, route_options)
+	  #   ActionController::Routing::Routes.routes.insert(0, new_route)
+	  # end
+
+	  # def remove_route
+	  #   ActionController::Routing::Routes.routes.reject! { |r| r.instance_variable_get(:@requirements)[:slug_id] == id }
+	  # end
+
+    catalog_path("fedora-system:FedoraObject-3.0").should == '/catalog/fedora-system:FedoraObject-3.0'
+    { :get => "/catalog/fedora-system:FedoraObject-3.0" }.should route_to(:controller => 'catalog', :action => 'show', :id=>'fedora-system:FedoraObject-3.0')
+  end
+  
+  describe "index" do
+    
+    describe "access controls" do
+      before(:all) do
+        @public_only_results = Blacklight.solr.find Hash[:phrases=>{:access_t=>"public"}]
+        @private_only_results = Blacklight.solr.find Hash[:phrases=>{:access_t=>"private"}]
+      end
+
+      it "should only return public documents if role does not have permissions" do
+        controller.stubs(:current_user).returns(nil)
+        get :index
+        assigns("response").docs.count.should == @public_only_results.docs.count
+      end
+      it "should return all documents if role does have permissions" do
+	User.any_instance.stubs(:login).returns("BigWig")
+        mock_user =  User.new
+        session[:superuser_mode] = true
+        controller.stubs(:current_user).returns(mock_user)
+        get :index
+        assigns["response"].docs.count.should > @public_only_results.docs.count
+      end
+    end
+  end
+
+
+  describe "load_fedora_document" do
+
+    it "should load @document_fedora and @file_assets" do
+      controller.stubs(:params).returns({:id=>"foo:id"})
+      stub_base_object = stub("Base Object")
+      ActiveFedora::ContentModel.expects(:known_models_for).with( stub_base_object ).returns( [ModsAsset] )
+      stub_mods_asset = stub("MODS Asset")
+      stub_mods_asset.expects(:file_objects).with(:response_format=>:solr).returns("file assets response")
+      
+      # Note: Had to stub Fedora::Repository.instance.find_model rather than stubbing ActiveFedora::Base.load_instance and ModsAsset.load_instance because 
+      # Mocha was not unstubbing the ModsAsset class
+      #
+      # ActiveFedora::Base.stubs(:load_instance).with("foo:id").returns( stub_base_object )
+      # ModsAsset.stubs(:load_instance).with("foo:id").returns( stub_mods_asset )
+      Fedora::Repository.instance.expects(:find_model).with("foo:id", ActiveFedora::Base).returns( stub_base_object )
+      Fedora::Repository.instance.expects(:find_model).with("foo:id", ModsAsset).returns( stub_mods_asset )
+      
+      controller.load_fedora_document
+      
+      assigns[:document_fedora].should == stub_mods_asset
+      assigns[:file_assets].should == "file assets response"
+    end
+  end
+  
+  describe "edit" do
+
+    it "should trigger show action" do
+      controller.expects(:show)
+      controller.stubs(:enforce_access_controls)
+      controller.stubs(:load_fedora_document)
+      get :edit, :id=>'hydrangea:fixture_mods_article1'
+    end
+    it "should render show template (which then delegates to edit partials)" do
+      controller.stubs(:enforce_access_controls)
+      controller.stubs(:load_fedora_document)
+      get :edit, :id=>'hydrangea:fixture_mods_article1'
+      response.should render_template("show")
+    end
+  end
+  
+  describe "filters" do
+    describe "index" do
+      it "should trigger enforce_index_permissions" do
+        controller.expects(:add_access_controls_to_solr_params)
+        controller.expects(:enforce_index_permissions)
+        get :index
+      end
+    end
+    describe "show" do
+      it "should trigger enforce_show_permissions and load_fedora_document" do
+        controller.expects(:load_fedora_document)
+        controller.expects(:enforce_show_permissions)
+        get :show, :id=>'test:3'
+      end
+    end
+    describe "edit" do
+      it "should trigger enforce_edit_permissions and load_fedora_document" do
+        controller.expects(:load_fedora_document)
+        controller.expects(:enforce_edit_permissions)
+        get :edit, :id=>'test:3'
+      end
+    end
+  end
+  
+end

data/test_support/spec/controllers/catalog_controller_viewing_context_spec.rb

@@ -0,0 +1,62 @@
+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 CatalogController do
+  
+  before do
+    controller.stubs(:load_fedora_document)
+  end
+
+  describe "show" do
+    it "should redirect to edit view if session is in edit context and user has edit permission" do
+      controller.stubs(:reader?).returns(true)            
+      controller.expects(:editor?).returns(true)
+      
+      controller.session[:viewing_context] = "edit"
+      get(:show, {:id=>"hydrangea:fixture_mods_article1"})
+      response.should redirect_to(:action => 'edit')
+    end
+    it "should allow you to reset the session context to browse using :viewing_context param" do
+      controller.stubs(:reader?).returns(true)            
+      controller.session[:viewing_context] = "edit"
+      get(:show, :id=>"hydrangea:fixture_mods_article1", :viewing_context=>"browse")
+      session[:viewing_context].should == "browse"
+      response.should_not redirect_to(:action => 'edit')
+    end
+    
+    it "should quietly switch session state to browse if user does not have edit permissions" do
+      controller.expects(:reader?).returns(true)
+      controller.expects(:editor?).returns(false)
+      controller.session[:viewing_context] = "edit"
+      get(:show, {:id=>"hydrangea:fixture_mods_article1"})
+      session[:viewing_context].should == "browse"
+      response.should_not redirect_to(:action => 'edit')
+    end
+  end
+  
+  describe "edit" do
+    it "should enforce edit permissions, redirecting to show action and resetting session context if user does not have edit permissions" do
+      mock_user = mock("User")
+      mock_user.stubs(:login).returns("patron1")
+      mock_user.stubs(:is_being_superuser?).returns(false)
+      controller.stubs(:current_user).returns(mock_user)
+      
+      get :edit, :id=>"hydrangea:fixture_mods_article1"
+      response.should redirect_to(:action => 'show')
+      flash[:notice].should == "You do not have sufficient privileges to edit this document. You have been redirected to the read-only view."
+    end
+    it "should render normally if user has edit permissions" do
+      controller.expects(:editor?).returns(true)
+      get :edit, :id=>"hydrangea:fixture_mods_article1"
+      response.should_not redirect_to(:action => 'show')
+    end
+  end
+  
+end

data/test_support/spec/controllers/contributors_controller_spec.rb

@@ -0,0 +1,47 @@
+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 ContributorsController do
+  
+  describe "create" do
+    it "should support adding new person / contributor / organization nodes" do
+      mock_document = mock("document")
+      ["person","conference","organization"].each do |type|
+        mock_document.expects(:insert_contributor).with(type).returns(["foo node",989])
+        mock_document.expects(:save)
+        ModsAsset.expects(:find).with("_PID_").returns(mock_document)
+        post :create, :asset_id=>"_PID_", :controller => "contributors", :content_type => "mods_asset", :contributor_type=>type
+        response.should redirect_to "http://test.host/catalog/_PID_/edit##{type}_989"
+      end
+    end
+    it "should return inline html if format is inline" do
+      mock_document = mock("document")
+      ["person","conference","organization"].each do |type|
+        mock_document.expects(:insert_contributor).with(type).returns(["foo node","foo index"])
+        mock_document.expects(:save)
+        ModsAsset.expects(:find).with("_PID_").returns(mock_document)
+        post :create, :asset_id=>"_PID_", :controller => "contributors", :content_type => "mods_asset", :contributor_type=>type, :format=>"inline"
+        response.should render_template "contributors/_edit_#{type}"
+      end
+    end
+  end
+  
+  describe "destroy" do
+    it "should delete the contributor corresponding to contributor_type and index" do
+      mock_dataset = mock("Dataset")
+      mock_dataset.expects(:remove_contributor).with("conference", "3")
+      mock_dataset.expects(:save)
+      ModsAsset.expects(:find).with("_PID_").returns(mock_dataset)
+      
+      delete :destroy, :asset_id=>"_PID_", :content_type => "mods_asset", :contributor_type=>"conference", :index=>"3"
+    end
+  end
+  
+end

data/test_support/spec/controllers/file_assets_controller_spec.rb

@@ -0,0 +1,189 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe FileAssetsController do
+  include Devise::TestHelpers
+
+  before do
+    session[:user]='bob'
+  end
+
+  ## Plugin Tests
+  it "should use FileAssetsController" do
+    controller.should be_an_instance_of(FileAssetsController)
+  end
+  it "should be restful" do
+    { :get => "/file_assets" }.should route_to(:controller=>'file_assets', :action=>'index')
+    { :get => "/file_assets/3" }.should route_to(:controller=>'file_assets', :action=>'show', :id=>"3")
+    { :delete=> "/file_assets/3" }.should route_to(:controller=>'file_assets', :action=>'destroy', :id=>"3")
+    { :put=>"/file_assets/3" }.should route_to(:controller=>'file_assets', :action=>'update', :id=>"3")
+    { :get => "/file_assets/3/edit" }.should route_to(:controller=>'file_assets', :action=>'edit', :id=>"3")
+    { :get =>"/file_assets/new" }.should route_to(:controller=>'file_assets', :action=>'new')
+    { :post => "/file_assets" }.should route_to(:controller=>'file_assets', :action=>'create')
+    
+  end
+  
+  describe "index" do
+    
+    it "should find all file assets in the repo if no container_id is provided" do
+      #FileAsset.expects(:find_by_solr).with(:all, {}).returns("solr result")
+      # Solr::Connection.any_instance.expects(:query).with('conforms_to_field:info\:fedora/afmodel\:FileAsset', {}).returns("solr result")
+      Solr::Connection.any_instance.expects(:query).with('active_fedora_model_s:FileAsset', {}).returns("solr result")
+      controller.stubs(:load_permissions_from_solr)
+      ActiveFedora::Base.expects(:new).never
+      xhr :get, :index
+      assigns[:solr_result].should == "solr result"
+    end
+    it "should find all file assets belonging to a given container object if container_id or container_id is provided" do
+      mock_container = mock("container")
+      mock_container.expects(:file_objects).with(:response_format => :solr).returns("solr result")
+      controller.expects(:get_search_results).with(:q=>'is_part_of_s:info\:fedora/_PID_').returns(["assets solr response","assets solr list"])
+      controller.expects(:get_solr_response_for_doc_id).with('_PID_').returns(["container solr response","container solr doc"])
+      controller.stubs(:load_permissions_from_solr)
+      
+      ActiveFedora::Base.expects(:load_instance).with("_PID_").returns(mock_container)
+      xhr :get, :index, :asset_id=>"_PID_"
+      assigns[:response].should == "assets solr response"
+      assigns[:document_list].should == "assets solr list"
+      
+      assigns[:container_response].should == "container solr response"
+      assigns[:document].should == "container solr doc"
+      assigns[:solr_result].should == "solr result"
+      assigns[:container].should == mock_container
+    end
+    
+    it "should find all file assets belonging to a given container object if container_id or container_id is provided" do
+      pending
+      # this was testing a hacked version
+      mock_solr_hash = {"has_collection_member_field"=>["info:fedora/foo:id"]}
+      mock_container = mock("container")
+      mock_container.expects(:collection_members).with(:response_format=>:solr).returns("solr result")
+      ActiveFedora::Base.expects(:load_instance).with("_PID_").returns(mock_container)
+      xhr :get, :index, :asset_id=>"_PID_"
+      assigns[:solr_result].should == "solr result"
+      assigns[:container].should == mock_container
+    end
+  end
+
+  describe "index" do
+    before(:each) do
+      Fedora::Repository.stubs(:instance).returns(stub_everything)
+    end
+
+    it "should be refined further!"
+    
+  end
+  describe "new" do
+    it "should return the file uploader view"
+    it "should set :container_id to value of :container_id if available" do
+      xhr :get, :new, :asset_id=>"_PID_"
+      @controller.params[:asset_id].should == "_PID_"
+    end
+  end
+
+  describe "show" do
+    it "should redirect to index view if current_user does not have read or edit permissions" do
+      mock_user = mock("User")
+      mock_user.stubs(:login).returns("fake_user")
+      mock_user.stubs(:is_being_superuser?).returns(false)
+      controller.stubs(:current_user).returns(mock_user)
+      get(:show, :id=>"hydrangea:fixture_file_asset1")
+      response.should redirect_to(:action => 'index')
+    end
+    it "should redirect to index view if the file does not exist" do
+      get(:show, :id=>"example:invalid_object")
+      response.should redirect_to(:action => 'index')
+    end
+  end
+  
+  describe "create" do
+    it "should create and save a file asset from the given params" do
+      mock_fa = mock("FileAsset")
+      mock_fa.stubs(:pid).returns("foo:pid")
+      controller.expects(:create_and_save_file_asset_from_params).returns(mock_fa)
+      xhr :post, :create, :Filedata=>mock("File"), :Filename=>"Foo File"
+    end
+    it "if container_id is provided, should associate the created file asset wtih the container" do
+      stub_fa = stub("FileAsset", :save)
+      stub_fa.stubs(:pid).returns("foo:pid")
+      controller.expects(:create_and_save_file_asset_from_params).returns(stub_fa)
+      controller.expects(:associate_file_asset_with_container)      
+      xhr :post, :create, :Filedata=>stub("File"), :Filename=>"Foo File", :asset_id=>"_PID_"
+    end
+    it "should redirect back to container edit view if no Filedata is provided but container_id is provided" do
+      xhr :post, :create, :asset_id=>"_PID_"
+      response.should redirect_to(:controller=>"catalog", :id=>"_PID_", :action => 'edit')
+      # flash[:notice].should == "You must specify a file to upload."
+    end
+    it "should display a message that you need to select a file to upload if no Filedata is provided" do
+      xhr :post, :create
+      # flash[:notice].should == "You must specify a file to upload."
+    end
+    
+  end
+
+  describe "destroy" do
+    it "should delete the asset identified by pid" do
+      mock_obj = mock("asset", :delete)
+      ActiveFedora::Base.expects(:load_instance).with("__PID__").returns(mock_obj)
+      delete(:destroy, :id => "__PID__")
+    end
+    it "should remove container relationship and perform proper garbage collection" do
+      pending "relies on ActiveFedora implementing Base.file_objects_remove"
+      mock_container = mock("asset")
+      mock_container.expects(:file_objects_remove).with("_file_asset_pid_")
+      FileAsset.expects(:garbage_collect).with("_file_asset_pid_")
+      ActiveFedora::Base.expects(:load_instance).with("_container_pid_").returns(mock_container)
+      delete(:destroy, :id => "_file_asset_pid_", :asset_id=>"_container_pid_")
+    end
+  end
+  
+  describe "integration tests - " do
+    before(:all) do
+      Fedora::Repository.register(ActiveFedora.fedora_config[:url])
+      ActiveFedora::SolrService.register(ActiveFedora.solr_config[:url])
+      @test_container = ActiveFedora::Base.new
+      @test_container.add_relationship(:is_member_of, "foo:1")
+      @test_container.add_relationship(:has_collection_member, "foo:2")
+      @test_container.save
+      
+      @test_fa = FileAsset.new
+      @test_fa.add_relationship(:is_part_of, @test_container)
+      @test_fa.save
+    end
+
+    after(:all) do
+     @test_container.delete
+     @test_fa.delete
+    end
+
+    describe "index" do
+      it "should retrieve the container object and its file assets" do
+        #xhr :get, :index, :container_id=>@test_container.pid
+        get :index, {:asset_id=>@test_container.pid}
+        @controller.params[:asset_id].should_not be_nil
+        assigns(:solr_result).should_not be_nil
+        #puts assigns(:solr_result).inspect
+        assigns(:container).file_objects(:response_format=>:id_array).should include(@test_fa.pid)
+        assigns(:container).file_objects(:response_format=>:id_array).should include("foo:2")
+      end
+    end
+    
+    describe "create" do
+      before :each do
+        mock_user = mock("User")
+        mock_warden = mock("Warden")
+        mock_warden.stubs(:authenticate).returns(mock_user)
+        request.env['warden'] = mock_warden
+      end
+
+      it "should set is_part_of relationship on the new File Asset pointing back at the container" do
+        
+        test_file = fixture("empty_file.txt")
+        filename = "My File Name"
+        post :create, {:Filedata=>test_file, :Filename=>filename, :asset_id=>@test_container.pid}
+        assigns(:file_asset).relationships[:self][:is_part_of].should == ["info:fedora/#{@test_container.pid}"] 
+        retrieved_fa = FileAsset.load_instance(@test_fa.pid).relationships[:self][:is_part_of].should == ["info:fedora/#{@test_container.pid}"]
+      end
+    end
+  end
+end