[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Regtest changes phase 1 (issue 6454121)
From: |
David Kastrup |
Subject: |
Re: Regtest changes phase 1 (issue 6454121) |
Date: |
Wed, 08 Aug 2012 21:17:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) |
"Phil Holmes" <address@hidden> writes:
>> b) Although the marks don't affect the test they do
>> tend to mask the real tests and make it harder to see
>> what is really being tested.
>>
>> I'd leave the code alone and expand the description if
>> further clarity is needed.
>>
>> Trevor
>
>
> Absolutely appreciate this comment. However, the theory of the
> regtests is that by looking at the description and the image, you can
> tell whether the regtest has been passed. No-one who looked at this
> regtest could do that. With the changes proposed, you can. The
> question is - do you need to inspect the code, or the image and
> description, to tell whether the regtest has been passed?
Perhaps instead of code comments and/or descriptions we should employ
footnotes to point out the details of the test coverage? That would
seem like the most directly accessible variant.
--
David Kastrup
- Regtest changes phase 1 (issue 6454121), PhilEHolmes, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), PhilEHolmes, 2012/08/10
- Re: Regtest changes phase 1 (issue 6454121), graham, 2012/08/10
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/13