Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Give feedback on file in error and line number when reporting console… #326

Closed
wants to merge 1 commit into from

Conversation

svjson
Copy link

@svjson svjson commented Sep 27, 2017

… errors

This addition saves a lot of headache when trying to discern what and where went wrong, ie, due to parse errors - in my case due to ES6 being used in source files.

@svjson
Copy link
Author

svjson commented Sep 27, 2017

Compare:

[ERROR] JavaScript Console Errors:
[ERROR]
[ERROR] * SyntaxError: Parse error

To:
[ERROR] JavaScript Console Errors:
[ERROR]
[ERROR] * SyntaxError: Parse error (http://localhost:41155/app/view/order/grid/OrderGridViewController.js?_dc=1506513677850:25)

@svjson
Copy link
Author

svjson commented Oct 13, 2017

This is particularly helpful, if not essential, in a larger project such as the one I'm working on right now where any use of ES6 will make phantomjs unable to parse a file and leave us looking for a needle in a haystack to find the problem.

@klieber klieber force-pushed the master branch 2 times, most recently from a622225 to 198a543 Compare September 16, 2020 04:03
@klieber klieber closed this in 31862a5 Sep 16, 2020
klieber pushed a commit to klieber/jasmine-maven-plugin that referenced this pull request Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant