hydra-collections 0.0.1

data/solr_conf/conf/solrconfig.xml

@@ -0,0 +1,163 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<config>
+  <!-- NOTE: various comments and unused configuration possibilities have been purged
+     from this file.  Please refer to http://wiki.apache.org/solr/SolrConfigXml,
+     as well as the default solrconfig file included with Solr -->
+  
+  <abortOnConfigurationError>${solr.abortOnConfigurationError:true}</abortOnConfigurationError>
+  
+  <luceneMatchVersion>LUCENE_40</luceneMatchVersion>
+
+  <directoryFactory name="DirectoryFactory" class="${solr.directoryFactory:solr.StandardDirectoryFactory}"/>
+
+  <!-- solr lib dirs -->
+  <lib dir="../lib/contrib/analysis-extras/lib" />
+  <lib dir="../lib/contrib/analysis-extras/lucene-libs" />
+  
+  <dataDir>${solr.data.dir:}</dataDir>
+
+  <requestHandler name="search" class="solr.SearchHandler" default="true">
+    <!-- default values for query parameters can be specified, these
+         will be overridden by parameters in the request
+      -->
+     <lst name="defaults">
+       <str name="defType">edismax</str>
+       <str name="echoParams">explicit</str>
+       <str name="q.alt">*:*</str>
+       <str name="mm">2&lt;-1 5&lt;-2 6&lt;90%</str>
+       <int name="qs">1</int>
+       <int name="ps">2</int>
+       <float name="tie">0.01</float>
+       <!-- this qf and pf are used by default, if not otherwise specified by
+            client. The default blacklight_config will use these for the
+            "keywords" search. See the author_qf/author_pf, title_qf, etc 
+            below, which the default blacklight_config will specify for
+            those searches. You may also be interested in:
+            http://wiki.apache.org/solr/LocalParams
+       -->
+        <str name="qf">
+          id
+          all_text_timv
+          active_fedora_model_ssi
+          object_type_si
+        </str>
+        <str name="pf">
+          all_text_timv^10
+        </str>
+
+       <str name="author_qf">
+       </str>
+       <str name="author_pf">
+       </str>
+       <str name="title_qf">
+       </str>
+       <str name="title_pf">
+       </str>
+       <str name="subject_qf">
+       </str>
+       <str name="subject_pf">
+       </str>
+       
+       <str name="fl">
+         *, 
+         score
+       </str>
+
+       <str name="facet">true</str>
+       <str name="facet.mincount">1</str>
+       <str name="facet.limit">10</str>
+       <str name="facet.field">active_fedora_model_ssi</str>
+       <str name="facet.field">object_type_si</str>
+       
+       <str name="spellcheck">true</str>
+       <str name="spellcheck.dictionary">default</str>
+       <str name="spellcheck.onlyMorePopular">true</str>
+       <str name="spellcheck.extendedResults">true</str>
+       <str name="spellcheck.collate">false</str>
+       <str name="spellcheck.count">5</str>
+
+     </lst>
+    <arr name="last-components">
+      <str>spellcheck</str>
+    </arr>
+  </requestHandler>
+
+  <requestHandler name="permissions" class="solr.SearchHandler" >
+    <lst name="defaults">
+      <str name="facet">off</str>
+      <str name="echoParams">all</str>
+      <str name="rows">1</str>
+      <str name="q">{!raw f=id v=$id}</str> <!-- use id=666 instead of q=id:666 -->
+      <str name="fl">
+        id,
+        access_ssim,
+        discover_access_group_ssim,discover_access_person_ssim,
+        read_access_group_ssim,read_access_person_ssim,
+        edit_access_group_ssim,edit_access_person_ssim,
+        depositor_ti,
+        embargo_release_date_dtsi
+        inheritable_access_ssim,
+        inheritable_discover_access_group_ssim,inheritable_discover_access_person_ssim,
+        inheritable_read_access_group_ssim,inheritable_read_access_person_ssim,
+        inheritable_edit_access_group_ssim,inheritable_edit_access_person_ssim,
+        inheritable_embargo_release_date_dtsi
+      </str>
+    </lst>
+  </requestHandler>
+  
+  <requestHandler name="standard" class="solr.SearchHandler">
+     <lst name="defaults">
+       <str name="echoParams">explicit</str>
+       <str name="defType">lucene</str>
+     </lst>
+  </requestHandler>
+
+  <!-- for requests to get a single document; use id=666 instead of q=id:666 -->
+  <requestHandler name="document" class="solr.SearchHandler" >
+    <lst name="defaults">
+      <str name="echoParams">all</str>
+      <str name="fl">*</str>
+      <str name="rows">1</str>
+      <str name="q">{!raw f=id v=$id}</str> <!-- use id=666 instead of q=id:666 -->
+    </lst>
+  </requestHandler>
+
+
+  <searchComponent name="spellcheck" class="solr.SpellCheckComponent">
+    <str name="queryAnalyzerFieldType">textSpell</str>
+    <!-- Multiple "Spell Checkers" can be declared and used by this component
+      (e.g. for title_spell field)
+      -->
+    <lst name="spellchecker">
+      <str name="name">default</str>
+      <str name="field">spell</str>
+      <str name="spellcheckIndexDir">./spell</str>
+      <str name="buildOnOptimize">true</str>
+    </lst>
+  </searchComponent>
+  
+  <requestHandler name="/replication" class="solr.ReplicationHandler" startup="lazy" /> 
+  
+  <requestDispatcher handleSelect="true" >
+    <requestParsers enableRemoteStreaming="true" multipartUploadLimitInKB="2048" />
+  </requestDispatcher>
+  
+  <requestHandler name="/analysis/field" startup="lazy" class="solr.FieldAnalysisRequestHandler" />
+  <requestHandler name="/update" class="solr.UpdateRequestHandler"  />
+  <requestHandler name="/admin/" class="org.apache.solr.handler.admin.AdminHandlers" />
+  
+  <requestHandler name="/admin/ping" class="solr.PingRequestHandler">
+    <lst name="invariants">
+      <str name="q">solrpingquery</str>
+    </lst>
+    <lst name="defaults">
+      <str name="echoParams">all</str>
+    </lst>
+  </requestHandler>
+  
+  <!-- config for the admin interface --> 
+  <admin>
+    <defaultQuery>search</defaultQuery>
+  </admin>
+  
+</config>

data/solr_conf/solr.xml

@@ -0,0 +1,35 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements.  See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License.  You may obtain a copy of the License at
+
+     http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+
+<!--
+ All (relative) paths are relative to the installation path
+  
+  persistent: Save changes made via the API to this file
+  sharedLib: path to a lib directory that will be shared across all cores
+-->
+<solr persistent="false" sharedLib="lib">
+
+  <!--
+  adminPath: RequestHandler path to manage cores.  
+    If 'null' (or absent), cores will not be manageable via REST
+  -->
+  <cores adminPath="/admin/cores" defaultCoreName="development">
+    <core name="development" instanceDir="development-core" />
+    <core name="test" instanceDir="test-core" />
+  </cores>
+</solr>

data/spec/controllers/accepts_batches_spec.rb

@@ -0,0 +1,72 @@
+require "spec_helper"
+
+class AcceptsBatchesController < ApplicationController
+  include Hydra::Collections::AcceptsBatches
+end
+
+describe AcceptsBatchesController do
+
+  describe "batch" do
+    it "should accept batch from parameters" do
+      controller.params["batch_document_ids"] = ["abc", "xyz"]
+      controller.batch.should == ["abc", "xyz"]
+    end
+    describe ":all" do
+      before do
+        doc1 = stub(:id=>123)
+        doc2 = stub(:id=>456)
+        Hydra::Collections::SearchService.any_instance.should_receive(:last_search_documents).and_return([doc1, doc2])
+        controller.stub(:current_user=>stub(:user_key=>'vanessa'))
+      end
+      it "should add every document in the current resultset to the batch" do
+        controller.params["batch_document_ids"] = "all"
+        controller.batch.should == [123, 456]
+      end
+    end
+  end
+  
+  describe "should allow filtering for access" do
+    before do
+      @allowed = [1,2,3]
+      @disallowed = [5,6,7]
+      subject.batch = @allowed + @disallowed
+    end
+    it "using filter_docs_with_access!" do
+      @allowed.each {|doc_id| subject.should_receive(:can?).with(:foo, doc_id).and_return(true)}
+      @disallowed.each {|doc_id| subject.should_receive(:can?).with(:foo, doc_id).and_return(false)}
+      subject.send(:filter_docs_with_access!, :foo)
+      subject.batch.should
+      flash[:notice].should == "You do not have permission to edit the documents: #{@disallowed.join(', ')}"
+    end
+    it "using filter_docs_with_edit_access!" do
+      @allowed.each {|doc_id| subject.should_receive(:can?).with(:edit, doc_id).and_return(true)}
+      @disallowed.each {|doc_id| subject.should_receive(:can?).with(:edit, doc_id).and_return(false)}
+      subject.send(:filter_docs_with_edit_access!)
+      subject.batch.should
+      flash[:notice].should == "You do not have permission to edit the documents: #{@disallowed.join(', ')}"
+    end
+    it "using filter_docs_with_read_access!" do
+      @allowed.each {|doc_id| subject.should_receive(:can?).with(:read, doc_id).and_return(true)}
+      @disallowed.each {|doc_id| subject.should_receive(:can?).with(:read, doc_id).and_return(false)}
+      subject.send(:filter_docs_with_read_access!)
+      subject.batch.should
+      flash[:notice].should == "You do not have permission to edit the documents: #{@disallowed.join(', ')}"
+    end
+    it "and be sassy if you didn't select anything" do
+      subject.batch = []
+      subject.send(:filter_docs_with_read_access!)
+      flash[:notice].should == "Select something first"
+    end
+    
+  end
+    
+  it "should check for empty" do
+    controller.batch = ["77826928", "94120425"]
+    controller.check_for_empty_batch?.should == false
+    controller.batch = []
+    controller.check_for_empty_batch?.should == true
+  end
+
+
+  
+end

data/spec/controllers/collections_controller_spec.rb

@@ -0,0 +1,93 @@
+# Copyright © 2012 The Pennsylvania State University
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'spec_helper'
+
+describe CollectionsController do
+  before do
+    controller.stub(:has_access?).and_return(true)
+
+    @user = FactoryGirl.find_or_create(:user)
+    sign_in @user
+    User.any_instance.stub(:groups).and_return([])
+    controller.stub(:clear_session_user) ## Don't clear out the authenticated session
+  end
+
+  describe '#new' do
+    it 'should assign @collection' do
+      get :new
+      expect(assigns(:collection)).to be_kind_of(Collection)
+    end
+    it "should pass through batch ids if provided and stick them in the form" do
+      pending "Couldn't get have_selector working before I had to move on.  - MZ"
+      get :new, batch_document_ids: ["test2", "test88"]
+      response.should have_selector("p[class='foo']")
+    end
+  end
+  
+  describe '#create' do
+    it "should create a Collection" do
+      old_count = Collection.count
+      post :create, collection: {title: "My First Collection ", description: "The Description\r\n\r\nand more"}
+      Collection.count.should == old_count+1
+      assigns[:collection].title.should == "My First Collection "
+      assigns[:collection].description.should == "The Description\r\n\r\nand more"
+      assigns[:collection].depositor.should == @user.user_key
+      response.should redirect_to Hydra::Collections::Engine.routes.url_helpers.collection_path(assigns[:collection].id)
+    end
+    it "should add docs to collection if batch ids provided" do
+      @asset1 = ActiveFedora::Base.create!
+      @asset2 = ActiveFedora::Base.create!
+      post :create, batch_document_ids: [@asset1, @asset2], collection: {title: "My Secong Collection ", description: "The Description\r\n\r\nand more"}
+      assigns[:collection].members.should == [@asset1, @asset2]
+    end
+  end
+  
+  describe "#update" do
+    before do
+      @collection = Collection.new
+      @collection.apply_depositor_metadata(@user.user_key)
+      @collection.save
+      @asset1 = ActiveFedora::Base.create!
+      @asset2 = ActiveFedora::Base.create!
+      @asset3 = ActiveFedora::Base.create!
+      controller.should_receive(:authorize!).and_return(true)
+    end
+    it "should update collection metadata" do
+      put :update, id: @collection.id, collection: {title: "New Title", description: "New Description"}
+      response.should redirect_to Hydra::Collections::Engine.routes.url_helpers.collection_path(@collection.id)
+      assigns[:collection].title.should == "New Title"
+      assigns[:collection].description.should == "New Description"
+    end
+    it "should support adding batches of members" do
+      @collection.members << @asset1
+      @collection.save
+      put :update, id: @collection.id, collection: {members:"add"}, batch_document_ids:[@asset2, @asset3]
+      response.should redirect_to Hydra::Collections::Engine.routes.url_helpers.collection_path(@collection.id)
+      assigns[:collection].members.should == [@asset2, @asset3, @asset1]
+    end
+    it "should support removing batches of members" do
+      @collection.members = [@asset1, @asset2, @asset3]
+      @collection.save
+      put :update, id: @collection.id, collection: {members:"remove"}, batch_document_ids:[@asset1, @asset3]
+      response.should redirect_to Hydra::Collections::Engine.routes.url_helpers.collection_path(@collection.id)
+      assigns[:collection].members.should == [@asset2]
+    end
+    it "should support setting members array" do
+      put :update, id: @collection.id, collection: {members:"add"}, batch_document_ids:[@asset2, @asset3, @asset1]
+      response.should redirect_to Hydra::Collections::Engine.routes.url_helpers.collection_path(@collection.id)
+      assigns[:collection].members.should == [@asset1,@asset2, @asset3]
+    end
+  end
+end

data/spec/factories.rb

@@ -0,0 +1,18 @@
+# Copyright © 2012 The Pennsylvania State University
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+FactoryGirl.define do
+  factory :collection 
+end
+

data/spec/factories/users.rb

@@ -0,0 +1,31 @@
+# Copyright © 2012 The Pennsylvania State University
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+FactoryGirl.define do
+  factory :user, :class => User do |u|
+    email 'jilluser@example.com'
+    password 'password'
+  end
+
+  factory :archivist, :class => User do |u|
+    email 'archivist1@example.com'
+    password 'password'
+  end
+
+  factory :curator, :class => User do |u|
+    email 'curator1@example.com'
+    password 'password'
+  end
+end
+

data/spec/helpers/collections_helper_spec.rb

@@ -0,0 +1,29 @@
+# Copyright © 2013 The Pennsylvania State University
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'spec_helper'
+
+include Hydra::Collections::Engine.routes.url_helpers
+
+describe CollectionsHelper do
+  it "button_for_create_collection should create a button to the collections new path" do
+    html = button_for_create_collection 
+    html.should have_selector("form[action='#{collections.new_collection_path}']")
+    html.should have_selector("input[type='submit']")
+  end
+  it "button_for_create_collection should create a button with my text" do
+    html = button_for_create_collection "Create My Button"
+    html.should have_selector("input[value='Create My Button']")
+  end
+end