robust_excel_ole 1.1.4 → 1.1.5

data/spec/book_specs/book_save_spec.rb

@@ -95,7 +95,7 @@ describe Book do
         Book.open(@simple_file) do |book|
           book.save_as(@simple_save_file1, :if_exists => :overwrite)
         end
-        File.exist?(@simple_save_file1).should be_true
+        File.exist?(@simple_save_file1).should be true
       end
 
       it "should raise error if filename is nil" do
@@ -129,7 +129,7 @@ describe Book do
           simple_save_file = @dir + '/simple_save_file.' + extensions_value
           File.delete simple_save_file rescue nil
           @book.save_as(simple_save_file, :if_exists => :overwrite)
-          File.exist?(simple_save_file).should be_true
+          File.exist?(simple_save_file).should be true
           new_book = Book.open(simple_save_file)
           new_book.should be_a Book
           new_book.close
@@ -148,7 +148,7 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         @book.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :forget)
       end
 
@@ -171,7 +171,7 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        File.exist?(@simple_file_other_path1).should be_true
+        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(WorkbookBlocked, /blocked by another workbook/)
@@ -184,7 +184,7 @@ describe Book do
         end
         @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :forget)
         @book.should_not be_alive
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         new_book = Book.open(@simple_file_other_path1)
         new_book.should be_a Book
         new_book.close
@@ -198,11 +198,11 @@ describe Book do
         sheet = @book.sheet(1)
         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
         @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :forget)
         @book.should_not be_alive
         @book2.should be_alive
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         new_book = Book.open(@simple_file_other_path1)
         new_book.should be_a Book
         new_book.close
@@ -220,11 +220,11 @@ describe Book do
         sheet = @book.sheet(1)
         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
         @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :save)
         @book.should_not be_alive
         @book2.should be_alive
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         new_book = Book.open(@simple_file_other_path1)
         new_book.should be_a Book
         new_book.close
@@ -239,11 +239,11 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        @book.Saved.should be_true
+        @book.Saved.should be true
         @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :close_if_saved)
         @book.should_not be_alive
         @book2.should be_alive
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         new_book = Book.open(@simple_file_other_path1)
         new_book.should be_a Book
         new_book.close
@@ -253,7 +253,7 @@ describe Book do
         sheet = @book.sheet(1)
         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      
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite, :if_obstructed => :close_if_saved)
         }.to raise_error(WorkbookBlocked, /blocking workbook is unsaved: "workbook.xls"/)
@@ -264,7 +264,7 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        File.exist?(@simple_file_other_path1).should be_true
+        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(OptionInvalid, ":if_obstructed: invalid option: :invalid")
@@ -275,7 +275,7 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        File.exist?(@simple_file_other_path1).should be_true
+        File.exist?(@simple_file_other_path1).should be true
         expect{
           @book2.save_as(@simple_file_other_path1, :if_exists => :overwrite)
         }.to raise_error(WorkbookBlocked, /blocked by another workbook/)
@@ -283,7 +283,7 @@ describe Book do
 
       it "should raise an error if the file does not exist and an workbook with the same name and other path exists" do
         File.delete @simple_file_other_path1 rescue nil
-        File.exist?(@simple_file_other_path1).should be_false
+        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(WorkbookBlocked, /blocked by another workbook/)
@@ -294,7 +294,7 @@ describe Book do
         File.open(@simple_file_other_path1,"w") do | file |
           file.puts "garbage"
         end
-        File.exist?(@simple_file_other_path1).should be_true
+        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(FileAlreadyExists, /file already exists: "workbook.xls"/)
@@ -332,7 +332,7 @@ describe Book do
             file.puts "garbage"
           end
           @book.save_as(@simple_save_file1, :if_exists => :overwrite)
-          File.exist?(@simple_save_file1).should be_true
+          File.exist?(@simple_save_file1).should be true
           new_book = Book.open(@simple_save_file1)
           new_book.should be_a Book
           new_book.close
@@ -355,12 +355,12 @@ describe Book do
           File.open(@simple_save_file1,"w") do | file |
             file.puts "garbage"
           end
-          File.exist?(@simple_save_file1).should be_true
+          File.exist?(@simple_save_file1).should be true
           booklength = File.size?(@simple_save_file1)
           expect {
             @book.save_as(@simple_save_file1, :if_exists => :raise)
             }.to raise_error(FileAlreadyExists, /file already exists: "workbook_save.xls"/)
-          File.exist?(@simple_save_file1).should be_true
+          File.exist?(@simple_save_file1).should be true
           File.size?(@simple_save_file1).should == booklength
         end
 
@@ -382,7 +382,7 @@ describe Book do
             # "Yes" is to the left of "No", which is the  default. --> language independent
             @key_sender.puts "{left}{enter}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :alert)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should > @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
             new_book = Book.open(@simple_save_file1, :excel => :new)
@@ -399,7 +399,7 @@ describe Book do
             @key_sender.puts "{enter}"
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :alert)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
           end
@@ -412,7 +412,7 @@ describe Book do
             @key_sender.puts "{right}{enter}"
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :alert)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
           end
@@ -437,7 +437,7 @@ describe Book do
             # "Yes" is to the left of "No", which is the  default. --> language independent
             @key_sender.puts "{left}{enter}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :excel)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should > @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
             new_book = Book.open(@simple_save_file1, :excel => :new)
@@ -454,7 +454,7 @@ describe Book do
             @key_sender.puts "{enter}"
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :excel)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
           end
@@ -467,7 +467,7 @@ describe Book do
             @key_sender.puts "{right}{enter}"
             #@key_sender.puts "%{n}" #, :initial_wait => 0.2, :if_target_missing=>"Excel window not found")
             @book.save_as(@simple_save_file1, :if_exists => :excel)
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
           end
@@ -478,7 +478,7 @@ describe Book do
             expect{
               @book.save_as(@simple_save_file1, :if_exists => :excel)
               }.to raise_error(ObjectNotAlive, "workbook is not alive")
-            File.exist?(@simple_save_file1).should be_true
+            File.exist?(@simple_save_file1).should be true
             File.size?(@simple_save_file1).should == @garbage_length
             @book.excel.DisplayAlerts.should == displayalert_value
           end
@@ -491,12 +491,12 @@ describe Book do
           File.open(@simple_save_file1,"w") do | file |
             file.puts "garbage"
           end
-          File.exist?(@simple_save_file1).should be_true
+          File.exist?(@simple_save_file1).should be true
           booklength = File.size?(@simple_save_file1)
           expect {
             @book.save_as(@simple_save_file1)
             }.to raise_error(FileAlreadyExists, /file already exists: "workbook_save.xls"/)
-          File.exist?(@simple_save_file1).should be_true
+          File.exist?(@simple_save_file1).should be true
           File.size?(@simple_save_file1).should == booklength
         end
 

data/spec/book_specs/book_unobtr_spec.rb

@@ -44,7 +44,7 @@ describe Book do
         sheet = book.sheet(1)
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         book.should be_alive
-        book.Saved.should be_false
+        book.Saved.should be false
       end
     end
 
@@ -52,6 +52,749 @@ describe Book do
       expect{unobtrusively_ok?}.to_not raise_error
     end
 
+    describe "writability" do
+
+      context "with no book" do
+
+        it "should open read-write" do
+          Book.unobtrusively(@simple_file1) do |book|            
+            book.ReadOnly.should be false
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end          
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file, :read_only => false) do |book|
+            book.ReadOnly.should be false
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file, :read_only => false, :writable => false) do |book|
+            book.ReadOnly.should be false
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file, :writable => true) do |book|
+            book.ReadOnly.should be false
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file, :writable => true, :read_only => false) do |book|
+            book.ReadOnly.should be false
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open read-only" do
+          Book.unobtrusively(@simple_file, :read_only => true) do |book|            
+            book.ReadOnly.should be true
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open read-only" do
+          Book.unobtrusively(@simple_file, :read_only => true, :writable => false) do |book|            
+            book.ReadOnly.should be true
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false
+          end
+          Excel.current.Workbooks.Count.should == 0
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open not writable" do
+          Book.unobtrusively(@simple_file, :writable => false) do |book|
+            book.ReadOnly.should be true
+            @old_value = book.sheet(1)[1,1].Value
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+            book.Saved.should be false            
+          end
+          Excel.current.Workbooks.Count.should == 0          
+          b1 = Book.open(@simple_file1)
+          b1.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should raise error if both options are true" do          
+          expect{
+            Book.unobtrusively(@simple_file, :writable => true, :read_only => true) {|book|}
+          }.to raise_error(OptionInvalid, "contradicting options")
+        end
+      end
+  
+      context "with open writable book" do
+
+        before do
+           @book = Book.open(@simple_file1)
+           @old_value = @book.sheet(1)[1,1].Value
+        end
+
+        it "should open as read-write by default" do
+          Book.unobtrusively(@simple_file1) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false, :writable => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do          
+          Book.unobtrusively(@simple_file1, :writable => true, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true, :writable => false) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open not writable" do
+          Book.unobtrusively(@simple_file1, :writable => false) do |book|
+            book.ReadOnly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+      end
+
+      context "with open read-only book" do
+
+        before do
+          @book = Book.open(@simple_file1, :read_only => true)
+          @old_value = @book.sheet(1)[1,1].Value 
+        end
+
+        it "should not change the read_only mode" do
+          Book.unobtrusively(@simple_file1) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false, :writable => false) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          e1 = Excel.create
+          Book.unobtrusively(@simple_file1, :writable => true, :rw_change_excel => e1) do |book|
+            book.Readonly.should be false
+            book.filename.should == @book.filename
+            book.excel.should == e1
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true, :rw_change_excel => :current) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true, :rw_change_excel => :new) do |book|
+            book.Readonly.should be false
+            book.filename.should == @book.filename
+            book.excel.should_not == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true, :writable => false) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should open not writable" do
+          Book.unobtrusively(@simple_file1, :writable => false) do |book|
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should == @old_value
+        end
+      end
+
+      context "with open unsaved writable book" do
+
+        before do
+           @book = Book.open(@simple_file1)           
+           @book.sheet(1)[1,1] = @book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"           
+           @old_value = @book.sheet(1)[1,1].Value
+        end
+
+        it "should open as read-write by default" do
+          Book.unobtrusively(@simple_file1) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do
+          Book.unobtrusively(@simple_file1, :read_only => false, :writable => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-write" do
+          Book.unobtrusively(@simple_file1, :writable => true) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-write" do          
+          Book.unobtrusively(@simple_file1, :writable => true, :read_only => false) do |book|
+            book.Readonly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-only (not implemented)" do
+          expect{
+            Book.unobtrusively(@simple_file1, :read_only => true) 
+          }.to raise_error(NotImplementedREOError)
+        end
+
+        it "should force to read-only (not implemented)" do
+          expect{
+            Book.unobtrusively(@simple_file1, :read_only => true, :writable => false) 
+          }.to raise_error(NotImplementedREOError)
+        end
+
+        #it "should force to read-only (not implemented)" do
+        #  expect{
+        #    Book.unobtrusively(@simple_file1, :read_only => true) do |book|            
+        #      book.ReadOnly.should be true
+        #      book.should == @book
+        #      book.filename.should == @book.filename
+        #      book.excel.should == @book.excel
+        #      book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+        #    end
+        #    @book.Saved.should be false
+        #    @book.sheet(1)[1,1].Value.should == @old_value
+        #  end
+        #  }.to raise_error(NotImplementedREOError)
+        #end
+
+        #it "should force to read-only" do
+        #  expect{
+        #    Book.unobtrusively(@simple_file1, :read_only => true, :writable => false) do |book|            
+        #      book.ReadOnly.should be true
+        #      book.should == @book
+        #      book.filename.should == @book.filename
+        #      book.excel.should == @book.excel
+        #      book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+        #    end
+        #    @book.Saved.should be false
+        #    @book.sheet(1)[1,1].Value.should == @old_value
+        #  end
+        #  }.to raise_error(NotImplementedREOError)
+        #end
+
+        it "should open not writable" do
+          Book.unobtrusively(@simple_file1, :writable => false) do |book|
+            book.ReadOnly.should be false
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close(:if_unsaved => :forget)
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+      end
+
+      context "with open unsaved read-only book" do
+
+        before do
+          @book = Book.open(@simple_file1, :read_only => true)          
+          @book.sheet(1)[1,1] = @book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          @old_value = @book.sheet(1)[1,1].Value
+        end
+
+        it "should open as read-only by default" do
+          Book.unobtrusively(@simple_file1) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => false) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open as read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => false, :writable => false) do |book|
+            book.Readonly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+    
+        it "should raise an error" do
+          expect{
+            Book.unobtrusively(@simple_file1, :writable => true)
+            }.to raise_error(NotImplementedREOError, "unsaved read-only workbook shall be written")
+        end
+
+        it "should raise an error" do
+           expect{
+            Book.unobtrusively(@simple_file1, :writable => true)
+            }.to raise_error(NotImplementedREOError, "unsaved read-only workbook shall be written")
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should force to read-only" do
+          Book.unobtrusively(@simple_file1, :read_only => true, :writable => false) do |book|            
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+
+        it "should open not writable" do
+          Book.unobtrusively(@simple_file1, :writable => false) do |book|
+            book.ReadOnly.should be true
+            book.should == @book
+            book.filename.should == @book.filename
+            book.excel.should == @book.excel
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo"
+          end
+          @book.Saved.should be false
+          @book.ReadOnly.should be true
+          @book.sheet(1)[1,1].Value.should_not == @old_value
+          @book.close
+          book2 = Book.open(@simple_file1)
+          book2.sheet(1)[1,1].Value.should_not == @old_value
+        end
+      end
+
+    end
+
+    describe "intertweened blocks" do
+
+      context "with no book" do
+      
+        it "should not close the book in the outer block" do
+          Book.unobtrusively(@simple_file1) do |book|
+            Book.unobtrusively(@simple_file1) do |book2|
+              book2.should == book
+            end
+            book.should be_alive
+            Book.unobtrusively(@simple_file1) do |book3|
+              book3.should == book
+            end
+            book.should be_alive
+          end
+        end
+
+        it "should write in the outer and inner block" do
+          Book.unobtrusively(@simple_file1) do |book|
+            @old_value = book.sheet(1)[1,1].Value
+            book.ReadOnly.should be false
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should_not == @old_value
+            Book.unobtrusively(@simple_file1) do |book2|
+              book2.should == book
+              book2.ReadOnly.should be false
+              book2.Saved.should be false
+              book2.sheet(1)[1,1].Value.should_not == @old_value
+              book2.sheet(1)[1,1] = book2.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+              book2.sheet(1)[1,1].Value.should == @old_value
+            end
+            book.should be_alive
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should == @old_value            
+          end
+          book = Book.open(@simple_file1)
+          book.sheet(1)[1,1].Value.should == @old_value 
+        end
+
+        it "should write in the outer and not in the inner block" do
+          expect{
+          Book.unobtrusively(@simple_file1) do |book|
+            @old_value = book.sheet(1)[1,1].Value
+            book.ReadOnly.should be false
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should_not == @old_value
+            Book.unobtrusively(@simple_file1, :read_only => true) do |book2|
+              book2.should == book
+              book2.ReadOnly.should be true
+              #book2.Saved.should be false
+              book2.sheet(1)[1,1].Value.should_not == @old_value
+              book2.sheet(1)[1,1] = book2.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+              book2.sheet(1)[1,1].Value.should == @old_value
+            end
+            book.should be_alive
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should_not == @old_value            
+          end
+          book = Book.open(@simple_file1)
+          book.sheet(1)[1,1].Value.should_not == @old_value
+          }.to raise_error(NotImplementedREOError)
+        end
+
+        it "should write in the outer and not in the inner block" do
+          Book.unobtrusively(@simple_file1) do |book|
+            @old_value = book.sheet(1)[1,1].Value
+            book.ReadOnly.should be false
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should_not == @old_value
+            Book.unobtrusively(@simple_file1, :writable => false) do |book2|
+              book2.should == book
+              book2.ReadOnly.should be false
+              book2.Saved.should be false
+              book2.sheet(1)[1,1].Value.should_not == @old_value
+              book2.sheet(1)[1,1] = book2.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+              book2.sheet(1)[1,1].Value.should == @old_value
+            end
+            book.should be_alive
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should == @old_value            
+          end
+          book = Book.open(@simple_file1)
+          book.sheet(1)[1,1].Value.should == @old_value
+        end
+
+        it "should be read-only in the outer and write in the inner block" do
+          Book.unobtrusively(@simple_file1, :read_only => true) do |book|
+            @old_value = book.sheet(1)[1,1].Value
+            book.ReadOnly.should be true
+            book.sheet(1)[1,1] = book.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+            book.Saved.should be false
+            book.sheet(1)[1,1].Value.should_not == @old_value
+            Book.unobtrusively(@simple_file1) do |book2|
+              book2.should == book
+              book2.ReadOnly.should be true
+              book2.Saved.should be false
+              book2.sheet(1)[1,1].Value.should_not == @old_value
+              book2.sheet(1)[1,1] = book2.sheet(1)[1,1].Value == "foo" ? "bar" : "foo" 
+              book2.sheet(1)[1,1].Value.should == @old_value
+            end
+            book.should be_alive
+            book.Saved.should be false
+            book.ReadOnly.should be true
+            book.sheet(1)[1,1].Value.should == @old_value            
+          end
+          book = Book.open(@simple_file1)
+          book.sheet(1)[1,1].Value.should == @old_value
+        end
+
+      end
+
+    end
+
     describe "unchanging" do
 
       context "with openess" do
@@ -75,18 +818,18 @@ describe Book do
 
       context "with writability" do
 
-        it "should remain writable" do
-          book1 = Book.open(@simple_file1, :read_only => false)
+        it "should remain read_only" do
+          book1 = Book.open(@simple_file1, :read_only => true)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.ReadOnly.should be_false
+          book1.ReadOnly.should be true
         end
 
-        it "should remain read_only" do
-          book1 = Book.open(@simple_file1, :read_only => true)
+        it "should remain writable" do
+          book1 = Book.open(@simple_file1, :read_only => false)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.ReadOnly.should be_true
+          book1.ReadOnly.should be false
         end
 
         it "should write and remain read_only and open the workbook in another Excel" do
@@ -95,23 +838,24 @@ describe Book do
           Book.unobtrusively(@simple_file1) do |book|
             sheet = book.sheet(1)
             sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
-            book.excel.should_not == book1.excel
+            book.excel.should == book1.excel
           end
-          book1.ReadOnly.should be_true
+          book1.ReadOnly.should be true
           book1.close
           book2 = Book.open(@simple_file1)
-          book2.sheet(1)[1,1].Value.should_not == old_value
+          book2.sheet(1)[1,1].Value.should == old_value
         end
 
         it "should write and remain read_only and open the workbook in the same Excel" do
           book1 = Book.open(@simple_file1, :read_only => true)
           old_value = book1.sheet(1)[1,1].Value
-          Book.unobtrusively(@simple_file1, :readonly_excel => true) do |book|
+          Book.unobtrusively(@simple_file1, :writable => true) do |book|
+            book.ReadOnly.should be false
             sheet = book.sheet(1)
             sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
             book.excel.should == book1.excel
           end
-          book1.ReadOnly.should be_true
+          book1.ReadOnly.should be true
           book1.close
           book2 = Book.open(@simple_file1)
           book2.sheet(1)[1,1].Value.should_not == old_value
@@ -125,15 +869,15 @@ describe Book do
           book1 = Book.open(@simple_file1, :visible => false)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.excel.Visible.should be_false
+          book1.excel.Visible.should be false
         end
 
         it "should remain visible" do
           book1 = Book.open(@simple_file1, :visible => true)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.excel.Visible.should be_true
-          book1.Windows(book1.Name).Visible.should be_true
+          book1.excel.Visible.should be true
+          book1.Windows(book1.Name).Visible.should be true
         end
 
       end
@@ -144,14 +888,14 @@ describe Book do
           book1 = Book.open(@simple_file1, :check_compatibility => false)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.CheckCompatibility.should be_false
+          book1.CheckCompatibility.should be false
         end
 
         it "should remain check-compatibility true" do
           book1 = Book.open(@simple_file1, :check_compatibility => true)
           Book.unobtrusively(@simple_file1) do |book|
           end
-          book1.CheckCompatibility.should be_true
+          book1.CheckCompatibility.should be true
         end
 
       end
@@ -189,62 +933,65 @@ describe Book do
         Excel.close_all
         Book.unobtrusively(@simple_file) do |book|
           book.should be_a Book
-          book.excel.Visible.should be_false
-          book.CheckCompatibility.should be_false
-          book.ReadOnly.should be_false
+          book.excel.Visible.should be false
+          book.CheckCompatibility.should be false
+          book.ReadOnly.should be false
         end
       end
 
       it "should open unobtrusively in a new Excel" do
         expect{ unobtrusively_ok? }.to_not raise_error
       end
-          
-      context "with two running excel instances" do
-        before :all do
-          Excel.close_all
-        end
+    end
 
-        before do
-          @excel1 = Excel.new(:reuse => false)
-          @excel2 = Excel.new(:reuse => false)
-        end
+    context "with two running excel instances" do
+      
+      before :all do
+        Excel.close_all
+      end
 
-        after do
-          Excel.kill_all
-        end
+      before do
+        @excel1 = Excel.new(:reuse => false)
+        @excel2 = Excel.new(:reuse => false)
+      end
+
+      after do
+        Excel.kill_all
+      end
 
       it "should open unobtrusively in the first opened Excel" do
         Book.unobtrusively(@simple_file) do |book|
-            book.excel.should     == @excel1
-            book.excel.should_not == @excel2
-          end
+          book.excel.should     == @excel1
+          book.excel.should_not == @excel2
+        end
       end
 
       it "should open unobtrusively in a new Excel" do
         Book.unobtrusively(@simple_file, :if_closed => :new) do |book|
           book.should be_a Book
           book.should be_alive
-            book.excel.should_not == @excel1
-            book.excel.should_not == @excel2
-          end
+          book.excel.should_not == @excel1
+          book.excel.should_not == @excel2
+        end
       end
 
       it "should open unobtrusively in a given Excel" do
         Book.unobtrusively(@simple_file, :if_closed => @excel2) do |book|
           book.should be_a Book
           book.should be_alive
-            book.excel.should_not == @excel1
-            book.excel.should     == @excel2
+          book.excel.should_not == @excel1
+          book.excel.should     == @excel2
         end
       end
-  
-      it "should raise an error if the excel instance is not alive" do
-        Excel.close_all
+
+      it "should open in another Excel instance if the given Excel instance is not alive" do
+        Excel.kill_all
+        sleep 2
         expect{
           Book.unobtrusively(@simple_file, :if_closed => @excel2) do |book|
+            book.should be_alive
           end
         }.to_not raise_error
-        end
       end
 
       it "should raise an error if the option is invalid" do
@@ -305,12 +1052,12 @@ describe Book do
       end
 
       it "should let a saved book saved" do
-        @book.Saved.should be_true
+        @book.Saved.should be true
         @book.should be_alive
         sheet = @book.sheet(1)
         old_cell_value = sheet[1,1].value
         unobtrusively_ok?
-        @book.Saved.should be_true
+        @book.Saved.should be true
         @book.should be_alive
         sheet = @book.sheet(1)
         sheet[1,1].value.should_not == old_cell_value
@@ -320,35 +1067,35 @@ describe Book do
         sheet = @book.sheet(1)
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         old_cell_value = sheet[1,1].value
-        @book.Saved.should be_false
+        @book.Saved.should be false
         unobtrusively_ok?
         @book.should be_alive
-        @book.Saved.should be_false
+        @book.Saved.should be false
         @book.close(:if_unsaved => :forget)
-        @book2 = Book.open(@simple_file)
+        @book2 = Book.open(@simple_file1)
         sheet2 = @book2.sheet(1)
         sheet2[1,1].value.should_not == old_cell_value
       end
 
       it "should modify unobtrusively the second, writable book" do
-        @book2 = Book.open(@simple_file, :force_excel => :new)
-        @book.ReadOnly.should be_false
-        @book2.ReadOnly.should be_true
+        @book2 = Book.open(@simple_file1, :force_excel => :new)
+        @book.ReadOnly.should be false
+        @book2.ReadOnly.should be true
         sheet = @book2.sheet(1)
         old_cell_value = sheet[1,1].value
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
         unobtrusively_ok?
         @book2.should be_alive
-        @book2.Saved.should be_false
+        @book2.Saved.should be false
         @book2.close(:if_unsaved => :forget)
         @book.close
         new_book = Book.open(@simple_file1)
         sheet2 = new_book.sheet(1)
         sheet2[1,1].value.should_not == old_cell_value
       end    
-
     end
 
+
     context "with a closed book" do
       
       before do
@@ -385,7 +1132,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         @book.should be_alive
         @book.close
@@ -409,7 +1156,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         @book.should_not be_alive
         new_book = Book.open(@simple_file1)
@@ -427,7 +1174,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         @book.should be_alive
         @book.close
@@ -450,7 +1197,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         @book.should_not be_alive
         new_book = Book.open(@simple_file1)
@@ -496,11 +1243,11 @@ describe Book do
           book.should be_a Book
           book.should be_alive
           book.excel.should == @book.excel
-          book.excel.Visible.should be_true
+          book.excel.Visible.should be true
         end        
         @book.should be_alive
         @book.should be_a Book
-        @book.excel.Visible.should be_true
+        @book.excel.Visible.should be true
       end
       
     end
@@ -527,14 +1274,14 @@ describe Book do
         Book.unobtrusively(@simple_file1, :if_closed => :new) do |book|
           book.excel.should_not == @book.excel
           book.excel.should_not == new_excel
-          book.excel.visible.should be_false
+          book.excel.visible.should be false
           book.excel.displayalerts.should == :if_visible
           @another_excel = book.excel
         end
         Book.unobtrusively(@simple_file1, :if_closed => :reuse) do |book|
           book.excel.should == @book.excel
           book.excel.should_not == new_excel
-          book.excel.visible.should be_false
+          book.excel.visible.should be false
           book.excel.displayalerts.should == :if_visible
           book.excel.should_not == @another_excel 
         end
@@ -569,25 +1316,25 @@ describe Book do
       end
 
       it "should let the saved book saved" do
-        @book.ReadOnly.should be_true
-        @book.Saved.should be_true
+        @book.ReadOnly.should be true
+        @book.Saved.should be true
         sheet = @book.sheet(1)
         old_cell_value = sheet[1,1].value
         unobtrusively_ok?
         @book.should be_alive
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
+        @book.Saved.should be true
+        @book.ReadOnly.should be true
         @book.close
         book2 = Book.open(@simple_file1)
         sheet2 = book2.sheet(1)
-        sheet2[1,1].value.should_not == old_cell_value
+        sheet2[1,1].value.should == old_cell_value
       end
 
       it "should let the unsaved book unsaved" do
-        @book.ReadOnly.should be_true
+        @book.ReadOnly.should be true
         sheet = @book.sheet(1)
         sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo" 
-        @book.Saved.should be_false
+        @book.Saved.should be false
         @book.should be_alive
         Book.unobtrusively(@simple_file1) do |book|
           book.should be_a Book
@@ -595,11 +1342,11 @@ describe Book do
           sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
           @cell_value = sheet[1,1].Value
           book.should be_alive
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         @book.should be_alive
-        @book.Saved.should be_false
-        @book.ReadOnly.should be_true
+        @book.Saved.should be false
+        @book.ReadOnly.should be true
         @book.close
         book2 = Book.open(@simple_file1)
         sheet2 = book2.sheet(1)
@@ -608,9 +1355,9 @@ describe Book do
       end
 
       it "should open unobtrusively by default the writable book" do
-        book2 = Book.open(@simple_file, :force_excel => :new, :read_only => false)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_false
+        book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => false)
+        @book.ReadOnly.should be true
+        book2.Readonly.should be false
         sheet = @book.sheet(1)
         cell_value = sheet[1,1].value
         Book.unobtrusively(@simple_file1, :if_closed => :new) do |book|
@@ -620,60 +1367,10 @@ describe Book do
           sheet = book.sheet(1)
           sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
           book.should be_alive
-          book.Saved.should be_false          
-        end  
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
-        @book.close
-        book2.close
-        book3 = Book.open(@simple_file1)
-        new_sheet = book3.sheet(1)
-        new_sheet[1,1].value.should_not == cell_value
-        book3.close
-      end
-
-      it "should open unobtrusively by default the book in a new Excel such that the book is writable" do
-        book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => true)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_true
-        sheet = @book.sheet(1)
-        cell_value = sheet[1,1].value
-        Book.unobtrusively(@simple_file, :if_closed => :new) do |book|
-          book.should be_a Book
-          book.excel.should_not == book2.excel
-          book.excel.should_not == @book.excel
-          sheet = book.sheet(1)
-          sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
-          book.should be_alive
-          book.Saved.should be_false          
+          book.Saved.should be false          
         end  
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
-        @book.close
-        book2.close
-        book3 = Book.open(@simple_file1)
-        new_sheet = book3.sheet(1)
-        new_sheet[1,1].value.should_not == cell_value
-        book3.close
-      end
-
-      it "should open unobtrusively the book in a new Excel such that the book is writable" do
-        book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => true)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_true
-        sheet = @book.sheet(1)
-        cell_value = sheet[1,1].value
-        Book.unobtrusively(@simple_file, :if_closed => :new) do |book|
-          book.should be_a Book
-          book.excel.should_not == book2.excel
-          book.excel.should_not == @book.excel
-          sheet = book.sheet(1)
-          sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
-          book.should be_alive
-          book.Saved.should be_false          
-        end  
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
+        @book.Saved.should be true
+        @book.ReadOnly.should be true
         @book.close
         book2.close
         book3 = Book.open(@simple_file1)
@@ -686,13 +1383,13 @@ describe Book do
         excel1 = Excel.new(:reuse => false)
         excel2 = Excel.new(:reuse => false)
         book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => true)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_true
+        @book.ReadOnly.should be true
+        book2.Readonly.should be true
         sheet = @book.sheet(1)
         cell_value = sheet[1,1].value
-        Book.unobtrusively(@simple_file1, :if_closed => :new, :readonly_excel => false) do |book|
+        Book.unobtrusively(@simple_file1, :writable => true, :if_closed => :new, :rw_change_excel => :new) do |book|
           book.should be_a Book
-          book.ReadOnly.should be_false
+          book.ReadOnly.should be false
           book.excel.should_not == book2.excel
           book.excel.should_not == @book.excel
           book.excel.should_not == excel1
@@ -700,10 +1397,10 @@ describe Book do
           sheet = book.sheet(1)
           sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
           book.should be_alive
-          book.Saved.should be_false          
+          book.Saved.should be false          
         end  
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
+        @book.Saved.should be true
+        @book.ReadOnly.should be true
         @book.close
         book2.close
         book3 = Book.open(@simple_file1)
@@ -716,21 +1413,21 @@ describe Book do
         excel1 = Excel.new(:reuse => false)
         excel2 = Excel.new(:reuse => false)
         book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => true)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_true
+        @book.ReadOnly.should be true
+        book2.Readonly.should be true
         sheet = @book.sheet(1)
         cell_value = sheet[1,1].value
-        Book.unobtrusively(@simple_file1, :if_closed => :new, :readonly_excel => true) do |book|
+        Book.unobtrusively(@simple_file1, :writable => true, :if_closed => :new, :rw_change_excel => :current) do |book|
           book.should be_a Book
           book.excel.should == book2.excel
-          book.ReadOnly.should be_false
+          book.ReadOnly.should be false
           sheet = book.sheet(1)
           sheet[1,1] = sheet[1,1].value == "foo" ? "bar" : "foo"
           book.should be_alive
-          book.Saved.should be_false          
+          book.Saved.should be false          
         end  
-        book2.Saved.should be_true
-        book2.ReadOnly.should be_true
+        book2.Saved.should be true
+        book2.ReadOnly.should be true
         @book.close
         book2.close
         book3 = Book.open(@simple_file1)
@@ -741,8 +1438,8 @@ describe Book do
 
       it "should open unobtrusively the book in the Excel where it was opened most recently" do
         book2 = Book.open(@simple_file1, :force_excel => :new, :read_only => true)
-        @book.ReadOnly.should be_true
-        book2.Readonly.should be_true
+        @book.ReadOnly.should be true
+        book2.Readonly.should be true
         sheet = @book.sheet(1)
         cell_value = sheet[1,1].value
         Book.unobtrusively(@simple_file1, :if_closed => :new, :read_only => true) do |book|
@@ -750,10 +1447,10 @@ describe Book do
           book.excel.should == book2.excel
           book.excel.should_not == @book.excel
           book.should be_alive
-          book.Saved.should be_true         
+          book.Saved.should be true         
         end  
-        @book.Saved.should be_true
-        @book.ReadOnly.should be_true
+        @book.Saved.should be true
+        @book.ReadOnly.should be true
         @book.close
         book2.close
       end
@@ -796,16 +1493,16 @@ describe Book do
       it "should save if the book was modified during unobtrusively" do
         m_time = File.mtime(@book1.stored_filename)
         Book.unobtrusively(@simple_file1, :if_closed => :new) do |book|
-          @book1.Saved.should be_true
-          book.Saved.should be_true  
+          @book1.Saved.should be true
+          book.Saved.should be true  
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          @book1.Saved.should be_false
-          book.Saved.should be_false
+          @book1.Saved.should be false
+          book.Saved.should be false
           sleep 1
         end
-        @book1.Saved.should be_true
+        @book1.Saved.should be true
         m_time2 = File.mtime(@book1.stored_filename)
         m_time2.should_not == m_time
       end      
@@ -813,8 +1510,8 @@ describe Book do
       it "should not save the book if it was not modified during unobtrusively" do
         m_time = File.mtime(@book1.stored_filename)
         Book.unobtrusively(@simple_file1) do |book|
-          @book1.Saved.should be_true
-          book.Saved.should be_true 
+          @book1.Saved.should be true
+          book.Saved.should be true 
           sleep 1
         end
         m_time2 = File.mtime(@book1.stored_filename)
@@ -834,7 +1531,7 @@ describe Book do
       it "should yield the block result true" do
         result = 
           Book.unobtrusively(@simple_file1) do |book| 
-            @book1.Saved.should be_true
+            @book1.Saved.should be true
           end
         result.should == true
       end
@@ -864,11 +1561,11 @@ describe Book do
           Book.unobtrusively(@simple_file1) do |book| 
             sheet = book.sheet(1)
             sheet[1,1] = 22
-            @book1.Saved.should be_false
+            @book1.Saved.should be false
             42
           end
         result.should == 42
-        @book1.Saved.should be_true
+        @book1.Saved.should be true
       end
     end
 
@@ -895,7 +1592,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         new_book = Book.open(@simple_file1)
         sheet = new_book.sheet(1)
@@ -910,7 +1607,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         new_book = Book.open(@simple_file1)
         sheet = new_book.sheet(1)
@@ -926,7 +1623,7 @@ describe Book do
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
         end
         new_book = Book.open(@simple_file1)
         sheet = new_book.sheet(1)
@@ -950,11 +1647,11 @@ describe Book do
         Book.for_reading(@simple_file1) do |book|
           book.should be_a Book
           book.should be_alive
-          book.Saved.should be_true  
+          book.Saved.should be true  
           sheet = book.sheet(1)
           cell = sheet[1,1]
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
-          book.Saved.should be_false
+          book.Saved.should be false
           book.excel.should == @book.excel
         end
         new_book = Book.open(@simple_file1, :visible => true)
@@ -984,7 +1681,7 @@ describe Book do
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
           book.excel.should_not == @book.excel
           book.excel.should_not == new_excel
-          book.excel.visible.should be_false
+          book.excel.visible.should be false
           book.excel.displayalerts.should == :if_visible
         end
         new_book = Book.open(@simple_file1, :visible => true)
@@ -1026,7 +1723,7 @@ describe Book do
           sheet[1,1] = cell.value == "foo" ? "bar" : "foo"
           book.excel.should_not == @book.excel
           book.excel.should_not == new_excel
-          book.excel.visible.should be_false
+          book.excel.visible.should be false
           book.excel.displayalerts.should == :if_visible
         end
         new_book = Book.open(@simple_file1, :visible => true)
@@ -1037,3 +1734,4 @@ describe Book do
   end
   
 end
+