Do not include src line if longer than 100 chars

danielstjules · Mar 29, 2017 · 1c8d47e · 1c8d47e
1 parent 9c21549
commit 1c8d47e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/lib/parser.js b/lib/parser.js
@@ -36,7 +36,7 @@ function _parse(src, filePath, sourceType) {
 }
 
 function getErrorContext(err, src) {
-  if (!err.loc || !err.loc.line) return '';
+  if (!err.loc || !err.loc.line || err.loc.column >= 100) return '';
 
   var line = src.split('\n')[err.loc.line - 1];
   var caret = ' '.repeat(err.loc.column) + '^';

diff --git a/spec/parserSpec.js b/spec/parserSpec.js
@@ -20,5 +20,13 @@ describe('parse', function() {
         expect(err.message).to.contain(`${src}\n    ^`);
       });
     });
+
+    it('does not include the src line if longer than 100 chars', function() {
+      var src = ' '.repeat(100) + ']';
+      var fn = () => Parser.parse(src, filePath);
+      expect(fn).to.throwException((err) => {
+        expect(err.message).not.to.contain(`^`);
+      });
+    });
   });
 });