Add test, fix example with tab color generation and fix a but due to a typo

3 files changed, 34 insertions, 8 deletions

diff --git a/examples/example.rb b/examples/example.rb
index fb4fc924..72066140 100755
--- a/examples/example.rb
+++ b/examples/example.rb
@@ -677,7 +677,7 @@ end
 
 ## Book Views
 #
-## Book views let you specify which sheet the show as active when the user opens the work book as well as a bunch of other 
+## Book views let you specify which sheet the show as active when the user opens the work book as well as a bunch of other
 ## tuning values for the UI @see Axlsx::WorkbookView
 ## ```ruby
 if examples.include? :book_view
@@ -849,9 +849,13 @@ end
 
 #```ruby
 if examples.include? :tab_color
+  p = Axlsx::Package.new
+  p.use_shared_strings = true
+  wb = p.workbook
   wb.add_worksheet(:name => "Change Tab Color") do |sheet|
     sheet.add_row ["Check", "out", "the", "Tab Color", "below!"]
     sheet.sheet_pr.tab_color = "FFFF6666"
   end
+  p.serialize 'tab_color.xlsx'
 end
-##```
\ No newline at end of file
+##```
diff --git a/lib/axlsx/workbook/worksheet/sheet_pr.rb b/lib/axlsx/workbook/worksheet/sheet_pr.rb
index e0056770..a299bc0f 100644
--- a/lib/axlsx/workbook/worksheet/sheet_pr.rb
+++ b/lib/axlsx/workbook/worksheet/sheet_pr.rb
@@ -8,7 +8,7 @@ module Axlsx
 
     serializable_attributes :sync_horizontal,
                             :sync_vertical,
-                            :transtion_evaluation,
+                            :transition_evaluation,
                             :transition_entry,
                             :published,
                             :filter_mode,
@@ -20,7 +20,7 @@ module Axlsx
     # waving magic show to set up the attriubte accessors
     boolean_attr_accessor :sync_horizontal,
                           :sync_vertical,
-                          :transtion_evaluation,
+                          :transition_evaluation,
                           :transition_entry,
                           :published,
                           :filter_mode,
diff --git a/test/workbook/worksheet/tc_sheet_pr.rb b/test/workbook/worksheet/tc_sheet_pr.rb
index 43f2941d..be45438b 100644
--- a/test/workbook/worksheet/tc_sheet_pr.rb
+++ b/test/workbook/worksheet/tc_sheet_pr.rb
@@ -5,23 +5,45 @@ class TestSheetPr < Test::Unit::TestCase
 
   def setup
     worksheet = Axlsx::Package.new.workbook.add_worksheet
-    @options = { 
+    @options = {
       :sync_horizontal => false,
       :sync_vertical => false,
-      :transtion_evaluation => true,
+      :transition_evaluation => true,
       :transition_entry => true,
       :published => false,
       :filter_mode => true,
       :enable_format_conditions_calculation => false,
       :code_name => '007',
-      :sync_ref => 'foo'
+      :sync_ref => 'foo',
+      :tab_color => 'FFFF6666'
     }
     @sheet_pr = Axlsx::SheetPr.new(worksheet, @options)
   end
 
   def test_initialization
     @options.each do |key, value|
-      assert_equal value, @sheet_pr.send(key)
+      if key==:tab_color
+        stored_value = @sheet_pr.send(key)
+        assert_equal Axlsx::Color, stored_value.class
+        assert_equal value, stored_value.rgb
+      else
+        assert_equal value, @sheet_pr.send(key)
+      end
     end
   end
+
+  def test_to_xml_string
+    doc = Nokogiri::XML(@sheet_pr.to_xml_string)
+    assert_equal(doc.xpath("//sheetPr[@syncHorizontal='0']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@syncVertical='0']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@transitionEvaluation='1']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@transitionEntry='1']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@published='0']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@filterMode='1']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@enableFormatConditionsCalculation='0']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@codeName='007']").size, 1)
+    assert_equal(doc.xpath("//sheetPr[@syncRef='foo']").size, 1)
+    assert_equal(doc.xpath("//sheetPr/tabColor[@rgb='FFFF6666']").size, 1)
+    assert_equal(doc.xpath("//sheetPr/pageSetUpPr[@fitToPage='0']").size, 1)
+  end
 end