active-fedora 13.1.2 → 13.1.3

data/spec/unit/files_hash_spec.rb

@@ -1,52 +0,0 @@
-require 'spec_helper'
-
-describe ActiveFedora::FilesHash do
-  before do
-    class FilesContainer; end
-    allow(FilesContainer).to receive(:child_resource_reflections).and_return(file: reflection)
-    allow(container).to receive(:association).with(:file).and_return(association)
-    allow(container).to receive(:undeclared_files).and_return([])
-  end
-
-  after { Object.send(:remove_const, :FilesContainer) }
-
-  subject(:file_hash) { described_class.new(container) }
-  let(:reflection) { instance_double(ActiveFedora::Reflection::MacroReflection) }
-  let(:association) { instance_double(ActiveFedora::Associations::SingularAssociation, reader: object) }
-  let(:object) { double('object') }
-  let(:container) { FilesContainer.new }
-
-  describe "#key?" do
-    context 'when the key is present' do
-      it "is true" do
-        expect(file_hash.key?(:file)).to be true
-      end
-      it "returns true if a string is passed" do
-        expect(file_hash.key?('file')).to be true
-      end
-    end
-
-    context 'when the key is not present' do
-      it "is false" do
-        expect(file_hash.key?(:foo)).to be false
-      end
-    end
-  end
-
-  describe "#[]" do
-    context 'when the key is present' do
-      it "returns the object" do
-        expect(file_hash[:file]).to eq object
-      end
-      it "returns the object if a string is passed" do
-        expect(file_hash['file']).to eq object
-      end
-    end
-
-    context 'when the key is not present' do
-      it "is nil" do
-        expect(file_hash[:foo]).to be_nil
-      end
-    end
-  end
-end

data/spec/unit/filter_spec.rb

@@ -1,133 +0,0 @@
-require 'spec_helper'
-
-describe ActiveFedora::Associations::FilterAssociation do
-  before do
-    class Image < ActiveFedora::Base
-      ordered_aggregation :members, through: :list_source, class_name: 'ActiveFedora::Base'
-
-      filters_association :members, as: :child_objects, condition: :pcdm_object?
-      filters_association :members, as: :child_collections, condition: :pcdm_collection?
-    end
-
-    class TestObject < ActiveFedora::Base
-      def pcdm_object?
-        true
-      end
-
-      def pcdm_collection?
-        false
-      end
-    end
-
-    class TestCollection < ActiveFedora::Base
-      def pcdm_object?
-        false
-      end
-
-      def pcdm_collection?
-        true
-      end
-    end
-  end
-
-  after do
-    Object.send(:remove_const, :Image)
-    Object.send(:remove_const, :TestObject)
-    Object.send(:remove_const, :TestCollection)
-  end
-
-  let(:image) { Image.new }
-  let(:test_object) { TestObject.new }
-  let(:test_collection) { TestCollection.new }
-
-  describe "setting" do
-    context "when an incorrect object type is sent" do
-      it "raises an error" do
-        image.child_collections
-        expect { image.child_collections = [test_object] }.to raise_error ArgumentError
-      end
-    end
-
-    context "when the parent is already loaded" do
-      let(:another_collection) { TestCollection.new }
-      before do
-        image.members = [test_object, test_collection]
-        image.child_collections = [another_collection]
-      end
-      it "overwrites existing matches" do
-        expect(image.members).to contain_exactly test_object, another_collection
-      end
-    end
-  end
-
-  describe "appending" do
-    context "when an incorrect object type is sent" do
-      it "raises an error" do
-        expect { image.child_collections << test_object }.to raise_error ArgumentError
-      end
-    end
-
-    context "when the parent is already loaded" do
-      let(:another_collection) { TestCollection.new }
-      before do
-        image.members = [test_object, test_collection]
-        image.child_collections << [another_collection]
-      end
-
-      it "updates the parent" do
-        expect(image.members).to contain_exactly test_object, test_collection, another_collection
-      end
-    end
-  end
-
-  describe "#size" do
-    it "returns the size" do
-      # Need to persist so that count_records will be called.
-      image.save
-      test_object.save
-      image.members = [test_object]
-
-      expect(image.reload.child_objects.size).to eq 1
-    end
-  end
-
-  describe "reading" do
-    before do
-      image.members = [test_object, test_collection]
-    end
-
-    it "returns the objects of the correct type" do
-      expect(image.child_objects).to eq [test_object]
-    end
-
-    describe "when the parent association is changed" do
-      before do
-        image.child_objects = [test_object]
-        image.child_objects.to_a # this would cause the @target of the association to be populated
-        image.members = [test_collection]
-      end
-
-      it "updates the filtered relation" do
-        expect(image.child_objects).to eq []
-      end
-    end
-
-    describe "#_ids" do
-      it "returns just the ids" do
-        expect(image.child_object_ids).to eq [test_object.id]
-      end
-    end
-  end
-
-  describe "#delete" do
-    subject { image.members }
-
-    let(:another_object) { TestObject.new }
-    before do
-      image.members = [test_object, test_collection, another_object]
-      image.child_objects.delete(test_object)
-    end
-
-    it { is_expected.to contain_exactly test_collection, another_object }
-  end
-end

data/spec/unit/finder_methods_spec.rb

@@ -1,154 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe ActiveFedora::FinderMethods do
-  let(:object_class) do
-    Class.new do
-      def self.delegated_attributes
-        {}
-      end
-
-      def self.solr_query_handler
-        'standard'
-      end
-
-      def self.default_sort_params
-        ["system_create_dtsi asc"]
-      end
-    end
-  end
-
-  let(:finder_class) do
-    this = self
-    Class.new do
-      include ActiveFedora::FinderMethods
-      @@klass = this.object_class
-      def initialize
-        @klass = @@klass
-      end
-    end
-  end
-
-  let(:finder) { finder_class.new }
-
-  describe '#equivalent_class?' do
-    let(:child_class) { Class.new(object_class) }
-    let(:object_class) { Class.new }
-    it 'is true for the exact class' do
-      expect(finder.send(:equivalent_class?, object_class)).to be true
-    end
-
-    it 'is true for child classes' do
-      expect(finder.send(:equivalent_class?, child_class)).to be true
-    end
-
-    it 'is falsey for non-decendants' do
-      expect(finder.send(:equivalent_class?, Class.new)).to be_nil
-    end
-  end
-
-  describe '#class_to_load' do
-    subject(:class_to_load) { finder.send(:class_to_load, resource, true) }
-
-    let(:best_model) { Class.new }
-    let(:mapper) { instance_double(ActiveFedora::DefaultModelMapper, classifier: classifier) }
-    let(:classifier) { instance_double(ActiveFedora::ModelClassifier, best_model: best_model) }
-    let(:resource) { instance_double(Hash) }
-
-    before do
-      allow(ActiveFedora).to receive(:model_mapper).and_return(mapper)
-    end
-
-    context 'when using the default implementation' do
-      it 'raises an error' do
-        expect { class_to_load }.to raise_error ActiveFedora::ModelMismatch
-      end
-    end
-
-    context 'when a custom finder overrides equivalent_class?' do
-      before do
-        allow(finder).to receive(:equivalent_class?).and_return(true)
-      end
-
-      it 'calls the overridden implementation' do
-        expect(class_to_load).to eq best_model
-        expect(finder).to have_received(:equivalent_class?).with(best_model)
-      end
-    end
-  end
-
-  describe "#condition_to_clauses" do
-    subject { finder.send(:condition_to_clauses, key, value) }
-    let(:key) { 'library_id' }
-
-    context "when value is nil" do
-      let(:value) { nil }
-      it { is_expected.to eq "-library_id:[* TO *]" }
-    end
-
-    context "when value is empty string" do
-      let(:value) { '' }
-      it { is_expected.to eq "-library_id:[* TO *]" }
-    end
-
-    context "when value is an id" do
-      let(:value) { 'one/two/three' }
-      it { is_expected.to eq "_query_:\"{!field f=library_id}one/two/three\"" }
-    end
-
-    context "when value is an array" do
-      let(:value) { ['one', 'four'] }
-      it { is_expected.to eq "(_query_:\"{!field f=library_id}one\" AND " \
-                             "_query_:\"{!field f=library_id}four\")" }
-    end
-  end
-
-  describe "#search_in_batches" do
-    let(:docs) { instance_double(RSolr::Response::PaginatedDocSet, has_next?: false) }
-    let(:select_handler) { 'select' }
-    let(:connection) { instance_double(RSolr::Client) }
-    before do
-      expect(finder).to receive(:create_query).with('age_t' => '21').and_return('dummy query')
-      allow(ActiveFedora::SolrService.instance).to receive(:conn).and_return(connection)
-      allow(ActiveFedora::SolrService).to receive(:select_path).and_return(select_handler)
-      expect(connection).to receive(:paginate) \
-        .with(1, 1000, select_handler, params: hash_including(other_opt: 'test')) \
-        .and_return('response' => { 'docs' => docs })
-    end
-    it "yields the docs" do
-      expect { |b|
-        finder.search_in_batches({ 'age_t' => '21' }, { other_opt: 'test' }, &b)
-      }.to yield_with_args(docs)
-    end
-
-    context "with custom select handler" do
-      let(:select_handler) { 'select_test' }
-      it "uses the custom select handler" do
-        finder.search_in_batches({ 'age_t' => '21' }, other_opt: 'test') do end
-      end
-    end
-  end
-
-  describe '#search_by_id' do
-    context 'with a document in solr' do
-      let(:doc) { instance_double(Hash) }
-
-      before do
-        expect(finder).to receive(:search_with_conditions).with({ id: 'x' }, hash_including(rows: 1)).and_return([doc])
-      end
-
-      it "returns the document" do
-        expect(finder.search_by_id('x')).to eq doc
-      end
-    end
-
-    context 'without a document in solr' do
-      before do
-        expect(finder).to receive(:search_with_conditions).with({ id: 'x' }, hash_including(rows: 1)).and_return([])
-      end
-
-      it "returns the document" do
-        expect { finder.search_by_id('x') }.to raise_error ActiveFedora::ObjectNotFoundError, "Object 'x' not found in solr"
-      end
-    end
-  end
-end

data/spec/unit/fixity_service_spec.rb

@@ -1,128 +0,0 @@
-require 'spec_helper'
-
-describe ActiveFedora::FixityService do
-  let(:service) { described_class.new(uri) }
-  let(:uri) { RDF::URI("http://path/to/resource") }
-
-  let(:passing_fedora44_response_body) {
-    <<-EOF
-@prefix premis: <http://www.loc.gov/premis/rdf/v1#> .
-
-<http://127.0.0.1:8080/rest/dev/0k/22/5b/04/0k225b04p/files/9f296a1f-10e7-44a3-83eb-4811d611edc6/fcr:versions/version1> premis:hasFixity <http://127.0.0.1:8080/rest/dev/0k/22/5b/04/0k225b04p/files/9f296a1f-10e7-44a3-83eb-4811d611edc6/fcr:versions/version1#fixity/1493843767961> .
-
-<http://127.0.0.1:8080/rest/dev/0k/22/5b/04/0k225b04p/files/9f296a1f-10e7-44a3-83eb-4811d611edc6/fcr:versions/version1#fixity/1493843767961> a premis:Fixity , premis:EventOutcomeDetail ;
-  premis:hasEventOutcome "SUCCESS"^^<http://www.w3.org/2001/XMLSchema#string> ;
-  premis:hasMessageDigest <urn:sha1:b995eef5262dd1c74f0ed9c96be1f404394d45dc> ;
-  premis:hasSize "103945"^^<http://www.w3.org/2001/XMLSchema#long> .
-  EOF
-  }
-
-  let(:failing_fedora44_response_body) {
-    <<-EOF
-@prefix premis: <http://www.loc.gov/premis/rdf/v1#> .
-
-<http://127.0.0.1:8080/rest/dev/ks/65/hc/20/ks65hc20t/files/e316b4b5-4627-44f8-9fdb-d2016e0e7380/fcr:versions/version3> premis:hasFixity <http://127.0.0.1:8080/rest/dev/ks/65/hc/20/ks65hc20t/files/e316b4b5-4627-44f8-9fdb-d2016e0e7380/fcr:versions/version3#fixity/1493844791463> .
-
-<http://127.0.0.1:8080/rest/dev/ks/65/hc/20/ks65hc20t/files/e316b4b5-4627-44f8-9fdb-d2016e0e7380/fcr:versions/version3#fixity/1493844791463> a premis:Fixity , premis:EventOutcomeDetail ;
-  premis:hasEventOutcome "BAD_CHECKSUM"^^<http://www.w3.org/2001/XMLSchema#string> , "BAD_SIZE"^^<http://www.w3.org/2001/XMLSchema#string> ;
-  premis:hasMessageDigest <urn:sha1:1a89571e25dd372563a10740a883e93f8af2d146> ;
-  premis:hasSize "1878582"^^<http://www.w3.org/2001/XMLSchema#long> .
-EOF
-  }
-
-  describe "the instance" do
-    subject { described_class.new(uri) }
-    it { is_expected.to respond_to(:response) }
-  end
-
-  describe "initialize" do
-    context "with a string" do
-      subject { service.target }
-      let(:uri) { 'http://path/to/resource' }
-      it { is_expected.to eq 'http://path/to/resource' }
-    end
-
-    context "with an RDF::URI" do
-      subject { service.target }
-      it { is_expected.to eq 'http://path/to/resource' }
-    end
-  end
-
-  describe "#verified?" do
-    before { allow(service).to receive(:fixity_response_from_fedora).and_return(response) }
-    subject { service.verified? }
-
-    context "with Fedora version >= 4.4.0" do
-      context "with a passing result" do
-        let(:response) do
-          instance_double("Response", body: passing_fedora44_response_body)
-        end
-        it { is_expected.to be true }
-      end
-
-      context "with a failing result" do
-        let(:response) do
-          instance_double("Response", body: failing_fedora44_response_body)
-        end
-        it { is_expected.to be false }
-      end
-    end
-
-    context "with Fedora version < 4.4.0" do
-      context "with a passing result" do
-        let(:response) do
-          instance_double("Response", body: '<subject> <http://fedora.info/definitions/v4/repository#status> "SUCCESS"^^<http://www.w3.org/2001/XMLSchema#string> .')
-        end
-        it { is_expected.to be true }
-      end
-
-      context "with a failing result" do
-        let(:response) do
-          instance_double("Response", body: '<subject> <http://fedora.info/definitions/v4/repository#status> "BAD_CHECKSUM"^^<http://www.w3.org/2001/XMLSchema#string> .')
-        end
-        it { is_expected.to be false }
-      end
-    end
-
-    context "with a non-existent predicate" do
-      let(:response) do
-        instance_double("Response", body: '<subject> <http://bogus.com/definitions/v1/bogusTerm#foo> "SUCCESS"^^<http://www.w3.org/2001/XMLSchema#string> .')
-      end
-      it { is_expected.to be false }
-    end
-  end
-
-  describe "expected_message_digest" do
-    before { allow(service).to receive(:fixity_response_from_fedora).and_return(response) }
-    subject { service.expected_message_digest }
-    context "with success response" do
-      let(:response) do
-        instance_double("Response", body: passing_fedora44_response_body)
-      end
-      it { is_expected.to match(/urn:sha1:[a-f0-9]+/) }
-    end
-    context "with failure response" do
-      let(:response) do
-        instance_double("Response", body: failing_fedora44_response_body)
-      end
-      it { is_expected.to match(/urn:sha1:[a-f0-9]+/) }
-    end
-  end
-
-  describe "expected_size" do
-    before { allow(service).to receive(:fixity_response_from_fedora).and_return(response) }
-    subject { service.expected_size }
-    context "with success response" do
-      let(:response) do
-        instance_double("Response", body: passing_fedora44_response_body)
-      end
-      it { is_expected.to be_kind_of Numeric }
-    end
-    context "with failure response" do
-      let(:response) do
-        instance_double("Response", body: failing_fedora44_response_body)
-      end
-      it { is_expected.to be_kind_of Numeric }
-    end
-  end
-end