robust_excel_ole 0.6 → 0.6.1

data/spec/book_specs/book_misc_spec.rb

@@ -49,6 +49,15 @@ describe Book do
 
   describe "with visible" do
 
+    it "should adapt its default value at the visible value of the Excel" do
+      excel1 = Excel.create
+      excel1.Visible = true
+      book1 = Book.open(@simple_file)
+      excel1.Visible.should be_true
+      excel1.visible.should be_true
+      book1.visible.should be_true
+    end
+
     it "should preserve :visible if it is not set" do
       book1 = Book.open(@simple_file)
       book1.excel.Visible.should be_false
@@ -289,7 +298,7 @@ describe Book do
 
       it "should report that workbook is not alive" do
         @book.close
-        expect{ @book.Nonexisting_method }.to raise_error(ExcelError, "method missing: workbook not alive")
+        expect{ @book.Nonexisting_method }.to raise_error(ObjectNotAlive, "method missing: workbook not alive")
       end
     end
 
@@ -370,22 +379,22 @@ describe Book do
     it "should raise an error if name not defined" do
       expect {
         @book1.nameval("foo")
-      }.to raise_error(ExcelError, /name "foo" not in "another_workbook.xls"/)
+      }.to raise_error(NameNotFound, /name "foo" not in "another_workbook.xls"/)
       expect {
           @book1.set_nameval("foo","bar")
-      }.to raise_error(ExcelError, /name "foo" not in "another_workbook.xls"/)
+      }.to raise_error(NameNotFound, /name "foo" not in "another_workbook.xls"/)
       expect {
           @book1["foo"] = "bar"
-      }.to raise_error(ExcelError, /name "foo" not in "another_workbook.xls"/)
+      }.to raise_error(NameNotFound, /name "foo" not in "another_workbook.xls"/)
     end    
 
     it "should raise an error if name was defined but contents is calcuated" do
       expect {
         @book1.set_nameval("named_formula","bar")
-      }.to raise_error(ExcelError, /cannot assign value to range named "named_formula" in "another_workbook.xls"/)
+      }.to raise_error(RangeNotEvaluatable, /cannot assign value to range named "named_formula" in "another_workbook.xls"/)
       expect {
         @book1["named_formula"] = "bar"
-      }.to raise_error(ExcelError, /cannot assign value to range named "named_formula" in "another_workbook.xls"/)
+      }.to raise_error(RangeNotEvaluatable, /cannot assign value to range named "named_formula" in "another_workbook.xls"/)
     end
 
     # Excel Bug: for local names without uqifier: takes the first sheet as default even if another sheet is activated
@@ -415,7 +424,7 @@ describe Book do
       @book1.nameval("five").should == [[1,2],[3,4]]
       expect {
         @book1.rename_range("four","five")
-      }.to raise_error(ExcelError, /name "four" not in "another_workbook.xls"/)
+      }.to raise_error(NameNotFound, /name "four" not in "another_workbook.xls"/)
     end
   end
 

data/spec/book_specs/book_open_spec.rb

@@ -335,7 +335,7 @@ describe Book do
       it "should raise an error if no Excel or Book is given" do
         expect{
           Book.open(@simple_file1, :force_excel => :b)
-          }.to raise_error(ExcelErrorOpen, "given object is neither an Excel, a Workbook, nor a Win32ole")
+          }.to raise_error(TypeErrorREO, "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
@@ -564,7 +564,7 @@ describe Book do
       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")
+          }.to raise_error(TypeErrorREO, "given object is neither an Excel, a Workbook, nor a Win32ole")
       end
       
     end
@@ -807,7 +807,7 @@ describe Book do
       it "should raise an error, if :if_unsaved is :raise" do
         expect {
           @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
-        }.to raise_error(ExcelErrorOpen, /workbook is already open but not saved: "workbook.xls"/)
+        }.to raise_error(WorkbookNotSaved, /workbook is already open but not saved: "workbook.xls"/)
       end
 
       it "should let the book open, if :if_unsaved is :accept" do
@@ -853,7 +853,7 @@ describe Book do
           @key_sender.puts "{right}{enter}"
           expect{
             Book.open(@simple_file1, :if_unsaved => :alert)
-            }.to raise_error(ExcelErrorOpen, "open: user canceled or open error")
+            }.to raise_error(WorkbookError, "open: user canceled or open error")
           @book.should be_alive
         end
       end
@@ -885,7 +885,7 @@ describe Book do
           @key_sender.puts "{right}{enter}"
           expect{
             Book.open(@simple_file1, :if_unsaved => :excel)
-            }.to raise_error(ExcelErrorOpen, "open: user canceled or open error")
+            }.to raise_error(WorkbookError, "open: user canceled or open error")
           @book.should be_alive
         end
       end
@@ -902,13 +902,13 @@ describe Book do
       it "should raise an error, if :if_unsaved is default" do
         expect {
           @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
-        }.to raise_error(ExcelErrorOpen, /workbook is already open but not saved: "workbook.xls"/)
+        }.to raise_error(WorkbookNotSaved, /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_file1, :if_unsaved => :invalid_option)
-        }.to raise_error(ExcelErrorOpen, ":if_unsaved: invalid option: :invalid_option")
+        }.to raise_error(OptionInvalid, ":if_unsaved: invalid option: :invalid_option")
       end
     end
 
@@ -937,7 +937,7 @@ describe Book do
           it "should raise an error, if :if_obstructed is :raise" do
             expect {
               @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/)
+            }.to raise_error(WorkbookBlocked, /blocked by a workbook 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
@@ -961,7 +961,7 @@ describe Book do
               if :if_obstructed is :close_if_saved" do
             expect{
               @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/)
+            }.to raise_error(WorkbookBlocked, /workbook with the same name in a different path is unsaved/)
             @book.save
             @new_book = Book.open(@simple_file1, :if_obstructed => :close_if_saved)
             @book.should_not be_alive
@@ -983,13 +983,13 @@ describe Book do
           it "should raise an error, if :if_obstructed is default" do
             expect {
               @new_book = Book.open(@simple_file1)              
-            }.to raise_error(ExcelErrorOpen, /blocked by a book with the same name in a different path/)
+            }.to raise_error(WorkbookBlocked, /blocked by a workbook 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_file1, :if_obstructed => :invalid_option)
-            }.to raise_error(ExcelErrorOpen, ":if_obstructed: invalid option: :invalid_option")
+            }.to raise_error(OptionInvalid, ":if_obstructed: invalid option: :invalid_option")
           end
         end
       end
@@ -1032,14 +1032,14 @@ describe Book do
       it "should raise error if filename is nil" do
         expect{
           Book.open(@nonexisting)
-          }.to raise_error(ExcelErrorOpen, "filename is nil")
+          }.to raise_error(FileNameNotGiven, "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)
-        }.to raise_error(ExcelErrorOpen, "file \"#{@simple_save_file}\" not found")
+        }.to raise_error(FileNotFound, "file \"#{@simple_save_file}\" not found")
       end
 
       it "should create a workbook" do
@@ -1054,7 +1054,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(FileNotFound, "file \"#{@simple_save_file}\" not found")
       end
 
     end
@@ -1079,29 +1079,24 @@ describe Book 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
@@ -1242,7 +1237,7 @@ describe Book do
         expected_path = Regexp.new(File.expand_path(path).gsub(/\//, "."))
         expect {
           Book.open(path)
-        }.to raise_error(ExcelErrorOpen, "file \"#{path}\" not found")
+        }.to raise_error(FileNotFound, "file \"#{path}\" not found")
       end
     end
   end

data/spec/book_specs/book_save_spec.rb

@@ -56,7 +56,7 @@ describe Book do
         @book = Book.open(@simple_file, :read_only => true)
         expect {
           @book.save
-        }.to raise_error(ExcelErrorSave, "Not opened for writing (opened with :read_only option)")
+        }.to raise_error(WorkbookReadOnly, "Not opened for writing (opened with :read_only option)")
         @book.close
       end
 
@@ -65,7 +65,7 @@ describe Book do
         @book.close
         expect{
           @book.save
-        }.to raise_error(ExcelErrorSave, "workbook is not alive")
+        }.to raise_error(ObjectNotAlive, "workbook is not alive")
       end
 
     end
@@ -82,7 +82,7 @@ describe Book do
       it {
         expect {
           @book.save_as(@simple_file)
-        }.to raise_error(ExcelErrorSave,
+        }.to raise_error(WorkbookReadOnly,
                      "Not opened for writing (opened with :read_only option)")
       }
     end
@@ -100,7 +100,7 @@ describe Book do
         book = Book.open(@simple_file)
         expect{
           book.save_as(@wrong_name)
-        }.to raise_error(ExcelErrorSave, "filename is nil")
+        }.to raise_error(FileNameNotGiven, "filename is nil")
       end
 
       it "should raise error if workbook is not alive" do
@@ -108,7 +108,7 @@ describe Book do
         book.close
         expect{
           book.save_as(@simple_save_file)
-        }.to raise_error(ExcelErrorSave, "workbook is not alive")
+        }.to raise_error(ObjectNotAlive, "workbook is not alive")
       end
     end
 
@@ -155,7 +155,7 @@ describe Book do
         File.exist?(@simple_file_other_path1).should be_true
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :raise)
-        }.to raise_error(ExcelErrorSave, /blocked by another workbook/)
+        }.to raise_error(WorkbookBlocked, /blocked by another workbook/)
       end
 
       it "should close the blocking workbook without saving, and save the current workbook with :if_obstructed => :forget" do
@@ -237,7 +237,7 @@ describe Book do
         @book.Saved.should be_false      
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :close_if_saved)
-        }.to raise_error(ExcelErrorSave, /blocking workbook is unsaved: "workbook.xls"/)
+        }.to raise_error(WorkbookBlocked, /blocking workbook is unsaved: "workbook.xls"/)
       end
 
       it "should raise an error with an invalid option" do
@@ -248,7 +248,7 @@ describe Book do
         File.exist?(@simple_file_other_path1).should be_true
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :invalid)
-        }.to raise_error(ExcelErrorSave, ":if_obstructed: invalid option: :invalid")
+        }.to raise_error(OptionInvalid, ":if_obstructed: invalid option: :invalid")
       end
 
       it "should raise an error by default" do
@@ -259,7 +259,7 @@ describe Book do
         File.exist?(@simple_file_other_path1).should be_true
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite)
-        }.to raise_error(ExcelErrorSave, /blocked by another workbook/)
+        }.to raise_error(WorkbookBlocked, /blocked by another workbook/)
       end
 
       it "should raise an error if the file does not exist and an workbook with the same name and other path exists" do
@@ -267,7 +267,7 @@ describe Book do
         File.exist?(@simple_file_other_path1).should be_false
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :raise)
-          }.to raise_error(ExcelErrorSave, /blocked by another workbook/)
+          }.to raise_error(WorkbookBlocked, /blocked by another workbook/)
       end
 
       it "should raise an error if the file exists and an workbook with the same name and other path exists" do
@@ -278,7 +278,7 @@ describe Book do
         File.exist?(@simple_file_other_path1).should be_true
         expect{
           @book.save_as(@simple_file_other_path1, :if_exists => :raise, :if_obstructed => :raise)
-        }.to raise_error(ExcelErrorSave, /file already exists: "workbook.xls"/)
+        }.to raise_error(FileAlreadyExists, /file already exists: "workbook.xls"/)
       end
 
     end
@@ -303,7 +303,7 @@ describe Book do
           book_save = Book.open(@simple_save_file1, :excel => :new)
           expect{
             @book.save_as(@simple_save_file1, :if_exists => :overwrite)
-            }.to raise_error(ExcelErrorSave, "workbook is open and used in Excel")
+            }.to raise_error(WorkbookBeingUsed, "workbook is open and used in Excel")
           book_save.close
         end        
 
@@ -340,7 +340,7 @@ describe Book do
           booklength = File.size?(@simple_save_file1)
           expect {
             @book.save_as(@simple_save_file1, :if_exists => :raise)
-            }.to raise_error(ExcelErrorSave, /file already exists: "workbook_save.xls"/)
+            }.to raise_error(FileAlreadyExists, /file already exists: "workbook_save.xls"/)
           File.exist?(@simple_save_file1).should be_true
           File.size?(@simple_save_file1).should == booklength
         end
@@ -381,7 +381,7 @@ describe Book do
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
+              }.to raise_error(WorkbookError, "not saved or canceled by user")
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -396,7 +396,7 @@ describe Book do
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
+              }.to raise_error(WorkbookError, "not saved or canceled by user")
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -407,7 +407,7 @@ describe Book do
             @book.ole_workbook.Close
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :alert)
-              }.to raise_error(ExcelErrorSave, "workbook is not alive")
+              }.to raise_error(ObjectNotAlive, "workbook is not alive")
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -451,7 +451,7 @@ describe Book do
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :excel)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
+              }.to raise_error(WorkbookError, "not saved or canceled by user")
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -466,7 +466,7 @@ describe Book do
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :excel)
-              }.to raise_error(ExcelErrorSave, "not saved or canceled by user")
+              }.to_not raise_error
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -477,7 +477,7 @@ describe Book do
             @book.ole_workbook.Close
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :excel)
-              }.to raise_error(ExcelErrorSave, "workbook is not alive")
+              }.to raise_error(ObjectNotAlive, "workbook is not alive")
             File.exist?(@simple_save_file1).should be_true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
@@ -495,7 +495,7 @@ describe Book do
           booklength = File.size?(@simple_save_file1)
           expect {
             @book.save_as(@simple_save_file1)
-            }.to raise_error(ExcelErrorSave, /file already exists: "workbook_save.xls"/)
+            }.to raise_error(FileAlreadyExists, /file already exists: "workbook_save.xls"/)
           File.exist?(@simple_save_file1).should be_true
           File.size?(@simple_save_file1).should == booklength
         end
@@ -505,7 +505,7 @@ describe Book do
           @book.save_as(@simple_save_file1)
           expect {
             @book.save_as(@simple_save_file1, :if_exists => :invalid)
-            }.to raise_error(ExcelErrorSave, ':if_exists: invalid option: :invalid')
+            }.to raise_error(OptionInvalid, ':if_exists: invalid option: :invalid')
         end
       end
     end

data/spec/book_specs/book_sheet_spec.rb

@@ -183,7 +183,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 "new_sheet" already exists/)
+          }.to raise_error(NameAlreadyExists, /sheet name "new_sheet" already exists/)
       end
     end
   end
@@ -281,7 +281,7 @@ describe Book do
       @book.copy_sheet(@sheet, :as => 'new_sheet')
       expect{
         @book.copy_sheet(@sheet, :as => 'new_sheet')
-        }.to raise_error(ExcelErrorSheet, /sheet name "new_sheet" already exists/)
+        }.to raise_error(NameAlreadyExists, /sheet name "new_sheet" already exists/)
     end
   end
 
@@ -381,7 +381,7 @@ describe Book do
         @book.add_or_copy_sheet(@sheet, :as => 'new_sheet')
         expect{
           @book.add_or_copy_sheet(@sheet, :as => 'new_sheet')
-          }.to raise_error(ExcelErrorSheet, /sheet name "new_sheet" already exists/)
+          }.to raise_error(NameAlreadyExists, /sheet name "new_sheet" already exists/)
       end
     end
   end

data/spec/book_specs/book_unobtr_spec.rb

@@ -113,7 +113,7 @@ describe Book do
         expect{
           Book.unobtrusively(@simple_file, :invalid_option) do |book|
           end
-        }.to raise_error(ExcelErrorOpen, "given object is neither an Excel, a Workbook, nor a Win32ole")
+        }.to raise_error(TypeErrorREO, "given object is neither an Excel, a Workbook, nor a Win32ole")
       end
 
       it "should be visible" do