enju_biblio 0.1.0.pre69 → 0.1.0.pre70

data/spec/models/manifestation_spec.rb

@@ -223,6 +223,22 @@ describe Manifestation, :solr => true do
   #  #manifestation.volume_number.should eq 1
   #end
 
+  context ".export" do
+    it "should export a header line" do
+      lines = Manifestation.export
+      (header, *lines) = lines.split(/\r?\n/)
+      header = header.split(/\t/)
+      expect(header.size).to eq lines.first.split(/\t/).size
+      expect(header).to include "manifestation_id"
+      expect(header).to include "manifestation_created_at"
+      expect(header).to include "manifestation_updated_at"
+      expect(header).to include "item_created_at"
+      expect(header).to include "item_updated_at"
+      expect(header).to include "item_id"
+      expect(header).to include "manifestation_identifier"
+    end
+  end
+
   if defined?(EnjuCirculation)
     it "should respond to is_checked_out_by?" do
       manifestations(:manifestation_00001).is_checked_out_by?(users(:admin)).should be_truthy

data/spec/models/resource_export_file_spec.rb

@@ -13,6 +13,25 @@ describe ResourceExportFile do
     Message.count.should eq message_count + 1
     Message.order(:id).last.subject.should eq 'エクスポートが完了しました'
   end
+
+  context "NCID export" do
+    it "should export NCID value" do
+      manifestation = FactoryGirl.create(:manifestation)
+      ncid = IdentifierType.where(name: "ncid").first
+      identifier = FactoryGirl.create(:identifier, identifier_type: ncid, body: "BA91833159")
+      manifestation.identifiers << identifier
+      manifestation.save!
+      export_file = ResourceExportFile.new
+      export_file.user = users(:admin)
+      export_file.save!
+      export_file.export!
+      file = export_file.resource_export
+      expect(file).to be_truthy
+      lines = File.open(file.path).readlines.map(&:chomp)
+      expect(lines.first.split(/\t/)).to include "ncid"
+      expect(lines.last.split(/\t/)).to include "BA91833159"
+    end
+  end
 end
 
 # == Schema Information

data/spec/models/resource_import_file_spec.rb

@@ -131,6 +131,47 @@ describe ResourceImportFile do
         Message.count.should eq old_message_count + 1
         Message.order(:id).last.subject.should eq 'インポートが完了しました'
       end
+
+      it "should be searchable right after the import", solr: true do
+        @file.import_start
+        Manifestation.search{ keywords "10101" }.total.should > 0
+        Manifestation.search{ keywords "10101", :fields => [:item_identifier] }.total.should > 0
+        Manifestation.search{ keywords "item_identifier_sm:10101" }.total.should > 0
+      end
+    end
+
+    describe "ISBN import" do
+      context "with record not found" do
+        it "should record an error message", vcr: true do
+          file = ResourceImportFile.create resource_import: StringIO.new("isbn\n9780007264551"), user: users(:admin)
+          result = file.import_start
+          expect(result[:failed]).to eq 1
+          resource_import_result = file.resource_import_results.last
+          expect(resource_import_result.error_message).not_to be_empty
+        end
+      end
+      context "with ISBN invalid" do
+        it "should record an error message", vcr: true do
+          file = ResourceImportFile.create resource_import: StringIO.new("isbn\n978000726455x"), user: users(:admin)
+          result = file.import_start
+          expect(result[:failed]).to eq 1
+          resource_import_result = file.resource_import_results.last
+          expect(resource_import_result.error_message).not_to be_empty
+        end
+      end
+    end
+
+    describe "NCID import" do
+      it "should import ncid value" do
+        file = ResourceImportFile.create resource_import: StringIO.new("original_title\tncid\noriginal_title_ncid\tBA67656964\n"), user: users(:admin)
+        result = file.import_start
+        expect(result[:manifestation_imported]).to eq 1
+        resource_import_result = file.resource_import_results.last
+        expect(resource_import_result.error_message).to be_blank
+        expect(resource_import_result.manifestation).not_to be_blank
+        manifestation = resource_import_result.manifestation
+        expect(manifestation.identifier_contents(:ncid).first).to eq "BA67656964"
+      end
     end
 
     describe "when it is written in shift_jis" do
@@ -190,6 +231,8 @@ describe ResourceImportFile do
     it "should update items", vcr: true do
       @file = ResourceImportFile.create resource_import: File.new("#{Rails.root.to_s}/../../examples/item_update_file.tsv"), edit_mode: 'update'
       @file.modify
+      expect(@file.resource_import_results.first).to be_truthy
+      expect(@file.resource_import_results.first.body).to match /item_identifier/
       item_00001 = Item.where(item_identifier: '00001').first
       item_00001.manifestation.creators.order('agents.id').collect(&:full_name).should eq ['たなべ', 'こうすけ']
       item_00001.binding_item_identifier.should eq '900001'
@@ -204,6 +247,7 @@ describe ResourceImportFile do
       item_00003.include_supplements.should be_truthy
 
       Item.where(item_identifier: '00004').first.include_supplements.should be_falsy
+      Item.where(item_identifier: '00025').first.call_number.should eq "547|ヤ"
     end
 
     #it "should update series_statement", vcr: true do
@@ -213,6 +257,20 @@ describe ResourceImportFile do
     #  manifestation.reload
     #  manifestation.series_statements.should eq [SeriesStatement.find(2)]
     #end
+
+    describe "NCID import" do
+      it "should import ncid value" do
+        file = ResourceImportFile.create resource_import: StringIO.new("manifestation_id\tncid\n1\tBA67656964\n"), user: users(:admin), edit_mode: 'update'
+        result = file.import_start
+        #expect(result[:manifestation_found]).to eq 1
+        expect(file.error_message).to be_nil
+        resource_import_result = file.resource_import_results.last
+        expect(resource_import_result.error_message).to be_blank
+        expect(resource_import_result.manifestation).not_to be_blank
+        manifestation = resource_import_result.manifestation
+        expect(manifestation.identifier_contents(:ncid).first).to eq "BA67656964"
+      end
+    end
   end
 
   describe "when its mode is 'destroy'" do

data/spec/views/resource_import_files/index.html.erb_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe "resource_import_files/index" do
+  fixtures :all
+
+  before(:each) do
+    assign(:resource_import_files, ResourceImportFile.page(1))
+    admin = User.find('enjuadmin')
+    view.stub(:current_resource).and_return(admin)
+    @ability = EnjuLeaf::Ability.new(admin, '0.0.0.0')
+    @ability.extend(CanCan::Ability)
+    controller.stub(:current_ability) { @ability }
+  end
+
+  it "renders a list of resource_import_files" do
+    render
+    expect(rendered).to match /MyString/
+  end
+end

data/spec/views/resource_import_files/show.html.erb_spec.erb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe "resource_import_files/show" do
+  fixtures :all
+
+  before(:each) do
+    file = resource_import_files(:resource_import_file_00001)
+    assign(:resource_import_file, file)
+    assign(:resource_import_results, 
+      Kaminari.paginate_array(file.resource_import_results).page(1))
+    admin = User.find('enjuadmin')
+    view.stub(:current_user).and_return(admin)
+    @ability = EnjuLeaf::Ability.new(admin, '0.0.0.0')
+    @ability.extend(CanCan::Ability)
+    controller.stub(:current_ability) { @ability }
+  end
+
+  it "renders a resource_import_file" do
+    render
+    expect(rendered).to match /MyString/
+  end
+  it "renders a list of resource_import_results" do
+    render
+    expect(rendered).to match /<table/
+    expect(rendered).to render_template "resource_import_files/_results"
+  end
+end

data/spec/views/resource_import_results/index.html.erb_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe "resource_import_results/index" do
+  fixtures :all
+
+  before(:each) do
+    assign(:resource_import_results, ResourceImportResult.page(1))
+    admin = User.find('enjuadmin')
+    view.stub(:current_resource).and_return(admin)
+    @ability = EnjuLeaf::Ability.new(admin, '0.0.0.0')
+    @ability.extend(CanCan::Ability)
+    controller.stub(:current_ability) { @ability }
+  end
+
+  it "renders a list of resource_import_results" do
+    render
+    expect(rendered).to match /MyString/
+  end
+
+  context "with @resource_import_file" do
+    before(:each) do
+      @resource_import_file = ResourceImportFile.find(1)
+      @resource_import_results = ResourceImportResult.where(resource_import_file_id: 1).page(1)
+    end
+    it "renders a list of resource_import_results for the resource_import_file" do
+      render
+      expect(view).to render_template "resource_import_results/_list_lines"
+      expect(rendered).to match /MyString/
+      expect(rendered).to match /<td>1<\/td>/
+    end
+  end
+end

data/spec/views/resource_import_results/show.html.erb_spec.erb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe "resource_import_results/show" do
+  fixtures :all
+
+  before(:each) do
+    @resource_import_result = resource_import_results(:one)
+    admin = User.find('enjuadmin')
+    view.stub(:current_user).and_return(admin)
+    @ability = EnjuLeaf::Ability.new(admin, '0.0.0.0')
+    @ability.extend(CanCan::Ability)
+    controller.stub(:current_ability) { @ability }
+  end
+
+  it "renders a resource_import_result" do
+    render
+    expect(rendered).to match /MyString/
+  end
+  it "renders even if a resource_import_file is deleted" do
+    #@resource_import_result.resource_import_file = nil
+    @resource_import_result.resource_import_file.delete
+    @resource_import_result.resource_import_file = nil
+    STDERR.puts @resource_import_result.inspect
+    render
+    expect(rendered).to match /MyText/
+  end
+end