Stringify: allow empty string for indent option (#151)

* test: write tests for indent behavior, including failing empty string test

* fix: allow empty string to be used in stringify indent option

Author: fritz-c

lib/stringify/identity.js

@@ -19,7 +19,7 @@ module.exports = Compiler;
 function Compiler(options) {
   options = options || {};
   Base.call(this, options);
-  this.indentation = options.indent;
+  this.indentation = typeof options.indent === 'string' ? options.indent : '  ';
 }
 
 /**
@@ -250,5 +250,5 @@ Compiler.prototype.indent = function(level) {
     return '';
   }
 
-  return Array(this.level).join(this.indentation || '  ');
+  return Array(this.level).join(this.indentation);
 };

test/stringify.js

@@ -110,3 +110,28 @@ describe('stringify(obj, {sourcemap: true})', function() {
     result.map.should.be.an.instanceOf(SourceMapGenerator);
   });
 });
+
+describe('stringify(obj, {indent: *})', function() {
+  var css =
+    '@media print {\n' +
+    '\tbody {\n' +
+    '\t\tbackground: #fff;\n' +
+    '\t}\n' +
+    '}';
+  var stylesheet = parse(css);
+
+  it('should default to two-space indent', function(){
+    var result = stringify(stylesheet);
+    result.should.eql(css.replace(/\t/g, '  '));
+  });
+
+  it('should indent according to the indent string', function(){
+    var result = stringify(stylesheet, { indent: '\t' });
+    result.should.eql(css);
+  });
+
+  it('should accept empty string for indent', function(){
+    var result = stringify(stylesheet, { indent: '' });
+    result.should.eql(css.replace(/\t/g, ''));
+  });
+});