robust_excel_ole 0.3.1 → 0.3.2

data/spec/book_spec.rb

@@ -19,11 +19,11 @@ describe Book do
 
   before do
     @dir = create_tmpdir
-    @simple_file = @dir + '/simple.xls'
-    @simple_save_file = @dir + '/simple_save.xls'
-    @different_file = @dir + '/different_simple.xls'
-    @simple_file_other_path = @dir + '/more_data/simple.xls'
-    @more_simple_file = @dir + '/more_simple.xls'
+    @simple_file = @dir + '/workbook.xls'
+    @simple_save_file = @dir + '/workbook_save.xls'
+    @different_file = @dir + '/different_workbook.xls'
+    @simple_file_other_path = @dir + '/more_data/workbook.xls'
+    @more_simple_file = @dir + '/more_workbook.xls'
   end
 
   after do
@@ -34,17 +34,25 @@ describe Book do
 
   describe "create file" do
     context "with standard" do
-      it "simple file with default" do
+      it "open an existing file" do
         expect {
           @book = Book.new(@simple_file)
         }.to_not raise_error
         @book.should be_a Book
         @book.close
       end
+
+      it "create a new file" do
+        File.delete @simple_save_file rescue nil
+        Book.create(@simple_save_file)
+        book = Book.open(@simple_save_file, :if_absent => :raise)
+        book.should be_a Book
+        book.close
+        File.exist?(@simple_save_file).should be_true
+      end
     end
   end
   
-
   describe "open" do
 
     context "standard use cases" do
@@ -116,7 +124,7 @@ describe Book do
       end
     end
 
-    context "with transperency identity" do
+    context "with identity transperence" do
 
       before do
         @book = Book.open(@simple_file)
@@ -128,19 +136,19 @@ describe Book do
 
       it "should yield identical Book objects for identical Excel books" do
         book2 = Book.open(@simple_file)
-        book2.should == @book
+        book2.should === @book
         book2.close
       end
 
       it "should yield different Book objects for different Excel books" do
         book2 = Book.open(@different_file)
-        book2.should_not == @book
+        book2.should_not === @book
         book2.close
       end
 
       it "should yield different Book objects when opened the same file in different Excel instances" do
         book2 = Book.open(@simple_file, :force_excel => :new)
-        book2.should_not == @book
+        book2.should_not === @book
         book2.close
       end
 
@@ -148,7 +156,7 @@ describe Book do
         @book.should be_alive
         @book.close
         book2 = Book.open(@simple_file)
-        book2.should == @book
+        book2.should === @book
         book2.should be_alive
         book2.close
       end
@@ -159,7 +167,7 @@ describe Book do
         Excel.close_all
         book2 = Book.open(@simple_file)
         book2.should be_alive
-        book2.should == @book
+        book2.should === @book
         book2.close
       end
 
@@ -169,7 +177,7 @@ describe Book do
         @book.close
         @book.should_not be_alive
         book2 = Book.open(@simple_file, :force_excel => :new)
-        book2.should_not == @book
+        book2.should_not === @book
         book2.should be_alive
         book2.excel.should_not == old_excel
         book2.close
@@ -181,7 +189,7 @@ describe Book do
         @book.close
         @book.should_not be_alive
         book2 = Book.open(@simple_file, :force_excel => new_excel)
-        book2.should_not == @book
+        book2.should_not === @book
         book2.should be_alive
         book2.excel.should == new_excel
         book2.excel.should_not == old_excel
@@ -194,7 +202,7 @@ describe Book do
         @book.close
         @book.should_not be_alive
         book2 = Book.open(@simple_file, :force_excel => old_excel)
-        book2.should == @book
+        book2.should === @book
         book2.should be_alive
         book2.excel.should == old_excel
         @book.should be_alive
@@ -270,6 +278,14 @@ describe Book do
       end
     end
 
+    context "with another :force_excel" do
+      it "should do force_excel even if both force_ and default_excel is given" do
+        book2 = Book.open(@simple_file, :force_excel => nil)
+        book2.should be_alive
+        book2.should be_a Book
+      end
+    end
+
     context "with :default_excel" do
 
       before do
@@ -327,9 +343,9 @@ describe Book do
         book2 = Book.open(@simple_file, :default_excel => :reuse)
         book2.should be_alive
         book2.should be_a Book
-        #book2.excel.should_not == excel
-        #book2.excel.should_not == new_excel2
-        #book2.excel.should == new_excel
+        book2.excel.should_not == excel
+        book2.excel.should_not == new_excel2
+        book2.excel.should == new_excel
         @book.should be_alive
         book2.should == @book
         book2.close
@@ -388,152 +404,6 @@ describe Book do
         book2.excel.should == @book.excel
         book2.should == @book
       end
-
-    end
-
-    context "with :if_locked" do
-
-      context "with an writable book first" do 
-        
-        before do
-          @book = Book.open(@simple_file)
-        end
-
-        after do
-          @book.close
-        end
-        
-        it "should use the already open Excel and book" do
-          @book.ReadOnly.should be_false
-          new_book1 = Book.open(@simple_file, :if_locked => :readonly)
-          new_book1.ReadOnly.should be_true
-          new_book1.excel.should_not == @book.excel
-          new_book1.should_not == @book
-          new_book1.close
-          new_book2 = Book.open(@simple_file, :if_locked => :take_writable)
-          new_book2.ReadOnly.should be_false
-          new_book2.excel == @book.excel
-          new_book2.should == @book
-          new_book2.close
-          new_book3 = Book.open(@simple_file)
-          new_book3.ReadOnly.should be_false
-          new_book3.excel == @book.excel
-          new_book3.should == @book
-          new_book3.close
-        end
-
-        it "should open in a new Excel as read_only" do
-          new_book = Book.open(@simple_file, :force_excel => :new, :if_locked => :readonly)
-          @book.ReadOnly.should be_false
-          new_book.ReadOnly.should be_true
-          new_book.excel.should_not == @book.excel
-          new_book.close
-          new_excel = Excel.new(:reuse => false)
-          new_book2 = Book.open(@simple_file, :force_excel => new_excel, :if_locked => :readonly)
-          new_book2.ReadOnly.should be_true
-          new_book2.excel.should_not == @book.excel
-          new_book2.close
-        end
-
-        it "should open in the Excel where the book is writable" do
-          new_book = Book.open(@simple_file, :force_excel => :new, :if_locked => :take_writable)
-          @book.ReadOnly.should be_false
-          new_book.ReadOnly.should be_false
-          new_book.excel.should == @book.excel
-          new_book.should == @book
-          new_book.close
-          new_excel = Excel.new(:reuse => false)
-          new_book2 = Book.open(@simple_file, :force_excel => new_excel, :if_locked => :take_writable)
-          new_book2.ReadOnly.should be_false
-          new_book2.excel.should == @book.excel
-          new_book2.close
-        end
-
-        it "should open in a new Excel as writable" do
-          new_book = Book.open(@simple_file, :force_excel => :new, :if_locked => :force_writability)
-          @book.ReadOnly.should be_false
-          new_book.ReadOnly.should be_false
-          new_book.excel.should_not == @book.excel
-          new_book.close
-          new_excel = Excel.new(:reuse => false)
-          new_book2 = Book.open(@simple_file, :force_excel => new_excel, :if_locked => :force_writability)
-          new_book2.ReadOnly.should be_false
-          new_book2.excel.should_not == @book.excel
-          new_book2.close
-        end
-
-        it "should open in a new Excel as read_only by default" do
-          new_book = Book.open(@simple_file, :force_excel => :new)
-          @book.ReadOnly.should be_false
-          new_book.ReadOnly.should be_true
-          new_book.excel.should_not == @book.excel
-          new_book.close
-          new_excel = Excel.new(:reuse => false)
-          new_book2 = Book.open(@simple_file, :force_excel => new_excel)
-          new_book2.ReadOnly.should be_true
-          new_book2.excel.should_not == @book.excel
-          new_book2.close
-        end
-      end
-
-      context "with read_only book first" do 
-
-        before do
-          @book = Book.open(@simple_file, :read_only => true)
-        end
-
-        after do
-          @book.close
-        end
-
-        it "should use the already open Excel and book" do
-          @book.ReadOnly.should be_true
-          new_book1 = Book.open(@simple_file, :if_locked => :readonly)
-          new_book1.ReadOnly.should be_true
-          new_book1.excel.should == @book.excel
-          new_book1.should == @book
-          new_book1.close
-          new_book2 = Book.open(@simple_file, :if_locked => :take_writable)
-          new_book2.ReadOnly.should be_true
-          new_book2.excel.should == @book.excel
-          new_book2.should == @book
-          new_book2.close
-          new_book3 = Book.open(@simple_file)
-          new_book3.ReadOnly.should be_true
-          new_book3.excel.should == @book.excel
-          new_book3.should == @book
-          new_book3.close
-        end
-
-        it "should open in the already open Excel" do
-          new_book = Book.open(@simple_file, :if_locked => :force_writability)
-          @book.ReadOnly.should be_true
-          new_book.ReadOnly.should be_false
-          new_book.excel.should_not == @book.excel
-          new_book.should_not == @book
-          new_book.close
-        end
-
-        it "should open in a new Excel as writable" do
-          @book.ReadOnly.should be_true
-          new_book1 = Book.open(@simple_file, :force_excel => :new, :if_locked => :readonly)
-          new_book1.ReadOnly.should be_false
-          new_book1.excel.should_not == @book.excel
-          new_book1.close
-          new_book2 = Book.open(@simple_file, :force_excel => :new, :if_locked => :take_writable)
-          new_book2.ReadOnly.should be_false
-          new_book2.excel.should_not == @book.excel
-          new_book2.close
-          new_book3 = Book.open(@simple_file, :force_excel => :new, :if_locked => :force_writability)
-          new_book3.ReadOnly.should be_false
-          new_book3.excel.should_not == @book.excel
-          new_book3.close
-          new_book4 = Book.open(@simple_file, :force_excel => :new)
-          new_book4.ReadOnly.should be_false
-          new_book4.excel.should_not == @book.excel
-          new_book4.close
-        end
-      end
     end
 
     context "with :if_unsaved" do
@@ -612,13 +482,10 @@ describe Book do
         @new_book.close
       end
 
-     it "should open the book in a new excel instance, if :if_unsaved is default" do
-        @new_book = Book.open(@simple_file)
-        @book.should be_alive
-        @new_book.should be_alive
-        @new_book.filename.should == @book.filename
-        @new_book.excel.should_not == @book.excel       
-        @new_book.close
+      it "should raise an error, if :if_unsaved is default" 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)})")
       end
 
       it "should raise an error, if :if_unsaved is invalid option" do
@@ -696,14 +563,11 @@ describe Book do
             @new_book.excel.should_not == @book.excel
           end
 
-          it "should open the book in a new excel instance, if :if_obstructed is default" do
-            @new_book = Book.open(@simple_file)
-            @book.should be_alive
-            @new_book.should be_alive
-            @new_book.filename.should_not == @book.filename
-            @new_book.excel.should_not == @book.excel
-          end
-         
+          it "should raise an error, if :if_obstructed is default" do
+            expect {
+              @new_book = Book.open(@simple_file)
+            }.to raise_error(ExcelErrorOpen, "blocked by a book with the same name in a different path")
+          end         
 
           it "should raise an error, if :if_obstructed is invalid option" do
             expect {
@@ -746,17 +610,35 @@ describe Book do
       end
     end      
     
-
     context "with non-existing file" do
+
       it "should raise an exception" do
         File.delete @simple_save_file rescue nil
         expect {
-          Book.open(@simple_save_file)
+          Book.open(@simple_save_file, :if_absent => :raise)
         }.to raise_error(ExcelErrorOpen, "file #{@simple_save_file} not found")
       end
+
+      it "should create a workbook" do
+        File.delete @simple_save_file rescue nil
+        book = Book.open(@simple_save_file, :if_absent => :create)
+        book.should be_a Book
+        book.close
+        File.exist?(@simple_save_file).should be_true
+      end
+
+      it "should create a workbook by default" do
+        File.delete @simple_save_file rescue nil
+        book = Book.open(@simple_save_file, :if_absent => :create)
+        book.should be_a Book
+        book.close
+        File.exist?(@simple_save_file).should be_true
+      end
+
     end
 
     context "with attr_reader excel" do
+     
       before do
         @new_book = Book.open(@simple_file)
       end
@@ -772,7 +654,7 @@ describe Book do
 
     context "with :read_only" do
       
-      it "should reopen the book with writable (unsaved changes from readonly are not saved)" do
+      it "should reopen the book with writable (unsaved changes from readonly will not be saved)" do
         book = Book.open(@simple_file, :read_only => true)
         book.ReadOnly.should be_true
         book.should be_alive
@@ -790,7 +672,7 @@ describe Book do
         new_cell_value.should == old_cell_value
       end
 
-      it "should reopen the book with readonly (unsaved changes of the writable should be saved)" do
+      it "should not raise an error when trying to reopen the book as read_only while the writable book had unsaved changes" do
         book = Book.open(@simple_file, :read_only => false)
         book.ReadOnly.should be_false
         book.should be_alive
@@ -799,24 +681,20 @@ describe Book do
         sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
         book.Saved.should be_false
         new_book = Book.open(@simple_file, :read_only => true, :if_unsaved => :accept)
-        new_book.ReadOnly.should be_true
-        new_book.should be_alive
-        book.should be_alive   
-        new_book.should == book 
-        new_sheet = new_book[0]
-        new_cell_value = new_sheet[0,0].value
-        new_cell_value.should_not == old_cell_value
+        new_book.ReadOnly.should be_false
+        new_book.Saved.should be_false
+        new_book.should == book
       end
 
-      it "should reopen the book with writable (unsaved changes from readonly are not saved)" do
-        book = Book.open(@simple_file, :force_excel => :new, :read_only => true)
+      it "should reopen the book with writable in the same Excel instance (unsaved changes from readonly will not be saved)" do
+        book = Book.open(@simple_file, :read_only => true)
         book.ReadOnly.should be_true
         book.should be_alive
         sheet = book[0]
         old_cell_value = sheet[0,0].value
         sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
         book.Saved.should be_false
-        new_book = Book.open(@simple_file, :force_excel => book.excel, :read_only => false, :if_unsaved => :accept)
+        new_book = Book.open(@simple_file, :if_unsaved => :accept, :force_excel => book.excel, :read_only => false)
         new_book.ReadOnly.should be_false 
         new_book.should be_alive
         book.should be_alive   
@@ -835,16 +713,11 @@ describe Book do
         sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
         book.Saved.should be_false
         new_book = Book.open(@simple_file, :force_excel => book.excel, :read_only => true, :if_unsaved => :accept)
-        new_book.ReadOnly.should be_true
-        new_book.should be_alive
-        book.should be_alive   
-        new_book.should == book 
-        new_sheet = new_book[0]
-        new_cell_value = new_sheet[0,0].value
-        new_cell_value.should_not == old_cell_value
+        new_book.ReadOnly.should be_false
+        new_book.Saved.should be_false
+        new_book.should == book
       end
 
-
       it "should open the second book in another Excel as writable" do
         book = Book.open(@simple_file, :read_only => true)
         book.ReadOnly.should be_true
@@ -895,7 +768,7 @@ describe Book do
         path = '~/Abrakadabra.xlsx'
         expected_path = Regexp.new(File.expand_path(path).gsub(/\//, "."))
         expect {
-          Book.open(path)
+          Book.open(path, :if_absent => :raise)
         }.to raise_error(ExcelErrorOpen, "file #{path} not found")
       end
     end
@@ -922,6 +795,28 @@ describe Book do
     end
   end
 
+  describe "hidden_excel" do
+    
+    context "with some open book" do
+
+      before do
+        @book = Book.open(@simple_file)
+      end
+
+      after do
+        @book.close
+      end
+
+      it "should create and use a hidden Excel instance" do
+        book2 = Book.open(@simple_file, :force_excel => @book.book_store.ensure_hidden_excel)
+        book2.excel.should_not == @book.excel
+        book2.excel.Visible.should be_false
+        book2.excel.DisplayAlerts.should be_false
+        book2.close 
+      end
+    end
+  end
+
   describe "unobtrusively" do
 
     def unobtrusively_ok? # :nodoc: #
@@ -943,13 +838,24 @@ describe Book do
       it "should open unobtrusively in the first opened Excel" do
         excel = Excel.new(:reuse => false)
         new_excel = Excel.new(:reuse => false)
-        Book.unobtrusively(@simple_file) do |book|
+        Book.unobtrusively(@simple_file, :if_closed => :reuse) do |book|
           book.should be_a Book
           book.should be_alive
           book.excel.should == excel
           book.excel.should_not == new_excel
         end
       end
+
+      it "should open unobtrusively in a new Excel" do
+        excel = Excel.new(:reuse => false)
+        new_excel = Excel.new(:reuse => false)
+        Book.unobtrusively(@simple_file, :if_closed => :hidden) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should_not == excel
+          book.excel.should_not == new_excel
+        end
+      end
     end
 
     context "with an open book" do
@@ -1030,13 +936,16 @@ describe Book do
       end
 
       # The bold reanimation of the @book
-      it "should keep open the book" do
+      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[0,0].value
         @book.close
         @book.should_not be_alive
-        Book.unobtrusively(@simple_file, :keep_open => true) do |book|
+        Book.unobtrusively(@simple_file, :if_closed => :reuse, :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[0,0]
           sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
@@ -1048,8 +957,77 @@ describe Book do
         sheet = new_book[0]
         sheet[0,0].value.should_not == old_cell_value
       end
+
+      # book shall be reanimated even with if_closed => 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[0,0].value
+        @book.close
+        @book.should_not be_alive
+        Book.unobtrusively(@simple_file) 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[0,0]
+          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
+          book.Saved.should be_false
+        end
+        @book.should_not be_alive
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[0,0].value.should_not == old_cell_value
+      end
+
+      it "should use another excel if the Excels are closed" do
+        excel = Excel.new(:reuse => false)
+        sheet = @book[0]
+        old_cell_value = sheet[0,0].value
+        @book.close
+        @book.should_not be_alive
+        Excel.close_all
+        Book.unobtrusively(@simple_file, :keep_open => true, :if_closed => :reuse) do |book|
+          book.should be_a Book
+          book.excel.should == @book.excel
+          book.excel.should_not == excel
+          sheet = book[0]
+          cell = sheet[0,0]
+          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
+          book.Saved.should be_false
+        end
+        @book.should be_alive
+        @book.close
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[0,0].value.should_not == old_cell_value
+      end
+
+      it "should use another excel if the Excels are closed" do
+        excel = Excel.new(:reuse => false)
+        sheet = @book[0]
+        old_cell_value = sheet[0,0].value
+        @book.close
+        @book.should_not be_alive
+        Excel.close_all
+        Book.unobtrusively(@simple_file, :keep_open => true) do |book|
+          book.should be_a Book
+          book.excel.should_not == @book.excel
+          book.excel.should_not == excel
+          sheet = book[0]
+          cell = sheet[0,0]
+          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
+          book.Saved.should be_false
+        end
+        @book.should_not be_alive
+        new_book = Book.open(@simple_file)
+        sheet = new_book[0]
+        sheet[0,0].value.should_not == old_cell_value
+      end
     end
 
+
     context "with a read_only book" do
 
       before do
@@ -1093,7 +1071,32 @@ describe Book do
         sheet2[0,0].value.should == cell_value        
       end
 
-      it "should modify one of the unobtrusively books" do
+      it "should open unobtrusively by default the writable book" do
+        book2 = Book.open(@simple_file, :force_excel => :new, :read_only => false)
+        @book.ReadOnly.should be_true
+        book2.Readonly.should be_false
+        sheet = @book[0]
+        cell_value = sheet[0,0].value
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.excel.should == book2.excel
+          book.excel.should_not == @book.excel
+          sheet = book[0]
+          sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
+          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[0,0].value.should_not == cell_value
+        book3.close
+      end
+
+      it "should open unobtrusively by default 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
@@ -1101,6 +1104,63 @@ describe Book do
         cell_value = sheet[0,0].value
         Book.unobtrusively(@simple_file) do |book|
           book.should be_a Book
+          book.excel.should_not == book2.excel
+          book.excel.should_not == @book.excel
+          sheet = book[0]
+          sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
+          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[0,0].value.should_not == cell_value
+        book3.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[0,0].value
+        Book.unobtrusively(@simple_file, :use_this => false) do |book|
+          book.should be_a Book
+          book.excel.should_not == book2.excel
+          book.excel.should_not == @book.excel
+          sheet = book[0]
+          sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
+          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[0,0].value.should_not == cell_value
+        book3.close
+      end
+
+      it "should open unobtrusively the book in a new Excel to open the book writable" do
+        excel1 = Excel.new(:reuse => false)
+        excel2 = Excel.new(:reuse => false)
+        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[0,0].value
+        Book.unobtrusively(@simple_file, :use_readonly_excel => false) do |book|
+          book.should be_a Book
+          book.ReadOnly.should be_false
+          book.excel.should_not == book2.excel
+          book.excel.should_not == @book.excel
+          book.excel.should_not == excel1
+          book.excel.should_not == excel2
           sheet = book[0]
           sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
           book.should be_alive
@@ -1115,6 +1175,53 @@ describe Book do
         new_sheet[0,0].value.should_not == cell_value
         book3.close
       end
+
+      it "should open unobtrusively the book in the same Excel to open the book writable" do
+        excel1 = Excel.new(:reuse => false)
+        excel2 = Excel.new(:reuse => false)
+        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[0,0].value
+        Book.unobtrusively(@simple_file, :use_readonly_excel => true) do |book|
+          book.should be_a Book
+          book.excel.should == book2.excel
+          book.ReadOnly.should be_false
+          sheet = book[0]
+          sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
+          book.should be_alive
+          book.Saved.should be_false          
+        end  
+        book2.Saved.should be_true
+        book2.ReadOnly.should be_false
+        @book.close
+        book2.close
+        book3 = Book.open(@simple_file)
+        new_sheet = book3[0]
+        new_sheet[0,0].value.should_not == cell_value
+        book3.close
+      end
+
+      it "should open unobtrusively the book in the Excel where it was opened most recently" 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[0,0].value
+        Book.unobtrusively(@simple_file, :read_only => true) do |book|
+          book.should be_a Book
+          book.excel.should == book2.excel
+          book.excel.should_not == @book.excel
+          book.should be_alive
+          book.Saved.should be_true         
+        end  
+        @book.Saved.should be_true
+        @book.ReadOnly.should be_true
+        @book.close
+        book2.close
+      end
+
     end
 
     context "with a virgin Book class" do
@@ -1162,6 +1269,7 @@ describe Book do
           book.Saved.should be_false
           sleep 1
         end
+        @book1.Saved.should be_true
         m_time2 = File.mtime(@book1.stored_filename)
         m_time2.should_not == m_time
       end      
@@ -1229,83 +1337,76 @@ describe Book do
     end
 
     context "with visible" do
-       before do
-        @book1 = Book.open(@simple_file)
-      end
 
       after do
-        @book1.close(:if_unsaved => :forget)
+        @book1.close
       end    
 
-      it "should unobtrusively use a book invisible" do
+      it "should let the book invisible" do
+        @book1 = Book.open(@simple_file)
+        @book1.excel.Visible.should be_false
+        Book.unobtrusively(@simple_file) do |book| 
+          book.excel.Visible.should be_false
+        end
         @book1.excel.Visible.should be_false
         Book.unobtrusively(@simple_file, :visible => false) do |book| 
-          @book1.excel.Visible.should be_false
           book.excel.Visible.should be_false
-          sheet = book[0]
-          cell = sheet[0,0]
-          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
-          sheet = book[0]
         end
         @book1.excel.Visible.should be_false
       end
       
-      it "should unobtrusively use a book visible" do
+      it "should let the book visible" do
+        @book1 = Book.open(@simple_file, :visible => true)
+        @book1.excel.Visible.should be_true
+        Book.unobtrusively(@simple_file) do |book| 
+          book.excel.Visible.should be_true
+        end
+        @book1.excel.Visible.should be_true
+        Book.unobtrusively(@simple_file, :visible => true) do |book| 
+          book.excel.Visible.should be_true
+        end
+        @book1.excel.Visible.should be_true
+      end
+
+      it "should open the book unobtrusively visible" do
+        @book1 = Book.open(@simple_file)
         @book1.excel.Visible.should be_false
         Book.unobtrusively(@simple_file, :visible => true) do |book| 
-          @book1.excel.Visible.should be_true
           book.excel.Visible.should be_true
-          sheet = book[0]
-          cell = sheet[0,0]
-          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
-          sheet = book[0]
         end
         @book1.excel.Visible.should be_false
       end
 
-      it "should unobtrusively use a book invisibe by default" do
-        @book1.excel.Visible.should be_false
-        Book.unobtrusively(@simple_file) do |book| 
-          @book1.excel.Visible.should be_false
+      it "should open the book unobtrusively invisible" do
+        @book1 = Book.open(@simple_file, :visible => true)
+        @book1.excel.Visible.should be_true
+        Book.unobtrusively(@simple_file, :visible => false) do |book| 
           book.excel.Visible.should be_false
-          sheet = book[0]
-          cell = sheet[0,0]
-          sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
-          sheet = book[0]
         end
-        @book1.excel.Visible.should be_false
+        @book1.excel.Visible.should be_true
       end
+
     end
 
     context "with several Excel instances" do
 
-      it "should open unobtrusively the closed book in the most recent Excel" do
-        book1 = Book.open(@simple_file)
-        book2 = Book.open(@simple_file, :force_excel => :new)
-        book1.Readonly.should == false
-        book2.Readonly.should == true
-        book1.close
-        book2.close
-        Book.unobtrusively(@simple_file) do |book| 
-          book.excel.should == book2.excel
-          book.excel.should_not == book1.excel
-          book.Readonly.should == false
-        end
+      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[0,0].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
-        book1 = Book.open(@simple_file)
-        book2 = Book.open(@simple_file, :force_excel => :new)
-        book2.Readonly.should == true
-        old_sheet = book1[0]
-        old_cell_value = old_sheet[0,0].value
-        book1.close
-        book2.close
-        book1.should_not be_alive
-        book2.should_not be_alive
-        Book.unobtrusively(@simple_file) do |book| 
-          book.excel.should == book2.excel
-          book.excel.should_not == book1.excel
+      it "should open unobtrusively the closed book in the most recent Excel where it was open before" do      
+        Book.unobtrusively(@simple_file, :if_closed => :reuse) do |book| 
+          book.excel.should == @book2.excel
+          book.excel.should_not == @book1.excel
           book.ReadOnly.should == false
           sheet = book[0]
           cell = sheet[0,0]
@@ -1314,24 +1415,14 @@ describe Book do
         end
         new_book = Book.open(@simple_file)
         sheet = new_book[0]
-        sheet[0,0].value.should_not == old_cell_value
+        sheet[0,0].value.should_not == @old_cell_value
       end
 
-      it "should open unobtrusively the closed book in a new Excel if the Excel is not alive anymore" do
-        book1 = Book.open(@simple_file)
-        book2 = Book.open(@simple_file, :force_excel => :new)
-        book2.Readonly.should == true
-        old_sheet = book1[0]
-        old_cell_value = old_sheet[0,0].value
-        book1.close
-        book2.close
-        book1.should_not be_alive
-        book2.should_not be_alive
-        Excel.close_all
+      it "should open unobtrusively the closed book in the new hidden Excel" do
         Book.unobtrusively(@simple_file) do |book| 
+          book.excel.should_not == @book2.excel
+          book.excel.should_not == @book1.excel
           book.ReadOnly.should == false
-          #book.excel.should_not == book1.excel
-          #book.excel.should_not == book2.excel
           sheet = book[0]
           cell = sheet[0,0]
           sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
@@ -1339,22 +1430,15 @@ describe Book do
         end
         new_book = Book.open(@simple_file)
         sheet = new_book[0]
-        sheet[0,0].value.should_not == old_cell_value
+        sheet[0,0].value.should_not == @old_cell_value
       end
 
-      it "should open unobtrusively the once opened book in a new Excel if the Excel is not alive anymore" do
-        book1 = Book.open(@simple_file)
-        book2 = Book.open(@simple_file, :force_excel => :new)
-        book2.Readonly.should == true
-        old_sheet = book1[0]
-        old_cell_value = old_sheet[0,0].value
-        book1.should be_alive
-        book2.should be_alive
+      it "should open unobtrusively the closed book in a new Excel if the Excel is not alive anymore" do
         Excel.close_all
         Book.unobtrusively(@simple_file) do |book| 
           book.ReadOnly.should == false
-          #book.excel.should_not == book1.excel
-          #book.excel.should_not == book2.excel
+          book.excel.should_not == @book1.excel
+          book.excel.should_not == @book2.excel
           sheet = book[0]
           cell = sheet[0,0]
           sheet[0,0] = cell.value == "simple" ? "complex" : "simple"
@@ -1362,9 +1446,96 @@ describe Book do
         end
         new_book = Book.open(@simple_file)
         sheet = new_book[0]
-        sheet[0,0].value.should_not == old_cell_value
+        sheet[0,0].value.should_not == @old_cell_value
+      end
+    end
+
+    context "with :hidden" do
+
+      before do
+        @book1 = Book.open(@simple_file)
+        @book1.close
+      end
+    
+      it "should create a new hidden Excel instance" do
+        Book.unobtrusively(@simple_file, :if_closed => :hidden) do |book| 
+          book.should be_a Book
+          book.should be_alive
+          book.excel.Visible.should be_false
+          book.excel.DisplayAlerts.should be_false
+        end
+      end
+
+      it "should create a new hidden Excel instance and use this afterwards" do
+        hidden_excel = nil
+        Book.unobtrusively(@simple_file, :if_closed => :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, :if_closed => :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
 
+      it "should create a new hidden Excel instance if the Excel is closed" do
+        Excel.close_all
+        Book.unobtrusively(@simple_file, :if_closed => :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_not == @book1.excel
+        end
+      end
+
+      it "should exclude hidden Excel when reuse in unobtrusively" do
+        hidden_excel = nil
+        Book.unobtrusively(@simple_file, :if_closed => :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_not == @book1.excel
+          hidden_excel = book.excel
+        end
+        Book.unobtrusively(@simple_file, :if_closed => :reuse) 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_not == hidden_excel
+        end
+      end
+
+      it "should exclude hidden Excel when reuse in open" do
+        hidden_excel = nil
+        Book.unobtrusively(@simple_file, :if_closed => :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_not == @book1.excel
+          hidden_excel = book.excel
+        end
+        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2.excel.should_not == hidden_excel
+      end
+
+      it "should exclude hidden Excel when reuse in open" do
+        book1 = Book.open(@simple_file)
+        book1.close
+        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2.excel.should == book1.excel
+        book1.should be_alive
+        book2.close
+      end
     end
   end
 
@@ -1389,16 +1560,14 @@ describe Book do
       it "should raise an error if name not defined" do
         expect {
           value = @book1.nvalue("foo")
-        }.to raise_error(ExcelErrorNValue, "name foo not in more_simple.xls")
+        }.to raise_error(ExcelErrorNValue, "name foo not in more_workbook.xls")
       end
 
       it "should raise an error if name was defined but contents is calcuated" do
         expect {
           value = @book1.nvalue("named_formula")
-        }.to raise_error(ExcelErrorNValue, "range error in more_simple.xls")
+        }.to raise_error(ExcelErrorNValue, "range error in more_workbook.xls")
       end
-
-
     end
   end
 
@@ -1453,6 +1622,12 @@ describe Book do
         }.to raise_error(ExcelErrorClose, "book is unsaved (#{File.basename(@simple_file)})")
       end
 
+      it "should raise error by default" do
+        expect{
+          @book.close(:if_unsaved => :raise)
+        }.to raise_error(ExcelErrorClose, "book is unsaved (#{File.basename(@simple_file)})")
+      end
+
       it "should close the book and leave its file untouched with option :forget" do
         ole_workbook = @book.workbook
         excel = @book.excel
@@ -1952,8 +2127,6 @@ describe Book do
           }.to raise_error(ExcelErrorSheet, "sheet name already exists")
       end
     end
-
-
   end
 
   describe 'access sheet' do