Merge pull request #7 from placemark/json-errors

Handle JSON errors

Author: tmcw

src/errors.ts

@@ -14,7 +14,12 @@ interface HintIssueType extends HintIssueBase {
   code: 'invalid_type';
 }
 
-export type HintIssue = HintIssueRootType | HintIssueType;
+interface HintJSONIssue {
+  code: 'invalid_json';
+  line: number;
+}
+
+export type HintIssue = HintIssueRootType | HintIssueType | HintJSONIssue;
 
 export class HintError extends Error {
   issues: HintIssue[] = [];

src/index.ts

@@ -168,19 +168,8 @@ export const check = (jsonStr: string): GeoJSON => {
     });
   } catch (e) {
     issues.push({
-      code: 'invalid_type',
-      loc: {
-        start: {
-          line: e.line,
-          column: e.column,
-          offset: 0,
-        },
-        end: {
-          line: e.line,
-          column: e.column,
-          offset: 0,
-        },
-      },
+      code: 'invalid_json',
+      line: e.line,
     });
   }
   if (ast) checkObject(issues, ast.body);

test/bad.test.ts

@@ -40,6 +40,22 @@ describe('check', () => {
     ).toThrow(HintError);
   });
 
+  it('bad json', () => {
+    try {
+      check(
+        `{
+    "type": "MultiPoint"
+    "coordinates": [["foo", "bar"]]
+}`
+      );
+    } catch (e) {
+      expect(e.issues[0]).toEqual({
+        code: 'invalid_json',
+        line: 3,
+      });
+    }
+  });
+
   describe('works with fixtures', () => {
     const fixtureNames = readdirSync(Path.join(__dirname, './fixture/bad/'));
     for (let name of fixtureNames) {