Add RuboCop Minitest

1 files changed, 143 insertions, 78 deletions

diff --git a/test/workbook/worksheet/tc_worksheet.rb b/test/workbook/worksheet/tc_worksheet.rb
index fde0611d..103d1641 100644
--- a/test/workbook/worksheet/tc_worksheet.rb
+++ b/test/workbook/worksheet/tc_worksheet.rb
@@ -8,14 +8,16 @@ class TestWorksheet < Test::Unit::TestCase
   end
 
   def test_pn
-    assert_equal(@ws.pn, "worksheets/sheet1.xml")
+    assert_equal("worksheets/sheet1.xml", @ws.pn)
     ws = @ws.workbook.add_worksheet
-    assert_equal(ws.pn, "worksheets/sheet2.xml")
+
+    assert_equal("worksheets/sheet2.xml", ws.pn)
   end
 
   def test_name_is_html_encoded
     @ws.name = '<foo> & <bar>'
-    assert_equal(@ws.name, '&lt;foo&gt; &amp; &lt;bar&gt;')
+
+    assert_equal('&lt;foo&gt; &amp; &lt;bar&gt;', @ws.name)
   end
 
   def test_name_exception_on_invalid_character
@@ -59,7 +61,7 @@ class TestWorksheet < Test::Unit::TestCase
   def test_page_margins_yeild
     @ws.page_margins do |pm|
       assert(pm.is_a?(Axlsx::PageMargins))
-      assert(@ws.page_margins == pm)
+      assert_equal(@ws.page_margins, pm)
     end
   end
 
@@ -70,7 +72,7 @@ class TestWorksheet < Test::Unit::TestCase
   def test_page_setup_yield
     @ws.page_setup do |ps|
       assert(ps.is_a?(Axlsx::PageSetup))
-      assert(@ws.page_setup == ps)
+      assert_equal(@ws.page_setup, ps)
     end
   end
 
@@ -81,7 +83,7 @@ class TestWorksheet < Test::Unit::TestCase
   def test_print_options_yield
     @ws.print_options do |po|
       assert(po.is_a?(Axlsx::PrintOptions))
-      assert(@ws.print_options == po)
+      assert_equal(@ws.print_options, po)
     end
   end
 
@@ -92,7 +94,7 @@ class TestWorksheet < Test::Unit::TestCase
   def test_header_footer_yield
     @ws.header_footer do |hf|
       assert(hf.is_a?(Axlsx::HeaderFooter))
-      assert(@ws.header_footer == hf)
+      assert_equal(@ws.header_footer, hf)
     end
   end
 
@@ -108,7 +110,8 @@ class TestWorksheet < Test::Unit::TestCase
   def test_no_autowidth
     @ws.workbook.use_autowidth = false
     @ws.add_row [1, 2, 3, 4]
-    assert_equal(@ws.column_info[0].width, nil)
+
+    assert_nil(@ws.column_info[0].width)
   end
 
   def test_initialization_options
@@ -117,6 +120,7 @@ class TestWorksheet < Test::Unit::TestCase
     print_options = { :grid_lines => true, :headings => true, :horizontal_centered => true, :vertical_centered => true }
     header_footer = { :different_first => false, :different_odd_even => false, :odd_header => 'Header' }
     optioned = @ws.workbook.add_worksheet(:name => 'bob', :page_margins => page_margins, :page_setup => page_setup, :print_options => print_options, :header_footer => header_footer)
+
     page_margins.each_key do |key|
       assert_equal(page_margins[key], optioned.page_margins.send(key))
     end
@@ -129,7 +133,7 @@ class TestWorksheet < Test::Unit::TestCase
     header_footer.each_key do |key|
       assert_equal(header_footer[key], optioned.header_footer.send(key))
     end
-    assert_equal(optioned.name, 'bob')
+    assert_equal('bob', optioned.name)
   end
 
   # def test_use_gridlines
@@ -145,9 +149,10 @@ class TestWorksheet < Test::Unit::TestCase
   # end
 
   def test_rels_pn
-    assert_equal(@ws.rels_pn, "worksheets/_rels/sheet1.xml.rels")
+    assert_equal("worksheets/_rels/sheet1.xml.rels", @ws.rels_pn)
     ws = @ws.workbook.add_worksheet
-    assert_equal(ws.rels_pn, "worksheets/_rels/sheet2.xml.rels")
+
+    assert_equal("worksheets/_rels/sheet2.xml.rels", ws.rels_pn)
   end
 
   def test_rId
@@ -161,11 +166,13 @@ class TestWorksheet < Test::Unit::TestCase
   def test_dimension
     @ws.add_row [1, 2, 3]
     @ws.add_row [4, 5, 6]
-    assert_equal @ws.dimension.sqref, "A1:C2"
+
+    assert_equal("A1:C2", @ws.dimension.sqref)
   end
 
   def test_dimension_with_empty_row
     @ws.add_row
+
     assert_equal "A1:AA200", @ws.dimension.sqref
   end
 
@@ -175,30 +182,34 @@ class TestWorksheet < Test::Unit::TestCase
     range = @ws["A1:C2"]
     first_row = @ws[0]
     last_row = @ws[1]
+
     assert_equal(@ws.rows[0], first_row)
     assert_equal(@ws.rows[1], last_row)
-    assert_equal(range.size, 6)
+    assert_equal(6, range.size)
     assert_equal(range.first, @ws.rows.first.cells.first)
     assert_equal(range.last, @ws.rows.last.cells.last)
   end
 
   def test_add_row
-    assert(@ws.rows.empty?, "sheet has no rows by default")
+    assert_empty(@ws.rows, "sheet has no rows by default")
     r = @ws.add_row([1, 2, 3])
-    assert_equal(@ws.rows.size, 1, "add_row adds a row")
+
+    assert_equal(1, @ws.rows.size, "add_row adds a row")
     assert_equal(@ws.rows.first, r, "the row returned is the row added")
   end
 
   def test_add_chart
-    assert(@ws.workbook.charts.empty?, "the sheet's workbook should not have any charts by default")
+    assert_empty(@ws.workbook.charts, "the sheet's workbook should not have any charts by default")
     @ws.add_chart Axlsx::Pie3DChart
-    assert_equal(@ws.workbook.charts.size, 1, "add_chart adds a chart to the workbook")
+
+    assert_equal(1, @ws.workbook.charts.size, "add_chart adds a chart to the workbook")
   end
 
   def test_add_page_break_with_string_cell_ref
-    assert(@ws.row_breaks.empty?)
-    assert(@ws.col_breaks.empty?)
+    assert_empty(@ws.row_breaks)
+    assert_empty(@ws.col_breaks)
     @ws.add_page_break("B1")
+
     assert_equal(1, @ws.row_breaks.size)
     assert_equal(1, @ws.col_breaks.size)
   end
@@ -207,23 +218,26 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_row [1, 2, 3, 4]
     @ws.add_row [1, 2, 3, 4]
 
-    assert(@ws.row_breaks.empty?)
-    assert(@ws.col_breaks.empty?)
+    assert_empty(@ws.row_breaks)
+    assert_empty(@ws.col_breaks)
     @ws.add_page_break(@ws.rows.last.cells[1])
+
     assert_equal(1, @ws.row_breaks.size)
     assert_equal(1, @ws.col_breaks.size)
   end
 
   def test_drawing
-    assert @ws.drawing.nil?
+    assert_nil @ws.drawing
     @ws.add_chart(Axlsx::Pie3DChart)
+
     assert @ws.drawing.is_a?(Axlsx::Drawing)
   end
 
   def test_add_pivot_table
-    assert(@ws.workbook.pivot_tables.empty?, "the sheet's workbook should not have any pivot tables by default")
+    assert_empty(@ws.workbook.pivot_tables, "the sheet's workbook should not have any pivot tables by default")
     @ws.add_pivot_table 'G5:G6', 'A1:D:10'
-    assert_equal(@ws.workbook.pivot_tables.size, 1, "add_pivot_tables adds a pivot_table to the workbook")
+
+    assert_equal(1, @ws.workbook.pivot_tables.size, "add_pivot_tables adds a pivot_table to the workbook")
   end
 
   def test_col_style
@@ -233,11 +247,11 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_row [1, 2, 3, 4]
     @ws.col_style((1..2), 1, :row_offset => 1)
     @ws.rows[(1..-1)].each do |r|
-      assert_equal(r.cells[1].style, 1)
-      assert_equal(r.cells[2].style, 1)
+      assert_equal(1, r.cells[1].style)
+      assert_equal(1, r.cells[2].style)
     end
-    assert_equal(@ws.rows.first.cells[1].style, 0)
-    assert_equal(@ws.rows.first.cells[0].style, 0)
+    assert_equal(0, @ws.rows.first.cells[1].style)
+    assert_equal(0, @ws.rows.first.cells[0].style)
   end
 
   def test_col_style_with_empty_column
@@ -253,14 +267,16 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_row [1, 2, 3]
     @ws.add_row [1, 2, 3, 4]
     c = @ws.cols[1]
-    assert_equal(c.size, 4)
-    assert_equal(c[0].value, 2)
+
+    assert_equal(4, c.size)
+    assert_equal(2, c[0].value)
   end
 
   def test_cols_with_block
     @ws.add_row [1, 2, 3]
     @ws.add_row [1]
     cols = @ws.cols { |_row, _column| :foo }
+
     assert_equal(:foo, cols[1][1])
   end
 
@@ -270,15 +286,16 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_row [1, 2, 3, 4]
     @ws.add_row [1, 2, 3, 4]
     @ws.row_style 1, 1, :col_offset => 1
+
     @ws.rows[1].cells[(1..-1)].each do |c|
-      assert_equal(c.style, 1)
+      assert_equal(1, c.style)
     end
-    assert_equal(@ws.rows[1].cells[0].style, 0)
-    assert_equal(@ws.rows[2].cells[1].style, 0)
+    assert_equal(0, @ws.rows[1].cells[0].style)
+    assert_equal(0, @ws.rows[2].cells[1].style)
     @ws.row_style(1..2, 1, :col_offset => 2)
     @ws.rows[(1..2)].each do |r|
       r.cells[(2..-1)].each do |c|
-        assert_equal(c.style, 1)
+        assert_equal(1, c.style)
       end
     end
   end
@@ -292,7 +309,8 @@ class TestWorksheet < Test::Unit::TestCase
   def test_to_xml_string_dimensions
     @ws.add_row [1, 2, 3]
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:dimension[@ref="A1:C1"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:dimension[@ref="A1:C1"]').size)
   end
 
   # def test_fit_to_page_assignation_does_nothing
@@ -315,20 +333,23 @@ class TestWorksheet < Test::Unit::TestCase
   def test_to_xml_string_auto_fit_data
     @ws.add_row [1, "two"]
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:cols/xmlns:col').size, 2)
+
+    assert_equal(2, doc.xpath('//xmlns:worksheet/xmlns:cols/xmlns:col').size)
   end
 
   def test_to_xml_string_sheet_data
     @ws.add_row [1, "two"]
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:sheetData/xmlns:row').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:sheetData/xmlns:row').size)
   end
 
   def test_to_xml_string_auto_filter
     @ws.add_row [1, "two"]
     @ws.auto_filter.range = "A1:B1"
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:autoFilter[@ref="A1:B1"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:autoFilter[@ref="A1:B1"]').size)
   end
 
   def test_to_xml_string_merge_cells
@@ -336,26 +357,30 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.merge_cells "A1:D1"
     @ws.merge_cells "E1:F1"
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="A1:D1"]').size, 1)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="E1:F1"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="A1:D1"]').size)
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="E1:F1"]').size)
   end
 
   def test_to_xml_string_merge_cells_row
     row = @ws.add_row [1, "two"]
     @ws.merge_cells row
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="A1:B1"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:mergeCells/xmlns:mergeCell[@ref="A1:B1"]').size)
   end
 
   def test_to_xml_string_row_breaks
     @ws.add_page_break("A1")
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:rowBreaks/xmlns:brk[@id="0"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:rowBreaks/xmlns:brk[@id="0"]').size)
   end
 
   def test_to_xml_string_sheet_protection
     @ws.sheet_protection.password = 'fish'
     doc = Nokogiri::XML(@ws.to_xml_string)
+
     assert(doc.xpath('//xmlns:sheetProtection'))
   end
 
@@ -365,7 +390,8 @@ class TestWorksheet < Test::Unit::TestCase
       pm.right = 7
     end
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:pageMargins[@left="9"][@right="7"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:pageMargins[@left="9"][@right="7"]').size)
   end
 
   def test_to_xml_string_page_setup
@@ -374,7 +400,8 @@ class TestWorksheet < Test::Unit::TestCase
       ps.scale = 80
     end
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:pageSetup[@paperWidth="210mm"][@scale="80"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:pageSetup[@paperWidth="210mm"][@scale="80"]').size)
   end
 
   def test_to_xml_string_print_options
@@ -383,7 +410,8 @@ class TestWorksheet < Test::Unit::TestCase
       po.horizontal_centered = true
     end
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:printOptions[@gridLines=1][@horizontalCentered=1]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:printOptions[@gridLines=1][@horizontalCentered=1]').size)
   end
 
   def test_to_xml_string_header_footer
@@ -393,12 +421,14 @@ class TestWorksheet < Test::Unit::TestCase
       hf.odd_header = 'Test Header'
     end
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:headerFooter[@differentFirst=0][@differentOddEven=0]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:headerFooter[@differentFirst=0][@differentOddEven=0]').size)
   end
 
   def test_to_xml_string_drawing
     @ws.add_chart Axlsx::Pie3DChart
     doc = Nokogiri::XML(@ws.to_xml_string)
+
     assert_equal @ws.send(:worksheet_drawing).relationship.Id, doc.xpath('//xmlns:worksheet/xmlns:drawing').first["r:id"]
   end
 
@@ -407,14 +437,16 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_row [1, 2]
     table = @ws.add_table "A1:B2"
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:tableParts[@count="1"]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:tableParts[@count="1"]').size)
     assert_equal table.rId, doc.xpath('//xmlns:worksheet/xmlns:tableParts/xmlns:tablePart').first["r:id"]
   end
 
   def test_to_xml_string
     schema = Nokogiri::XML::Schema(File.open(Axlsx::SML_XSD))
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert(schema.validate(doc).map { |e| puts e.message; e }.empty?, "error free validation")
+
+    assert_empty(schema.validate(doc).map { |e| puts e.message; e }, "error free validation")
   end
 
   def test_styles
@@ -426,14 +458,16 @@ class TestWorksheet < Test::Unit::TestCase
     Axlsx::trust_input = false
     nasties = "\v\u2028\u0001\u0002\u0003\u0004\u0005\u0006\u0007\u0008\u001f"
     @ws.add_row [nasties]
+
     assert_equal(0, @ws.rows.last.cells.last.value.index("\v"))
-    assert_equal(nil, @ws.to_xml_string.index("\v"))
+    assert_nil(@ws.to_xml_string.index("\v"))
     Axlsx::trust_input = old
   end
 
   def test_to_xml_string_with_newlines
     cell_with_newline = "foo\n\r\nbar"
     @ws.add_row [cell_with_newline]
+
     assert_equal("foo\n\r\nbar", @ws.rows.last.cells.last.value)
     assert_not_nil(@ws.to_xml_string.index("foo\n\r\nbar"))
   end
@@ -451,40 +485,50 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.add_pivot_table 'G5:G6', 'A1:D10'
     schema = Nokogiri::XML::Schema(File.open(Axlsx::SML_XSD))
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert(schema.validate(doc).map { |e| puts e.message; e }.empty?, schema.validate(doc).map(&:message).join('\n'))
+
+    assert_empty(schema.validate(doc).map { |e| puts e.message; e }, schema.validate(doc).map(&:message).join('\n'))
   end
 
   def test_relationships
     @ws.add_row [1, 2, 3]
-    assert(@ws.relationships.empty?, "No Drawing relationship until you add a chart")
+
+    assert_empty(@ws.relationships, "No Drawing relationship until you add a chart")
     @ws.add_chart Axlsx::Pie3DChart
-    assert_equal(@ws.relationships.size, 1, "adding a chart creates the relationship")
+
+    assert_equal(1, @ws.relationships.size, "adding a chart creates the relationship")
     @ws.add_chart Axlsx::Pie3DChart
-    assert_equal(@ws.relationships.size, 1, "multiple charts still only result in one relationship")
+
+    assert_equal(1, @ws.relationships.size, "multiple charts still only result in one relationship")
     @ws.add_comment :text => 'builder', :author => 'bob', :ref => @ws.rows.last.cells.last
-    assert_equal(@ws.relationships.size, 3, "adding a comment adds 2 relationships")
+
+    assert_equal(3, @ws.relationships.size, "adding a comment adds 2 relationships")
     @ws.add_comment :text => 'not that is a comment!', :author => 'travis', :ref => "A1"
-    assert_equal(@ws.relationships.size, 3, "adding multiple comments in the same worksheet should not add any additional comment relationships")
+
+    assert_equal(3, @ws.relationships.size, "adding multiple comments in the same worksheet should not add any additional comment relationships")
     @ws.add_pivot_table 'G5:G6', 'A1:D10'
-    assert_equal(@ws.relationships.size, 4, "adding a pivot table adds 1 relationship")
+
+    assert_equal(4, @ws.relationships.size, "adding a pivot table adds 1 relationship")
   end
 
   def test_set_fixed_width_column
     @ws.add_row ["mule", "donkey", "horse"], :widths => [20, :ignore, nil]
-    assert(@ws.column_info.size == 3, "a data item for each column")
+
+    assert_equal(3, @ws.column_info.size, "a data item for each column")
     assert_equal(20, @ws.column_info[0].width, "adding a row with fixed width updates :fixed attribute")
-    assert_equal(@ws.column_info[1].width, nil, ":ignore does not set any data")
+    assert_nil(@ws.column_info[1].width, ":ignore does not set any data")
   end
 
   def test_fixed_height
     @ws.add_row [1, 2, 3], :height => 40
+
     assert_equal(40, @ws.rows[-1].height)
   end
 
   def test_set_column_width
     @ws.add_row ["chasing windmills", "penut"]
     @ws.column_widths nil, 0.5
-    assert_equal(@ws.column_info[1].width, 0.5, 'eat my width')
+
+    assert_in_delta(@ws.column_info[1].width, 0.5, 0.001, 'eat my width')
     assert_raise(ArgumentError, 'only accept unsigned ints') { @ws.column_widths 2, 7, -1 }
     assert_raise(ArgumentError, 'only accept Integer or Float') { @ws.column_widths 2, 7, "-1" }
   end
@@ -524,6 +568,7 @@ class TestWorksheet < Test::Unit::TestCase
     assert(@ws.send(:protected_ranges).is_a?(Axlsx::SimpleTypedList))
     assert_equal(0, @ws.send(:protected_ranges).size)
     @ws.protect_range('A1:A3')
+
     assert_equal('A1:A3', @ws.send(:protected_ranges).last.sqref)
   end
 
@@ -540,8 +585,9 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.merge_cells "A1:A2"
     @ws.merge_cells "B2:C3"
     @ws.merge_cells @ws.rows.last.cells[(0..1)]
-    assert_equal(@ws.send(:merged_cells).size, 3)
-    assert_equal(@ws.send(:merged_cells).last, "A3:B3")
+
+    assert_equal(3, @ws.send(:merged_cells).size)
+    assert_equal("A3:B3", @ws.send(:merged_cells).last)
   end
 
   def test_merge_cells_sorts_correctly_by_row_when_given_array
@@ -549,37 +595,43 @@ class TestWorksheet < Test::Unit::TestCase
       @ws.add_row [i]
     end
     @ws.merge_cells [@ws.rows[8].cells.first, @ws.rows[9].cells.first]
+
     assert_equal "A9:A10", @ws.send(:merged_cells).first
   end
 
   def test_auto_filter
-    assert(@ws.auto_filter.range.nil?)
+    assert_nil(@ws.auto_filter.range)
     assert(@wb.defined_names.none? { |defined_name| defined_name.name == '_xlnm._FilterDatabase' })
     assert_raise(ArgumentError) { @ws.auto_filter = 123 }
     @ws.auto_filter.range = "A1:D9"
-    assert_equal(@ws.auto_filter.range, "A1:D9")
+
+    assert_equal("A1:D9", @ws.auto_filter.range)
     @ws.to_xml_string
+
     assert(@wb.defined_names.any? { |defined_name| defined_name.name == '_xlnm._FilterDatabase' })
   end
 
   def test_auto_filter_assign
     other_ws = @wb.add_worksheet
 
-    assert(@ws.auto_filter.range.nil?)
-    assert(other_ws.auto_filter.range.nil?)
+    assert_nil(@ws.auto_filter.range)
+    assert_nil(other_ws.auto_filter.range)
     assert(@wb.defined_names.none? { |defined_name| defined_name.name == '_xlnm._FilterDatabase' })
     assert_raise(ArgumentError) { @ws.auto_filter = 123 }
 
     @ws.auto_filter = "A1:D9"
-    assert_equal(@ws.auto_filter.range, "A1:D9")
+
+    assert_equal("A1:D9", @ws.auto_filter.range)
 
     other_ws.auto_filter = "A1:D2"
-    assert_equal(other_ws.auto_filter.range, "A1:D2")
+
+    assert_equal("A1:D2", other_ws.auto_filter.range)
 
     @ws.to_xml_string
     other_ws.to_xml_string
 
     filter_database = @wb.defined_names.select { |defined_name| defined_name.name == '_xlnm._FilterDatabase' }
+
     assert_equal(2, filter_database.size)
     assert_equal(@ws.index, filter_database[0].local_sheet_id)
     assert_equal(other_ws.index, filter_database[1].local_sheet_id)
@@ -589,23 +641,26 @@ class TestWorksheet < Test::Unit::TestCase
     @ws.auto_filter.range = 'A1:D9'
     @ws.auto_filter.add_column 0, :filters, :filter_items => [1]
     doc = Nokogiri::XML(@ws.to_xml_string)
+
     assert(doc.xpath('//sheetPr[@filterMode=1]'))
   end
 
   def test_outline_level_rows
     3.times { @ws.add_row [1, 2, 3] }
     @ws.outline_level_rows 0, 2
+
     assert_equal(1, @ws.rows[0].outline_level)
-    assert_equal(true, @ws.rows[2].hidden)
-    assert_equal(true, @ws.sheet_view.show_outline_symbols)
+    assert(@ws.rows[2].hidden)
+    assert(@ws.sheet_view.show_outline_symbols)
   end
 
   def test_outline_level_columns
     3.times { @ws.add_row [1, 2, 3] }
     @ws.outline_level_columns 0, 2
+
     assert_equal(1, @ws.column_info[0].outline_level)
-    assert_equal(true, @ws.column_info[2].hidden)
-    assert_equal(true, @ws.sheet_view.show_outline_symbols)
+    assert(@ws.column_info[2].hidden)
+    assert(@ws.sheet_view.show_outline_symbols)
   end
 
   def test_worksheet_does_not_get_added_to_workbook_on_initialize_failure
@@ -616,13 +671,15 @@ class TestWorksheet < Test::Unit::TestCase
 
   def test_worksheet_only_includes_outline_pr_when_set
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr').size, 0)
+
+    assert_equal(0, doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr').size)
 
     @ws.sheet_pr.outline_pr.summary_below = false
     @ws.sheet_pr.outline_pr.summary_right = true
     doc = Nokogiri::XML(@ws.to_xml_string)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr').size, 1)
-    assert_equal(doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr[@summaryBelow=0][@summaryRight=1]').size, 1)
+
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr').size)
+    assert_equal(1, doc.xpath('//xmlns:worksheet/xmlns:sheetPr/xmlns:outlinePr[@summaryBelow=0][@summaryRight=1]').size)
   end
 
   def test_merge_styles_1
@@ -706,7 +763,7 @@ class TestWorksheet < Test::Unit::TestCase
 
     wb.apply_styles
 
-    assert_equal true, wb.styles_applied
+    assert wb.styles_applied
     assert_equal 12, wb.styles.style_index.count
     assert_equal 12 + 2, wb.styles.style_index.keys.max
   end
@@ -731,7 +788,7 @@ class TestWorksheet < Test::Unit::TestCase
 
     wb.apply_styles
 
-    assert_equal true, wb.styles_applied
+    assert wb.styles_applied
     assert_equal 17, wb.styles.style_index.count
   end
 
@@ -751,7 +808,7 @@ class TestWorksheet < Test::Unit::TestCase
 
     wb.apply_styles
 
-    assert_equal true, wb.styles_applied
+    assert wb.styles_applied
     assert_equal 8, wb.styles.style_index.count
     assert_equal 8, wb.styled_cells.count
   end
@@ -771,7 +828,7 @@ class TestWorksheet < Test::Unit::TestCase
 
     wb.apply_styles
 
-    assert_equal true, wb.styles_applied
+    assert wb.styles_applied
     assert_equal 6, wb.styles.style_index.count
     assert_equal 6, wb.styled_cells.count
 
@@ -786,6 +843,7 @@ class TestWorksheet < Test::Unit::TestCase
       sz: 11,
       family: 1
     }
+
     assert_equal b2_cell_style, (wb.styles.style_index.values.find { |x| x == b2_cell_style })
 
     d3_cell_style = {
@@ -799,6 +857,7 @@ class TestWorksheet < Test::Unit::TestCase
       sz: 11,
       family: 1
     }
+
     assert_equal d3_cell_style, (wb.styles.style_index.values.find { |x| x == d3_cell_style })
   end
 
@@ -895,6 +954,7 @@ class TestWorksheet < Test::Unit::TestCase
   def test_escape_formulas
     @wb.escape_formulas = false
     @ws = @wb.add_worksheet
+
     assert_false @ws.escape_formulas
     assert_false @ws.add_row(['']).cells.first.escape_formulas
     assert_false @ws.add_row([''], escape_formulas: false).cells.first.escape_formulas
@@ -902,10 +962,12 @@ class TestWorksheet < Test::Unit::TestCase
     assert_equal [true, false], @ws.add_row(['', ''], escape_formulas: [true, false]).cells.map(&:escape_formulas)
 
     @ws = Axlsx::Worksheet.new(@wb)
+
     assert_false @ws.escape_formulas
 
     @wb.escape_formulas = true
     @ws = @wb.add_worksheet
+
     assert @ws.escape_formulas
     assert @ws.add_row(['']).cells.first.escape_formulas
     assert_false @ws.add_row([''], escape_formulas: false).cells.first.escape_formulas
@@ -913,9 +975,11 @@ class TestWorksheet < Test::Unit::TestCase
     assert_equal [true, false], @ws.add_row(['', ''], escape_formulas: [true, false]).cells.map(&:escape_formulas)
 
     @ws = Axlsx::Worksheet.new(@wb)
+
     assert @ws.escape_formulas
 
     @ws.escape_formulas = false
+
     assert_false @ws.escape_formulas
     assert_false @ws.add_row(['']).cells.first.escape_formulas
     assert_false @ws.add_row([''], escape_formulas: false).cells.first.escape_formulas
@@ -923,6 +987,7 @@ class TestWorksheet < Test::Unit::TestCase
     assert_equal [true, false], @ws.add_row(['', ''], escape_formulas: [true, false]).cells.map(&:escape_formulas)
 
     @ws.escape_formulas = true
+
     assert @ws.escape_formulas
     assert @ws.add_row(['']).cells.first.escape_formulas
     assert_false @ws.add_row([''], escape_formulas: false).cells.first.escape_formulas