Re: [PATCH] Make display_lines() spawn diff

Stefan Sperling wrote on Sat, Jul 24, 2010 at 15:55:19 +0200:
> On Sat, Jul 24, 2010 at 06:46:11PM +0530, Ramkumar Ramachandra wrote:
> > Hi,
> >
> > This patch is not intended for inclusion, but I hope it conveys the
> > idea- I want display_lines() to display a diff instead of the full
> > actual and expected outputs.
>
> That would be nice indeed, but ...
>

+1

> > + exit_code, stdout_output, stderr_output = \
> > + svntest.main.run_command("diff", 1, 0, '-a', '-u', \
> > + actual_file.name, expected_file.name)
> > + print "".join(stdout_output)
>
> ... this won't work on windows.
>
> We'll need a utility function in our test suite that prints diffs.
>

The test suite could use tools/diff/diff?

> > def compare_and_display_lines(message, label, expected, actual,
> > raisable=None):
Received on 2010-07-24 15:58:51 CEST