code review 4368063: gofmt: added test framework in Go

gofmt: add test framework in Go

- replaced existing testdata/test.sh with new gofmt_test
- added initial test case for rewrite tests

TODO: Need to add more tests.

[gri, 2011-04-10 00:18:49]

Hello rsc (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[rsc, 2011-04-13 13:34:03]

Exec'ing gofmt is probably not the right approach except
for testing command-line processing.  If you refactor main
to provide a suitable function and then call that function
from the test loop, the test will be less code and run much
faster.

[gri, 2011-04-13 20:45:39]

PTAL
- gri

On Wed, Apr 13, 2011 at 6:34 AM,  <rsc@golang.org> wrote:
> Exec'ing gofmt is probably not the right approach except
> for testing command-line processing.  If you refactor main
> to provide a suitable function and then call that function
> from the test loop, the test will be less code and run much
> faster.
>
>
>
> http://codereview.appspot.com/4368063/
>

[rsc, 2011-04-13 20:54:20]

LGTM

[rsc, 2011-04-13 20:54:48]

if you get this before the submit, s/added/add/ in CL description

[gri, 2011-04-13 21:00:03]

*** Submitted as http://code.google.com/p/go/source/detail?r=b56f308973d8 ***

gofmt: add test framework in Go

- replaced existing testdata/test.sh with new gofmt_test
- added initial test case for rewrite tests

TODO: Need to add more tests.

R=rsc
CC=golang-dev
http://codereview.appspot.com/4368063