Of course that is the reason but what changed the model? Before the tests were green so the model changed or the importer of the model changed. I will add the check something like:
self lookUpPropertyNamed: #PRIS computedAs: [
(self mooseModel allModelPackages size > 1) ifFalse: [0] ifTrue:[
self numberOfClientPackages / (self mooseModel allModelPackages size - 1)]
]
it is fine?
Cheers, Fabrizio
On 11 Oct 2011, at 17:56, Tudor Girba wrote:
Is it because the model has only one package? :)
In any case, the metric should have a check for size = 1.
Cheers, Doru
On 11 Oct 2011, at 16:22, Fabrizio Perin wrote:
Another question: basically all the Fuel tests end with an error because of a division by zero in this method:
relativeImportanceForSystem <MSEProperty: #relativeImportanceForSystem type: #Number> <MSEComment: 'The number of client packages normalized by the total number of packages'>
^ self lookUpPropertyNamed: #PRIS computedAs: [ self numberOfClientPackages / (self mooseModel allModelPackages size - 1)]
Something else changed lately that can make (self mooseModel allModelPackages size - 1) = 0?
Cheers,
Fabrizio
Moose-dev mailing list Moose-dev@iam.unibe.ch https://www.iam.unibe.ch/mailman/listinfo/moose-dev
-- www.tudorgirba.com
"Beauty is where we see it."
Moose-dev mailing list Moose-dev@iam.unibe.ch https://www.iam.unibe.ch/mailman/listinfo/moose-dev