Fix indent bug for tables without headers (issue piotrmurach/tty-table/issues/45)

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change log
 
+### Fixed
+* Fix indentation of tables without headers by Roger Marlow(@rogermarlow)
+
 ## [v0.12.0] - 2020-09-20
 
 ### Changed

lib/tty/table/indentation.rb

@@ -30,7 +30,7 @@ def indent(part, indentation)
       #
       # @api public
       def insert_indentation(line, indentation)
-        line ? " " * indentation + line.to_s : ""
+        line ? " " * indentation + line.to_s : line
       end
       module_function :insert_indentation
     end # Indentation

lib/tty/table/renderer/basic.rb

@@ -199,7 +199,6 @@ def render
             operations.add(*op)
           end
           operations.apply_to(table, *select_operations)
-
           render_data.compact.join("\n")
         end
 
@@ -267,7 +266,9 @@ def render_data
         # @api private
         def render_header(row, data_border)
           top_line = data_border.top_line
-          return top_line unless row.is_a?(TTY::Table::Header)
+          unless row.is_a?(TTY::Table::Header)
+            return Indentation.indent(top_line, @indent)
+          end
           header = [top_line, data_border.row_line(row)]
           if !border.separator || border.separator?(0)
             header << data_border.middle_line

spec/unit/renderer/ascii/indentation_spec.rb

@@ -37,3 +37,17 @@
     end
   end
 end
+
+RSpec.describe TTY::Table::Renderer::ASCII, "indentation without headers" do
+  it "does not regress on issue 45" do
+    table = TTY::Table.new
+    table << %w(a1 a2 a3)
+    table << %w(b1 b2 b3)
+    expect(table.render(:ascii, indent: 3)).to eql([
+      "   +--+--+--+",
+      "   |a1|a2|a3|",
+      "   |b1|b2|b3|",
+      "   +--+--+--+"
+    ].join("\n"))
+  end
+end