[Test] Stop ignoring exceptions thrown in snapshot tests

Summary:
This logic can hide breakages and cause tests to pass when the
underlying logic is actually broken. I encountered these hidden
breakages while working on some further feature changes, and figured
it was appropriate to fix this first.

As all existing tests pass without the try/catch, this change is
effectively a no-op.

Test Plan:
`yarn install && yarn run test ./tests/`

Author: ckwalsh

test-setup/run_spec.mjs

@@ -45,7 +45,9 @@ function run_spec(dirname, parsers, options) {
                         raw(source + '~'.repeat(80) + '\n' + result),
                     ).toMatchSnapshot(filename);
                 } catch (e) {
-                    console.error(e, path);
+                    throw new Error(
+                        `Problem occurred in ${path} file: ${error.name}`,
+                    );
                 }
             });