robust_excel_ole 0.3.0 → 0.3.1

data/spec/book_store_spec.rb

@@ -13,6 +13,7 @@ end
 $VERBOSE = nil
 
 include RobustExcelOle
+
 module RobustExcelOle
   class MockBookstore
     def fetch(filename, options = { })
@@ -86,7 +87,7 @@ describe BookStore do
       end
 
       after do
-        @book.close
+        @book.close rescue nil
       end
 
       it "should do simple store and fetch" do        
@@ -118,6 +119,14 @@ describe BookStore do
         new_book.should == nil
       end
 
+      it "should fetch a closed book" do
+        @bookstore.store(@book)
+        @book.close
+        book1 = @bookstore.fetch(@simple_file)
+        book1.should be_a Book
+        book1.should_not be_alive
+      end
+
       it "should fetch nothing when fetching a different book" do
         @bookstore.store(@book)
         new_book = @bookstore.fetch(@different_file)
@@ -171,20 +180,49 @@ describe BookStore do
         new_book.close
       end
 
-      it "should fetch the writable book even if the readonly book has unsaved changes" do
+      it "should fetch the last book with :prefer_writeable => false" do
         @book2 = Book.open(@simple_file, :force_excel => :new)
-        sheet = @book2[0]
         @bookstore.store(@book2)
-        sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
         @book.ReadOnly.should be_false
         @book2.ReadOnly.should be_true
-        @book2.Saved. should be_false
-        new_book = @bookstore.fetch(@simple_file)
-        new_book.should == @book
-        new_book.should_not == @book2
+        new_book = @bookstore.fetch(@simple_file, :prefer_writable => false)
+        new_book.should_not == @book
+        new_book.should == @book2
         new_book.close
       end
 
+      it "should fetch the second, open book, if the first book is closed" do
+        @book2 = Book.open(@simple_file, :force_excel => :new)
+        @bookstore.store(@book2)
+        @book.ReadOnly.should be_false
+        @book2.ReadOnly.should be_true
+        @book.close
+        new_book = @bookstore.fetch(@simple_file, :prefer_writable => false)
+        new_book2 = @bookstore.fetch(@simple_file)
+        new_book.should_not == @book
+        new_book2.should_not == @book
+        new_book.should == @book2
+        new_book2.should == @book2
+        new_book.close
+        new_book2.close
+      end
+
+      it "should fetch the first, open book, if the second book is closed, even with :prefer_writeable => false" do
+        @book2 = Book.open(@simple_file, :force_excel => :new)
+        @bookstore.store(@book2)
+        @book.ReadOnly.should be_false
+        @book2.ReadOnly.should be_true
+        @book2.close
+        new_book = @bookstore.fetch(@simple_file, :prefer_writable => false)
+        new_book2 = @bookstore.fetch(@simple_file)
+        new_book.should_not == @book2
+        new_book2.should_not == @book2
+        new_book.should == @book
+        new_book2.should == @book
+        new_book.close
+        new_book2.close
+      end
+     
     end
 
     context "with readonly book" do
@@ -205,9 +243,17 @@ describe BookStore do
         @book.ReadOnly.should be_true
         @book2.ReadOnly.should be_false
         new_book = @bookstore.fetch(@simple_file)
+        new_book2 = @bookstore.fetch(@simple_file, :prefer_writable => true)
+        new_book3 = @bookstore.fetch(@simple_file, :prefer_writable => false)
         new_book.should == @book2
+        new_book2.should == @book2
+        new_book3.should == @book2
         new_book.should_not == @book
+        new_book2.should_not == @book
+        new_book3.should_not == @book
         new_book.close
+        new_book2.close
+        new_book3.close
       end
 
       it "should fetch the recent readonly book when there are only readonly books" do
@@ -221,20 +267,6 @@ describe BookStore do
         new_book.close
       end
 
-      it "should fetch the second readonly book with unsaved changes" do
-        @book2 = Book.open(@simple_file, :force_excel => :new, :read_only => true)
-        sheet = @book2[0]
-        @bookstore.store(@book2)
-        sheet[0,0] = sheet[0,0].value == "simple" ? "complex" : "simple"
-        @book.ReadOnly.should be_true
-        @book2.ReadOnly.should be_true
-        @book2.Saved.should be_false
-        new_book = @bookstore.fetch(@simple_file)
-        new_book.should == @book2
-        new_book.should_not == @book        
-        new_book.close
-      end
-
       it "should fetch the second, writable book, if a writable, a readonly and an unsaved readonly book exist" do
         @book2 = Book.open(@simple_file, :force_excel => :new)
         @book3 = Book.open(@simple_file, :force_excel => :new)
@@ -247,19 +279,44 @@ describe BookStore do
         @book3.ReadOnly.should be_true
         @book3.Saved.should be_false
         new_book = @bookstore.fetch(@simple_file)
+        new_book2 = @bookstore.fetch(@simple_file, :prefer_writable => false)
         new_book.should == @book2
+        new_book2.should == @book3
         new_book.should_not == @book        
-        new_book.should_not == @book3  
+        new_book.should_not == @book3
+        new_book2.should_not == @book        
+        new_book2.should_not == @book2  
         new_book.close
+        new_book2.close
       end
     end
    
+    context "with several closed books" do
+      
+      before do
+        @book = Book.open(@simple_file)
+        @bookstore.store(@book)
+        @book2 = Book.open(@simple_file, :force_excel => :new)
+        @bookstore.store(@book2)
+        @book.close
+        @book2.close
+      end
+
+      it "should fetch the recent closed book" do
+        new_book = @bookstore.fetch(@simple_file)
+        new_book.should == @book2
+        new_book.should_not == @book
+      end
+      
+    end
+
     context "with changing file name" do
 
       before do
         @book = Book.open(@simple_file)
         @book.save_as(@simple_save_file, :if_exists => :overwrite)      
-        @bookstore = @book.book_store
+        @bookstore.store(@book)
+        #@bookstore = @book.book_store
       end
 
       after do
@@ -277,6 +334,7 @@ describe BookStore do
       end
     end
 
+
     context "with given excel instance and fetching readonly" do
       
       before do
@@ -293,14 +351,110 @@ describe BookStore do
       it "should fetch the book in the given excel instance" do
         @book.ReadOnly.should be_false
         @book2.ReadOnly.should be_true
-        book_new = @bookstore.fetch(@simple_file, :excel => @book2.excel)
+        book_new = @bookstore.fetch(@simple_file, :prefer_excel => @book2.excel)
         book_new.should be_a Book
         book_new.should be_alive
         book_new.should == @book2
       end
+    end
+  end
+  
+  describe "book life cycle" do
+    
+    context "with an open book" do
+
+      before do
+        @book = Book.open(@simple_file)
+        @bookstore.store(@book)
+      end
+
+      after do
+        @book.close rescue nil
+      end
+
+      it "should find the book if the book has astill got a reference" do
+        GC.start
+        @bookstore.fetch(@simple_file).should == @book
+      end
+
+      it "should have forgotten the book if there is no reference anymore" do
+        @book = nil
+        GC.start
+        @bookstore.fetch(@simple_file).should == nil
+      end
 
+      it "should have forgotten some books if they have no reference anymore" do
+        book_new = Book.open(@different_file)
+        @bookstore.store(book_new)
+        @book = nil
+        GC.start
+        @bookstore.fetch(@simple_file).should == nil
+        @bookstore.fetch(@different_file).should == book_new
+      end
     end
+  end
+
+  describe "excel_list" do
+
+    context "with no books" do
+      
+      it "should yield nil" do
+        @bookstore.excel_list.should == {}
+      end
+    
+    end
+
+    context "with open books" do
+    
+      before do
+        @book = Book.open(@simple_file)
+        @bookstore.store(@book)
+      end
 
+      after do
+        @book.close
+      end
 
+      it "should yield an excel and the workbook" do
+        excels = @bookstore.excel_list
+        excels.size.should == 1
+        excels.each do |excel,workbooks|
+          excel.should be_a Excel
+          workbooks.size.should == 1
+          workbooks.each do |workbook|
+            workbook.should == @book.workbook
+          end
+        end
+      end
+
+      it "should yield an excel with two books" do
+        book1 = Book.open(@different_file)
+        @bookstore.store(book1)
+        excels = @bookstore.excel_list
+        excels.size.should == 1
+        excels.each do |excel,workbooks|
+          excel.should be_a Excel
+          workbooks.size.should == 2
+          #workbooks[0].should == @book.workbook
+          #workbooks[1].should == book1.workbook
+        end
+      end
+
+      it "should yield two excels and two books" do
+        e = Excel.create
+        book1 = Book.open(@simple_file, :force_excel => :new)
+        @bookstore.store(book1)
+        excels = @bookstore.excel_list
+        excels.size.should == 2
+        num = 0
+        excels.each do |excel,workbooks|
+          num = num + 1
+          excel.should be_a Excel
+          workbooks.size.should == 1
+          #workbooks[0].should == @book.workbook if num == 1          
+          #workbooks[0].should == book1.workbook if num == 2
+        end
+      end
+    end
   end
 end

data/spec/excel_spec.rb

@@ -60,6 +60,7 @@ module RobustExcelOle
 
       it "should reuse existing excel with default options for 'new'" do
         excel2 = Excel.new
+        excel2.should be_a Excel
         #puts "@excel1 #{@excel1.Hwnd}"
         #puts "excel2  #{excel2.Hwnd}"
         excel2.Hwnd.should == @excel1.Hwnd
@@ -226,8 +227,6 @@ module RobustExcelOle
     end
   end
 
-
-
   describe "RobustExcelOle" do
     context "#absolute_path" do
       it "should work" do
@@ -242,9 +241,7 @@ module RobustExcelOle
         RobustExcelOle::absolute_path(@filename).gsub("\\","/").should == @filename
       end
     end
-
   end
-
 end
 
 class TestError < RuntimeError

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: robust_excel_ole
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: false
   segments: 
   - 0
   - 3
-  - 0
-  version: 0.3.0
+  - 1
+  version: 0.3.1
 platform: ruby
 authors: 
 - traths
@@ -15,29 +15,13 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2015-04-22 00:00:00 +02:00
+date: 2015-05-09 00:00:00 +02:00
 default_executable: 
 dependencies: 
-- !ruby/object:Gem::Dependency 
-  name: rake
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 63
-        segments: 
-        - 0
-        - 9
-        - 2
-        version: 0.9.2
-  type: :development
-  version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: rspec
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -49,72 +33,8 @@ dependencies:
         - 0
         version: 2.6.0
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rb-fchange
-  prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 21
-        segments: 
-        - 0
-        - 0
-        - 5
-        version: 0.0.5
-  type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: wdm
-  prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 25
-        segments: 
-        - 0
-        - 0
-        - 3
-        version: 0.0.3
-  type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: win32console
-  prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 31
-        segments: 
-        - 1
-        - 3
-        - 2
-        version: 1.3.2
-  type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
-  name: guard-rspec
-  prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 9
-        segments: 
-        - 2
-        - 1
-        - 1
-        version: 2.1.1
-  type: :development
-  version_requirements: *id006
-description: RobustExcelOle wraps the win32ole library, and allows to perform various operation in Excel with ruby in a reliable way. Detailed description please see the README.
+  version_requirements: *id001
+description: RobustExcelOle processes Excel files, provides all win32ole operations, convenient methods for opening, saving and closing, and implements an Excel file management system.
 email: 
 - Thomas.Raths@gmx.net
 executables: []
@@ -130,6 +50,7 @@ files:
 - Guardfile
 - LICENSE
 - README.rdoc
+- README_detail.rdoc
 - Rakefile
 - TodoList.md
 - examples/edit_sheets/example_access_sheets_and_cells.rb
@@ -221,6 +142,6 @@ rubyforge_project: robust_excel_ole
 rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
-summary: RobustExcelOle wraps the win32ole library and implements various operations in Excel
+summary: RobustExcelOle processes Excel files and wraps the win32ole library.
 test_files: []