fedora-migrate 0.2.0 → 0.3.0

data/spec/unit/migration_options_spec.rb

@@ -58,4 +58,22 @@ describe FedoraMigrate::MigrationOptions do
     end
   end
 
+  describe "#blacklist" do
+    context "by default" do
+      subject { TestCase.new.blacklist }
+      it { is_expected.to be_empty }
+    end
+    context "with a list of pids" do
+      let(:blacklist) { ["pid1, pid2"] }
+      subject do
+        TestCase.new.tap do |example| 
+          example.options = { blacklist: blacklist }
+        end
+      end
+      it "returns the list of pids" do
+        expect(subject.blacklist).to eql blacklist
+      end
+    end
+  end
+
 end

data/spec/unit/migration_report_spec.rb

@@ -2,7 +2,9 @@ require 'spec_helper'
 
 describe FedoraMigrate::MigrationReport do
 
-  let(:existing_report) { FedoraMigrate::MigrationReport.new("spec/fixtures/sample-report.json") }
+  let(:path)            { "spec/fixtures/reports/sample" }
+  let(:default_path)    { "migration_report" }
+  let(:existing_report) { FedoraMigrate::MigrationReport.new(path) }
   let(:new_report)      { FedoraMigrate::MigrationReport.new }
 
   context "with an existing report" do
@@ -12,6 +14,10 @@ describe FedoraMigrate::MigrationReport do
       subject { existing_report.results }
       it { is_expected.to be_kind_of(Hash) }
     end
+    describe "::path" do
+      subject { existing_report.path }
+      it { is_expected.to eql path }
+    end
     describe "::failed_objects" do
       subject { existing_report.failed_objects }
       it { is_expected.to include("scholarsphere:6395wb555", "scholarsphere:x346dm27k") }
@@ -31,17 +37,11 @@ describe FedoraMigrate::MigrationReport do
       it { is_expected.to be_kind_of(String) }
     end
     describe "::save" do
-      context "with the default path" do
-        it "should write the report" do
-          expect(File).to receive(:write).with("report.json", "{\n}")
-          new_report.save
-        end
-      end
-      context "with a user-provided path" do
-        it "should write the report" do
-          expect(File).to receive(:write).with("foo/path/report.json", "{\n}")
-          new_report.save("foo/path")
-        end
+      let(:individual_report) { Hash.new }
+      let(:pid) { "some:pid" }
+      it "should write the report" do
+        expect(File).to receive(:write).with("migration_report/some_pid.json", "{\n}")
+        new_report.save(pid, individual_report)
       end
     end
   end
@@ -53,6 +53,9 @@ describe FedoraMigrate::MigrationReport do
       subject { new_report.results }
       it { is_expected.to be_kind_of(Hash) }
     end
+    describe "::path" do
+      subject { new_report.path }
+      it { is_expected.to eql default_path }
+    end
   end
-
 end

data/spec/unit/repository_migrator_spec.rb

@@ -8,16 +8,10 @@ describe FedoraMigrate::RepositoryMigrator do
   it { is_expected.to respond_to(:report) }
   it { is_expected.to respond_to(:namespace) }
 
-  def mock_report content
-    report = FedoraMigrate::MigrationReport.new 
-    report.results = JSON.parse(content.to_json)
-    report
-  end
-
   describe "#failures" do
     context "when objects have failed to migrate" do
       let(:failing_report) { { "sufia:rb68xc089" => FedoraMigrate::RepositoryMigrator::SingleObjectReport.new(false, "objects", "relationships") } }
-      before { allow_any_instance_of(FedoraMigrate::RepositoryMigrator).to receive(:report).and_return(mock_report(failing_report)) }
+      before { allow_any_instance_of(FedoraMigrate::MigrationReport).to receive(:results).and_return(failing_report) }
       subject do
         migrator = FedoraMigrate::RepositoryMigrator.new(namespace)
         migrator.failures
@@ -26,7 +20,7 @@ describe FedoraMigrate::RepositoryMigrator do
     end
     context "when all objects have migrated" do
       let(:passing_report) { { "sufia:rb68xc089" => FedoraMigrate::RepositoryMigrator::SingleObjectReport.new(true, "objects", "relationships") } }
-      before { allow_any_instance_of(FedoraMigrate::RepositoryMigrator).to receive(:report).and_return(mock_report(passing_report)) }
+      before { allow_any_instance_of(FedoraMigrate::MigrationReport).to receive(:results).and_return(passing_report) }
       subject do
         migrator = FedoraMigrate::RepositoryMigrator.new(namespace)
         migrator.failures

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fedora-migrate
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Adam Wead
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-24 00:00:00.000000000 Z
+date: 2015-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hydra-head
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.0'
+        version: '9.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.0'
+        version: '9.5'
 - !ruby/object:Gem::Dependency
   name: rubydora
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: jettywrapper
   requirement: !ruby/object:Gem::Requirement
@@ -180,7 +180,6 @@ files:
 - lib/fedora_migrate/version.rb
 - lib/tasks/fedora-migrate.rake
 - spec/fixtures/datastreams/sufia-rb68xc089-characterization.xml
-- spec/fixtures/failed-report.json
 - spec/fixtures/objects/f3-migration-a.xml
 - spec/fixtures/objects/gf-versioned-content.xml
 - spec/fixtures/objects/scholarsphere_5712mc568.xml
@@ -196,7 +195,20 @@ files:
 - spec/fixtures/objects/sufia_5m60qr95r.xml
 - spec/fixtures/objects/sufia_5m60qr961.xml
 - spec/fixtures/objects/sufia_5m60qr979.xml
-- spec/fixtures/sample-report.json
+- spec/fixtures/reports/failed/sufia_5m60qr94g.json
+- spec/fixtures/reports/failed/sufia_5m60qr95r.json
+- spec/fixtures/reports/failed/sufia_5m60qr961.json
+- spec/fixtures/reports/failed/sufia_5m60qr979.json
+- spec/fixtures/reports/failed/sufia_rb68xc089.json
+- spec/fixtures/reports/failed/sufia_rb68xc09k.json
+- spec/fixtures/reports/failed/sufia_rb68xc10b.json
+- spec/fixtures/reports/failed/sufia_rb68xc11m.json
+- spec/fixtures/reports/failed/sufia_xp68km39w.json
+- spec/fixtures/reports/sample/scholarsphere_000000000.json
+- spec/fixtures/reports/sample/scholarsphere_000000018.json
+- spec/fixtures/reports/sample/scholarsphere_05741r698.json
+- spec/fixtures/reports/sample/scholarsphere_6395wb555.json
+- spec/fixtures/reports/sample/scholarsphere_x346dm27k.json
 - spec/integration/content_versions_spec.rb
 - spec/integration/fedora3_interface_spec.rb
 - spec/integration/missing_relationships_spec.rb
@@ -244,13 +256,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.4.3
 signing_key: 
 specification_version: 4
 summary: Migrate Hydra-based repository data from Fedora3 to Fedora4
 test_files:
 - spec/fixtures/datastreams/sufia-rb68xc089-characterization.xml
-- spec/fixtures/failed-report.json
 - spec/fixtures/objects/f3-migration-a.xml
 - spec/fixtures/objects/gf-versioned-content.xml
 - spec/fixtures/objects/scholarsphere_5712mc568.xml
@@ -266,7 +277,20 @@ test_files:
 - spec/fixtures/objects/sufia_5m60qr95r.xml
 - spec/fixtures/objects/sufia_5m60qr961.xml
 - spec/fixtures/objects/sufia_5m60qr979.xml
-- spec/fixtures/sample-report.json
+- spec/fixtures/reports/failed/sufia_5m60qr94g.json
+- spec/fixtures/reports/failed/sufia_5m60qr95r.json
+- spec/fixtures/reports/failed/sufia_5m60qr961.json
+- spec/fixtures/reports/failed/sufia_5m60qr979.json
+- spec/fixtures/reports/failed/sufia_rb68xc089.json
+- spec/fixtures/reports/failed/sufia_rb68xc09k.json
+- spec/fixtures/reports/failed/sufia_rb68xc10b.json
+- spec/fixtures/reports/failed/sufia_rb68xc11m.json
+- spec/fixtures/reports/failed/sufia_xp68km39w.json
+- spec/fixtures/reports/sample/scholarsphere_000000000.json
+- spec/fixtures/reports/sample/scholarsphere_000000018.json
+- spec/fixtures/reports/sample/scholarsphere_05741r698.json
+- spec/fixtures/reports/sample/scholarsphere_6395wb555.json
+- spec/fixtures/reports/sample/scholarsphere_x346dm27k.json
 - spec/integration/content_versions_spec.rb
 - spec/integration/fedora3_interface_spec.rb
 - spec/integration/missing_relationships_spec.rb

data/spec/fixtures/failed-report.json

@@ -1,339 +0,0 @@
-{
-  "sufia:5m60qr94g": {
-    "status": false,
-    "object": "Sample object migration failure",
-    "relationships": [
-
-    ]
-  },
-  "sufia:5m60qr95r": {
-    "status": true,
-    "object": {
-      "id": "5m60qr95r",
-      "class": "GenericFile",
-      "content_datastreams": [
-        {
-          "ds": "content",
-          "versions": [
-            {
-              "name": "file2.txt",
-              "mime_type": "text/plain",
-              "original_date": "2015-01-19T21:32:47Z"
-            }
-          ]
-        },
-        {
-          "ds": "thumbnail",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        },
-        {
-          "ds": "characterization",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        }
-      ],
-      "rdf_datastreams": [
-        {
-          "ds": "descMetadata",
-          "status": [
-
-          ]
-        }
-      ],
-      "permissions": [
-        "read_groups = []",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"awead@psu.edu\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-      "http://localhost:8983/fedora/rest/test/5m60qr95r--info:fedora/fedora-system:def/relations-external#isPartOf--http://localhost:8983/fedora/rest/test/5m60qr94g"
-    ]
-  },
-  "sufia:5m60qr961": {
-    "status": true,
-    "object": {
-      "id": "5m60qr961",
-      "class": "GenericFile",
-      "content_datastreams": [
-        {
-          "ds": "content",
-          "versions": [
-            {
-              "name": "file1.txt",
-              "mime_type": "text/plain",
-              "original_date": "2015-01-19T21:32:49Z"
-            }
-          ]
-        },
-        {
-          "ds": "thumbnail",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        },
-        {
-          "ds": "characterization",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        }
-      ],
-      "rdf_datastreams": [
-        {
-          "ds": "descMetadata",
-          "status": [
-
-          ]
-        }
-      ],
-      "permissions": [
-        "read_groups = []",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"awead@psu.edu\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-      "http://localhost:8983/fedora/rest/test/5m60qr961--info:fedora/fedora-system:def/relations-external#isPartOf--http://localhost:8983/fedora/rest/test/5m60qr94g"
-    ]
-  },
-  "sufia:5m60qr979": {
-    "status": true,
-    "object": {
-      "id": "5m60qr979",
-      "class": "Collection",
-      "content_datastreams": [
-
-      ],
-      "rdf_datastreams": [
-        {
-          "ds": "descMetadata",
-          "status": [
-
-          ]
-        }
-      ],
-      "permissions": [
-        "read_groups = [\"public\"]",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"awead@psu.edu\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": "2015-01-19T21:34:23.805Z",
-        "modified": "2015-03-03T18:48:51.16Z"
-      }
-    },
-    "relationships": [
-      "http://localhost:8983/fedora/rest/test/5m60qr979--info:fedora/fedora-system:def/relations-external#hasCollectionMember--http://localhost:8983/fedora/rest/test/5m60qr95r",
-      "http://localhost:8983/fedora/rest/test/5m60qr979--info:fedora/fedora-system:def/relations-external#hasCollectionMember--http://localhost:8983/fedora/rest/test/5m60qr961"
-    ]
-  },
-  "sufia:rb68xc089": {
-    "status": false,
-    "object": "Sample object migration failure",
-    "relationships": [
-
-    ]
-  },
-  "sufia:rb68xc09k": {
-    "status": true,
-    "object": {
-      "id": "rb68xc09k",
-      "class": "Batch",
-      "content_datastreams": [
-
-      ],
-      "rdf_datastreams": [
-
-      ],
-      "permissions": null,
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-
-    ]
-  },
-  "sufia:rb68xc10b": {
-    "status": true,
-    "object": {
-      "id": "rb68xc10b",
-      "class": "GenericFile",
-      "content_datastreams": [
-        {
-          "ds": "content",
-          "versions": [
-
-          ]
-        },
-        {
-          "ds": "thumbnail",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        },
-        {
-          "ds": "characterization",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        }
-      ],
-      "rdf_datastreams": [
-
-      ],
-      "permissions": [
-        "read_groups = []",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"jilluser@example.com\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-      "http://localhost:8983/fedora/rest/test/rb68xc10b--info:fedora/fedora-system:def/relations-external#isPartOf--http://localhost:8983/fedora/rest/test/rb68xc09k"
-    ]
-  },
-  "sufia:rb68xc11m": {
-    "status": true,
-    "object": {
-      "id": "rb68xc11m",
-      "class": "GenericFile",
-      "content_datastreams": [
-        {
-          "ds": "content",
-          "versions": [
-
-          ]
-        },
-        {
-          "ds": "thumbnail",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        },
-        {
-          "ds": "characterization",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        }
-      ],
-      "rdf_datastreams": [
-
-      ],
-      "permissions": [
-        "read_groups = []",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"otherUser\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-      "http://localhost:8983/fedora/rest/test/rb68xc11m--info:fedora/fedora-system:def/relations-external#isPartOf--http://localhost:8983/fedora/rest/test/rb68xc09k"
-    ]
-  },
-  "sufia:xp68km39w": {
-    "status": true,
-    "object": {
-      "id": "xp68km39w",
-      "class": "GenericFile",
-      "content_datastreams": [
-        {
-          "ds": "content",
-          "versions": [
-
-          ]
-        },
-        {
-          "ds": "thumbnail",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        },
-        {
-          "ds": "characterization",
-          "versions": [
-            {
-              "error": "Nil source -- it's probably defined in the target but not present in the source"
-            }
-          ]
-        }
-      ],
-      "rdf_datastreams": [
-        {
-          "ds": "descMetadata",
-          "status": [
-
-          ]
-        }
-      ],
-      "permissions": [
-        "read_groups = []",
-        "edit_groups = []",
-        "discover_groups = []",
-        "read_users = []",
-        "edit_users = [\"awead@psu.edu\"]",
-        "discover_users = []"
-      ],
-      "dates": {
-        "uploaded": null,
-        "modified": null
-      }
-    },
-    "relationships": [
-
-    ]
-  }
-}