Return original input when specified range is unformattable (prettier…

…#2250) * Add failing test for range-formatting whitespace See prettier#2247 (comment) * Return original input when specified range is unformattable Fixes prettier#2247
roryokane · Jun 24, 2017 · 4b955a1 · 4b955a1
1 parent b0413fc
commit 4b955a1
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 0 deletions.
diff --git a/index.js b/index.js
@@ -219,6 +219,14 @@ function calculateRange(text, opts, ast) {
     endNonWhitespace,
     isSourceElement
   );
+
+  if (!startNodeAndParents || !endNodeAndParents) {
+    return {
+      rangeStart: 0,
+      rangeEnd: 0
+    };
+  }
+
   const siblingAncestors = findSiblingAncestors(
     startNodeAndParents,
     endNodeAndParents

diff --git a/tests/range/__snapshots__/jsfmt.spec.js.snap b/tests/range/__snapshots__/jsfmt.spec.js.snap
@@ -223,4 +223,11 @@ catch (err) {}
 try {
 } catch (err) {}
 
+`;
+
+exports[`whitespace.js 1`] = `
+ 
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ 
+
 `;
diff --git a/tests/range/whitespace.js b/tests/range/whitespace.js
@@ -0,0 +1 @@
+<<<PRETTIER_RANGE_START>>> <<<PRETTIER_RANGE_END>>>
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		<<<PRETTIER_RANGE_START>>> <<<PRETTIER_RANGE_END>>>