A little better RegExps and refactor (#130)

* A little better RegExps and refactor
* Return early
* Remove \r from regexps

Author: lilliliilil

Diff for: src/stripIndent/stripIndent.test.js

@@ -48,3 +48,9 @@ test('does nothing if there are no indents', async (t) => {
   const actual = stripIndent`wow such doge`
   t.is(actual, expected)
 })
+
+test('does nothing if minimal indent has zero length', async (t) => {
+  const expected = 'wow\n such\n doge'
+  const actual = stripIndent`wow\n such\n doge`
+  t.is(actual, expected)
+})

Diff for: src/stripIndentTransformer/stripIndentTransformer.js

@@ -9,21 +9,19 @@ const stripIndentTransformer = (type = 'initial') => ({
   onEndResult (endResult) {
     if (type === 'initial') {
       // remove the shortest leading indentation from each line
-      const match = endResult.match(/^[ \t]*(?=\S)/gm)
-      // return early if there's nothing to strip
-      if (match === null) {
-        return endResult
+      const match = endResult.match(/^[^\S\n]*(?=\S)/gm)
+      const indent = match && Math.min(...match.map(el => el.length))
+      if (indent) {
+        const regexp = new RegExp(`^.{${indent}}`, 'gm')
+        return endResult.replace(regexp, '')
       }
-      const indent = Math.min(...match.map(el => el.length))
-      const regexp = new RegExp('^[ \\t]{' + indent + '}', 'gm')
-      endResult = indent > 0 ? endResult.replace(regexp, '') : endResult
-    } else if (type === 'all') {
+      return endResult
+    }
+    if (type === 'all') {
       // remove all indentation from each line
-      endResult = endResult.replace(/(?:\n[^\S\n]*)/g, '\n')
-    } else {
-      throw new Error(`Unknown type: ${type}`)
+      return endResult.replace(/^[^\S\n]+/gm, '')
     }
-    return endResult
+    throw new Error(`Unknown type: ${type}`)
   }
 })