Strange, the Fuel tests are green on my image and I have the last version of all packages.
Alexandre On 23 Nov 2011, at 18:27, [hidden email] wrote: > See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> > > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Hi,
I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :). I am describing the situation, because it will probably be useful for others that want to extend Moose with properties. You added: FAMIXPackage>>weightedMethodCount <MSEProperty: #weightedMethodCount type: #Number> <MSEComment: 'The sum of the complexity in a package'> ... This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter. So, one solution is to add FAMIXPackage>>weightedMethodCount: However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived: FAMIXPackage>>weightedMethodCount <MSEProperty: #weightedMethodCount type: #Number> <MSEComment: 'The sum of the complexity in a package'> <derived> ... Cheers, Doru On 23 Nov 2011, at 22:33, Alexandre Bergel wrote: > Strange, the Fuel tests are green on my image and I have the last version of all packages. > > Alexandre > > > On 23 Nov 2011, at 18:27, [hidden email] wrote: > >> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> >> >> > > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev -- www.tudorgirba.com "We are all great at making mistakes." _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Thanks for the clarification :D
Cheers, Santiago 2011/11/24 Tudor Girba <[hidden email]> Hi, Santiago Vidal _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
In reply to this post by Tudor Girba-2
Thanks!
It makes perfect sense. Alexandre On 24 Nov 2011, at 04:20, Tudor Girba wrote: > Hi, > > I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :). > > I am describing the situation, because it will probably be useful for others that want to extend Moose with properties. > > You added: > FAMIXPackage>>weightedMethodCount > <MSEProperty: #weightedMethodCount type: #Number> > <MSEComment: 'The sum of the complexity in a package'> > ... > > This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter. > > So, one solution is to add FAMIXPackage>>weightedMethodCount: > > However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived: > FAMIXPackage>>weightedMethodCount > <MSEProperty: #weightedMethodCount type: #Number> > <MSEComment: 'The sum of the complexity in a package'> > <derived> > ... > > Cheers, > Doru > > > On 23 Nov 2011, at 22:33, Alexandre Bergel wrote: > >> Strange, the Fuel tests are green on my image and I have the last version of all packages. >> >> Alexandre >> >> >> On 23 Nov 2011, at 18:27, [hidden email] wrote: >> >>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> >>> >>> >> >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev > > -- > www.tudorgirba.com > > "We are all great at making mistakes." > > > > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
In reply to this post by Tudor Girba-2
I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins?
Cheers, Alexandre On 24 Nov 2011, at 04:20, Tudor Girba wrote: > Hi, > > I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :). > > I am describing the situation, because it will probably be useful for others that want to extend Moose with properties. > > You added: > FAMIXPackage>>weightedMethodCount > <MSEProperty: #weightedMethodCount type: #Number> > <MSEComment: 'The sum of the complexity in a package'> > ... > > This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter. > > So, one solution is to add FAMIXPackage>>weightedMethodCount: > > However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived: > FAMIXPackage>>weightedMethodCount > <MSEProperty: #weightedMethodCount type: #Number> > <MSEComment: 'The sum of the complexity in a package'> > <derived> > ... > > Cheers, > Doru > > > On 23 Nov 2011, at 22:33, Alexandre Bergel wrote: > >> Strange, the Fuel tests are green on my image and I have the last version of all packages. >> >> Alexandre >> >> >> On 23 Nov 2011, at 18:27, [hidden email] wrote: >> >>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> >>> >>> >> >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev > > -- > www.tudorgirba.com > > "We are all great at making mistakes." > > > > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
I explained above.
Perhaps what I did not say is that MSEProperty: pragmas only take effect when you reset the meta description. So, I guess that you added the MSEProperty and ran the tests without reseting the meta descriptions. Given that the tests do not reset this meta description, they worked locally. When the server started fresh, the meta description got initialized and the new pragma took effect. Cheers, Doru On Thu, Nov 24, 2011 at 2:59 PM, Alexandre Bergel <[hidden email]> wrote: > I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins? > > Cheers, > Alexandre > > > On 24 Nov 2011, at 04:20, Tudor Girba wrote: > >> Hi, >> >> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :). >> >> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties. >> >> You added: >> FAMIXPackage>>weightedMethodCount >> <MSEProperty: #weightedMethodCount type: #Number> >> <MSEComment: 'The sum of the complexity in a package'> >> ... >> >> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter. >> >> So, one solution is to add FAMIXPackage>>weightedMethodCount: >> >> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived: >> FAMIXPackage>>weightedMethodCount >> <MSEProperty: #weightedMethodCount type: #Number> >> <MSEComment: 'The sum of the complexity in a package'> >> <derived> >> ... >> >> Cheers, >> Doru >> >> >> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote: >> >>> Strange, the Fuel tests are green on my image and I have the last version of all packages. >>> >>> Alexandre >>> >>> >>> On 23 Nov 2011, at 18:27, [hidden email] wrote: >>> >>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> >>>> >>>> >>> >>> -- >>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>> Alexandre Bergel http://www.bergel.eu >>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>> >>> >>> >>> >>> >>> _______________________________________________ >>> Moose-dev mailing list >>> [hidden email] >>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev >> >> -- >> www.tudorgirba.com >> >> "We are all great at making mistakes." >> >> >> >> >> >> >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev > > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev > -- www.tudorgirba.com "Every thing has its own flow" _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Ah yes!
Alexandre On 24 Nov 2011, at 11:29, Tudor Girba wrote: > I explained above. > > Perhaps what I did not say is that MSEProperty: pragmas only take > effect when you reset the meta description. So, I guess that you added > the MSEProperty and ran the tests without reseting the meta > descriptions. Given that the tests do not reset this meta description, > they worked locally. When the server started fresh, the meta > description got initialized and the new pragma took effect. > > Cheers, > Doru > > On Thu, Nov 24, 2011 at 2:59 PM, Alexandre Bergel > <[hidden email]> wrote: >> I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins? >> >> Cheers, >> Alexandre >> >> >> On 24 Nov 2011, at 04:20, Tudor Girba wrote: >> >>> Hi, >>> >>> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :). >>> >>> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties. >>> >>> You added: >>> FAMIXPackage>>weightedMethodCount >>> <MSEProperty: #weightedMethodCount type: #Number> >>> <MSEComment: 'The sum of the complexity in a package'> >>> ... >>> >>> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter. >>> >>> So, one solution is to add FAMIXPackage>>weightedMethodCount: >>> >>> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived: >>> FAMIXPackage>>weightedMethodCount >>> <MSEProperty: #weightedMethodCount type: #Number> >>> <MSEComment: 'The sum of the complexity in a package'> >>> <derived> >>> ... >>> >>> Cheers, >>> Doru >>> >>> >>> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote: >>> >>>> Strange, the Fuel tests are green on my image and I have the last version of all packages. >>>> >>>> Alexandre >>>> >>>> >>>> On 23 Nov 2011, at 18:27, [hidden email] wrote: >>>> >>>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/> >>>>> >>>>> >>>> >>>> -- >>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>>> Alexandre Bergel http://www.bergel.eu >>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>>> >>>> >>>> >>>> >>>> >>>> _______________________________________________ >>>> Moose-dev mailing list >>>> [hidden email] >>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev >>> >>> -- >>> www.tudorgirba.com >>> >>> "We are all great at making mistakes." >>> >>> >>> >>> >>> >>> >>> >>> >>> _______________________________________________ >>> Moose-dev mailing list >>> [hidden email] >>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev >> >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev >> > > > > -- > www.tudorgirba.com > "Every thing has its own flow" > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Interesting! happily it's fixed!
Martín
On Thu, Nov 24, 2011 at 11:31 AM, Alexandre Bergel <[hidden email]> wrote: Ah yes! _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Free forum by Nabble | Edit this page |