Skip to content
This repository has been archived by the owner on Sep 7, 2018. It is now read-only.

Guidance on modifying buster to filter AssertionError stacktrace #409

Open
lgtout opened this issue May 13, 2014 · 6 comments
Open

Guidance on modifying buster to filter AssertionError stacktrace #409

lgtout opened this issue May 13, 2014 · 6 comments
Assignees
Labels
Milestone

Comments

@lgtout
Copy link

lgtout commented May 13, 2014

If I wanted to modify Buster such that, when an AssertionError occurs, the only lines shown in the stack trace output would be lines belonging to the "thing" being tested, or the test itself (filtering out all lines referencing buster or assertion framework code), where would I look in the buster source? Thanks much!

@dkl-ppi
Copy link
Member

dkl-ppi commented May 14, 2014

@dkl-ppi dkl-ppi self-assigned this May 14, 2014
@dkl-ppi
Copy link
Member

dkl-ppi commented May 15, 2014

@jaabio, what is the current state? Have you made any progress?

@lgtout
Copy link
Author

lgtout commented May 16, 2014

@dwittner

thanks for following up. i did take a look at the buster-test stack filter.

is it possible that the docs at the url you provided are out of date? i only had the buster npm package installed, so i installed the buster-test npm package to try out the instructions in the docs. (was that necessary, or is there some way of getting a reference to stackFilter with only the buster npm package installed?) it appears there is no longer a stackFilter field on buster-test. does that sound right?

@lgtout
Copy link
Author

lgtout commented May 16, 2014

@dwittner

just did some more exploring and see that buster-test is actually included as a dependency of the buster npm package, which seems to indicate that installing buster-test separately wasn't necessary. is that correct? if so, do you know how i might be able to get a reference to the stackFilter from within a node file module that contains buster tests? or is stackFilter configurable globally? thanks.

@dkl-ppi
Copy link
Member

dkl-ppi commented May 20, 2014

@jaabio, you are right. It seems the described way at http://docs.busterjs.org/en/latest/modules/buster-test/stack-filter/ isn't working at the moment, but it should. I will fix it asap and let you know.

@lgtout
Copy link
Author

lgtout commented May 20, 2014

Excellent. Thanks!
On May 20, 2014 4:31 AM, "Daniel Wittner" notifications@github.com wrote:

@jaabio https://github.com/jaabio, you are right. It seems the
described way at
http://docs.busterjs.org/en/latest/modules/buster-test/stack-filter/isn't working at the moment, but it should. I will fix it asap and let you
know.


Reply to this email directly or view it on GitHubhttps://github.com//issues/409#issuecomment-43614045
.

@dkl-ppi dkl-ppi added this to the 1.0 release milestone Jul 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants