{"action":"created","issue":{"url":"https://api.github.com/repos/Khan/KaTeX/issues/297","labels_url":"https://api.github.com/repos/Khan/KaTeX/issues/297/labels{/name}","comments_url":"https://api.github.com/repos/Khan/KaTeX/issues/297/comments","events_url":"https://api.github.com/repos/Khan/KaTeX/issues/297/events","html_url":"https://github.com/Khan/KaTeX/issues/297","id":93651593,"number":297,"title":"Systematic testing of error messages","user":{"login":"gagern","id":190759,"avatar_url":"https://avatars.githubusercontent.com/u/190759?v=3","gravatar_id":"","url":"https://api.github.com/users/gagern","html_url":"https://github.com/gagern","followers_url":"https://api.github.com/users/gagern/followers","following_url":"https://api.github.com/users/gagern/following{/other_user}","gists_url":"https://api.github.com/users/gagern/gists{/gist_id}","starred_url":"https://api.github.com/users/gagern/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gagern/subscriptions","organizations_url":"https://api.github.com/users/gagern/orgs","repos_url":"https://api.github.com/users/gagern/repos","events_url":"https://api.github.com/users/gagern/events{/privacy}","received_events_url":"https://api.github.com/users/gagern/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-07T23:17:39Z","updated_at":"2015-07-07T23:51:22Z","closed_at":null,"body":"We have a lot of error messages a.k.a. `ParseError` exceptions all over the place, and I feel like many of them don't receive any testing at all. Would it make sense to systematically add unit tests for all of them, ensuring that the right message gets generated, with the right error position indication? Should that go into `katex-spec.js` or some other file instead?\r\n\r\nCome to think of it, such a systematic testing of rare code paths might well be the first step towards some code coverage control, e.g. using [istanbul](https://www.npmjs.com/package/istanbul). But that's a different issue, so let's stick to exceptions for now."},"comment":{"url":"https://api.github.com/repos/Khan/KaTeX/issues/comments/119376231","html_url":"https://github.com/Khan/KaTeX/issues/297#issuecomment-119376231","issue_url":"https://api.github.com/repos/Khan/KaTeX/issues/297","id":119376231,"user":{"login":"kevinb7","id":1044413,"avatar_url":"https://avatars.githubusercontent.com/u/1044413?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinb7","html_url":"https://github.com/kevinb7","followers_url":"https://api.github.com/users/kevinb7/followers","following_url":"https://api.github.com/users/kevinb7/following{/other_user}","gists_url":"https://api.github.com/users/kevinb7/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinb7/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinb7/subscriptions","organizations_url":"https://api.github.com/users/kevinb7/orgs","repos_url":"https://api.github.com/users/kevinb7/repos","events_url":"https://api.github.com/users/kevinb7/events{/privacy}","received_events_url":"https://api.github.com/users/kevinb7/received_events","type":"User","site_admin":false},"created_at":"2015-07-07T23:51:22Z","updated_at":"2015-07-07T23:51:22Z","body":"Good idea.  We should probably split `katex-spec.js` into separate spec files at some point.  Coverage is a good idea too."}}