PATCH: tests for MI commands

[Daniel Jacobowitz]

On Thu, Jul 28, 2005 at 10:25:48AM +1200, Nick Roberts wrote:
>  >    Given the scarce resources for this unpaid work I ask you again to
>  >    consider removing the mi2-*.exp files and just test the current version
>  >    of MI.  The reality is that they serve no useful purpose (mi-*.exp tests
>  >    would still pass if GDB was invoked with "-i=mi2") and they require
>  >    effort that could be better spent on other things.
>  > 
>  > We can discuss this *after* you either:
>  > a) Fixed the test failures.
>  > 
>  > b) Backed out your patch.
> 
> I'd like to do a) by removing the mi2-*.exp files ;-).  There doesn't seem
> much point in making a load of changes if we are going to just remove the
> files afterwards.

Obviously this is not uncontroversial, so it is not appropriate as a
short term solution.

>  > You promised not to do this again, and yet you did.
> 
> Let's remember that the testsuite is there to prevent regressions in GDB and
> not as an end in itself.  Presumably the focus should be on having the right
> tests in the suite and not just on ensuring that everything passes.

I will echo everyone else's comments here - in particular Stan's.  Lots
of people run the testsuite, either manually during development or in
automated test harnesses, and in either case there's someone who has to
look into the new failures when they appear.  Leaving things broken is
a problem.

-- 
Daniel Jacobowitz
CodeSourcery, LLC