Add RuboCop Minitest

1 files changed, 21 insertions, 9 deletions

diff --git a/test/drawing/tc_drawing.rb b/test/drawing/tc_drawing.rb
index 282a03da..a0cf9f98 100644
--- a/test/drawing/tc_drawing.rb
+++ b/test/drawing/tc_drawing.rb
@@ -7,23 +7,26 @@ class TestDrawing < Test::Unit::TestCase
   end
 
   def test_initialization
-    assert(@ws.workbook.drawings.empty?)
+    assert_empty(@ws.workbook.drawings)
   end
 
   def test_add_chart
     chart = @ws.add_chart(Axlsx::Pie3DChart, :title => "bob", :start_at => [0, 0], :end_at => [1, 1])
+
     assert(chart.is_a?(Axlsx::Pie3DChart), "must create a chart")
     assert_equal(@ws.workbook.charts.last, chart, "must be added to workbook charts collection")
     assert_equal(@ws.drawing.anchors.last.object.chart, chart, "an anchor has been created and holds a reference to this chart")
     anchor = @ws.drawing.anchors.last
-    assert_equal([anchor.from.row, anchor.from.col], [0, 0], "options for start at are applied")
-    assert_equal([anchor.to.row, anchor.to.col], [1, 1], "options for start at are applied")
-    assert_equal(chart.title.text, "bob", "option for title is applied")
+
+    assert_equal([0, 0], [anchor.from.row, anchor.from.col], "options for start at are applied")
+    assert_equal([1, 1], [anchor.to.row, anchor.to.col], "options for start at are applied")
+    assert_equal("bob", chart.title.text, "option for title is applied")
   end
 
   def test_add_image
     src = File.dirname(__FILE__) + "/../fixtures/image1.jpeg"
     image = @ws.add_image(:image_src => src, :start_at => [0, 0], :width => 600, :height => 400)
+
     assert(@ws.drawing.anchors.last.is_a?(Axlsx::OneCellAnchor))
     assert(image.is_a?(Axlsx::Pic))
     assert_equal(600, image.width)
@@ -33,37 +36,45 @@ class TestDrawing < Test::Unit::TestCase
   def test_add_two_cell_anchor_image
     src = File.dirname(__FILE__) + "/../fixtures/image1.jpeg"
     image = @ws.add_image(:image_src => src, :start_at => [0, 0], :end_at => [15, 0])
+
     assert(@ws.drawing.anchors.last.is_a?(Axlsx::TwoCellAnchor))
     assert(image.is_a?(Axlsx::Pic))
   end
 
   def test_charts
     chart = @ws.add_chart(Axlsx::Pie3DChart, :title => "bob", :start_at => [0, 0], :end_at => [1, 1])
+
     assert_equal(@ws.drawing.charts.last, chart, "add chart is returned")
     chart = @ws.add_chart(Axlsx::Pie3DChart, :title => "nancy", :start_at => [1, 5], :end_at => [5, 10])
+
     assert_equal(@ws.drawing.charts.last, chart, "add chart is returned")
   end
 
   def test_pn
     @ws.add_chart(Axlsx::Pie3DChart)
-    assert_equal(@ws.drawing.pn, "drawings/drawing1.xml")
+
+    assert_equal("drawings/drawing1.xml", @ws.drawing.pn)
   end
 
   def test_rels_pn
     @ws.add_chart(Axlsx::Pie3DChart)
-    assert_equal(@ws.drawing.rels_pn, "drawings/_rels/drawing1.xml.rels")
+
+    assert_equal("drawings/_rels/drawing1.xml.rels", @ws.drawing.rels_pn)
   end
 
   def test_index
     @ws.add_chart(Axlsx::Pie3DChart)
+
     assert_equal(@ws.drawing.index, @ws.workbook.drawings.index(@ws.drawing))
   end
 
   def test_relationships
     @ws.add_chart(Axlsx::Pie3DChart, :title => "bob", :start_at => [0, 0], :end_at => [1, 1])
-    assert_equal(@ws.drawing.relationships.size, 1, "adding a chart adds a relationship")
+
+    assert_equal(1, @ws.drawing.relationships.size, "adding a chart adds a relationship")
     @ws.add_chart(Axlsx::Pie3DChart, :title => "nancy", :start_at => [1, 5], :end_at => [5, 10])
-    assert_equal(@ws.drawing.relationships.size, 2, "adding a chart adds a relationship")
+
+    assert_equal(2, @ws.drawing.relationships.size, "adding a chart adds a relationship")
   end
 
   def test_to_xml
@@ -75,6 +86,7 @@ class TestDrawing < Test::Unit::TestCase
       errors.push error
       puts error.message
     end
-    assert(errors.empty?, "error free validation")
+
+    assert_empty(errors, "error free validation")
   end
 end