We're working on it, making small changes and creating code standards tests and code quality tests.
It would be great to have a way of tracking the results of these tests in a distant future and make them mandatory before integrate new things into the image.
Francisco Three points: - I agree on the principle. Normally I read all the code that is integrated - now we should pay attention. I prefer to put code that make progress but is not perfect versus doing nothing and waiting for a better one - then people can comment on the submission made on the tracker but really few do it. So what is the solution let the code rot there? And kill our good energy. So you see nothing is simple. This is clear that having a nice bunch of SmallLint rules should not hurt :) Stef
Francisco
On Sat, May 22, 2010 at 7:17 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
What I was thinking mariano is that we could move one class at a time in classTests and make the tests green. This way we do not have red tests and still make progress Stef
On May 22, 2010, at 8:06 PM, Mariano Abel Coca wrote:
It is a must. We have to got all the tests in green. But actually that isn't tested.
I'll make the tests, I'll fix them, and then I'll send the whole changes to merge in the baseline.
Cheers,
Mariano.
On Sat, May 22, 2010 at 2:58 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote: the key point is that we would like to avoid to have red tests all the time so fixing the as yet unclassified should be done before.
BTW Does anybody use BadEqualer HashTester PrototypeTester
because so far I do not see anybody user of them. They look like experiment code to me.
Stef
On May 22, 2010, at 7:31 PM, Mariano Abel Coca wrote:
Hi, I want to make global the checks included in ClassTestCase, and remove it. That is because actually it's only being tested for it's subclasses. And also, it's declaring that a subclass will be a class test instead of just a test, but no one of it's subclasses actually test the class, but the instances, therefore is a TestCase, not a ClassTestCase. See TimespanTest as an example.
What Im saying is that I'm trying to force a run of a suit of Code Standards and Code Quality tests before sending something to merge to a baseline of a project.
What do you think? Are you agree with that?
Cheers,
Mariano. _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project