Parser-removeUnusedTemps enhancement

Bert Freudenberg bert at freudenbergs.de
Wed Oct 3 10:15:28 UTC 2007


On Oct 3, 2007, at 11:08 , Damien Cassou wrote:

> 2007/10/3, Romain Robbes <romain.robbes at gmail.com>:
>> Well, writing a unit test for this will be a pain. It's a one- 
>> method change,
>> should I really write a new test class compiling several methods,  
>> somehow
>> bypassing user input to return an accurate answer, decompiling and  
>> comparing
>> the source code with an expected result, and remove the temporary  
>> methods
>> just for this?
>
> I'm just telling you what the release team is waiting for. But your
> arguments are right :-).

I thought the requirement was to not break any unit tests. Requiring  
every enhancement to come with a unit test is silly.


- Bert -





More information about the Squeak-dev mailing list