LGTM with some extra tests.
https://codereview.appspot.com/12748044/diff/1/names/relation.go File names/relation.go (right):
https://codereview.appspot.com/12748044/diff/1/names/relation.go#newcode18 names/relation.go:18: func RelationTag(relationId string) string { if !IsRelation(relationId) { panic(fmt.Sprintf("%q is not a valid relation id", relationId)) }
https://codereview.appspot.com/12748044/diff/1/state/relation.go File state/relation.go (right):
https://codereview.appspot.com/12748044/diff/1/state/relation.go#newcode66 state/relation.go:66: func (r *Relation) Tag() string { This should have a test.
https://codereview.appspot.com/12748044/diff/1/state/state_test.go File state/state_test.go (right):
https://codereview.appspot.com/12748044/diff/1/state/state_test.go#newcode1438 state/state_test.go:1438: tag: "service-ser-vice2", We need a relation tag test here.
https://codereview.appspot.com/12748044/
« Back to merge proposal
LGTM with some extra tests.
https:/ /codereview. appspot. com/12748044/ diff/1/ names/relation. go
File names/relation.go (right):
https:/ /codereview. appspot. com/12748044/ diff/1/ names/relation. go#newcode18 go:18: func RelationTag( relationId string) string { relationId) { fmt.Sprintf( "%q is not a valid relation id", relationId))
names/relation.
if !IsRelation(
panic(
}
https:/ /codereview. appspot. com/12748044/ diff/1/ state/relation. go
File state/relation.go (right):
https:/ /codereview. appspot. com/12748044/ diff/1/ state/relation. go#newcode66 go:66: func (r *Relation) Tag() string {
state/relation.
This should have a test.
https:/ /codereview. appspot. com/12748044/ diff/1/ state/state_ test.go
File state/state_test.go (right):
https:/ /codereview. appspot. com/12748044/ diff/1/ state/state_ test.go# newcode1438 test.go: 1438: tag: "service- ser-vice2" ,
state/state_
We need a relation tag test here.
https:/ /codereview. appspot. com/12748044/