search

testng-team / testng

TestNG testing framework
https://testng.org
Apache License 2.0
1.99k stars 1.02k forks source link

No SAX Parser #558

Open smaudet opened 10 years ago

smaudet commented 10 years ago

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 commented 10 years ago

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 commented 6 years ago

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 commented 6 years ago

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.