Code review comment for lp:~thumper/juju-core/user-display-name

Revision history for this message
Jesse Meek (waigani) wrote :

https://codereview.appspot.com/102970043/diff/1/state/user_test.go
File state/user_test.go (right):

https://codereview.appspot.com/102970043/diff/1/state/user_test.go#newcode75
state/user_test.go:75: user := s.makeUser(c)
It seems you are making a user for every test. Why not makeUser in
SetUpTest?

https://codereview.appspot.com/102970043/

« Back to merge proposal