enju_library 0.2.0.beta.1 → 0.2.0.beta.2

data/spec/fixtures/user_export_files.yml

@@ -0,0 +1,25 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+
+user_export_file_00001:
+  id: 1
+  user_id: 1
+user_export_file_00002:
+  id: 2
+  user_id: 1
+user_export_file_00003:
+  id: 3
+  user_id: 1
+
+# == Schema Information
+#
+# Table name: user_export_files
+#
+#  id                   :integer          not null, primary key
+#  user_id              :integer
+#  executed_at          :datetime
+#  created_at           :datetime
+#  updated_at           :datetime
+#  user_export_id       :string
+#  user_export_size     :integer
+#  user_import_filename :string
+#

data/spec/fixtures/user_groups.yml

@@ -29,3 +29,22 @@ user_group_00003:
   created_at: 2007-12-23 03:15:05.126457 +09:00
   position: 3
   valid_period_for_new_user: 730
+
+# == Schema Information
+#
+# Table name: user_groups
+#
+#  id                               :integer          not null, primary key
+#  name                             :string
+#  display_name                     :text
+#  note                             :text
+#  position                         :integer
+#  created_at                       :datetime
+#  updated_at                       :datetime
+#  deleted_at                       :datetime
+#  valid_period_for_new_user        :integer          default(0), not null
+#  expired_at                       :datetime
+#  number_of_day_to_notify_overdue  :integer          default(1), not null
+#  number_of_day_to_notify_due_date :integer          default(7), not null
+#  number_of_time_to_notify_overdue :integer          default(3), not null
+#

data/spec/fixtures/user_import_files.yml

@@ -0,0 +1,51 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  id: 1
+  user_id: 1
+  note: MyText
+  executed_at: 2014-01-10 21:22:16
+  user_import_file_name: MyString
+  user_import_content_type: text/csv
+  user_import_file_size: 1
+  user_import_updated_at: 2014-01-10 21:22:16
+  user_import_fingerprint: MyString
+  edit_mode: MyString
+  error_message: MyText
+
+two:
+  id: 2
+  user_id: 1
+  note: MyText
+  executed_at: 2014-01-10 21:22:16
+  user_import_file_name: MyString
+  user_import_content_type: text/csv
+  user_import_file_size: 1
+  user_import_updated_at: 2014-01-10 21:22:16
+  user_import_fingerprint: MyString
+  edit_mode: MyString
+  error_message: MyText
+
+# == Schema Information
+#
+# Table name: user_import_files
+#
+#  id                       :integer          not null, primary key
+#  user_id                  :integer
+#  note                     :text
+#  executed_at              :datetime
+#  user_import_file_name    :string
+#  user_import_content_type :string
+#  user_import_file_size    :integer
+#  user_import_updated_at   :datetime
+#  user_import_fingerprint  :string
+#  edit_mode                :string
+#  error_message            :text
+#  created_at               :datetime
+#  updated_at               :datetime
+#  user_encoding            :string
+#  default_library_id       :integer
+#  default_user_group_id    :integer
+#  user_import_id           :string
+#  user_import_size         :integer
+#

data/spec/fixtures/user_import_results.yml

@@ -0,0 +1,26 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  id: 1
+  user_import_file_id: 1
+  user_id: 1
+  body: MyText
+
+two:
+  id: 2
+  user_import_file_id: 1
+  user_id: 2
+  body: MyText
+
+# == Schema Information
+#
+# Table name: user_import_results
+#
+#  id                  :integer          not null, primary key
+#  user_import_file_id :integer
+#  user_id             :integer
+#  body                :text
+#  created_at          :datetime
+#  updated_at          :datetime
+#  error_message       :text
+#

data/spec/fixtures/user_reserve_stats.yml

@@ -20,9 +20,8 @@ two:
 #  start_date   :datetime
 #  end_date     :datetime
 #  note         :text
-#  created_at   :datetime         not null
-#  updated_at   :datetime         not null
+#  created_at   :datetime
+#  updated_at   :datetime
 #  started_at   :datetime
 #  completed_at :datetime
 #
-

data/spec/models/user_export_file_spec.rb

@@ -0,0 +1,30 @@
+# -*- encoding: utf-8 -*-
+require 'spec_helper'
+  
+describe UserExportFile do
+  fixtures :all
+  
+  it "should export in background" do
+    message_count = Message.count
+    file = UserExportFile.new
+    file.user = users(:admin)
+    file.save
+    UserExportFileJob.perform_later(file).should be_truthy
+    Message.count.should eq message_count + 1
+    Message.order(:id).last.subject.should eq 'エクスポートが完了しました'
+  end
+end
+
+# == Schema Information
+#
+# Table name: user_export_files
+#
+#  id                   :integer          not null, primary key
+#  user_id              :integer
+#  executed_at          :datetime
+#  created_at           :datetime
+#  updated_at           :datetime
+#  user_export_id       :string
+#  user_export_size     :integer
+#  user_import_filename :string
+#

data/spec/models/user_group_spec.rb

@@ -0,0 +1,37 @@
+# -*- encoding: utf-8 -*-
+require 'spec_helper'
+
+describe UserGroup do
+  fixtures :user_groups
+
+  it "should contain string in its display_name" do
+    user_group = user_groups(:user_group_00001)
+    user_group.display_name = "en:test"
+    user_group.valid?.should be_truthy
+  end
+
+  it "should not contain invalid yaml in its display_name" do
+    user_group = user_groups(:user_group_00001)
+    user_group.display_name = "en:test\r\nja: テスト"
+    user_group.valid?.should be_falsy
+  end
+end
+
+# == Schema Information
+#
+# Table name: user_groups
+#
+#  id                               :integer          not null, primary key
+#  name                             :string
+#  display_name                     :text
+#  note                             :text
+#  position                         :integer
+#  created_at                       :datetime
+#  updated_at                       :datetime
+#  deleted_at                       :datetime
+#  valid_period_for_new_user        :integer          default(0), not null
+#  expired_at                       :datetime
+#  number_of_day_to_notify_overdue  :integer          default(1), not null
+#  number_of_day_to_notify_due_date :integer          default(7), not null
+#  number_of_time_to_notify_overdue :integer          default(3), not null
+#

data/spec/models/user_import_file_spec.rb

@@ -0,0 +1,220 @@
+# -*- encoding: utf-8 -*-
+require 'rails_helper'
+
+describe UserImportFile do
+  fixtures :all
+
+  describe "when its mode is 'create'" do
+    before(:each) do
+      @file = UserImportFile.new user_import: File.new("#{Rails.root}/../../examples/user_import_file_sample.tsv")
+      @file.default_user_group = UserGroup.find(2)
+      @file.default_library = Library.find(3)
+      @file.user = users(:admin)
+      @file.save
+    end
+
+    it "should be imported" do
+      old_users_count = User.count
+      old_import_results_count = UserImportResult.count
+      @file.current_state.should eq 'pending'
+      @file.import_start.should eq({:user_imported => 5, :user_found => 0, :failed => 0, error: 3})
+      User.order('id DESC')[1].username.should eq 'user005'
+      User.order('id DESC')[2].username.should eq 'user003'
+      User.count.should eq old_users_count + 5
+
+      user001 = User.where(username: 'user001').first
+      user001.profile.keyword_list.should eq "日本史\n地理"
+      user001.profile.full_name.should eq '田辺 浩介'
+      user001.profile.full_name_transcription.should eq 'たなべ こうすけ'
+      user001.profile.required_role.name.should eq 'User'
+      user001.locked_at.should be_truthy
+
+      user002 = User.where(username: 'user002').first
+      user002.profile.user_group.name.should eq 'faculty'
+      user002.profile.expired_at.to_i.should eq Time.zone.parse('2013-12-01').end_of_day.to_i
+      user002.valid_password?('4NsxXPLy')
+      user002.profile.user_number.should eq '001002'
+      user002.profile.library.name.should eq 'hachioji'
+      user002.profile.locale.should eq 'en'
+      user002.profile.required_role.name.should eq 'Librarian'
+      user002.locked_at.should be_nil
+
+      user003 = User.where(username: 'user003').first
+      user003.profile.note.should eq 'テストユーザ'
+      user003.role.name.should eq 'Librarian'
+      user003.profile.user_number.should eq '001003'
+      user003.profile.library.name.should eq 'kamata'
+      user003.profile.locale.should eq 'ja'
+      user003.profile.checkout_icalendar_token.should eq 'secrettoken'
+      user003.profile.save_checkout_history.should be_truthy
+      user003.profile.save_search_history.should be_falsy
+      user003.profile.share_bookmarks.should be_falsy
+      User.where(username: 'user000').first.should be_nil
+      UserImportResult.count.should eq old_import_results_count + 10
+      UserImportResult.order('id DESC')[0].error_message.should eq 'line 10: 利用者番号はすでに存在します。'
+      UserImportResult.order('id DESC')[1].error_message.should eq 'line 9: 利用者番号は不正な値です。'
+      UserImportResult.order('id DESC')[2].error_message.should eq 'line 8: パスワードは6文字以上で入力してください。'
+
+      user005 = User.where(username: 'user005').first
+      user005.role.name.should eq 'User'
+      user005.profile.library.name.should eq 'hachioji'
+      user005.profile.locale.should eq 'en'
+      user005.profile.user_number.should eq '001005'
+      user005.profile.user_group.name.should eq 'faculty'
+
+      user006 = User.where(username: 'user006').first
+      user006.role.name.should eq 'User'
+      user006.profile.library.name.should eq 'hachioji'
+      user006.profile.locale.should eq 'en'
+      user006.profile.user_number.should be_nil
+      user006.profile.user_group.name.should eq UserGroup.find(2).name
+
+      @file.user_import_fingerprint.should be_truthy
+      @file.executed_at.should be_truthy
+
+      @file.reload
+      @file.error_message.should eq "次の列は無視されました。 save_search_history, share_bookmarks, invalid\nline 8: パスワードは6文字以上で入力してください。\nline 9: 利用者番号は不正な値です。\nline 10: 利用者番号はすでに存在します。"
+      @file.current_state.should eq 'failed'
+    end
+
+    it "should send message when import is completed" do
+      old_message_count = Message.count
+      @file.user = User.where(username: 'librarian1').first
+      @file.import_start
+      Message.count.should eq old_message_count + 1
+      Message.order(:id).last.subject.should eq 'インポートが完了しました'
+    end
+
+    it "should not import users that have higher roles than current user's role" do
+      old_users_count = User.count
+      old_import_results_count = UserImportResult.count
+      @file.user = User.where(username: 'librarian1').first
+      @file.import_start.should eq({:user_imported => 4, :user_found => 0, :failed => 1, error: 3})
+      User.order('id DESC')[1].username.should eq 'user005'
+      User.count.should eq old_users_count + 4
+      UserImportResult.count.should eq old_import_results_count + 10
+    end
+  end
+
+  describe "when its mode is 'update'" do
+    before(:each) do
+      FactoryGirl.create(:user,
+        username: 'user001',
+        profile: FactoryGirl.create(:profile)
+      )
+    end
+    it "should update users" do
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_update_file.tsv"), user: users(:admin)
+      old_message_count = Message.count
+      result = @file.modify
+      result.should have_key(:user_updated)
+      user001 = User.where(username: 'user001').first
+      user001.email.should eq 'user001@example.jp'
+      user001.profile.full_name.should eq '田辺 浩介'
+      user001.profile.full_name_transcription.should eq 'たなべこうすけ'
+      user001.profile.user_number.should eq 'user_number_1'
+      user001.profile.note.should eq 'test'
+      user001.profile.keyword_list.should eq 'keyword1 keyword2'
+      Message.count.should eq old_message_count + 1
+    end
+
+    it "should not overwrite with null value" do
+      user = User.where(username: 'user001').first
+      user.profile = FactoryGirl.create(:profile,
+        user_number: '001',
+        full_name: 'User 001',
+        full_name_transcription: 'User 001',
+        locale: 'ja',
+        note: 'Note',
+        keyword_list: 'keyword1 keyword2',
+        date_of_birth: 10.years.ago)
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_update_file2.tsv"), user: users(:admin)
+      result = @file.modify
+      result.should have_key(:user_updated)
+      user001 = User.find('user001')
+      user001.email.should eq 'user001@example.jp'
+      user001.profile.user_number.should eq '001'
+      user001.profile.full_name.should eq 'User 001'
+      user001.profile.full_name_transcription.should eq 'User 001'
+      user001.profile.keyword_list.should eq 'keyword1 keyword2'
+    end
+    it "should update user_number" do
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_update_file3.tsv"), user: users(:admin)
+      result = @file.modify
+      result.should have_key(:user_updated)
+      user001 = User.where(username: 'user001').first
+      user001.profile.user_number.should eq '0001'
+    end
+    it "should update user's lock status" do
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_update_file4.tsv"), user: users(:admin)
+      result = @file.modify
+      result.should have_key(:user_updated)
+      user001 = User.where(username: 'user001').first
+      user001.access_locked?.should be_truthy
+    end
+  end
+
+  describe "when its mode is 'destroy'" do
+    before(:each) do
+      @file = UserImportFile.new user_import: File.new("#{Rails.root}/../../examples/user_import_file_sample.tsv"), user: users(:admin)
+      @file.user = users(:admin)
+      @file.default_user_group = UserGroup.find(2)
+      @file.default_library = Library.find(3)
+      @file.save
+      @file.import_start
+    end
+
+    it "should remove users" do
+      old_count = User.count
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_delete_file.tsv"), user: users(:admin)
+      @file.user = users(:admin)
+      old_message_count = Message.count
+      @file.remove
+      User.count.should eq old_count - 2
+      Message.count.should eq old_message_count + 1
+    end
+
+    it "should not remove users if there are checkouts" do
+      user001 = User.where(username: 'user001').first
+      FactoryGirl.create(:checkout, user: user001, item: FactoryGirl.create(:item))
+      old_count = User.count
+      @file = UserImportFile.create user_import: File.new("#{Rails.root}/../../examples/user_delete_file.tsv"), user: users(:admin)
+      @file.remove
+      User.where(username: 'user001').should_not be_blank
+      User.count.should eq old_count - 2
+    end
+  end
+
+  it "should import in background" do
+    file = UserImportFile.new user_import: File.new("#{Rails.root}/../../examples/user_import_file_sample.tsv"), user: users(:admin)
+    file.user = users(:admin)
+    file.default_user_group = UserGroup.find(2)
+    file.default_library = Library.find(3)
+    file.save
+    UserImportFileJob.perform_later(file).should be_truthy
+  end
+end
+
+# == Schema Information
+#
+# Table name: user_import_files
+#
+#  id                       :integer          not null, primary key
+#  user_id                  :integer
+#  note                     :text
+#  executed_at              :datetime
+#  user_import_file_name    :string
+#  user_import_content_type :string
+#  user_import_file_size    :integer
+#  user_import_updated_at   :datetime
+#  user_import_fingerprint  :string
+#  edit_mode                :string
+#  error_message            :text
+#  created_at               :datetime
+#  updated_at               :datetime
+#  user_encoding            :string
+#  default_library_id       :integer
+#  default_user_group_id    :integer
+#  user_import_id           :string
+#  user_import_size         :integer
+#

data/spec/models/user_import_result_spec.rb

@@ -0,0 +1,20 @@
+# -*- encoding: utf-8 -*-
+require 'spec_helper'
+
+describe UserImportResult do
+  #pending "add some examples to (or delete) #{__FILE__}"
+
+end
+
+# == Schema Information
+#
+# Table name: user_import_results
+#
+#  id                  :integer          not null, primary key
+#  user_import_file_id :integer
+#  user_id             :integer
+#  body                :text
+#  created_at          :datetime
+#  updated_at          :datetime
+#  error_message       :text
+#

data/spec/support/devise.rb

@@ -1,4 +1,4 @@
 RSpec.configure do |config|
-  config.include Devise::TestHelpers, :type => :controller
-  config.include Devise::TestHelpers, :type => :view
+  config.include Devise::Test::ControllerHelpers, type: :controller
+  config.include Devise::Test::ControllerHelpers, type: :view
 end

data/spec/views/user_export_files/new.html.erb_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe "user_export_files/new" do
+  before(:each) do
+    assign(:user_export_file, stub_model(
+      UserExportFile
+    ).as_new_record)
+    view.stub(:current_user).and_return(User.find('enjuadmin'))
+  end
+
+  it "renders new user form" do
+    render
+
+    # Run the generator again with the --webrat flag if you want to use webrat matchers
+    assert_select "form", action: user_export_files_path, method: "post" do
+      assert_select "input", name: "commit"
+    end
+  end
+end

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

@@ -0,0 +1,29 @@
+require 'rails_helper'
+
+describe "user_import_results/index" do
+  fixtures :all
+
+  before(:each) do
+    assign(:user_import_results, UserImportResult.page(1))
+    admin = User.find('enjuadmin')
+    view.stub(:current_user).and_return(admin)
+  end
+
+  it "renders a list of user_import_results" do
+    render
+    expect(rendered).to match /MyString/
+  end
+
+  context "with @user_import_file" do
+    before(:each) do
+      @user_import_file = UserImportFile.find(1)
+      @user_import_results = UserImportResult.where(user_import_file_id: 1).page(1)
+    end
+    it "renders a list of user_import_results for the user_import_file" do
+      render
+      expect(view).to render_template "user_import_results/_list_lines"
+      expect(rendered).to match /MyString/
+      expect(rendered).to match /<td>1<\/td>/
+    end
+  end
+end