robust_excel_ole 1.1.4 → 1.1.5

data/spec/book_specs/book_open_spec.rb

@@ -29,6 +29,7 @@ describe Book do
     @simple_file_xlsx = @dir + '/workbook.xlsx'
     @simple_file1 = @simple_file
     @different_file1 = @different_file
+    @simple_file_other_path1 = @simple_file_other_path
   end
 
   after do
@@ -36,6 +37,40 @@ describe Book do
     #rm_tmp(@dir)
   end
 
+  describe "simple open" do
+
+    it "should simply open" do
+      book = Workbook.open(@simple_file, :v => true, :f => {:e => :new})
+    end
+
+  end
+
+  describe "new" do
+
+    it "should simply create a new one" do
+      book = Workbook.new(@simple_file)
+      book.should be_alive
+      book.should be_a Book
+    end
+
+    it "should set options" do
+      book = Workbook.new(@simple_file, :visible => true, :read_only => true, :force => {:excel => :new})
+      book.should be_alive
+      book.should be_a Book
+      book.excel.Visible.should be true
+      book.Windows(book.Name).Visible.should be true
+      book.ReadOnly.should be true
+      book2 = Workbook.new(@different_file, :force => {:excel => :new}, :v => true)
+      book2.should be_alive
+      book2.should be_a Book
+      book2.excel.Visible.should be true
+      book2.Windows(book2.Name).Visible.should be true
+      book2.ReadOnly.should be false
+      book2.excel.should_not == book.excel
+    end
+
+  end
+
   describe "open" do
 
     context "with calculation mode" do
@@ -111,8 +146,8 @@ describe Book do
         book2.should be_a Book
         book2.excel.should_not == @book.excel 
         book2.should_not == @book
-        @book.Readonly.should be_false
-        book2.Readonly.should be_true
+        @book.Readonly.should be false
+        book2.Readonly.should be true
         book2.close
       end
     end
@@ -135,8 +170,8 @@ describe Book do
         new_book.should == @book
         new_book.filename.should == @book.filename
         new_book.excel.should == @book.excel
-        new_book.excel.Visible.should be_false
-        new_book.excel.DisplayAlerts.should be_false
+        new_book.excel.Visible.should be false
+        new_book.excel.DisplayAlerts.should be false
         new_book.should === @book
         new_book.close
       end
@@ -149,8 +184,8 @@ describe Book do
         new_book.should == @book
         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_true
+        new_book.excel.Visible.should be true
+        new_book.excel.DisplayAlerts.should be true
         new_book.should === @book
         new_book.close
       end
@@ -165,8 +200,8 @@ describe Book do
         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
 
@@ -252,7 +287,8 @@ describe Book do
         @book.close
         @book.should_not be_alive
         book2 = Book.open(@simple_file1, :force => {:excel => new_excel})
-        book2.should_not === @book
+        #@book.should be_alive
+        #book2.should === @book
         book2.should be_alive
         book2.excel.should == new_excel
         book2.excel.should_not == old_excel
@@ -274,6 +310,91 @@ describe Book do
 
     end
 
+    context "with abbrevations" do
+
+      before do
+        @book = Book.open(@simple_file1)
+      end
+
+      after do
+        @book.close rescue nil
+      end
+
+      it "should work as force" do
+        book2 = Book.open(@another_simple_file, :excel => :new)
+        book2.excel.should_not == @book.excel
+        book3 = Book.open(@different_file, :excel => book2.excel)
+        book3.excel.should == book2.excel
+      end
+
+      it "should work with abbrevation of force and excel" do
+        book2 = Book.open(@another_simple_file, :f => {:e => :new})
+        book2.excel.should_not == @book.excel
+        book3 = Book.open(@different_file, :f => {:e => book2.excel})
+        book3.excel.should == book2.excel
+      end
+
+      it "should work with abbrevation of force" do
+        book2 = Book.open(@another_simple_file, :f => {:excel => :new})
+        book2.excel.should_not == @book.excel
+        book3 = Book.open(@different_file, :f => {:excel => book2.excel})
+        book3.excel.should == book2.excel
+      end
+
+      it "should work with abbrevation of force" do
+        book2 = Book.open(@another_simple_file, :force => {:e => :new})
+        book2.excel.should_not == @book.excel
+        book3 = Book.open(@different_file, :force => {:e => book2.excel})
+        book3.excel.should == book2.excel
+      end
+
+      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_file1, :excel => book2.excel)
+        book3.excel.should === book2.excel
+        book4 = Book.open(@simple_file1, :excel => @book) 
+        book4.excel.should === @book.excel
+        book3.close
+        book4.close
+        book5 = Book.open(@simple_file1, :excel => book2.ole_workbook)
+        book5.excel.should ===  book2.excel
+        win32ole_excel1 = WIN32OLE.connect(@book.ole_workbook.Fullname).Application
+        book6 = Book.open(@simple_file1, :excel => win32ole_excel1)
+        book6.excel.should === @book.excel
+      end
+
+      it "should use abbreviations of default" do
+        book2 = Book.open(@simple_file1, :d => {:excel => :current})
+        book2.excel.should == @book.excel
+      end
+
+      it "should use abbreviations of default" do
+        book2 = Book.open(@simple_file1, :d => {:e => :current})
+        book2.excel.should == @book.excel
+      end
+
+      it "should use abbreviations of default" do
+        book2 = Book.open(@simple_file1, :default => {:e => :current})
+        book2.excel.should == @book.excel
+      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, :d => {:e => :current})
+        book2.excel.should == excel1
+        book2.close
+        book3 = Book.open(@simple_file1, :e => excel2)
+        book3.close
+        book3 = Book.open(@simple_file1, :d => {:e => :current})
+        book3.excel.should == excel2
+        book3.close
+      end
+
+    end
+
     context "with :force => {:excel}" do
 
       before do
@@ -300,15 +421,14 @@ describe Book do
         book6.excel.should === @book.excel
       end
 
-
       it "should open in a new Excel" do
         book2 = Book.open(@simple_file1, :force => {:excel => :new})
         book2.should be_alive
         book2.should be_a Book
         book2.excel.should_not == @book.excel 
         book2.should_not == @book
-        @book.Readonly.should be_false
-        book2.Readonly.should be_true
+        @book.Readonly.should be false
+        book2.Readonly.should be true
         book2.close
       end
 
@@ -323,14 +443,14 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.Readonly.should == true
+        #book4.Readonly.should == true
         book4.should_not == book2 
         book4.close
         book5 = Book.open(@simple_file1, :force => {:excel => book2})
         book5.should be_alive
         book5.should be_a Book
         book5.excel.should == book2.excel
-        book5.Readonly.should == true
+        #book5.Readonly.should == true
         book5.should_not == book2 
         book5.close
         book3.close
@@ -349,14 +469,14 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.ReadOnly.should be_true
+        book4.ReadOnly.should be true
         book4.should == book2
         book4.close
         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
-        book5.ReadOnly.should be_true
+        book5.ReadOnly.should be true
         book5.should == book2
         book5.close
         book3.close
@@ -371,7 +491,7 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.ReadOnly.should be_true
+        book4.ReadOnly.should be true
         book4.should == book2
         book4.close
       end
@@ -465,8 +585,8 @@ describe Book do
         book2.should be_a Book
         book2.excel.should_not == @book.excel 
         book2.should_not == @book
-        @book.Readonly.should be_false
-        book2.Readonly.should be_true
+        @book.Readonly.should be false
+        book2.Readonly.should be true
         book2.close
       end
 
@@ -481,14 +601,14 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.Readonly.should == true
+        #book4.Readonly.should == false
         book4.should_not == book2 
         book4.close
         book5 = Book.open(@simple_file1, :force_excel => book2)
         book5.should be_alive
         book5.should be_a Book
         book5.excel.should == book2.excel
-        book5.Readonly.should == true
+        #book5.Readonly.should == false
         book5.should_not == book2 
         book5.close
         book3.close
@@ -507,14 +627,14 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.ReadOnly.should be_true
+        book4.ReadOnly.should be true
         book4.should == book2
         book4.close
         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
-        book5.ReadOnly.should be_true
+        book5.ReadOnly.should be true
         book5.should == book2
         book5.close
         book3.close
@@ -529,7 +649,7 @@ describe Book do
         book4.should be_alive
         book4.should be_a Book
         book4.excel.should == book2.excel
-        book4.ReadOnly.should be_true
+        book4.ReadOnly.should be true
         book4.should == book2
         book4.close
       end
@@ -1175,40 +1295,65 @@ describe Book do
 
     end
 
+    it "should new_excel" do
+      book = Book.open(@simple_file1)
+      book.sheet(1)[1,1].Value = "f"
+      book.Saved.should be false
+      book2 = Book.open(@simple_file1, :if_unsaved => :new_excel)
+    end
+
     context "with :if_unsaved" do
 
       before do
         @book = Book.open(@simple_file1)
-        #@book.Windows(@book.Name).Visible = true
-        @sheet = @book.sheet(1)
-        @book.add_sheet(@sheet, :as => 'a_name')
+        sheet = @book.sheet(1)
+        #@book.add_sheet(@sheet, :as => 'a_name')
+        @old_value = sheet[1,1].Value
+        sheet[1,1] = (sheet[1,1].value == "foo" ? "bar" : "foo")
+        @new_value = sheet[1,1].Value
+        @book.Saved.should be false
       end
 
       after do
         @book.close(:if_unsaved => :forget)
-        @new_book.close rescue nil
+      end
+
+      it "should open the book in a new excel instance, if :if_unsaved is :new_excel" do
+        new_book = Book.open(@simple_file1, :if_unsaved => :new_excel)
+        new_book.excel.should_not == @book.excel
+        @book.should be_alive
+        @book.Saved.should be false
+        new_book.should be_alive
+        new_book.Saved.should be true
+        new_book.sheet(1)[1,1].Value.should == @old_value
+        #new_book.filename.should == @book.filename
+        new_book.excel.should_not == @book.excel       
+        new_book.close
       end
 
       it "should raise an error, if :if_unsaved is :raise" do
         expect {
-          @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
+          new_book = Book.open(@simple_file1, :if_unsaved => :raise)
         }.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
-        expect {
-          @new_book = Book.open(@simple_file1, :if_unsaved => :accept)
-          }.to_not raise_error
+        new_book = Book.open(@simple_file1, :if_unsaved => :accept)
         @book.should be_alive
-        @new_book.should be_alive
-        @new_book.should == @book
+        new_book.should be_alive
+        new_book.Saved.should be false      
+        @book.Saved.should be false  
+        new_book.sheet(1)[1,1].Value.should == @new_value
+        new_book.should == @book
       end
 
       it "should open book and close old book, if :if_unsaved is :forget" do
-        @new_book = Book.open(@simple_file1, :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
+        new_book.should be_alive
+        new_book.Saved.should be true
+        new_book.sheet(1)[1,1].Value.should == @old_value
+        #@new_book.filename.downcase.should == @simple_file.downcase
       end
 
       context "with :if_unsaved => :alert" do
@@ -1223,10 +1368,12 @@ 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_file1, :if_unsaved => :alert)
-          @new_book.should be_alive
-          @new_book.filename.downcase.should == @simple_file.downcase
-          @book.should_not be_alive
+          new_book = Book.open(@simple_file1, :if_unsaved => :alert)
+          new_book.should be_alive
+          #@book.should_not be_alive
+          #new_book.filename.downcase.should == @simple_file.downcase
+          new_book.Saved.should be true
+          new_book.sheet(1)[1,1].Value.should == @old_value
         end
       end
 =begin
@@ -1258,10 +1405,11 @@ 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_file1, :if_unsaved => :excel)
-          @new_book.should be_alive
-          @new_book.filename.downcase.should == @simple_file1.downcase
-          @book.should_not be_alive
+          new_book = Book.open(@simple_file1, :if_unsaved => :excel)
+          new_book.should be_alive
+          #@book.should_not be_alive
+          new_book.Saved.should be true
+          new_book.sheet(1)[1,1].Value.should == @old_value
         end
 
 =begin
@@ -1281,24 +1429,15 @@ describe Book do
 
       end
 
-      it "should open the book in a new excel instance, if :if_unsaved is :new_excel" do
-        @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
-        @new_book.excel.should_not == @book.excel       
-        @new_book.close
-      end
-
       it "should raise an error, if :if_unsaved is default" do
         expect {
-          @new_book = Book.open(@simple_file1, :if_unsaved => :raise)
+          new_book = Book.open(@simple_file1, :if_unsaved => :raise)
         }.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)
+          new_book = Book.open(@simple_file1, :if_unsaved => :invalid_option)
         }.to raise_error(OptionInvalid, ":if_unsaved: invalid option: :invalid_option")
       end
     end
@@ -1314,79 +1453,90 @@ describe Book do
               book_before = Book.open(@simple_file1)
               book_before.close
             end
-            @book = Book.open(@simple_file_other_path)
+            @book = Book.open(@simple_file_other_path1)
             #@book.Windows(@book.Name).Visible = true
-            @sheet_count = @book.ole_workbook.Worksheets.Count
-            @sheet = @book.sheet(1)
-            @book.add_sheet(@sheet, :as => 'a_name')
+            #@sheet_count = @book.ole_workbook.Worksheets.Count
+            sheet = @book.sheet(1)
+            #@book.add_sheet(@sheet, :as => 'a_name')
+            @old_value = sheet[1,1].Value
+            sheet[1,1] = (sheet[1,1].value == "foo" ? "bar" : "foo")
+            @new_value = sheet[1,1].Value
+            @book.Saved.should be false
           end
 
           after do
             @book.close(:if_unsaved => :forget)
-            @new_book.close rescue nil
+            #@new_book.close rescue nil
           end
 
           it "should raise an error, if :if_obstructed is :raise" do
             expect {
-              @new_book = Book.open(@simple_file1, :if_obstructed => :raise)
+              new_book = Book.open(@simple_file1, :if_obstructed => :raise)
             }.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
-            @new_book = Book.open(@simple_file1, :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
+            new_book.should be_alive
+            new_book.filename.downcase.should == @simple_file.downcase
+            old_book = Book.open(@simple_file_other_path1, :if_obstructed => :forget)
+            old_book.sheet(1)[1,1].Value.should == @old_value
           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_file1, :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_file1.downcase
-            old_book = Book.open(@simple_file_other_path, :if_obstructed => :forget)
-            old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
+            new_book.should be_alive
+            new_book.filename.downcase.should == @simple_file1.downcase
+            old_book = Book.open(@simple_file_other_path1, :if_obstructed => :forget)
+            old_book.sheet(1)[1,1].Value.should == @new_value
+            #old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
             old_book.close
           end
 
           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_file1, :if_obstructed => :close_if_saved)
+              new_book = Book.open(@simple_file1, :if_obstructed => :close_if_saved)
             }.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)
+            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_file1.downcase
-            old_book = Book.open(@simple_file_other_path, :if_obstructed => :forget)
-            old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
+            new_book.should be_alive
+            new_book.filename.downcase.should == @simple_file1.downcase
+            old_book = Book.open(@simple_file_other_path1, :if_obstructed => :forget)
+            old_book.sheet(1)[1,1].Value.should == @new_value
+            #old_book.ole_workbook.Worksheets.Count.should ==  @sheet_count + 1
             old_book.close
           end
 
           it "should close the old book and open the new book, if :if_obstructed is :close_if_saved" do
             @book.close(:if_unsaved => :forget)
             book = Book.open(@simple_file_other_path)
-            Book.open(@simple_file1, :if_obstructed => :close_if_saved)
+            book2 = Book.open(@simple_file1, :if_obstructed => :close_if_saved)
           end
 
           it "should open the book in a new excel instance, if :if_obstructed is :new_excel" do
-            @new_book = Book.open(@simple_file1, :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
-            @new_book.excel.should_not == @book.excel
+            @book.Saved.should be false
+            @book.sheet(1)[1,1].Value.should == @new_value
+            new_book.should be_alive
+            new_book.filename.should_not == @book.filename
+            new_book.excel.should_not == @book.excel
+            new_book.sheet(1)[1,1].Value.should == @old_value
           end
 
           it "should raise an error, if :if_obstructed is default" do
             expect {
-              @new_book = Book.open(@simple_file1)              
+              new_book = Book.open(@simple_file1)              
             }.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)
+              new_book = Book.open(@simple_file1, :if_obstructed => :invalid_option)
             }.to raise_error(OptionInvalid, ":if_obstructed: invalid option: :invalid_option")
           end
         end
@@ -1451,7 +1601,7 @@ describe Book do
         book = Book.open(@simple_save_file, :if_absent => :create)
         book.should be_a Book
         book.close
-        File.exist?(@simple_save_file).should be_true
+        File.exist?(@simple_save_file).should be true
       end
 
       it "should raise an exception by default" do
@@ -1485,7 +1635,7 @@ describe Book do
       it "should set update_links to :alert" do
         book = Book.open(@simple_file, :update_links => :alert)
         book.UpdateLinks.should == XlUpdateLinksUserSetting
-        book.Saved.should be_true
+        book.Saved.should be true
       end
 
       it "should set update_links to :never" do
@@ -1512,14 +1662,14 @@ describe Book do
       
       it "should reopen the book with writable (unsaved changes from readonly will not be saved)" do
         book = Book.open(@simple_file1, :read_only => true)
-        book.ReadOnly.should be_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
+        book.Saved.should be false
         new_book = Book.open(@simple_file1, :read_only => false, :if_unsaved => :accept)
-        new_book.ReadOnly.should be_false 
+        new_book.ReadOnly.should be false 
         new_book.should be_alive
         book.should be_alive   
         new_book.should == book 
@@ -1530,28 +1680,28 @@ describe Book 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_file1, :read_only => false)
-        book.ReadOnly.should be_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
+        book.Saved.should be false
         new_book = Book.open(@simple_file1, :read_only => true, :if_unsaved => :accept)
-        new_book.ReadOnly.should be_true
-        new_book.Saved.should be_true
+        new_book.ReadOnly.should be true
+        new_book.Saved.should be true
         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_file1, :read_only => true)
-        book.ReadOnly.should be_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
+        book.Saved.should be 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.ReadOnly.should be false 
         new_book.should be_alive
         book.should be_alive   
         new_book.should == book 
@@ -1562,23 +1712,26 @@ describe Book do
 
       it "should reopen the book with readonly (unsaved changes of the writable should be saved)" do
         book = Book.open(@simple_file1, :force => {:excel => :new}, :read_only => false)
-        book.ReadOnly.should be_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
+        book.Saved.should be false
         new_book = Book.open(@simple_file1, :force => {:excel => book.excel}, :read_only => true, :if_unsaved => :accept)
-        new_book.ReadOnly.should be_true
-        new_book.Saved.should be_true
+        new_book.ReadOnly.should be true
+        new_book.Saved.should be true
         new_book.should == book
+        new_sheet = new_book.sheet(1)
+        new_cell_value = new_sheet[1,1].value
+        new_cell_value.should_not == old_cell_value
       end
 
       it "should open the second book in another Excel as writable" do
         book = Book.open(@simple_file1, :read_only => true)
-        book.ReadOnly.should be_true
+        book.ReadOnly.should be true
         new_book = Book.open(@simple_file1, :force => {:excel => :new}, :read_only => false)
-        new_book.ReadOnly.should be_false
+        new_book.ReadOnly.should be false
         new_book.close
         book.close
       end
@@ -1618,10 +1771,10 @@ describe Book do
         book.close
       end
 
-      it "should open xlsm file" do
-        book = Book.open(@simple_file_xlsm, :visible => true)
-        book.close
-      end
+      #it "should open xlsm file" do
+      #  book = Book.open(@simple_file_xlsm, :visible => true)
+      #  book.close
+      #end
 
       it "should open xlsx file" do
         book = Book.open(@simple_file_xlsx, :visible => true)