TechnionYP5777 / Bugquery

Bug query
9 stars 1 forks source link

Add tests to com.bugquery.serverside.stacktrace #27

Closed ZivIzhar closed 7 years ago

ZivIzhar commented 7 years ago

Add tests and increase the test coverage rate.

ZivIzhar commented 7 years ago

com.bugquery.serverside.stacktrace test coverage is not 97.6% (the uncoveraged lines are static classes default constructures).

AdiOmari commented 7 years ago

@ZivIzhar So the algorithm misses some stack trace lines? that is what you mean. Please discuss it with @yonilx, if they are not important (at the end of the trace) and hard to include without breaking other stuff (affecting precision and including noise) then leave it as is.

ZivIzhar commented 7 years ago

What I meant is that because these classes only have static methods, there are no objects of this class, therefore the coverage drops. It's basicly means 100% coverage.

AdiOmari commented 7 years ago

👍