code review 6446099: leveldb: rewrite memfs_test's TestBasics as a table-dri...

leveldb: rewrite memfs_test's TestBasics as a table-driven test.

[nigeltao, 2012-08-08 05:22:59]

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

I'd like you to review this change to
https://code.google.com/p/leveldb-go

[dsymonds, 2012-08-08 05:30:22]

LGTM

http://codereview.appspot.com/6446099/diff/4001/leveldb/memfs/memfs_test.go
File leveldb/memfs/memfs_test.go (right):

http://codereview.appspot.com/6446099/diff/4001/leveldb/memfs/memfs_test.go#n...
leveldb/memfs/memfs_test.go:29: "1a: create /foo",
the "1a", "2a", etc. prefixes probably don't matter so much any more. If there's
so much going on here that it would help, it would probably be even more useful
to t.Log the whole name every time.

[nigeltao, 2012-08-09 03:46:38]

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

leveldb: rewrite memfs_test's TestBasics as a table-driven test.

R=dsymonds
CC=golang-dev
http://codereview.appspot.com/6446099