robust_excel_ole 0.5.1 → 0.6

data/spec/book_specs/book_open_spec.rb

@@ -14,7 +14,7 @@ describe Book do
     excel = Excel.new(:reuse => true)
     open_books = excel == nil ? 0 : excel.Workbooks.Count
     puts "*** open books *** : #{open_books}" if open_books > 0
-    Excel.close_all
+    Excel.kill_all
   end
 
   before do
@@ -27,15 +27,30 @@ describe Book do
     @linked_file = @dir + '/workbook_linked.xlsm'
     @simple_file_xlsm = @dir + '/workbook.xls'
     @simple_file_xlsx = @dir + '/workbook.xlsx'
+    @simple_file1 = @simple_file
+    @different_file1 = @different_file
   end
 
   after do
     Excel.kill_all
-    rm_tmp(@dir)
+    #rm_tmp(@dir)
   end
 
   describe "open" do
 
+    context "with causing warning dead excel without window handle" do
+
+      it "combined" do
+        Excel.kill_all
+        book1 = Book.open(@simple_file, :visible => true)
+        book2 = Book.open(@different_file)
+        Excel.kill_all
+        #sleep 1
+        book3 = Book.open(@another_simple_file)
+      end
+
+    end
+
     context "with class identifier 'Workbook'" do
 
       before do
@@ -91,22 +106,23 @@ describe Book do
         new_book.filename.should == @book.filename
         new_book.excel.should == @book.excel
         new_book.excel.Visible.should be_true
-        new_book.excel.DisplayAlerts.should be_false
+        new_book.excel.DisplayAlerts.should be_true
         new_book.should === @book
         new_book.close
       end
 
-      it "should yield an identical Book and set visible and displayalerts values" do
+      it "should yield an identical Book and set visible value" do
         workbook = @book.ole_workbook
-        new_book = Book.new(workbook, :visible => true, :displayalerts => true)
+        new_book = Book.new(workbook, :visible => true)
+        new_book.excel.displayalerts = true
         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.excel.visible.should be_true
-        new_book.excel.displayalerts.should be_true
+        new_book.excel.Visible.should be_true
+        new_book.excel.DisplayAlerts.should be_true
         new_book.close
       end
 
@@ -129,7 +145,7 @@ describe Book do
     context "with identity transperence" do
 
       before do
-        @book = Book.open(@simple_file)
+        @book = Book.open(@simple_file1)
       end
 
       after do
@@ -137,7 +153,7 @@ describe Book do
       end
 
       it "should yield identical Book objects for identical Excel books" do
-        book2 = Book.open(@simple_file)
+        book2 = Book.open(@simple_file1)
         book2.should === @book
         book2.close
       end
@@ -158,7 +174,7 @@ describe Book do
         @book.should be_alive
         @book.close
         @book.should_not be_alive
-        book2 = Book.open(@simple_file)
+        book2 = Book.open(@simple_file1)
         book2.should === @book
         book2.should be_alive
         book2.close
@@ -168,7 +184,7 @@ describe Book do
         @book.should be_alive
         @book.close
         Excel.close_all
-        book2 = Book.open(@simple_file)
+        book2 = Book.open(@simple_file1)
         book2.should be_alive
         book2.should === @book
         book2.close
@@ -179,7 +195,7 @@ describe Book do
         old_excel = @book.excel
         @book.close
         @book.should_not be_alive
-        book2 = Book.open(@simple_file, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :new)
         book2.should_not === @book
         book2.should be_alive
         book2.excel.should_not == old_excel
@@ -191,7 +207,7 @@ describe Book do
         new_excel = Excel.new(:reuse => false)
         @book.close
         @book.should_not be_alive
-        book2 = Book.open(@simple_file, :force_excel => new_excel)
+        book2 = Book.open(@simple_file1, :force_excel => new_excel)
         book2.should_not === @book
         book2.should be_alive
         book2.excel.should == new_excel
@@ -204,7 +220,7 @@ describe Book do
         new_excel = Excel.new(:reuse => false)
         @book.close
         @book.should_not be_alive
-        book2 = Book.open(@simple_file, :force_excel => old_excel)
+        book2 = Book.open(@simple_file1, :force_excel => old_excel)
         book2.should === @book
         book2.should be_alive
         book2.excel.should == old_excel
@@ -217,7 +233,7 @@ describe Book do
     context "with :force_excel" do
 
       before do
-        @book = Book.open(@simple_file)
+        @book = Book.open(@simple_file1)
       end
 
       after do
@@ -227,22 +243,22 @@ describe Book do
       it "should open in a given Excel provided as Excel, Book, or WIN32OLE representing an Excel or Workbook" do
         book2 = Book.open(@another_simple_file)
         book3 = Book.open(@different_file)
-        book3 = Book.open(@simple_file, :force_excel => book2.excel)
+        book3 = Book.open(@simple_file1, :force_excel => book2.excel)
         book3.excel.should === book2.excel
-        book4 = Book.open(@simple_file, :force_excel => @book) 
+        book4 = Book.open(@simple_file1, :force_excel => @book) 
         book4.excel.should === @book.excel
         book3.close
         book4.close
-        book5 = Book.open(@simple_file, :force_excel => book2.ole_workbook)
+        book5 = Book.open(@simple_file1, :force_excel => book2.ole_workbook)
         book5.excel.should ===  book2.excel
         win32ole_excel1 = WIN32OLE.connect(@book.ole_workbook.Fullname).Application
-        book6 = Book.open(@simple_file, :force_excel => win32ole_excel1)
+        book6 = Book.open(@simple_file1, :force_excel => win32ole_excel1)
         book6.excel.should === @book.excel
       end
 
 
       it "should open in a new Excel" do
-        book2 = Book.open(@simple_file, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :new)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should_not == @book.excel 
@@ -253,20 +269,20 @@ describe Book do
       end
 
       it "should open in a given Excel, not provide identity transparency, because old book readonly, new book writable" do
-        book2 = Book.open(@simple_file, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :new)
         book2.excel.should_not == @book.excel
-        book3 = Book.open(@simple_file, :force_excel => :new)
+        book3 = Book.open(@simple_file1, :force_excel => :new)
         book3.excel.should_not == book2.excel
         book3.excel.should_not == @book.excel
         book2.close
-        book4 = Book.open(@simple_file, :force_excel => book2.excel)
+        book4 = Book.open(@simple_file1, :force_excel => book2.excel)
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
         book4.Readonly.should == true
         book4.should_not == book2 
         book4.close
-        book5 = Book.open(@simple_file, :force_excel => book2)
+        book5 = Book.open(@simple_file1, :force_excel => book2)
         book5.should be_alive
         book5.should be_a Book
         book5.excel.should == book2.excel
@@ -277,22 +293,22 @@ describe Book do
       end
 
       it "should open in a given Excel, provide identity transparency, because book can be readonly, such that the old and the new book are readonly" do
-        book2 = Book.open(@simple_file, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :new)
         book2.excel.should_not == @book.excel
-        book3 = Book.open(@simple_file, :force_excel => :new)
+        book3 = Book.open(@simple_file1, :force_excel => :new)
         book3.excel.should_not == book2.excel
         book3.excel.should_not == @book.excel
         book2.close
         book3.close
         @book.close
-        book4 = Book.open(@simple_file, :force_excel => book2.excel, :read_only => true)
+        book4 = Book.open(@simple_file1, :force_excel => book2.excel, :read_only => true)
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
         book4.ReadOnly.should be_true
         book4.should == book2
         book4.close
-        book5 = Book.open(@simple_file, :force_excel => book2, :read_only => true)
+        book5 = Book.open(@simple_file1, :force_excel => book2, :read_only => true)
         book5.should be_alive
         book5.should be_a Book
         book5.excel.should == book2.excel
@@ -303,11 +319,11 @@ describe Book do
       end
 
       it "should open in a given Excel, provide identity transparency, because book can be readonly, such that the old and the new book are readonly" do
-        book2 = Book.open(@simple_file, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :new)
         book2.excel.should_not == @book.excel
         book2.close
         @book.close
-        book4 = Book.open(@simple_file, :force_excel => book2, :read_only => true)
+        book4 = Book.open(@simple_file1, :force_excel => book2, :read_only => true)
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
@@ -318,12 +334,12 @@ describe Book do
 
       it "should raise an error if no Excel or Book is given" do
         expect{
-          Book.open(@simple_file, :force_excel => :b)
+          Book.open(@simple_file1, :force_excel => :b)
           }.to raise_error(ExcelErrorOpen, "given object is neither an Excel, a Workbook, nor a Win32ole")
       end
 
       it "should do force_excel even if both force_ and default_excel is given" do
-        book2 = Book.open(@simple_file, :default_excel => @book.excel, :force_excel => :new)
+        book2 = Book.open(@simple_file1, :default_excel => @book.excel, :force_excel => :new)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should_not == @book.excel 
@@ -338,34 +354,32 @@ describe Book do
       end
 
       it "should force_excel with :reuse" do
-        book2 = Book.open(@different_file, :force_excel => :reuse)
+        book2 = Book.open(@different_file, :force_excel => :current)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should == @book.excel
       end
 
       it "should force_excel with :reuse even if :default_excel says sth. else" do
-        book2 = Book.open(@different_file, :force_excel => :reuse, :default_excel => :new)
+        book2 = Book.open(@different_file, :force_excel => :current, :default_excel => :new)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should == @book.excel
       end
 
-      it "should open force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
+      it "should force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
         excel2 = Excel.new(:reuse => false)
-        excel1_hwnd = @book.excel.hwnd
         @book.excel.close
-        book2 = Book.open(@simple_file, :force_excel => :reuse, :default_excel => :new)
+        book2 = Book.open(@simple_file1, :force_excel => :current, :default_excel => :new)
         book2.should be_alive
         book2.should be_a Book
-        book2.excel.should_not == excel2
-        book2.excel.hwnd.should == excel1_hwnd
+        book2.excel.should === excel2
       end
 
       it "should force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
-        book2 = Book.open(@different_file, :force_excel => :new)
+        book2 = Book.open(@different_file1, :force_excel => :new)
         book2.excel.close
-        book3 = Book.open(@different_file, :force_excel => :reuse, :default_excel => :new)
+        book3 = Book.open(@different_file1, :force_excel => :current, :default_excel => :new)
         book3.should be_alive
         book3.should be_a Book
         book3.excel.should == @book.excel
@@ -376,7 +390,7 @@ describe Book do
     context "with :default_excel" do
 
       before do
-        @book = Book.open(@simple_file, :visible => true)
+        @book = Book.open(@simple_file1, :visible => true)
       end
 
       after do
@@ -384,7 +398,7 @@ describe Book do
       end
 
       it "should use the open book" do
-        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2 = Book.open(@simple_file1, :default_excel => :current)
         book2.excel.should == @book.excel
         book2.should be_alive
         book2.should be_a Book
@@ -395,7 +409,7 @@ describe Book do
       it "should reopen the book in the excel instance where it was opened before" do
         excel = Excel.new(:reuse => false)
         @book.close
-        book2 = Book.open(@simple_file)
+        book2 = Book.open(@simple_file1)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should == @book.excel
@@ -412,7 +426,7 @@ describe Book do
         fn = @book.filename
         @book.close
         Excel.close_all
-        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2 = Book.open(@simple_file1, :default_excel => :current)
         book2.should be_alive
         book2.should be_a Book
         book2.filename.should == fn
@@ -426,7 +440,7 @@ describe Book do
         Excel.close_all
         new_excel = Excel.new(:reuse => false)
         new_excel2 = Excel.new(:reuse => false)
-        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2 = Book.open(@simple_file1, :default_excel => :current)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should_not == excel
@@ -442,7 +456,7 @@ describe Book do
         Excel.close_all
         excel1 = Excel.new(:reuse => false)
         excel2 = Excel.new(:reuse => false)
-        book2 = Book.open(@different_file, :default_excel => :reuse)
+        book2 = Book.open(@different_file, :default_excel => :current)
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should == excel1
@@ -454,12 +468,12 @@ describe Book do
         excel1 = @book.excel
         excel2 = Excel.new(:reuse => false)
         @book.close
-        book2 = Book.open(@simple_file, :default_excel => :reuse)
+        book2 = Book.open(@simple_file1, :default_excel => :current)
         book2.excel.should == excel1
         book2.close
-        book3 = Book.open(@simple_file, :force_excel => excel2)
+        book3 = Book.open(@simple_file1, :force_excel => excel2)
         book3.close
-        book3 = Book.open(@simple_file, :default_excel => :reuse)
+        book3 = Book.open(@simple_file1, :default_excel => :current)
         book3.excel.should == excel2
         book3.close
       end
@@ -468,7 +482,7 @@ describe Book do
         book2 = Book.open(@simple_file, :force_excel => :new)
         @book.close
         book2.close
-        book3 = Book.open(@simple_file)
+        book3 = Book.open(@simple_file1)
         book2.should be_alive
         book2.should be_a Book
         book3.excel.should == book2.excel
@@ -478,7 +492,7 @@ describe Book do
       end
 
       it "should open the book in a new excel if the book was not opened before" do
-        book2 = Book.open(@different_file, :default_excel => :reuse)
+        book2 = Book.open(@different_file, :default_excel => :current)
         book2.excel.should == @book.excel
         book3 = Book.open(@another_simple_file, :default_excel => :new)
         book3.excel.should_not == @book.excel
@@ -488,13 +502,13 @@ describe Book do
         excel = @book.excel
         excel2 = Excel.new(:reuse => false)
         excel.close
-        book2 = Book.open(@simple_file, :default_excel => :new)
+        book2 = Book.open(@simple_file1, :default_excel => :new)
         book2.excel.should_not == excel2
         book2.close
       end
 
       it "should open the book in a given excel if the book was not opened before" do
-        book2 = Book.open(@different_file, :default_excel => :reuse)
+        book2 = Book.open(@different_file, :default_excel => :current)
         book2.excel.should == @book.excel
         excel = Excel.new(:reuse => false)
         book3 = Book.open(@another_simple_file, :default_excel => excel)
@@ -504,7 +518,7 @@ describe Book do
       it "should open the book in a given excel if the book was opened before but the excel has been closed" do
         excel2 = Excel.new(:reuse => false, :visible => true)
         @book.excel.close        
-        book2 = Book.open(@simple_file, :visible => true, :default_excel => excel2)
+        book2 = Book.open(@simple_file1, :visible => true, :default_excel => excel2)
         book2.excel.should == excel2
       end
 
@@ -542,16 +556,245 @@ describe Book do
       end
 
       it "should reuse an open book by default" do
-        book2 = Book.open(@simple_file)
+        book2 = Book.open(@simple_file1)
+        book2.excel.should == @book.excel
+        book2.should == @book
+      end
+
+      it "should raise an error if no Excel or Book is given" do
+        expect{
+          Book.open(@different_file, :default_excel => :a)
+          }.to raise_error(ExcelErrorOpen, "given object is neither an Excel, a Workbook, nor a Win32ole")
+      end
+      
+    end
+
+    context "with :active instead of :current" do
+      
+      before do
+        @book = Book.open(@simple_file1)
+      end
+
+      after do
+        @book.close rescue nil
+      end
+
+      it "should force_excel with :active" do
+        book2 = Book.open(@different_file, :force_excel => :active)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == @book.excel
+      end
+
+      it "should force_excel with :reuse even if :default_excel says sth. else" do
+        book2 = Book.open(@different_file, :force_excel => :active, :default_excel => :new)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == @book.excel
+      end
+
+      it "should open force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
+        excel2 = Excel.new(:reuse => false)
+        @book.excel.close
+        book2 = Book.open(@simple_file1, :force_excel => :active, :default_excel => :new)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should === excel2
+      end
+
+      it "should force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
+        book2 = Book.open(@different_file1, :force_excel => :new)
+        book2.excel.close
+        book3 = Book.open(@different_file1, :force_excel => :active, :default_excel => :new)
+        book3.should be_alive
+        book3.should be_a Book
+        book3.excel.should == @book.excel
+      end
+
+      it "should use the open book" do
+        book2 = Book.open(@simple_file1, :default_excel => :active)
+        book2.excel.should == @book.excel
+        book2.should be_alive
+        book2.should be_a Book
+        book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in a new Excel if all Excel instances are closed" do
+        excel = Excel.new(:reuse => false)
+        excel2 = @book.excel
+        fn = @book.filename
+        @book.close
+        Excel.close_all
+        book2 = Book.open(@simple_file1, :default_excel => :active)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.filename.should == fn
+        @book.should be_alive
+        book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in the first opened Excel if the old Excel is closed" do
+        excel = @book.excel
+        Excel.close_all
+        new_excel = Excel.new(:reuse => false)
+        new_excel2 = Excel.new(:reuse => false)
+        book2 = Book.open(@simple_file1, :default_excel => :active)
+        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
+        @book.should be_alive
+        book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in the first opened excel, if the book cannot be reopened" do
+        @book.close
+        Excel.close_all
+        excel1 = Excel.new(:reuse => false)
+        excel2 = Excel.new(:reuse => false)
+        book2 = Book.open(@different_file, :default_excel => :active)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == excel1
+        book2.excel.should_not == excel2
+        book2.close
+      end
+
+      it "should reopen the book in the Excel where it was opened most recently" do
+        excel1 = @book.excel
+        excel2 = Excel.new(:reuse => false)
+        @book.close
+        book2 = Book.open(@simple_file1, :default_excel => :active)
+        book2.excel.should == excel1
+        book2.close
+        book3 = Book.open(@simple_file1, :force_excel => excel2)
+        book3.close
+        book3 = Book.open(@simple_file1, :default_excel => :active)
+        book3.excel.should == excel2
+        book3.close
+      end
+
+    end
+
+    context "with :reuse instead of :current" do
+      
+      before do
+        @book = Book.open(@simple_file1)
+      end
+
+      after do
+        @book.close rescue nil
+      end
+
+      it "should force_excel with :reuse" do
+        book2 = Book.open(@different_file, :force_excel => :reuse)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == @book.excel
+      end
+
+      it "should force_excel with :reuse even if :default_excel says sth. else" do
+        book2 = Book.open(@different_file, :force_excel => :reuse, :default_excel => :new)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == @book.excel
+      end
+
+      it "should open force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
+        excel2 = Excel.new(:reuse => false)
+        @book.excel.close
+        book2 = Book.open(@simple_file1, :force_excel => :reuse, :default_excel => :new)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should === excel2
+      end
+
+      it "should force_excel with :reuse when reopening and the Excel is not alive even if :default_excel says sth. else" do
+        book2 = Book.open(@different_file1, :force_excel => :new)
+        book2.excel.close
+        book3 = Book.open(@different_file1, :force_excel => :reuse, :default_excel => :new)
+        book3.should be_alive
+        book3.should be_a Book
+        book3.excel.should == @book.excel
+      end
+
+      it "should use the open book" do
+        book2 = Book.open(@simple_file1, :default_excel => :reuse)
         book2.excel.should == @book.excel
+        book2.should be_alive
+        book2.should be_a Book
+        book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in a new Excel if all Excel instances are closed" do
+        excel = Excel.new(:reuse => false)
+        excel2 = @book.excel
+        fn = @book.filename
+        @book.close
+        Excel.close_all
+        book2 = Book.open(@simple_file1, :default_excel => :reuse)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.filename.should == fn
+        @book.should be_alive
+        book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in the first opened Excel if the old Excel is closed" do
+        excel = @book.excel
+        Excel.close_all
+        new_excel = Excel.new(:reuse => false)
+        new_excel2 = Excel.new(:reuse => false)
+        book2 = Book.open(@simple_file1, :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
+        @book.should be_alive
         book2.should == @book
+        book2.close
+      end
+
+      it "should reopen a book in the first opened excel, if the book cannot be reopened" do
+        @book.close
+        Excel.close_all
+        excel1 = Excel.new(:reuse => false)
+        excel2 = Excel.new(:reuse => false)
+        book2 = Book.open(@different_file, :default_excel => :reuse)
+        book2.should be_alive
+        book2.should be_a Book
+        book2.excel.should == excel1
+        book2.excel.should_not == excel2
+        book2.close
+      end
+
+      it "should reopen the book in the Excel where it was opened most recently" do
+        excel1 = @book.excel
+        excel2 = Excel.new(:reuse => false)
+        @book.close
+        book2 = Book.open(@simple_file1, :default_excel => :reuse)
+        book2.excel.should == excel1
+        book2.close
+        book3 = Book.open(@simple_file1, :force_excel => excel2)
+        book3.close
+        book3 = Book.open(@simple_file1, :default_excel => :reuse)
+        book3.excel.should == excel2
+        book3.close
       end
+
     end
 
     context "with :if_unsaved" do
 
       before do
-        @book = Book.open(@simple_file)
+        @book = Book.open(@simple_file1)
         @sheet = @book.sheet(1)
         @book.add_sheet(@sheet, :as => 'a_name')
       end
@@ -563,13 +806,13 @@ describe Book do
 
       it "should raise an error, if :if_unsaved is :raise" do
         expect {
-          @new_book = Book.open(@simple_file, :if_unsaved => :raise)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
         }.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
         expect {
-          @new_book = Book.open(@simple_file, :if_unsaved => :accept)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :accept)
           }.to_not raise_error
         @book.should be_alive
         @new_book.should be_alive
@@ -577,7 +820,7 @@ describe Book do
       end
 
       it "should open book and close old book, if :if_unsaved is :forget" do
-        @new_book = Book.open(@simple_file, :if_unsaved => :forget)
+        @new_book = Book.open(@simple_file1, :if_unsaved => :forget)
         @book.should_not be_alive
         @new_book.should be_alive
         @new_book.filename.downcase.should == @simple_file.downcase
@@ -595,7 +838,7 @@ describe Book do
         it "should open the new book and close the unsaved book, if user answers 'yes'" do
           # "Yes" is the  default. --> language independent
           @key_sender.puts "{enter}"
-          @new_book = Book.open(@simple_file, :if_unsaved => :alert)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :alert)
           @new_book.should be_alive
           @new_book.filename.downcase.should == @simple_file.downcase
           @book.should_not be_alive
@@ -609,7 +852,7 @@ describe Book do
           @key_sender.puts "{right}{enter}"
           @key_sender.puts "{right}{enter}"
           expect{
-            Book.open(@simple_file, :if_unsaved => :alert)
+            Book.open(@simple_file1, :if_unsaved => :alert)
             }.to raise_error(ExcelErrorOpen, "open: user canceled or open error")
           @book.should be_alive
         end
@@ -627,9 +870,9 @@ describe Book do
         it "should open the new book and close the unsaved book, if user answers 'yes'" do
           # "Yes" is the  default. --> language independent
           @key_sender.puts "{enter}"
-          @new_book = Book.open(@simple_file, :if_unsaved => :excel)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :excel)
           @new_book.should be_alive
-          @new_book.filename.downcase.should == @simple_file.downcase
+          @new_book.filename.downcase.should == @simple_file1.downcase
           @book.should_not be_alive
         end
 
@@ -641,14 +884,14 @@ describe Book do
           @key_sender.puts "{right}{enter}"
           @key_sender.puts "{right}{enter}"
           expect{
-            Book.open(@simple_file, :if_unsaved => :excel)
+            Book.open(@simple_file1, :if_unsaved => :excel)
             }.to raise_error(ExcelErrorOpen, "open: user canceled or open error")
           @book.should be_alive
         end
       end
 
       it "should open the book in a new excel instance, if :if_unsaved is :new_excel" do
-        @new_book = Book.open(@simple_file, :if_unsaved => :new_excel)
+        @new_book = Book.open(@simple_file1, :if_unsaved => :new_excel)
         @book.should be_alive
         @new_book.should be_alive
         @new_book.filename.should == @book.filename
@@ -658,13 +901,13 @@ describe Book do
 
       it "should raise an error, if :if_unsaved is default" do
         expect {
-          @new_book = Book.open(@simple_file, :if_unsaved => :raise)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
         }.to raise_error(ExcelErrorOpen, /workbook is already open but not saved: "workbook.xls"/)
       end
 
       it "should raise an error, if :if_unsaved is invalid option" do
         expect {
-          @new_book = Book.open(@simple_file, :if_unsaved => :invalid_option)
+          @new_book = Book.open(@simple_file1, :if_unsaved => :invalid_option)
         }.to raise_error(ExcelErrorOpen, ":if_unsaved: invalid option: :invalid_option")
       end
     end
@@ -677,7 +920,7 @@ describe Book do
 
           before do        
             if i == 1 then 
-              book_before = Book.open(@simple_file)
+              book_before = Book.open(@simple_file1)
               book_before.close
             end
             @book = Book.open(@simple_file_other_path)
@@ -693,22 +936,22 @@ describe Book do
 
           it "should raise an error, if :if_obstructed is :raise" do
             expect {
-              @new_book = Book.open(@simple_file, :if_obstructed => :raise)
+              @new_book = Book.open(@simple_file1, :if_obstructed => :raise)
             }.to raise_error(ExcelErrorOpen, /blocked by a book with the same name in a different path/)
           end
 
           it "should close the other book and open the new book, if :if_obstructed is :forget" do
-            @new_book = Book.open(@simple_file, :if_obstructed => :forget)
+            @new_book = Book.open(@simple_file1, :if_obstructed => :forget)
             @book.should_not be_alive
             @new_book.should be_alive
             @new_book.filename.downcase.should == @simple_file.downcase
           end
 
           it "should save the old book, close it, and open the new book, if :if_obstructed is :save" do
-            @new_book = Book.open(@simple_file, :if_obstructed => :save)
+            @new_book = Book.open(@simple_file1, :if_obstructed => :save)
             @book.should_not be_alive
             @new_book.should be_alive
-            @new_book.filename.downcase.should == @simple_file.downcase
+            @new_book.filename.downcase.should == @simple_file1.downcase
             old_book = Book.open(@simple_file_other_path, :if_obstructed => :forget)
             old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
             old_book.close
@@ -717,20 +960,20 @@ describe Book do
           it "should raise an error, if the old book is unsaved, and close the old book and open the new book, 
               if :if_obstructed is :close_if_saved" do
             expect{
-              @new_book = Book.open(@simple_file, :if_obstructed => :close_if_saved)
+              @new_book = Book.open(@simple_file1, :if_obstructed => :close_if_saved)
             }.to raise_error(ExcelErrorOpen, /workbook with the same name in a different path is unsaved/)
             @book.save
-            @new_book = Book.open(@simple_file, :if_obstructed => :close_if_saved)
+            @new_book = Book.open(@simple_file1, :if_obstructed => :close_if_saved)
             @book.should_not be_alive
             @new_book.should be_alive
-            @new_book.filename.downcase.should == @simple_file.downcase
+            @new_book.filename.downcase.should == @simple_file1.downcase
             old_book = Book.open(@simple_file_other_path, :if_obstructed => :forget)
             old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
             old_book.close
           end
 
           it "should open the book in a new excel instance, if :if_obstructed is :new_excel" do
-            @new_book = Book.open(@simple_file, :if_obstructed => :new_excel)
+            @new_book = Book.open(@simple_file1, :if_obstructed => :new_excel)
             @book.should be_alive
             @new_book.should be_alive
             @new_book.filename.should_not == @book.filename
@@ -739,13 +982,13 @@ describe Book do
 
           it "should raise an error, if :if_obstructed is default" do
             expect {
-              @new_book = Book.open(@simple_file)              
+              @new_book = Book.open(@simple_file1)              
             }.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 {
-              @new_book = Book.open(@simple_file, :if_obstructed => :invalid_option)
+              @new_book = Book.open(@simple_file1, :if_obstructed => :invalid_option)
             }.to raise_error(ExcelErrorOpen, ":if_obstructed: invalid option: :invalid_option")
           end
         end
@@ -765,8 +1008,8 @@ describe Book do
       possible_options.each do |options_value|
         context "with :if_unsaved => #{options_value} and in the same and different path" do
           before do
-            @new_book = Book.open(@simple_file, :reuse=> true, :if_unsaved => options_value)
-            @different_book = Book.new(@different_file, :reuse=> true, :if_unsaved => options_value)
+            @new_book = Book.open(@simple_file1, :reuse => true, :if_unsaved => options_value)
+            @different_book = Book.new(@different_file, :reuse => true, :if_unsaved => options_value)
           end
           after do
             @new_book.close
@@ -786,7 +1029,13 @@ describe Book do
     
     context "with non-existing file" do
 
-      it "should raise an exception" do
+      it "should raise error if filename is nil" do
+        expect{
+          Book.open(@nonexisting)
+          }.to raise_error(ExcelErrorOpen, "filename is nil")
+      end
+
+      it "should raise error if file does not exist" do
         File.delete @simple_save_file rescue nil
         expect {
           Book.open(@simple_save_file, :if_absent => :raise)
@@ -825,17 +1074,49 @@ describe Book do
       end
     end
 
+    context "with :update_links" do
+      
+      it "should set update_links to :alert" do
+        book = Book.open(@simple_file, :update_links => :alert)
+        book.UpdateLinks.should == XlUpdateLinksUserSetting
+        #book.UpdateRemoteReferences.should be_true
+        book.Saved.should be_true
+      end
+
+      it "should set update_links to :never" do
+        book = Book.open(@simple_file, :update_links => :never)
+        book.UpdateLinks.should == XlUpdateLinksNever
+        #book.UpdateRemoteReferences.should be_false
+        book = Book.open(@simple_file, :update_links => :foo)
+        book.UpdateLinks.should == XlUpdateLinksNever
+        #book.UpdateRemoteReferences.should be_false
+      end
+
+      it "should set update_links to :always" do
+        book = Book.open(@simple_file, :update_links => :always)
+        book.UpdateLinks.should == XlUpdateLinksAlways
+        #book.UpdateRemoteReferences.should be_true
+      end
+
+      it "should set update_links to :never per default" do
+        book = Book.open(@simple_file)
+        book.UpdateLinks.should == XlUpdateLinksNever
+        #book.UpdateRemoteReferences.should be_false
+      end
+
+    end
+
     context "with :read_only" 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 = Book.open(@simple_file1, :read_only => true)
         book.ReadOnly.should be_true
         book.should be_alive
         sheet = book.sheet(1)
         old_cell_value = sheet[1,1].value
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         book.Saved.should be_false
-        new_book = Book.open(@simple_file, :read_only => false, :if_unsaved => :accept)
+        new_book = Book.open(@simple_file1, :read_only => false, :if_unsaved => :accept)
         new_book.ReadOnly.should be_false 
         new_book.should be_alive
         book.should be_alive   
@@ -846,28 +1127,28 @@ describe Book do
       end
 
       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 = Book.open(@simple_file1, :read_only => false)
         book.ReadOnly.should be_false
         book.should be_alive
         sheet = book.sheet(1)
         old_cell_value = sheet[1,1].value        
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         book.Saved.should be_false
-        new_book = Book.open(@simple_file, :read_only => true, :if_unsaved => :accept)
+        new_book = Book.open(@simple_file1, :read_only => true, :if_unsaved => :accept)
         new_book.ReadOnly.should be_false
         new_book.Saved.should be_false
         new_book.should == book
       end
 
       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 = Book.open(@simple_file1, :read_only => true)
         book.ReadOnly.should be_true
         book.should be_alive
         sheet = book.sheet(1)
         old_cell_value = sheet[1,1].value
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         book.Saved.should be_false
-        new_book = Book.open(@simple_file, :if_unsaved => :accept, :force_excel => book.excel, :read_only => false)
+        new_book = Book.open(@simple_file1, :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   
@@ -878,30 +1159,30 @@ describe Book do
       end
 
       it "should reopen the book with readonly (unsaved changes of the writable should be saved)" do
-        book = Book.open(@simple_file, :force_excel => :new, :read_only => false)
+        book = Book.open(@simple_file1, :force_excel => :new, :read_only => false)
         book.ReadOnly.should be_false
         book.should be_alive
         sheet = book.sheet(1)
         old_cell_value = sheet[1,1].value        
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         book.Saved.should be_false
-        new_book = Book.open(@simple_file, :force_excel => book.excel, :read_only => true, :if_unsaved => :accept)
+        new_book = Book.open(@simple_file1, :force_excel => book.excel, :read_only => true, :if_unsaved => :accept)
         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 = Book.open(@simple_file1, :read_only => true)
         book.ReadOnly.should be_true
-        new_book = Book.open(@simple_file, :force_excel => :new, :read_only => false)
+        new_book = Book.open(@simple_file1, :force_excel => :new, :read_only => false)
         new_book.ReadOnly.should be_false
         new_book.close
         book.close
       end
 
       it "should be able to save, if :read_only => false" do
-        book = Book.open(@simple_file, :read_only => false)
+        book = Book.open(@simple_file1, :read_only => false)
         book.should be_a Book
         expect {
           book.save_as(@simple_save_file, :if_exists => :overwrite)
@@ -910,7 +1191,7 @@ describe Book do
       end
 
       it "should be able to save, if :read_only is default" do
-        book = Book.open(@simple_file)
+        book = Book.open(@simple_file1)
         book.should be_a Book
         expect {
           book.save_as(@simple_save_file, :if_exists => :overwrite)