fix: do not throw on parse errors (#34)

* fix: do not throw when cannot parse source file

* do not crash when spec has bad syntax

* add readme note

README.md

@@ -97,6 +97,8 @@ $ npx find-ids --specs 'cypress/e2e/**/*.cy.{js,ts}' --command getBy --test-ids
 
 This module uses [debug](https://github.com/debug-js/debug#readme) to output verbose logs. To see the logs, run with the following environment variable `DEBUG=changed-test-ids`
 
+If a source file or a spec have syntax that cannot be parsed, they are ignored. You can see a debug message `changed-test-ids ⚠️ could not parse spec <filename> +0ms`
+
 ## Examples
 
 - repo [bahmutov/taste-the-sauce-test-ids](https://github.com/bahmutov/taste-the-sauce-test-ids)

src/index.js

@@ -164,7 +164,13 @@ function findTestQueries(source, options = {}) {
 function findTestAttributesInFile(filename) {
   debug('searching %s', filename)
   const source = fs.readFileSync(filename, 'utf8')
-  return findTestAttributes(source)
+
+  try {
+    return findTestAttributes(source)
+  } catch (e) {
+    debug('⚠️ could not parse source code in file %s', filename)
+    return []
+  }
 }
 
 /**
@@ -191,7 +197,12 @@ function findTestAttributesInFiles(filenames) {
  */
 function findTestQueriesInFile(filename, options = {}) {
   const source = fs.readFileSync(filename, 'utf8')
-  return findTestQueries(source, options)
+  try {
+    return findTestQueries(source, options)
+  } catch (e) {
+    debug('⚠️ could not parse spec %s', filename)
+    return []
+  }
 }
 
 /**

tests/fixtures/bad.cy.js

@@ -0,0 +1,5 @@
+// this spec has bad syntax on purpose
+// the parser should not crash
+it('tests', () => {
+  cy.getTest('
+})

tests/fixtures/bad.jsx

@@ -0,0 +1,3 @@
+// this source file has bad syntax on purpose
+// the parser should not crash, just move on
+export default function something