hydra-file-access 5.0.0.pre1

data/spec/fixtures/small_file.txt

@@ -0,0 +1 @@
+small

data/spec/helpers/access_controls_evaluation_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe Hydra::AccessControlsEvaluation do
+  
+  describe "editor?" do
+    it "should return false if the session[:user] is not logged in" do
+      controller.stub(:current_user).and_return(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.stub(:email).and_return "nobody_special@example.com"
+      mock_user.stub(:new_record?).and_return(false)
+      controller.stub(:current_user).and_return(mock_user)
+      helper.editor?.should be_false
+    end
+  end  
+  
+end

data/spec/helpers/assets_controller_helper_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe Hydra::AssetsControllerHelper do
+  describe "sanitize_update_params" do
+    it "should create a hash appropriate for passing into ActiveFedora::Base.update_datastream_attributes" do
+      sample_params = {
+        "content_type"=>"mods_asset", 
+        "action"=>"update", 
+        "_method"=>"put", 
+        "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"], "subject_topic" => ["subject", "topic", "subject", "topic"]}, 
+          "rightsMetadata"=>{"embargo_embargo_release_date"=>["embargo", "embargo_release_date"]}, 
+          "properties"=>{"release_to"=>["release_to"]}
+          }, 
+        "id"=>"hydrangea:fixture_mods_article3", 
+        "controller"=>"assets", 
+        "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"}, "subject_topic" => {"0"=>"Topic1", "1"=>"Topic2"}}, "rightsMetadata"=>{"embargo_embargo_release_date"=>{"0"=>""}}, "properties"=>{"released"=>{"0"=>"true"}, "release_to"=>{"0"=>"public"}}}
+        }
+      
+      expected_sane_params = {
+        "descMetadata"=>{[{:person=>0}, :institution]=>{"0"=>"FACULTY, UNIVERSITY"}, [{:journal=>0}, :title_info, :main_title]=>{"0"=>"The Journal of Mock Object"}, [{:journal=>0}, :issue, :volume]=>{"0"=>"2               "}, [{:journal=>0}, :issn]=>{"0"=>"1234-5678"}, [{:person=>1}, :first_name]=>{"0"=>"Henrietta"}, [{:person=>0}, :last_name]=>{"0"=>"Smith"}, [:location, :url]=>{"0"=>"http://example.com/foo"}, [:subject, :topic]=>{"0"=>"TOPIC 1", "1"=>"TOPIC 2", "2"=>"CONTROLLED TERM"}, [{:person=>0}, :first_name]=>{"0"=>"John"}, [{:person=>1}, :computing_id]=>{"0"=>""}, [{:journal=>0}, :issue, :end_page]=>{"0"=>"230"}, [:note]=>{"0"=>""}, [:title_info, :main_title]=>{"0"=>"Test Article"}, [{:journal=>0}, :issue, :publication_date]=>{"0"=>"FEB. 2007"}, [{:person=>1}, :description]=>{"0"=>""}, [:abstract]=>{"0"=>"ABSTRACT"}, [{:journal=>0}, :issue, :level]=>{"0"=>""}, [{:person=>1}, :institution]=>{"0"=>"Baltimore"}, [{:person=>0}, :computing_id]=>{"0"=>""}, [{:person=>1}, :last_name]=>{"0"=>"Lacks"}, [{:journal=>0}, :issue, :start_page]=>{"0"=>"195"}, [{:person=>0}, :description]=>{"0"=>""}, [{:journal=>0}, :origin_info, :publisher]=>{"0"=>"PUBLISHER"}, [:subject, :topic]=>{"0"=>"Topic1", "1"=>"Topic2"} }, "rightsMetadata"=>{[:embargo, :embargo_release_date]=>{"0"=>""}}, 
+        "properties"=>{[:release_to]=>{"0"=>"public"}, :released=>{"0"=>"true"}}
+        }
+      helper.stub(:params).and_return(sample_params)
+      sanitized_params = helper.sanitize_update_params
+      sanitized_params.should == expected_sane_params
+      sanitized_params["descMetadata"].should == expected_sane_params["descMetadata"]
+    end
+  end
+  
+  describe "tidy_response_from_update" do
+    it "should put response values into an array with fieldname / index / value Hash for each field updated" do
+      sample_input = {"descMetadata"=>{"journal_0_issue_start_page"=>{"0"=>"195"}, "person_0_computing_id"=>{"-1"=>""}, "journal_0_origin_info_publisher"=>{"0"=>"PUBLISHER"}, "person_1_institution"=>{"0"=>"Baltimore"}, "person_1_description"=>{"-1"=>""}, "abstract"=>{"0"=>"ABSTRACT"}, "person_0_description"=>{"-1"=>""}, "person_0_last_name"=>{"0"=>"Smith"}, "journal_0_issue_volume"=>{"0"=>"2               "}, "title_info_main_title"=>{"0"=>"Matt's Test Article! Again!?"}, "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_level"=>{"0"=>"55       "}, "journal_0_issue_end_page"=>{"0"=>"230"}, "person_1_first_name"=>{"0"=>"Henrietta"}, "person_0_institution"=>{"0"=>"FACULTY, UNIVERSITY"}, "journal_0_title_info_main_title"=>{"0"=>"The Journal of Mock Object"}, "journal_0_issue_publication_date"=>{"0"=>"FEB. 2007"}, "person_1_computing_id"=>{"-1"=>""}, "person_0_first_name"=>{"0"=>"John"}, "journal_0_issn"=>{"0"=>"1234-5678"}}, "rightsMetadata"=>{"embargo_embargo_release_date"=>{"-1"=>""}}, "properties"=>{:released=>{"0"=>"true"}, :release_to=>{"0"=>"public"}, [:release_to]=>{}}}
+      expected_output = {"updated"=>[{"value"=>{"0"=>"195"}, "index"=>"journal_0_issue_start_page", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"person_0_computing_id", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"PUBLISHER"}, "index"=>"journal_0_origin_info_publisher", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"Baltimore"}, "index"=>"person_1_institution", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"person_1_description", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"ABSTRACT"}, "index"=>"abstract", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"person_0_description", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"Smith"}, "index"=>"person_0_last_name", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"2               "}, "index"=>"journal_0_issue_volume", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"Matt's Test Article! Again!?"}, "index"=>"title_info_main_title", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"note", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"http://example.com/foo"}, "index"=>"location_url", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"Lacks"}, "index"=>"person_1_last_name", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"TOPIC 1", "1"=>"TOPIC 2", "2"=>"CONTROLLED TERM"}, "index"=>"subject_topic", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"55       "}, "index"=>"journal_0_issue_level", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"230"}, "index"=>"journal_0_issue_end_page", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"Henrietta"}, "index"=>"person_1_first_name", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"FACULTY, UNIVERSITY"}, "index"=>"person_0_institution", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"The Journal of Mock Object"}, "index"=>"journal_0_title_info_main_title", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"FEB. 2007"}, "index"=>"journal_0_issue_publication_date", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"person_1_computing_id", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"John"}, "index"=>"person_0_first_name", "field_name"=>"descMetadata"}, {"value"=>{"0"=>"1234-5678"}, "index"=>"journal_0_issn", "field_name"=>"descMetadata"}, {"value"=>{"-1"=>""}, "index"=>"embargo_embargo_release_date", "field_name"=>"rightsMetadata"}, {"value"=>{"0"=>"true"}, "index"=>:released, "field_name"=>"properties"}, {"value"=>{"0"=>"public"}, "index"=>:release_to, "field_name"=>"properties"}, {"value"=>{}, "index"=>[:release_to], "field_name"=>"properties"}]}
+      result = helper.tidy_response_from_update(sample_input)
+      (expected_output["updated"] - result["updated"]).should == []
+    end
+    it "if handling submission from jeditable (which will only submit one value at a time), return the value it submitted" do
+      sample_input = {"descMetadata"=>{"journal_0_issue_start_page"=>{"0"=>"195"}} }
+      expected_output = {"0"=>"195"}
+      helper.stub(:params).and_return({:field_id => "my field id"})
+      helper.tidy_response_from_update(sample_input).should == expected_output
+    end
+  end
+  
+  describe "update_document" do
+    it "should update the document with the provided params" do
+      sample_params = "sample params"
+      mock_document = mock("document")
+      mock_document.should_receive(:update_datastream_attributes).with(sample_params)
+      helper.update_document(mock_document, sample_params)
+    end
+  end
+  
+  describe "send_datastream" do
+    it "should return the requested datastream with content disposition & mime type set from datastream attributes" do
+      test_ds = ModsAsset.find("hydrangea:fixture_file_asset1").datastreams["DS1"]
+      helper.should_receive(:send_data).with(test_ds.content, :filename=>"bali.jpg", :type=>"image/jpeg")
+      helper.send(:send_datastream, test_ds)
+    end
+  end
+  
+end

data/spec/helpers/file_assets_helper_spec.rb

@@ -0,0 +1,116 @@
+require 'spec_helper'
+
+describe Hydra::Controller::UploadBehavior do
+
+  it "should respond to datastream_id" do
+    helper.should respond_to :datastream_id  ### API method, test that it's there to be overridden
+  end
+
+  describe "create_and_save_file_asset_from_params" do
+    it "should create the file asset, add posted blob to it and save the file asset" do
+      mock_fa = mock("file asset")
+      helper.stub(:params).and_return( { :Filedata => [mock_fa] } )
+      mock_fa.should_receive(:save)
+      helper.should_receive(:create_asset_from_file).and_return(mock_fa)
+      helper.should_receive(:add_posted_blob_to_asset)
+      helper.create_and_save_file_assets_from_params
+    end
+  end
+  
+  describe "add_posted_blob_to_asset" do
+    it "should set object title and label, relying on datastream_id to set dsId" do
+      mock_file = mock("File")
+      file_name = "Posted Filename.foo"
+      helper.stub(:params).and_return( :Filedata=>[mock_file], :Filename=>file_name, "container_id"=>"hydrangea:2973" )      
+      mock_fa = mock("file asset")
+      mock_file.should_receive(:original_filename).and_return(file_name)
+      helper.stub(:datastream_id).and_return('bar')
+      mock_fa.should_receive(:add_file_datastream).with(mock_file, :label=>file_name, :mimeType=>"mymimetype", :dsid=>'bar')
+      mock_fa.should_receive(:set_title_and_label).with( file_name, :only_if_blank=>true )
+      helper.should_receive(:mime_type).with(file_name).and_return("mymimetype")
+      helper.add_posted_blob_to_asset(mock_fa,mock_file)
+    end
+    
+    it "should support submissions from swfupload" do
+      mock_file = mock("File")
+      file_name = "Posted Filename.foo"
+      helper.stub(:params).and_return( :Filedata=>[mock_file], :Filename=>file_name, "container_id"=>"hydrangea:2973" )      
+      mock_fa = mock("file asset")
+      mock_file.should_receive(:original_filename).and_return(file_name)
+      mock_fa.should_receive(:add_file_datastream).with(mock_file, :label=>file_name, :mimeType=>"mymimetype", :dsid => 'content')
+      mock_fa.stub(:set_title_and_label)
+      helper.should_receive(:mime_type).with(file_name).and_return("mymimetype")
+      helper.add_posted_blob_to_asset(mock_fa,mock_file)
+    end
+    it "should support submissions from single-file uploader, defaulting to dsId of content" do
+      mock_file = mock("File")
+      file_name = "Posted Filename.foo"
+#      helper.should_receive(:filename_from_params).and_return(file_name)
+      helper.stub(:params).and_return( :Filedata=>[mock_file], :container_id=>"hydrangea:2973" )      
+      mock_file.should_receive(:original_filename).and_return(file_name)
+      mock_fa = mock("file asset")
+      helper.should_receive(:mime_type).with(file_name).and_return("mymimetype")
+      mock_fa.should_receive(:add_file_datastream).with(mock_file, :label=>file_name, :mimeType=>"mymimetype", :dsid => 'content')
+      mock_fa.stub(:set_title_and_label)
+      helper.add_posted_blob_to_asset(mock_fa,mock_file)
+    end
+  end
+  
+  describe "create_asset_from_params" do
+    it "should create a new file asset and set the label from params[:Filename]" do
+      helper.stub(:params).and_return( { :Filename => "Test Filename" } )
+      result = helper.create_asset_from_params
+      result.should be_kind_of FileAsset
+      result.label.should == "Test Filename"
+    end
+    it "should choose model by filename" do
+      pending "this is currently disabled"
+      helper.should_receive(:choose_model_by_filename)
+      helper.create_asset_from_params
+    end
+  end
+  
+  describe "posted_file" do
+    it "should return the posted file" do
+      helper.should_receive(:params).and_return(:Filedata=>"test posted file")
+      helper.posted_file.should == "test posted file"
+    end
+    it "should return nil if no file was posted" do
+      helper.posted_file.should == nil
+    end
+  end
+  
+  describe "filename_from_params" do
+    it "should return the value of params[:Filename] if it was submitted" do
+      helper.stub(:params).and_return(:Filename => "Test Filename")
+      helper.filename_from_params.should == "Test Filename"
+    end
+    it "should default to using the original filename of the posted file" do
+      helper.stub(:params).and_return({})
+      helper.should_receive(:posted_file).and_return(mock("File", :original_filename=>"Test Original Filename"))
+      helper.filename_from_params.should == "Test Original Filename"
+    end
+  end
+  
+  describe "choose_model_by_filename" do
+    it "should return model classes based on filename extensions" do
+      pending "This can only be enabled if/when we adopt replacements for ImageAsset, AudioAsset, etc. as default primitives."
+      
+      ["filename.wav","filename.mp3","filename.aiff"].each do |fn|
+        helper.choose_model_by_filename(fn).should == AudioAsset
+      end
+      
+      ["filename.mov","filename.flv","filename.mp4", "filename.m4v"].each do |fn|
+        helper.choose_model_by_filename(fn).should == VideoAsset
+      end
+      
+      ["filename.jpeg","filename.jpg","filename.gif", "filename.png"].each do |fn|
+        helper.choose_model_by_filename(fn).should == ImageAsset
+      end
+      
+      ["filename.doc","filename.pdf","filename.jp2", "filename.zip"].each do |fn|
+        helper.choose_model_by_filename(fn).should == FileAsset
+      end
+    end
+  end
+end

data/spec/helpers/hydra_helper_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+
+describe HydraHelper do
+  
+  describe "submit_name" do
+    it "should return 'Save' when the scripts session variable is set" do
+      helper.stub(:session => {:scripts=>true})
+      helper.submit_name.should == "Save"
+    end
+    it "should return 'Continue' when the new_asset param is set" do
+      helper.stub(:params => {:new_asset=>true})
+      helper.submit_name.should == "Continue"
+    end
+    it "should return 'Save and Continue' if all else fails" do
+      helper.stub(:params => {})
+      helper.submit_name.should == "Save and Continue"
+    end
+  end
+  
+end

data/spec/helpers/hydra_model_methods_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe Hydra::ModelMethods do
+  
+  
+  describe "set_title_and_label" do
+    before(:each) do
+      dm = mock("descMetadata")
+      helper.stub(:datastreams).and_return("descMetadata"=>dm)
+    end
+    it "should set the title and the label" do
+      helper.should_receive(:set_title)
+      helper.stub(:label).and_return(nil)
+      helper.should_receive(:label=).with("My title")
+      helper.set_title_and_label("My title")
+    end
+    it "should skip updating if the label is set already" do
+      helper.should_receive(:set_title).never
+      helper.should_receive(:label=).never
+      helper.stub(:label).and_return("pre existing label")
+      helper.set_title_and_label("My title", :only_if_blank=>true)
+    end
+  end
+  
+  describe "set_title" do
+    it "should set the title if the descMetadata is a NokogiriDatastream that responds to :title term" do
+      obj = ActiveFedora::Base.new
+      dm = Hydra::Datastream::Properties.new(obj.inner_object, nil)
+      dm.stub(:content).and_return('')
+      helper.stub(:datastreams).and_return("descMetadata"=>dm)
+      helper.set_title("My title")
+      dm.term_values(:title).should == ["My title"]
+    end
+    it "should set the title if the descMetadata is a MetadataDatastream with a title field defined" do
+      obj = ActiveFedora::Base.new
+      dm = ActiveFedora::QualifiedDublinCoreDatastream.new(obj.inner_object, nil)
+      dm.stub(:content).and_return('')
+      #dm = ActiveFedora::QualifiedDublinCoreDatastream.new  nil, nil 
+      helper.stub(:datastreams).and_return("descMetadata"=>dm)
+      helper.set_title("My title")
+      dm.title.should == ["My title"]
+    end
+  end
+  
+  describe "set_collection_type" do
+    it "should set the collection metadata field" do
+      prop_ds = mock("properties ds")
+      prop_ds.should_receive(:respond_to?).with(:collection_values).and_return(true)
+      prop_ds.should_receive(:collection_values=).with("mods_asset")
+
+      helper.stub(:datastreams).and_return({"properties"=>prop_ds})
+      helper.set_collection_type("mods_asset")
+    end
+  end
+
+  describe "#destroyable_child_assets" do
+    it "should return an array of file assets available for deletion" do
+      ma = ModsAsset.load_instance_from_solr("hydrangea:fixture_mods_article1")
+      deletable_assets = ma.destroyable_child_assets
+      deletable_assets.should be_a_kind_of Array
+      deletable_assets.length.should >= 1
+      deletable_assets.select {|a| a.pid == "hydrangea:fixture_uploaded_svg1"}[0].pid.should == "hydrangea:fixture_uploaded_svg1"
+    end
+    it "should return an empty array if there are now file assets" do
+      ma = ModsAsset.load_instance_from_solr("hydrangea:fixture_mods_article2")
+      deletable_assets = ma.destroyable_child_assets
+      deletable_assets.should be_a_kind_of Array
+      deletable_assets.should be_empty
+    end
+  end
+
+  describe "#destroy_child_assets" do
+    it "should destroy any child assets and return an array listing the child assets" do
+      ma = ModsAsset.load_instance_from_solr("hydrangea:fixture_mods_article1")
+      file_asset = mock("file object")
+      file_asset.should_receive(:pid).and_return("hydrangea:fixture_uploaded_svg1")
+      file_asset.should_receive(:delete).and_return(true)
+      ma.stub(:destroyable_child_assets).and_return([file_asset])
+      ma.destroy_child_assets.should == ["hydrangea:fixture_uploaded_svg1"]
+    end
+    it "should do nothing and return an empty array for an object with no child assets" do
+      ma = ModsAsset.load_instance_from_solr("hydrangea:fixture_mods_article2")
+      ma.destroy_child_assets.should == []
+    end
+  end
+
+
+end

data/spec/integration/file_asset_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+class DummyFileAsset < ActiveFedora::Base
+  def initialize(attr={})
+    super(attr)
+    add_relationship(:has_model, "info:fedora/afmodel:FileAsset")
+  end
+end
+
+describe FileAsset do
+  before(:each) do
+    @file_asset = FileAsset.new
+  end
+  after(:each) do
+    begin
+    @file_asset.delete
+    rescue
+    end
+  end
+  describe "with parts" do
+    before(:each) do
+      @asset1 = ActiveFedora::Base.new
+      @asset1.save
+    end
+
+    after(:each) do
+      @asset1.delete
+    end
+
+    describe ".container_id" do    
+      it "should return asset container objects via is_part_of relationships" do
+        #none
+        @file_asset.container_id.should be_nil
+        #is_part_of
+        @file_asset.container = @asset1
+        @file_asset.container_id.should == @asset1.pid
+      end
+    end
+  end
+
+  describe ".to_solr" do
+    it "should load base fields correctly if active_fedora_model is FileAsset" do
+      @file_asset.update_indexed_attributes({:title=>{0=>"testing"}})
+      solr_doc = @file_asset.to_solr
+      solr_doc["title_t"].should == ["testing"]
+    end
+    it "should load base fields for FileAsset for model_only if active_fedora_model is not FileAsset but is not child of FileAsset" do
+      pending "I'm unconvinced as to the usefullness of this test. Why create as one type then reload as another? - Justin"
+      @dummy_file_asset = DummyFileAsset.new
+      @dummy_file_asset.save
+      file_asset = FileAsset.find(@dummy_file_asset.pid)
+      ENABLE_SOLR_UPDATES = false
+      #it should save change to Fedora, but not solr
+      file_asset.update_indexed_attributes({:title=>{0=>"testing"}})
+      file_asset.save
+      ENABLE_SOLR_UPDATES = true
+      solr_doc = DummyFileAsset.find_by_solr(@dummy_file_asset.pid).hits.first
+      solr_doc["title_t"].should be_nil
+      @dummy_file_asset.update_index
+      solr_doc = DummyFileAsset.find_by_solr(@dummy_file_asset.pid).hits.first
+      solr_doc["title_t"].should == ["testing"]
+      begin
+      @dummy_file_asset.delete
+      rescue
+      end
+    end
+  end
+end

data/spec/lib/hydra_submission_workflow_spec.rb

@@ -0,0 +1,141 @@
+require File.expand_path( File.join( File.dirname(__FILE__),'..','spec_helper') )
+
+mods_asset_model = "info:fedora/afmodel:ModsAsset"
+
+describe Hydra::SubmissionWorkflow do
+  before(:each) do
+    @document = SolrDocument.new({:has_model_s => [mods_asset_model]})
+  end
+  include Hydra::SubmissionWorkflow
+  
+  describe "first step in workflow" do
+    it "should return the first step of a given workflow" do
+      first_step_in_workflow.should == "contributor"
+    end
+  end
+  
+  describe "next in workflow" do
+    it "should provide the next step based on the provided step" do
+      next_step_in_workflow(:contributor).should == "publication"
+    end
+    it "should return nil if there is no step (denoting the last step)" do
+      next_step_in_workflow(:permissions).should be_nil
+    end
+    it "should return the step after the first if a blank current step is provided (e.g. started from the first step so no wf_step param)" do
+      next_step_in_workflow(nil).should == "publication"
+      next_step_in_workflow("").should == "publication"
+    end
+  end
+
+  describe "partial for step" do
+    it "should return the partial for the given step" do
+      workflow_partial_for_step(:contributor).should match(/contributor_form/)
+    end
+  end
+
+  describe "previous show partials" do
+    it "should return all of the previous show partials for display in the HTML-only workflow" do
+      prev_partials = previous_show_partials(:permissions)
+      prev_partials.should be_kind_of(Array)
+      prev_partials.length.should == 4
+      prev_partials.first.should match(/mods.*contributors/)
+      prev_partials.last.should match(/mods.*file_assets/)
+    end
+    it "should return an empty array for the first step in the workflow" do
+      previous_show_partials(:contributor).should == []
+      previous_show_partials("").should == []
+    end
+  end
+
+  describe "model specific configurations" do
+    it "should return the appropriate configuration when an @document object is available" do
+      config = model_config
+      config.should be_kind_of(Array)
+      config.length.should == 5
+      partial_is_mods = []
+      config.each do |c|
+        c.is_a?(Hash).should be_true
+        c.has_key?(:name).should be_true
+        c.has_key?(:edit_partial).should be_true
+        partial_is_mods << c[:edit_partial].include?("mods_assets")
+      end
+      partial_is_mods.include?(true).should be_true
+    end
+    it "should return the appropriate config when a model is available in the params hash" do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return({:content_type => "generic_content"})
+      config = to.model_config
+      config.should be_kind_of(Array)
+      config.length.should == 4
+      partial_is_generic = []
+      config.each do |c|
+        c.is_a?(Hash).should be_true
+        c.has_key?(:name).should be_true
+        c.has_key?(:edit_partial).should be_true
+        partial_is_generic << c[:edit_partial].include?("generic_content")
+      end
+      partial_is_generic.include?(true).should be_true
+    end
+    it "should return the appropriate config when the ID of an object is available in the params hash" do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return({:id=>"hydrangea:fixture_mods_article1"})
+      config = to.model_config
+      config.should be_kind_of(Array)
+      config.length.should == 5
+      config.each do |c|
+        c.is_a?(Hash).should be_true
+        c.has_key?(:name).should be_true
+        c.has_key?(:edit_partial).should be_true
+      end
+    end
+    it "should return the configuration for non mods assets (generic_content)" do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return(:id=>"hydra:test_generic_content")
+      config = to.model_config
+      config.should be_kind_of(Array)
+      config.length.should == 4
+      partial_is_generic = []
+      config.each do |c|
+        c.is_a?(Hash).should be_true
+        c.has_key?(:name).should be_true
+        c.has_key?(:edit_partial).should be_true
+        partial_is_generic << c[:edit_partial].include?("generic_content")
+      end
+      partial_is_generic.include?(true).should be_true
+    end
+    it "should return nil if an ID is the only context available and it is not a valid object" do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return(:id=>"hydra:not_an_actual_record")
+      to.model_config.should be_nil
+    end
+  end
+  describe "before_filter validation" do
+    it "should redirect back when the validation method returns false." do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return({:id=>"hydrangea:fixture_mods_article1",:action=>"create"})
+      to.should_receive(:redirect_to).with(:back)
+      to.validate_workflow_step
+    end
+    it "should not redirect when the validation method returns true." do
+      @document = nil
+      to = SubmissionWorkflowObject.new
+      to.stub(:params).and_return({:id=>"hydrangea:fixture_mods_article1",:action=>"update"})
+      to.should_receive(:redirect_to).never
+      to.validate_workflow_step
+    end
+  end
+end
+class SubmissionWorkflowObject
+  include Hydra::SubmissionWorkflow
+  def mods_assets_create_validation
+    return false
+  end
+  def mods_assets_update_validation
+    return true
+  end
+end