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

No SAX Parser #558

Open
smaudet opened this issue Oct 17, 2014 · 3 comments
Open

No SAX Parser #558

smaudet opened this issue Oct 17, 2014 · 3 comments

Comments

@smaudet
Copy link

smaudet commented Oct 17, 2014

There is currently no SAX Parser for results in TestNG.

This is important as currently large test suites (>1k tests) will either fail or have difficulty generating test results thanks to all test parsing being done in memory.

It is not too difficult to write one, and the default Reporter classes do not provide the necessary api to be easily inheritable.

Additionally, the whole reporter system is wasteful, this system should be re-architect-ed to allow runnning reporters as part of a report loop with shared resources.

@smaudet
Copy link
Author

smaudet commented Oct 21, 2014

Am working on a Fix, current strategy is to use JAXB and add a separate Reporter class that uses org.testng.xml.jaxb generated classes. I have generated an xsd from the provided dtd, and generated the necessary classes, working on creating a modified reporter and testing the new reporter class.

@krmahadevan
Copy link
Member

ping @cbeust @juherr - I am not sure as to what needs to be done here. Would either of you please let me know a bit more in-detail on what is being expected ?

@juherr
Copy link
Member

juherr commented May 31, 2018

As I understand it, it is a feature request for the reporter api where we should not store all the result in memory but call listeners on the fly.

It is something I've already tried without success.
I think we should find a way because reporters are often the root cause of perf issues with huge suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants