robust_excel_ole 0.3.5 → 0.3.6

data/spec/{book_save_spec.rb → book_specs/book_save_spec.rb}

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 
-require File.join(File.dirname(__FILE__), './spec_helper')
+require File.join(File.dirname(__FILE__), './../spec_helper')
 
 
 $VERBOSE = nil
@@ -70,8 +70,8 @@ describe Book do
       it {
         expect {
           @book.save_as(@simple_file)
-        }.to raise_error(IOError,
-                     "Not opened for writing(open with :read_only option)")
+        }.to raise_error(ExcelErrorSave,
+                     "Not opened for writing (opened with :read_only option)")
       }
     end
 
@@ -110,6 +110,155 @@ describe Book do
       end
     end
 
+    context "with blocked by another file" do
+
+      before do
+        @book = Book.open(@simple_file)
+        @book2 = Book.open(@another_simple_file)
+      end
+
+      after do
+        @book.close(:if_unsaved => :forget)
+        @book2.close
+      end
+
+      it "should raise an error with :obstructed => :raise" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        File.exist?(@simple_file_other_path).should be_true
+        expect{
+          @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :raise)
+        }.to raise_error(ExcelErrorSave, /blocked by another workbook: "workbook.xls"/)
+      end
+
+      it "should close the blocking workbook without saving, and save the current workbook with :if_obstructed => :forget" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :forget)
+        @book.should_not be_alive
+        File.exist?(@simple_file_other_path).should be_true
+        new_book = Book.open(@simple_file_other_path)
+        new_book.should be_a Book
+        new_book.close
+      end
+
+      it "should close the blocking workbook without saving even if it is unsaved with :if_obstructed => :forget" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        sheet = @book[0]
+        cell_value = sheet[1,1].value
+        sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        @book.Saved.should be_false
+        @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :forget)
+        @book.should_not be_alive
+        @book2.should be_alive
+        File.exist?(@simple_file_other_path).should be_true
+        new_book = Book.open(@simple_file_other_path)
+        new_book.should be_a Book
+        new_book.close
+        old_book = Book.open(@simple_file)
+        old_sheet = old_book[0]
+        old_sheet[1,1].value.should == cell_value
+        old_book.close
+      end
+
+      it "should save and close the blocking workbook, and save the current workbook with :if_obstructed => :save" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        sheet = @book[0]
+        cell_value = sheet[1,1].value
+        sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        @book.Saved.should be_false
+        @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :save)
+        @book.should_not be_alive
+        @book2.should be_alive
+        File.exist?(@simple_file_other_path).should be_true
+        new_book = Book.open(@simple_file_other_path)
+        new_book.should be_a Book
+        new_book.close
+        old_book = Book.open(@simple_file)
+        old_sheet = old_book[0]
+        old_sheet[1,1].value.should_not == cell_value
+        old_book.close
+      end
+
+      it "should close the blocking workbook if it was saved, and save the current workbook with :if_obstructed => :close_if_saved" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        @book.Saved.should be_true
+        @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :close_if_saved)
+        @book.should_not be_alive
+        @book2.should be_alive
+        File.exist?(@simple_file_other_path).should be_true
+        new_book = Book.open(@simple_file_other_path)
+        new_book.should be_a Book
+        new_book.close
+      end
+
+      it "should raise an error if the blocking workbook was unsaved with :if_obstructed => :close_if_saved" do
+        sheet = @book[0]
+        cell_value = sheet[1,1].value
+        sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        @book.Saved.should be_false      
+        expect{
+          @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :close_if_saved)
+        }.to raise_error(ExcelErrorSave, /blocking workbook is unsaved: "workbook.xls"/)
+      end
+
+      it "should raise an error with an invalid option" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        File.exist?(@simple_file_other_path).should be_true
+        expect{
+          @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :invalid)
+        }.to raise_error(ExcelErrorSave, ":if_obstructed: invalid option: :invalid")
+      end
+
+      it "should raise an error by default" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        File.exist?(@simple_file_other_path).should be_true
+        expect{
+          @book2.save_as(@simple_file_other_path, :if_exists => :overwrite)
+        }.to raise_error(ExcelErrorSave, /blocked by another workbook: "workbook.xls"/)
+      end
+
+      it "should raise an error if the file does not exist and an workbook with the same name and other path exists" do
+        File.delete @simple_file_other_path rescue nil
+        File.exist?(@simple_file_other_path).should be_false
+        expect{
+          @book2.save_as(@simple_file_other_path, :if_exists => :overwrite, :if_obstructed => :raise)
+          }.to raise_error(ExcelErrorSave, /blocked by another workbook: "workbook.xls"/)
+      end
+
+      it "should raise an error if the file exists and an workbook with the same name and other path exists" do
+        File.delete @simple_file_other_path rescue nil
+        File.open(@simple_file_other_path,"w") do | file |
+          file.puts "garbage"
+        end
+        File.exist?(@simple_file_other_path).should be_true
+        expect{
+          @book.save_as(@simple_file_other_path, :if_exists => :raise, :if_obstructed => :raise)
+        }.to raise_error(ExcelErrorSave, /file already exists: "workbook.xls"/)
+      end
+
+    end
+
+
     # options :overwrite, :raise, :excel, no option, invalid option
     possible_displayalerts = [true, false]
     possible_displayalerts.each do |displayalert_value|
@@ -128,9 +277,9 @@ describe Book do
           book_save = Book.open(@simple_save_file, :excel => :new)
           expect{
             @book.save_as(@simple_save_file, :if_exists => :overwrite)
-            }.to raise_error(ExcelErrorSave, "book is open and used in Excel")
+            }.to raise_error(ExcelErrorSave, "workbook is open and used in Excel")
           book_save.close
-        end
+        end        
 
         it "should save to simple_save_file.xls with :if_exists => :overwrite" do
           File.delete @simple_save_file rescue nil
@@ -143,6 +292,18 @@ describe Book do
           new_book.should be_a Book
           new_book.close
         end
+
+        it "should simple save if file name is equal to the old one with :if_exists => :overwrite" do
+          sheet = @book[0]
+          sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+          new_value = sheet[1,1].value
+          @book.save_as(@simple_file, :if_exists => :overwrite)
+          new_book = Book.open(@simple_file)
+          new_sheet = new_book[0]
+          new_sheet[1,1].value.should == new_value
+          new_book.close
+        end
+
         it "should save to 'simple_save_file.xls' with :if_exists => :raise" do
           dirname, basename = File.split(@simple_save_file)
           File.delete @simple_save_file rescue nil
@@ -153,7 +314,7 @@ describe Book do
           booklength = File.size?(@simple_save_file)
           expect {
             @book.save_as(@simple_save_file, :if_exists => :raise)
-            }.to raise_error(ExcelErrorSave, 'book already exists: ' + basename)
+            }.to raise_error(ExcelErrorSave, /file already exists: "workbook_save.xls"/)
           File.exist?(@simple_save_file).should be_true
           File.size?(@simple_save_file).should == booklength
         end
@@ -165,7 +326,7 @@ describe Book do
               file.puts "garbage"
             end
             @garbage_length = File.size?(@simple_save_file)
-            @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
+            @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '../helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
           end
 
           after do
@@ -220,7 +381,7 @@ describe Book do
             @book.workbook.Close
             expect{
               @book.save_as(@simple_save_file, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSaveUnknown)
+              }.to raise_error(ExcelErrorSave, "Workbook is not alive")
             File.exist?(@simple_save_file).should be_true
             File.size?(@simple_save_file).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -238,17 +399,17 @@ describe Book do
           booklength = File.size?(@simple_save_file)
           expect {
             @book.save_as(@simple_save_file)
-            }.to raise_error(ExcelErrorSave, 'book already exists: ' + basename)
+            }.to raise_error(ExcelErrorSave, /file already exists: "workbook_save.xls"/)
           File.exist?(@simple_save_file).should be_true
           File.size?(@simple_save_file).should == booklength
         end
 
-        it "should save to 'simple_save_file.xls' with :if_exists => :invalid_option" do
+        it "should save to 'simple_save_file.xls' with :if_exists => :invalid" do
           File.delete @simple_save_file rescue nil
           @book.save_as(@simple_save_file)
           expect {
-            @book.save_as(@simple_save_file, :if_exists => :invalid_option)
-            }.to raise_error(ExcelErrorSave, ':if_exists: invalid option: invalid_option')
+            @book.save_as(@simple_save_file, :if_exists => :invalid)
+            }.to raise_error(ExcelErrorSave, ':if_exists: invalid option: :invalid')
         end
       end
     end

data/spec/{book_sheet_spec.rb → book_specs/book_sheet_spec.rb}

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 
-require File.join(File.dirname(__FILE__), './spec_helper')
+require File.join(File.dirname(__FILE__), './../spec_helper')
 
 
 $VERBOSE = nil
@@ -45,7 +45,9 @@ describe Book do
 
     context "only first argument" do
       it "should add worksheet" do
-        expect { @book.add_sheet @sheet }.to change{ @book.workbook.Worksheets.Count }.from(3).to(4)
+        @book.workbook.Worksheets.Count == 3
+        @book.add_sheet @sheet
+        @book.workbook.Worksheets.Count == 4
       end
 
       it "should return copyed sheet" do
@@ -104,7 +106,9 @@ describe Book do
 
     context "without argument" do
       it "should add empty sheet" do
-        expect { @book.add_sheet }.to change{ @book.workbook.Worksheets.Count }.from(3).to(4)
+        @book.workbook.Worksheets.Count.should == 3
+        @book.add_sheet
+        @book.workbook.Worksheets.Count.should == 4
       end
 
       it "should return copyed sheet" do
@@ -119,7 +123,7 @@ describe Book do
         @book.add_sheet(@sheet, :as => 'new_sheet')
         expect{
           @book.add_sheet(@sheet, :as => 'new_sheet')
-          }.to raise_error(ExcelErrorSheet, "sheet name already exists")
+          }.to raise_error(ExcelErrorSheet, /sheet name "new_sheet" already exists/)
       end
     end
   end

data/spec/{book_spec.rb → book_specs/book_spec.rb}

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 
-require File.join(File.dirname(__FILE__), './spec_helper')
+require File.join(File.dirname(__FILE__), './../spec_helper')
 
 
 $VERBOSE = nil
@@ -47,17 +47,35 @@ describe Book do
 
   describe "open" do
 
+    context "lift a workbook to a Book object" do
+
+      before do
+        @book = Book.open(@simple_file)
+      end
+
+      after do
+        @book.close
+      end
+
+      it "should yield an identical Book" do
+        workbook = @book.workbook
+        new_book = Book.new(workbook)
+        new_book.should be_a Book
+        new_book.should be_alive
+        new_book.should == @book
+        new_book.filename.should == @book.filename
+        new_book.excel.should == @book.excel
+        new_book.should === @book
+        new_book.close
+      end
+    end
+
     context "with various file formats" do
 
       it "should open linked workbook" do
         book = Book.open(@linked_file, :visible => true)
         book.close
       end
-
-      it "should open xlsm file" do
-        book = Book.open(@simple_file_xlsm, :visible => true)
-        book.close
-      end
     end
 
     context "with standard options" do
@@ -208,7 +226,7 @@ describe Book do
       it "should raise an error, if :if_unsaved is :raise" do
         expect {
           @new_book = Book.open(@simple_file, :if_unsaved => :raise)
-        }.to raise_error(ExcelErrorOpen, "book is already open but not saved (#{File.basename(@simple_file)})")
+        }.to raise_error(ExcelErrorOpen, /workbook is already open but not saved: "workbook.xls"/)
       end
 
       it "should let the book open, if :if_unsaved is :accept" do
@@ -222,7 +240,7 @@ describe Book do
 
       context "with :if_unsaved => :alert" do
         before do
-         @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Office Excel" '  , "w"
+         @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '../helpers/key_sender.rb') + '" "Microsoft Office Excel" '  , "w"
         end
 
         after do
@@ -289,7 +307,7 @@ describe Book do
               if :if_obstructed is :close_if_saved" do
             expect{
               @new_book = Book.open(@simple_file, :if_obstructed => :close_if_saved)
-            }.to raise_error(ExcelErrorOpen, "book with the same name in a different path is unsaved: workbook.xls")
+            }.to raise_error(ExcelErrorOpen, /workbook with the same name in a different path is unsaved: "workbook.xls"/)
             @book.save
             @new_book = Book.open(@simple_file, :if_obstructed => :close_if_saved)
             @book.should_not be_alive
@@ -317,7 +335,7 @@ describe Book do
         File.delete @simple_save_file rescue nil
         expect {
           Book.open(@simple_save_file)
-        }.to raise_error(ExcelErrorOpen, "file #{@simple_save_file} not found")
+        }.to raise_error(ExcelErrorOpen, /file "#{@simple_save_file}" not found/)
       end
     end
 
@@ -375,9 +393,10 @@ describe Book do
     end
   end
 
-  describe "send methods to workbook" do
+  describe "uplifting" do
 
     context "with standard" do
+
       before do
         @book = Book.open(@simple_file)
       end
@@ -386,20 +405,429 @@ describe Book do
         @book.close
       end
 
-      it "should send Saved to workbook" do
+      it "should uplift a workbook to a book with an open book" do
+        workbook = @book.workbook
+        book1 = Book.new(workbook)
+        book1.should be_a Book
+        book1.should be_alive
+        book1.should == @book
+      end
+    end
+  end
+
+  describe "unobtrusively" do
+
+    def unobtrusively_ok? # :nodoc: #
+      Book.unobtrusively(@simple_file) do |book|
+        book.should be_a Book
+        sheet = book[0]
+        sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        book.should be_alive
+        book.Saved.should be_false
+      end
+    end
+
+    context "with no open book" do
+
+      it "should open unobtrusively in a new Excel" do
+        expect{ unobtrusively_ok? }.to_not raise_error
+      end
+    end
+
+    context "with two running excel instances" do
+      before :all do
+        Excel.close_all
+      end
+
+      before do
+        @excel1 = Excel.new(:reuse => false)
+        @excel2 = Excel.new(:reuse => false)
+      end
+
+      after do
+        #Excel.close_all
+        begin
+          @excel1.close
+          @excel2.close 
+        rescue ExcelErrorOpen => msg
+          puts "ExcelError: #{msg.message}" if msg.message =~ /Excel instance not alive or damaged/
+        end
+      end
+
+      it "should open unobtrusively in a new Excel" do
+        Book.unobtrusively(@simple_file, :hidden) do |book|
+          book.should be_a Book
+          book.should be_alive
+            book.excel.should_not == @excel1
+            book.excel.should_not == @excel2
+          end
+      end
+
+      it "should open unobtrusively in a given Excel" do
+        Book.unobtrusively(@simple_file, @excel2) do |book|
+          book.should be_a Book
+          book.should be_alive
+            book.excel.should_not == @excel1
+            book.excel.should     == @excel2
+        end
+      end
+    end
+  
+    context "with an open book" do
+
+      before do
+        @book = Book.open(@simple_file)
+      end
+
+      after do
+        @book.close(:if_unsaved => :forget)
+        @book2.close(:if_unsaved => :forget) rescue nil
+      end
+
+      it "should let an open Book open if two books have been opened and one has been closed and opened again" do
+        book2 = Book.open(@different_file, :force_excel => :new)
+        @book.close
+        book2.close
+        @book.reopen
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should == @book.excel
+        end        
+        @book.should be_alive
+        @book.should be_a Book
+      end
+
+      it "should let a saved book saved" do
+        @book.Saved.should be_true
+        @book.should be_alive
+        sheet = @book[0]
+        old_cell_value = sheet[1,1].value
+        unobtrusively_ok?
         @book.Saved.should be_true
+        @book.should be_alive
+        sheet = @book[0]
+        sheet[1,1].value.should_not == old_cell_value
       end
 
-      it "should send Fullname to workbook" do
-        @book.Fullname.tr('\\','/').should == @simple_file
+     it "should let the unsaved book unsaved" do
+        sheet = @book[0]
+        sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        old_cell_value = sheet[1,1].value
+        @book.Saved.should be_false
+        unobtrusively_ok?
+        @book.should be_alive
+        @book.Saved.should be_false
+        @book.close(:if_unsaved => :forget)
+        @book2 = Book.open(@simple_file)
+        sheet2 = @book2[0]
+        sheet2[1,1].value.should_not == old_cell_value
+      end
+    end
+    
+    context "with a closed book" do
+      
+      before do
+        @book = Book.open(@simple_file)
+      end
+
+      after do
+        @book.close(:if_unsaved => :forget)
+      end
+
+      it "should let the closed book closed by default" do
+        sheet = @book[0]
+        old_cell_value = sheet[1,1].value
+        @book.close
+        @book.should_not be_alive
+        unobtrusively_ok?
+        @book.should_not be_alive
+        @book = Book.open(@simple_file)
+        sheet = @book[0]
+        sheet[1,1].Value.should_not == old_cell_value
+      end
+
+      # The bold reanimation of the @book
+      it "should use the excel of the book and keep open the book" do
+        excel = Excel.new(:reuse => false)
+        sheet = @book[0]
+        old_cell_value = sheet[1,1].value
+        @book.close
+        @book.should_not be_alive
+        Book.unobtrusively(@simple_file, :keep_open => true) do |book|
+          book.should be_a Book
+          book.excel.should == @book.excel
+          book.excel.should_not == excel
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.Saved.should be_false
+        end
+        @book.should be_alive
+        @book.close
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[1,1].value.should_not == old_cell_value
+      end
+
+      # book shall be reanimated even with :hidden
+      it "should use the excel of the book and keep open the book" do
+        excel = Excel.new(:reuse => false)
+        sheet = @book[0]
+        old_cell_value = sheet[1,1].value
+        @book.close
+        @book.should_not be_alive
+        Book.unobtrusively(@simple_file, :hidden) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should_not == @book.excel
+          book.excel.should_not == excel
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.Saved.should be_false
+        end
+        @book.should_not be_alive
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[1,1].value.should_not == old_cell_value
+      end
+    end
+
+    context "with a visible book" do
+
+      before do
+        @book = Book.open(@simple_file, :visible => true)
+      end
+
+      after do
+        @book.close(:if_unsaved => :forget)
+        @book2.close(:if_unsaved => :forget) rescue nil
+      end
+
+      it "should let an open Book open" do
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should == @book.excel
+          book.excel.Visible.should be_true
+        end        
+        @book.should be_alive
+        @book.should be_a Book
+        @book.excel.Visible.should be_true
+      end
+    end
+
+    context "with a read_only book" do
+
+      before do
+        @book = Book.open(@simple_file, :read_only => true)
+      end
+
+      after do
+        @book.close
+      end
+
+      it "should open unobtrusively the book in a new Excel such that the book is writable" do
+        book2 = Book.open(@simple_file, :force_excel => :new, :read_only => true)
+        @book.ReadOnly.should be_true
+        book2.Readonly.should be_true
+        sheet = @book[0]
+        cell_value = sheet[1,1].value
+        Book.unobtrusively(@simple_file, :hidden) do |book|
+          book.should be_a Book
+          book.excel.should_not == book2.excel
+          book.excel.should_not == @book.excel
+          sheet = book[0]
+          sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+          book.should be_alive
+          book.Saved.should be_false          
+        end  
+        @book.Saved.should be_true
+        @book.ReadOnly.should be_true
+        @book.close
+        book2.close
+        book3 = Book.open(@simple_file)
+        new_sheet = book3[0]
+        new_sheet[1,1].value.should_not == cell_value
+        book3.close
+      end
+    end
+
+    context "with a virgin Book class" do
+      before do
+        class Book
+          @@bookstore = nil
+        end
+      end
+      it "should work" do
+        expect{ unobtrusively_ok? }.to_not raise_error
+      end
+    end
+
+    context "with a book never opened before" do
+      before do
+        class Book
+          @@bookstore = nil
+        end
+        other_book = Book.open(@different_file)
+      end
+      it "should open the book" do
+        expect{ unobtrusively_ok? }.to_not raise_error
+      end
+    end
+
+    context "with block result" do
+      before do
+        @book1 = Book.open(@simple_file)
+      end
+
+      after do
+        @book1.close(:if_unsaved => :forget)
+      end      
+
+      it "should yield the block result true" do
+        result = 
+          Book.unobtrusively(@simple_file) do |book| 
+            @book1.Saved.should be_true
+          end
+        result.should == true
+      end
+    end
+
+    context "with several Excel instances" do
+
+      before do
+        @book1 = Book.open(@simple_file)
+        @book2 = Book.open(@simple_file, :force_excel => :new)
+        @book1.Readonly.should == false
+        @book2.Readonly.should == true
+        old_sheet = @book1[0]
+        @old_cell_value = old_sheet[1,1].value
+        @book1.close
+        @book2.close
+        @book1.should_not be_alive
+        @book2.should_not be_alive
+      end
+
+      it "should open unobtrusively the closed book in the most recent Excel where it was open before" do      
+        Book.unobtrusively(@simple_file) do |book| 
+          book.excel.should == @book2.excel
+          book.excel.should_not == @book1.excel
+          book.ReadOnly.should == false
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.Saved.should be_false
+        end
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[1,1].value.should_not == @old_cell_value
+      end
+
+      it "should open unobtrusively the closed book in the new hidden Excel" do
+        Book.unobtrusively(@simple_file, :hidden) do |book| 
+          book.excel.should_not == @book2.excel
+          book.excel.should_not == @book1.excel
+          book.ReadOnly.should == false
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.Saved.should be_false
+        end
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[1,1].Value.should_not == @old_cell_value
       end
     end
-  end
 
-  describe "hidden_excel" do
+    context "with :hidden" do
+
+      before do
+        @book1 = Book.open(@simple_file)
+        @book1.close
+      end
     
-    context "with some open book" do
+      it "should create a new hidden Excel instance and use this afterwards" do
+        hidden_excel = nil
+        Book.unobtrusively(@simple_file, :hidden) do |book| 
+          book.should be_a Book
+          book.should be_alive
+          book.excel.Visible.should be_false
+          book.excel.DisplayAlerts.should be_false
+          hidden_excel = book.excel
+        end
+        Book.unobtrusively(@different_file, :hidden) do |book| 
+          book.should be_a Book
+          book.should be_alive
+          book.excel.Visible.should be_false
+          book.excel.DisplayAlerts.should be_false
+          book.excel.should == hidden_excel
+        end
+      end
+    end
+  end
+  
+  describe "for_reading, for_modifying" do
+
+    context "open unobtrusively for reading and modifying" do
+
+      before do
+        @book = Book.open(@simple_file)
+        sheet = @book[0]
+        @old_cell_value = sheet[1,1].value
+        @book.close
+      end
+
+      it "should not change the value" do
+        Book.for_reading(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.Saved.should be_true  
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.Saved.should be_false
+          book.excel.should == @book.excel
+        end
+        new_book = Book.open(@simple_file, :visible => true)
+        sheet = new_book[0]
+        sheet[1,1].Value.should == @old_cell_value
+      end
+
+      it "should not change the value and use the hidden Excel instance" do
+        new_excel = Excel.new(:reuse => false)
+        Book.for_reading(@simple_file, :hidden) do |book|
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.excel.should_not == @book.excel
+          book.excel.should_not == new_excel
+          book.excel.visible.should be_false
+          book.excel.displayalerts.should be_false
+        end
+        new_book = Book.open(@simple_file, :visible => true)
+        sheet = new_book[0]
+        sheet[1,1].Value.should == @old_cell_value
+      end
+
+      it "should change the value" do
+        Book.for_modifying(@simple_file) do |book|
+          sheet = book[0]
+          cell = sheet[1,1]
+          sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
+          book.excel.should == @book.excel
+        end
+        new_book = Book.open(@simple_file, :visible => true)
+        sheet = new_book[0]
+        sheet[1,1].Value.should_not == @old_cell_value
+      end
+    end
+  end
 
+  describe "send methods to workbook" do
+
+    context "with standard" do
       before do
         @book = Book.open(@simple_file)
       end
@@ -408,12 +836,8 @@ describe Book do
         @book.close
       end
 
-      it "should create and use a hidden Excel instance" do
-        book2 = Book.open(@simple_file, :force_excel => @book.bookstore.hidden_excel)
-        book2.excel.should_not == @book.excel
-        book2.excel.visible.should be_false
-        book2.excel.displayalerts.should be_false
-        book2.close 
+      it "should send Saved to workbook" do
+        @book.Saved.should be_true
       end
     end
   end
@@ -455,12 +879,6 @@ describe Book do
         @book1.close(:if_unsaved => :forget)
       end   
 
-      it "should set value of a range" do
-        @book1.nvalue("new").should == "foo"
-        @book1.set_nvalue("new","bar")
-        @book1.nvalue("new").should == "bar"
-      end
-
       it "should set value of a range" do
         @book1.nvalue("new").should == "foo"
         @book1["new"] = "bar"
@@ -503,15 +921,15 @@ describe Book do
       it "should raise error by default" do
         expect{
           @book.close(:if_unsaved => :raise)
-        }.to raise_error(ExcelErrorClose, "book is unsaved (#{File.basename(@simple_file)})")
+        }.to raise_error(ExcelErrorClose, /workbook is unsaved: "workbook.xls"/)
       end
 
       it "should save the book before close with option :save" do
         ole_workbook = @book.workbook
         excel = @book.excel
-        expect {
-          @book.close(:if_unsaved => :save)
-        }.to change {excel.Workbooks.Count }.by(-1)
+        excel.Workbooks.Count.should == 1
+        @book.close(:if_unsaved => :save)
+        excel.Workbooks.Count.should == 0
         @book.workbook.should == nil
         @book.should_not be_alive
         expect{
@@ -542,23 +960,6 @@ describe Book do
       end
     end
 
-    context "with open with read only" do
-      before do
-        @book = Book.open(@simple_file, :read_only => true)
-      end
-
-      after do
-        @book.close
-      end
-
-      it {
-        expect {
-          @book.save_as(@simple_file)
-        }.to raise_error(IOError,
-                     "Not opened for writing(open with :read_only option)")
-      }
-    end
-
     context "with argument" do
       before do
         Book.open(@simple_file) do |book|
@@ -570,111 +971,6 @@ describe Book do
         File.exist?(@simple_save_file).should be_true
       end
     end
-
-    context "with different extensions" do
-      before do
-        @book = Book.open(@simple_file)
-      end
-
-      after do
-        @book.close
-      end
-
-      possible_extensions = ['xls', 'xlsm', 'xlsx']
-      possible_extensions.each do |extensions_value|
-        it "should save to 'simple_save_file.#{extensions_value}'" do
-          simple_save_file = @dir + '/simple_save_file.' + extensions_value
-          File.delete simple_save_file rescue nil
-          @book.save_as(simple_save_file, :if_exists => :overwrite)
-          File.exist?(simple_save_file).should be_true
-          new_book = Book.open(simple_save_file)
-          new_book.should be_a Book
-          new_book.close
-        end
-      end
-    end
-
-    # options :overwrite, :raise, :excel, no option, invalid option
-    possible_displayalerts = [true, false]
-    possible_displayalerts.each do |displayalert_value|
-      context "with displayalerts=#{displayalert_value}" do
-        before do
-          @book = Book.open(@simple_file, :displayalerts => displayalert_value)
-        end
-
-        after do
-          @book.close
-        end
-
-        it "should raise an error if the book is open" do
-          File.delete @simple_save_file rescue nil
-          FileUtils.copy @simple_file, @simple_save_file
-          book_save = Book.open(@simple_save_file, :excel => :new)
-          expect{
-            @book.save_as(@simple_save_file, :if_exists => :overwrite)
-            }.to raise_error(ExcelErrorSave, "book is open and used in Excel")
-          book_save.close
-        end
-
-        context "with :if_exists => :alert" do
-          before do
-            File.delete @simple_save_file rescue nil
-            File.open(@simple_save_file,"w") do | file |
-              file.puts "garbage"
-            end
-            @garbage_length = File.size?(@simple_save_file)
-            @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
-          end
-
-          after do
-            @key_sender.close
-          end
-
-          it "should save if user answers 'yes'" do
-            # "Yes" is to the left of "No", which is the  default. --> language independent
-            @key_sender.puts "{left}{enter}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
-            @book.save_as(@simple_save_file, :if_exists => :alert)
-            File.exist?(@simple_save_file).should be_true
-            File.size?(@simple_save_file).should > @garbage_length
-            @book.excel.DisplayAlerts.should == displayalert_value
-            new_book = Book.open(@simple_save_file, :excel => :new)
-            new_book.should be_a Book
-            new_book.close
-            @book.excel.DisplayAlerts.should == displayalert_value
-          end
-
-          it "should not save if user answers 'no'" do
-            # Just give the "Enter" key, because "No" is the default. --> language independent
-            # strangely, in the "no" case, the question will sometimes be repeated three times
-            @key_sender.puts "{enter}"
-            @key_sender.puts "{enter}"
-            @key_sender.puts "{enter}"
-            #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
-            expect{
-              @book.save_as(@simple_save_file, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
-            File.exist?(@simple_save_file).should be_true
-            File.size?(@simple_save_file).should == @garbage_length
-            @book.excel.DisplayAlerts.should == displayalert_value
-          end
-
-          it "should not save if user answers 'cancel'" do
-            # 'Cancel' is right from 'yes'
-            # strangely, in the "no" case, the question will sometimes be repeated three times
-            @key_sender.puts "{right}{enter}"
-            @key_sender.puts "{right}{enter}"
-            @key_sender.puts "{right}{enter}"
-            #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
-            expect{
-              @book.save_as(@simple_save_file, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
-            File.exist?(@simple_save_file).should be_true
-            File.size?(@simple_save_file).should == @garbage_length
-            @book.excel.DisplayAlerts.should == displayalert_value
-          end
-        end
-      end
-    end
   end
 
   describe "alive?, filename, ==, visible, displayalerts, activate, saved" do
@@ -731,22 +1027,12 @@ describe Book do
         @new_book = Book.open(@simple_file)
         @new_book.should == @book
       end
-
-      it "should be false with two different books" do
-        @new_book = Book.new(@different_file)
-        @new_book.should_not == @book
-      end
-
-      it "should be false with same book names but different paths" do       
-        @new_book = Book.new(@simple_file_other_path, :excel => :new)
-        @new_book.should_not == @book
-      end
     end
 
     context "with activate" do
 
       before do
-        @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Office Excel" '  , "w"        
+        @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '../helpers/key_sender.rb') + '" "Microsoft Office Excel" '  , "w"        
         @book = Book.open(@simple_file, :visible => true)
         @book2 = Book.open(@another_simple_file, :force_excel => :new, :visible => true)
       end
@@ -791,7 +1077,10 @@ describe Book do
 
     context "only first argument" do
       it "should add worksheet" do
-        expect { @book.add_sheet @sheet }.to change{ @book.workbook.Worksheets.Count }.from(3).to(4)
+        @book.workbook.Worksheets.Count.should == 3
+        @book.add_sheet @sheet
+        @book.workbook.Worksheets.Count.should == 4
+        #expect { @book.add_sheet @sheet }.to change{ @book.workbook.Worksheets.Count }.from(3).to(4)
       end
 
       it "should return copyed sheet" do
@@ -802,23 +1091,6 @@ describe Book do
     end
 
     context "with first argument" do
-      context "with second argument is {:as => 'copyed_name'}" do
-        it "copyed sheet name should be 'copyed_name'" do
-          @book.add_sheet(@sheet, :as => 'copyed_name').name.should eq 'copyed_name'
-        end
-      end
-
-      context "with second argument is {:before => @sheet}" do
-        it "should add the first sheet" do
-          @book.add_sheet(@sheet, :before => @sheet).name.should eq @book[0].name
-        end
-      end
-
-      context "with second argument is {:after => @sheet}" do
-        it "should add the first sheet" do
-          @book.add_sheet(@sheet, :after => @sheet).name.should eq @book[1].name
-        end
-      end
 
       context "with second argument is {:before => @book[2], :after => @sheet}" do
         it "should arguments in the first is given priority" do
@@ -848,9 +1120,6 @@ describe Book do
     end
 
     context "without argument" do
-      it "should add empty sheet" do
-        expect { @book.add_sheet }.to change{ @book.workbook.Worksheets.Count }.from(3).to(4)
-      end
 
       it "should return copyed sheet" do
         sheet = @book.add_sheet