robust_excel_ole 0.3.5 → 0.3.6

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

@@ -1,13 +1,13 @@
 # -*- coding: utf-8 -*-
 
-require File.join(File.dirname(__FILE__), './spec_helper')
+require File.join(File.dirname(__FILE__), './../spec_helper')
 
 
-module My
-  class Excel < RobustExcelOle::Excel
+module My # :nodoc: #
+  class Excel < RobustExcelOle::Excel # :nodoc: #
   end
 
-  class Book < RobustExcelOle::Book
+  class Book < RobustExcelOle::Book   # :nodoc: #
   end
 end
 

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

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 
-require File.join(File.dirname(__FILE__), './spec_helper')
+require File.join(File.dirname(__FILE__), './../spec_helper')
 
 
 $VERBOSE = nil
@@ -74,8 +74,9 @@ describe Book do
           begin
             @excel1.close
             @excel2.close 
-          rescue ExcelErrorOpen => msg
-            puts "ExcelError: #{msg.message}" if msg.message =~ /Excel instance not alive or damaged/
+          rescue RuntimeError => msg
+            #puts "RuntimeError: #{msg.message}" if msg.message =~ /Excel instance not alive or damaged/
+            nil
           end
         end
 
@@ -119,7 +120,7 @@ describe Book do
         expect{
           Book.unobtrusively(@simple_file, :invalid_option) do |book|
           end
-        }.to raise_error(ExcelErrorOpen, "provided instance is neither an Excel nor a Book")
+        }.to raise_error(ExcelError, "receiver instance is neither an Excel nor a Book")
       end
 
     end
@@ -135,6 +136,42 @@ describe Book do
         @book2.close(:if_unsaved => :forget) rescue nil
       end
 
+      #it "should let an open Book open" do
+      #  Book.unobtrusively(@simple_file) do |book|
+      #    book.should be_a Book
+      #    book.should be_alive
+      #    book.excel.should == @book.excel
+      #  end        
+      #  @book.should be_alive
+      #  @book.should be_a Book
+      #end
+
+      it "should let an open Book open if it has been closed and opened again" do
+        @book.close
+        @book.reopen
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should == @book.excel
+        end        
+        @book.should be_alive
+        @book.should be_a Book
+      end
+
+      it "should let an open Book open if two books have been opened and one has been closed and opened again" do
+        book2 = Book.open(@different_file, :force_excel => :new)
+        @book.close
+        book2.close
+        @book.reopen
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should == @book.excel
+        end        
+        @book.should be_alive
+        @book.should be_a Book
+      end
+
       it "should open in the Excel of the given Book" do
         #book1 = Book.open(@different_file)
         @book2 = Book.open(@another_simple_file, :force_excel => :new)
@@ -323,8 +360,31 @@ describe Book do
         new_sheet[1,1].Value.should_not == old_cell_value
         new_book.close
       end
+    end
+
+    context "with a visible book" do
 
+      before do
+        @book = Book.open(@simple_file, :visible => true)
+      end
 
+      after do
+        @book.close(:if_unsaved => :forget)
+        @book2.close(:if_unsaved => :forget) rescue nil
+      end
+
+      it "should let an open Book open" do
+        Book.unobtrusively(@simple_file) do |book|
+          book.should be_a Book
+          book.should be_alive
+          book.excel.should == @book.excel
+          book.excel.Visible.should be_true
+        end        
+        @book.should be_alive
+        @book.should be_a Book
+        @book.excel.Visible.should be_true
+      end
+      
     end
 
     context "with various options for an Excel instance in which to open a closed book" do

data/spec/bookstore_spec.rb

@@ -2,13 +2,14 @@
 
 require File.join(File.dirname(__FILE__), './spec_helper')
 
+=begin
 RSpec.configure do |config|
 
   config.mock_with :rspec do |mocks|
     mocks.syntax = :should
   end
 end
-
+=end
 
 $VERBOSE = nil
 
@@ -27,7 +28,6 @@ module RobustExcelOle
   end
 end
 
-
 $mock_bookstore = MockBookstore.new
 
 class Book
@@ -54,8 +54,15 @@ describe Bookstore do
   end
 
   after do
-    Excel.close_all
-    rm_tmp(@dir)
+    begin
+      Excel.close_all
+    rescue WeakRef::RefError => msg
+      puts "#{msg.message}"
+      Excel.kill_all
+    end
+    begin
+      rm_tmp(@dir) rescue nil
+    end
   end
 
   describe "create bookstore" do

data/spec/cell_spec.rb

@@ -1,13 +1,15 @@
 # -*- coding: utf-8 -*-
 require File.join(File.dirname(__FILE__), './spec_helper')
 
-describe RobustExcelOle::Cell do
+include RobustExcelOle
+
+describe Cell do
 
   before(:all) do
-    excel = RobustExcelOle::Excel.new(:reuse => true)
+    excel = Excel.new(:reuse => true)
     open_books = excel == nil ? 0 : excel.Workbooks.Count
     puts "*** open books *** : #{open_books}" if open_books > 0
-    RobustExcelOle::Excel.close_all
+    Excel.close_all
   end
 
   before do
@@ -20,7 +22,7 @@ describe RobustExcelOle::Cell do
 
   context "open simple.xls" do
     before do
-      @book = RobustExcelOle::Book.open(@dir + '/workbook.xls', :read_only => true)
+      @book = Book.open(@dir + '/workbook.xls', :read_only => true)
       @sheet = @book[1]
       @cell = @sheet[1, 1]
     end
@@ -52,7 +54,7 @@ describe RobustExcelOle::Cell do
 
   context "open merge_cells.xls" do
     before do
-      @book = RobustExcelOle::Book.open(@dir + '/merge_cells.xls', :read_only => true)
+      @book = Book.open(@dir + '/merge_cells.xls', :read_only => true)
       @sheet = @book[0]
     end
 

data/spec/excel_spec.rb

@@ -4,12 +4,15 @@ require File.join(File.dirname(__FILE__), './spec_helper')
 
 $VERBOSE = nil
 
+include RobustExcelOle
+
 module RobustExcelOle
 
   describe Excel do
 
     before(:all) do
-      Excel.close_all
+      Excel.kill_all
+      #Excel.close_all
     end
 
     before do
@@ -22,7 +25,8 @@ module RobustExcelOle
     end
 
     after do
-      Excel.close_all
+      #Excel.close_all
+      Excel.kill_all
       rm_tmp(@dir)
     end
 
@@ -51,37 +55,147 @@ module RobustExcelOle
         creation_ok?
       end
 
+      it "should work with 'reuse' " do
+        excel = Excel.create
+        @excel = Excel.new(:reuse => true)
+        creation_ok?
+        @excel.should === excel
+        @excel = Excel.current
+        creation_ok?
+        @excel.should === excel
+      end
+
       it "should work with 'create' " do
+        excel = Excel.create
+        @excel = Excel.new(:reuse => false)
+        creation_ok?
+        @excel.should_not == excel
         @excel = Excel.create
         creation_ok?
+        @excel.should_not == excel
       end
 
+      it "should work with reusing (uplifting) an Excel given as WIN32Ole object" do
+        excel0 = Excel.create
+        book = Book.open(@simple_file)
+        excel = book.excel
+        win32ole_excel = WIN32OLE.connect(book.workbook.Fullname).Application
+        @excel = Excel.new(:reuse => win32ole_excel)
+        creation_ok?
+        @excel.should be_a Excel
+        @excel.should be_alive
+        @excel.should == excel
+      end
     end
 
-    context "with existing excel" do
+    context "with identity transparence" do
 
       before do
         @excel1 = Excel.create
       end
 
-      it "should create different excel" do
+      it "should create different Excel instances" do
         excel2 = Excel.create
+        excel2.should_not == @excel1
         excel2.Hwnd.should_not == @excel1.Hwnd
       end
 
-      it "should reuse existing excel" do
+      it "should reuse the existing Excel instances" do
         excel2 = Excel.current
+        excel2.should === @excel1
         excel2.Hwnd.should == @excel1.Hwnd
       end
 
-      it "should reuse existing excel with default options for 'new'" do
+      it "should reuse existing Excel instance with default options for 'new'" do
         excel2 = Excel.new
-        excel2.should be_a Excel
+        excel2.should === @excel1
         excel2.Hwnd.should == @excel1.Hwnd
       end
 
+      it "should yield the same Excel instances for the same Excel objects" do
+        excel2 = @excel1
+        excel2.Hwnd.should == @excel1.Hwnd
+        excel2.should === @excel1
+      end
+    end
+
+    context "excel_processes" do
+        
+      before do
+        @excel1 = Excel.create
+        @excel2 = Excel.create
+      end
+
+      it "should yield Excel objects" do        
+        excels = Excel.excel_processes    
+        excels[0].should == @excel1
+        excels[1].should == @excel2
+      end
+
+    end
+
+    context "kill Excel processes hard" do
+
+      before do
+        @excel1 = Excel.create
+        @excel2 = Excel.create
+      end
+
+      it "should kill Excel processes" do
+        Excel.kill_all
+        @excel1.alive?.should be_false
+        @excel2.alive?.should be_false
+      end
+
     end
 
+    context "with recreating Excel instances" do
+
+      it "should recreate a single Excel instance" do
+        book = Book.open(@simple_file)
+        excel = book.excel
+        excel.close
+        excel.should_not be_alive
+        excel.recreate
+        excel.should be_a Excel
+        excel.should be_alive
+        book.should be_alive
+        excel.close
+        excel.should_not be_alive
+      end
+
+      it "should recreate several Excel instances" do  
+        book = Book.open(@simple_file)      
+        book2 = Book.open(@another_simple_file, :force_excel => book)
+        book3 = Book.open(@different_file, :force_excel => :new)
+        excel = book.excel
+        excel3 = book3.excel
+        excel.close
+        excel3.close
+        excel.should_not be_alive
+        excel3.should_not be_alive
+        excel.recreate(:visible => true)
+        excel.should be_alive
+        excel.should be_a Excel
+        excel.visible.should be_true
+        excel.displayalerts.should be_false
+        book.should be_alive
+        book2.should be_alive
+        book.excel.should == excel
+        book2.excel.should == excel
+        excel3.recreate(:visible => true, :displayalerts => true)
+        excel3.should be_alive
+        excel3.should be_a Excel
+        excel3.visible.should be_true
+        excel3.displayalerts.should be_true
+        book3.should be_alive
+        excel.close
+        excel.should_not be_alive
+        excel3.close
+        excel3.should_not be_alive
+      end
+    end    
+
     context "close excel instances" do
       def direct_excel_creation_helper  # :nodoc: #
         expect { WIN32OLE.connect("Excel.Application") }.to raise_error
@@ -102,50 +216,162 @@ module RobustExcelOle
       end
     end
 
-=begin
-      # testing private methods
-      context "close_excel" do
+    describe "close_all" do
+
+      context "with saved workbooks" do
 
         before do
-          @book = Book.open(@simple_file, :visible => true)
-          @excel = @book.excel
-          @book2 = Book.open(@simple_file, :force_excel => :new, :visible => true)
-          @excel2 = @book2.excel
+          book = Book.open(@simple_file)
+          book2 = Book.open(@simple_file, :force_excel => :new)
+          @excel = book.excel
+          @excel2 = book2.excel
         end
 
-        it "should close one Excel" do
+        it "should close the Excel instances" do
           @excel.should be_alive
           @excel2.should be_alive
-          @book.should be_alive
-          @book2.should be_alive
-          @excel.close_excel(:hard => false)
+          Excel.close_all
           @excel.should_not be_alive
-          @book.should_not be_alive
-          @excel2.should be_alive
-          @book2.should be_alive
+          @excel2.should_not be_alive
         end
       end
 
-=end
+      context "with unsaved_workbooks" do
+        
+        before do
+          book1 = Book.open(@simple_file, :read_only => true)
+          book2 = Book.open(@simple_file, :force_excel => :new)
+          book3 = Book.open(@another_simple_file, :force_excel => book2.excel)
+          book4 = Book.open(@different_file, :force_excel => :new)
+          @excel1 = book1.excel
+          @excel2 = book2.excel
+          @excel4 = book4.excel
+          sheet2 = book2[0]
+          @old_cell_value2 = sheet2[1,1].value
+          sheet2[1,1] = sheet2[1,1].value == "foo" ? "bar" : "foo"
+          sheet3 = book3[0]
+          @old_cell_value3 = sheet3[1,1].value
+          sheet3[1,1] = sheet3[1,1].value == "foo" ? "bar" : "foo"
+        end
+
+        it "should close the first Excel without unsaved workbooks and then raise an error" do
+          expect{
+            Excel.close_all
+          }.to raise_error(ExcelErrorClose, "Excel contains unsaved workbooks")
+          @excel1.should_not be_alive
+        end
 
-    describe "close_all" do
+        it "should close the Excel instances without saving the unsaved workbooks" do
+          Excel.close_all(:if_unsaved => :forget)
+          @excel1.should_not be_alive
+          @excel2.should_not be_alive
+          @excel4.should_not be_alive
+          new_book2 = Book.open(@simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should == @old_cell_value2
+          new_book2.close   
+          new_book3 = Book.open(@another_simple_file)
+          new_sheet3 = new_book3[0]
+          new_sheet3[1,1].value.should == @old_cell_value3
+          new_book3.close   
+        end
 
-      context "with saved workbooks" do
+        it "should close the Excel instances with saving the unsaved workbooks" do
+          Excel.close_all(:if_unsaved => :save)
+          @excel1.should_not be_alive
+          @excel2.should_not be_alive
+          @excel4.should_not be_alive
+          new_book2 = Book.open(@simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should_not == @old_cell_value2
+          new_book2.close   
+          new_book3 = Book.open(@another_simple_file)
+          new_sheet3 = new_book3[0]
+          new_sheet3[1,1].value.should_not == @old_cell_value3
+          new_book3.close
+        end
+
+        it "should raise an error for invalid option" do
+          expect {
+            Excel.close_all(:if_unsaved => :invalid_option)
+          }.to raise_error(ExcelErrorClose, ":if_unsaved: invalid option: :invalid_option") 
+        end
+
+        it "should raise an error by default" do
+          expect{
+            Excel.close_all
+          }.to raise_error(ExcelErrorClose, "Excel contains unsaved workbooks")
+          @excel1.should_not be_alive
+        end
+      end
 
+      context "with :if_unsaved => :alert" do
         before do
-          book = Book.open(@simple_file, :visible => true)
-          book2 = Book.open(@simple_file, :force_excel => :new, :visible => true)
-          @excel = book.excel
+          @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
+          #book1 = Book.open(@simple_file, :read_only => true)
+          book2 = Book.open(@simple_file, :force_excel => :new)
+          #book3 = Book.open(@another_simple_file, :force_excel => book2.excel)
+          #book4 = Book.open(@different_file, :force_excel => :new)
+          #@excel1 = book1.excel
           @excel2 = book2.excel
+          #@excel4 = book4.excel
+          sheet2 = book2[0]
+          @old_cell_value2 = sheet2[1,1].value
+          sheet2[1,1] = sheet2[1,1].value == "foo" ? "bar" : "foo"
+          #sheet3 = book3[0]
+          #@old_cell_value3 = sheet3[1,1].value
+          #sheet3[1,1] = sheet3[1,1].value == "foo" ? "bar" : "foo"          
         end
 
-        it "should close the Excel instances" do
-          @excel.should be_alive
-          @excel2.should be_alive
-          Excel.close_all
-          @excel.should_not be_alive
+        after do
+          @key_sender.close
+        end
+
+
+        it "should save if user answers 'yes'" do
+          @key_sender.puts "{enter}"
+          Excel.close_all(:if_unsaved => :alert)
+          #@excel1.should_not be_alive
           @excel2.should_not be_alive
+          #@excel4.should_not be_alive
+          #@excel5.should_not be_alive
+          new_book2 = Book.open(@simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should_not == @old_cell_value2
+          new_book2.close   
+          #new_book3 = Book.open(@another_simple_file)
+          #new_sheet3 = new_book3[0]
+          #new_sheet3[1,1].value.should == @old_cell_value3
+          #new_book3.close       
         end
+
+        it "should not save if user answers 'no'" do            
+          @key_sender.puts "{right}{enter}"
+          @key_sender.puts "{right}{enter}"
+          Excel.close_all(:if_unsaved => :alert)
+          #@excel1.should_not be_alive
+          @excel2.should_not be_alive
+          #@excel4.should_not be_alive
+          #@excel5.should_not be_alive
+          new_book2 = Book.open(@simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should == @old_cell_value2
+          new_book2.close   
+          #new_book4 = Book.open(@different_file)
+          #new_sheet4 = new_book4[0]
+          #new_sheet4[1,1].value.should_not == @old_cell_value4
+          #new_book4.close   
+        end
+
+      #  it "should not save if user answers 'cancel'" do
+      #    @key_sender.puts "{left}{enter}"
+      #    @key_sender.puts "{left}{enter}"
+      #    @key_sender.puts "{left}{enter}"
+      #    @key_sender.puts "{left}{enter}"
+      #    expect{
+      #      Excel.close_all(:if_unsaved => :alert)
+      #      }.to raise_error(ExcelUserCanceled, "close: canceled by user")
+      #  end
       end
     end
 
@@ -167,7 +393,7 @@ module RobustExcelOle
         end
       end
 
-      context "with an unsaved workbook" do
+      context "with unsaved workbooks" do
 
         before do
           @excel = Excel.create
@@ -175,82 +401,155 @@ module RobustExcelOle
           sheet = @book[0]
           @old_cell_value = sheet[1,1].value
           sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+          @book2 = Book.open(@another_simple_file)
+          sheet2 = @book2[0]
+          @old_cell_value2 = sheet2[1,1].value
+          sheet2[1,1] = sheet2[1,1].value == "foo" ? "bar" : "foo"
         end
 
         it "should raise an error" do
           @excel.should be_alive
           @book.should be_alive
           @book.saved.should be_false
+          @book2.should be_alive
+          @book2.saved.should be_false
           expect{
             @excel.close(:if_unsaved => :raise)
           }.to raise_error(ExcelErrorClose, "Excel contains unsaved workbooks")
-          @excel.should be_alive
-          @book.should be_alive
         end
 
         it "should close the Excel without saving the workbook" do
           @excel.should be_alive
-          @book.should be_alive
-          @book.saved.should be_false
           @excel.close(:if_unsaved => :forget)
           @excel.should_not be_alive
-          @book.should_not be_alive
           new_book = Book.open(@simple_file)
           new_sheet = new_book[0]
           new_sheet[1,1].value.should == @old_cell_value
           new_book.close          
+          new_book2 = Book.open(@another_simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should == @old_cell_value2
+          new_book2.close 
         end
 
         it "should close the Excel with saving the workbook" do
           @excel.should be_alive
-          @book.should be_alive
-          @book.saved.should be_false
           @excel.close(:if_unsaved => :save)
           @excel.should_not be_alive
-          @book.should_not be_alive
           new_book = Book.open(@simple_file)
           new_sheet = new_book[0]
           new_sheet[1,1].value.should_not == @old_cell_value
           new_book.close          
+          new_book2 = Book.open(@another_simple_file)
+          new_sheet2 = new_book2[0]
+          new_sheet2[1,1].value.should_not == @old_cell_value2
+          new_book2.close   
         end
 
         it "should raise an error for invalid option" do
           expect {
             @excel.close(:if_unsaved => :invalid_option)
-          }.to raise_error(ExcelErrorClose, ":if_unsaved: invalid option: invalid_option") 
+          }.to raise_error(ExcelErrorClose, ":if_unsaved: invalid option: :invalid_option") 
         end
 
         it "should raise an error by default" do
           @excel.should be_alive
-          @book.should be_alive
-          @book.saved.should be_false
           expect{
             @excel.close
           }.to raise_error(ExcelErrorClose, "Excel contains unsaved workbooks")
+        end
+  
+        it "should close the Excel without saving the workbook hard" do
           @excel.should be_alive
           @book.should be_alive
+          @book.saved.should be_false
+          @excel.close(:if_unsaved => :forget, :hard => true)
+          @excel.should_not be_alive
+          @book.should_not be_alive
+          new_book = Book.open(@simple_file)
+          new_sheet = new_book[0]
+          new_sheet[1,1].value.should == @old_cell_value
+          new_book.close    
+          new_book.excel.close(:hard => true)
+          procs = WIN32OLE.connect("winmgmts:\\\\.")
+          processes = procs.InstancesOf("win32_process")     
+          result = []
+          processes.each do |p|
+            result << p if p.name == "EXCEL.EXE"
+          end
+          result.should be_empty
+        end
+      end
+
+      context "with :if_unsaved => :alert" do
+
+        before do
+          @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
+          @excel = Excel.create
+          @book = Book.open(@simple_file)
+          sheet = @book[0]
+          @old_cell_value = sheet[1,1].value
+          sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
+        end
+
+        after do
+          @key_sender.close
+        end
+
+        it "should save if user answers 'yes'" do
+          # "Yes" is to the left of "No", which is the  default. --> language independent
+          @excel.should be_alive
+          @key_sender.puts "{enter}" 
+          @excel.close(:if_unsaved => :alert)
+          @excel.should_not be_alive
+          new_book = Book.open(@simple_file)
+          new_sheet = new_book[0]
+          new_sheet[1,1].value.should_not == @old_cell_value
+          new_book.close   
+        end
+
+        it "should not save if user answers 'no'" do            
+          @excel.should be_alive
+          @book.should be_alive
+          @book.saved.should be_false
+          @key_sender.puts "{right}{enter}"
+          @excel.close(:if_unsaved => :alert)
+          @excel.should_not be_alive
+          @book.should_not be_alive
+          new_book = Book.open(@simple_file)
+          new_sheet = new_book[0]
+          new_sheet[1,1].value.should == @old_cell_value
+          new_book.close     
+        end
+
+        it "should not save if user answers 'cancel'" do
+          # strangely, in the "cancel" case, the question will sometimes be repeated twice            
+          @excel.should be_alive
+          @book.should be_alive
+          @book.saved.should be_false
+          @key_sender.puts "{left}{enter}"
+          @key_sender.puts "{left}{enter}"
+          expect{
+            @excel.close(:if_unsaved => :alert)
+            }.to raise_error(ExcelUserCanceled, "close: canceled by user")
         end
       end
     end
 
-#        context "with :if_unsaved => :alert" do
-#          before do
-#            @key_sender = IO.popen  'ruby "' + File.join(File.dirname(__FILE__), '/helpers/key_sender.rb') + '" "Microsoft Excel" '  , "w"
-#          end
-#
-#          after do
-#            @key_sender.close
-#          end
-#
-#         possible_answers = [:yes, :no, :cancel]
-#          possible_answers.each_with_index do |answer, position|
-#            it "should" + (answer == :yes ? "" : " not") + " the unsaved book and" + (answer == :cancel ? " not" : "") + " close it" + "if user answers '#{answer}'" do
-#            # "Yes" is the  default. "No" is right of "Yes", "Cancel" is right of "No" --> language independent
-#            @key_sender.puts  "{right}" * position + "{enter}"
-#          end
-#        end
-#      end
+    describe "alive" do
 
+      it "should yield alive" do
+        excel = Excel.create
+        excel.alive?.should be_true
+      end
+
+      it "should yield not alive" do
+        excel = Excel.create
+        excel.close
+        excel.alive?.should be_false
+      end
+
+    end
 
     describe "==" do
       before do
@@ -365,12 +664,18 @@ module RobustExcelOle
       end
 
       it "should raise WIN32OLERuntimeError" do
-        expect{ @excel1.NonexistingMethod }.to raise_error(VBAMethodMissingError)
+        expect{ @excel1.NonexistingMethod }.to raise_error(VBAMethodMissingError, /unknown VBA property or method :NonexistingMethod/)
       end
 
       it "should raise NoMethodError for uncapitalized methods" do
         expect{ @excel1.nonexisting_method }.to raise_error(NoMethodError)
       end
+
+      it "should report that Excel is not alive" do
+        @excel1.close
+        expect{ @excel1.Nonexisting_method }.to raise_error(ExcelError, "method missing: Excel not alive")
+      end
+
     end
 
     context "with hwnd and hwnd2excel" do
@@ -485,31 +790,6 @@ module RobustExcelOle
       end
     end
   end
-
-  describe "RobustExcelOle" do
-    context "#absolute_path" do
-      before do
-        @previous_dir = Dir.getwd
-      end
-      
-      after do
-        Dir.chdir @previous_dir
-      end
-
-      it "should work" do
-        RobustExcelOle::absolute_path("C:/abc").should == "C:\\abc"
-        RobustExcelOle::absolute_path("C:\\abc").should == "C:\\abc"
-        Dir.chdir "C:/windows"
-        RobustExcelOle::absolute_path("C:abc").downcase.should == Dir.pwd.gsub("/","\\").downcase + "\\abc"
-        RobustExcelOle::absolute_path("C:abc").upcase.should   == File.expand_path("abc").gsub("/","\\").upcase
-      end
-
-      it "should return right absolute path name" do
-        @filename = 'C:/Dokumente und Einstellungen/Zauberthomas/Eigene Dateien/robust_excel_ole/spec/book_spec.rb'
-        RobustExcelOle::absolute_path(@filename).gsub("\\","/").should == @filename
-      end
-    end
-  end
 end
 
 class TestError < RuntimeError  # :nodoc: #