Fix gap width validator for bar charts (#108)

`gap_width` and `gap_depth` now allow only integers in the range of 0–500. The previous behaviour (requiring a percentage value) was according to the current version of the OOXML spec, but Excel seems to rely on an older version, where the gap amount was required to be a simple integer.

Also, `gapDepth` is only allowed in 3D bar charts, so it is now no longer available for 2D bar charts.

1 files changed, 13 insertions, 12 deletions

diff --git a/test/drawing/tc_bar_chart.rb b/test/drawing/tc_bar_chart.rb
index d064e5be..c8bdc8d0 100644
--- a/test/drawing/tc_bar_chart.rb
+++ b/test/drawing/tc_bar_chart.rb
@@ -32,18 +32,12 @@ class TestBarChart < Test::Unit::TestCase
    assert(@chart.grouping == :standard)
  end
 
-
- def test_gapWidth
-   assert_raise(ArgumentError, "require valid gap width") { @chart.gap_width = 200 }
-   assert_nothing_raised("allow valid gapWidth") { @chart.gap_width = "200%" }
-   assert(@chart.gap_width == "200%")
- end
-
- def test_gapDepth
-   assert_raise(ArgumentError, "require valid gap_depth") { @chart.gap_depth = 200 }
-   assert_nothing_raised("allow valid gap_depth") { @chart.gap_depth = "200%" }
-   assert(@chart.gap_depth == "200%")
- end
+  def test_gap_width
+    assert_raise(ArgumentError, "require valid gap width") { @chart.gap_width = -1 }
+    assert_raise(ArgumentError, "require valid gap width") { @chart.gap_width = 501 }
+    assert_nothing_raised("allow valid gap width") { @chart.gap_width = 200 }
+    assert_equal(@chart.gap_width, 200, 'gap width is incorrect')
+  end
 
   def test_overlap
     assert_raise(ArgumentError, "require valid overlap") { @chart.overlap = -101 }
@@ -76,6 +70,13 @@ class TestBarChart < Test::Unit::TestCase
     assert(cat_axis_position < val_axis_position, "cat_axis must occur earlier than val_axis in the XML")
   end
 
+  def test_to_xml_string_has_gap_width
+    gap_width_value = rand(0..500)
+    @chart.gap_width = gap_width_value
+    doc = Nokogiri::XML(@chart.to_xml_string)
+    assert_equal(doc.xpath("//c:barChart/c:gapWidth").first.attribute('val').value, gap_width_value.to_s)
+  end
+
   def test_to_xml_string_has_overlap
     overlap_value = rand(-100..100)
     @chart.overlap = overlap_value