search

mezuro / kalibro_processor

Reimplementation of Kalibro processing
GNU Affero General Public License v3.0
2 stars 10 forks source link

Create acceptance test for ModuleResult retrieval with Hotspot results #216

Closed rafamanzo closed 8 years ago

rafamanzo commented 8 years ago

We once thought we had this covered (fa7b986) but figured out it was working because of inner bugs (https://github.com/mezuro/kalibro_processor/pull/214#discussion_r71339138).

We have to create a new acceptance test that covers this.

danielkza commented 8 years ago

I think changing the step to check for the first descendant hotspot metric result from the root should fix it. It doesn't matter which result we use, as long as we can find one.