Please take a look.
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 { On 2013/08/14 15:02:49, rog wrote: > if !IsRelation(relationId) { > panic(fmt.Sprintf("%q is not a valid relation id", relationId)) > }
Done.
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 { On 2013/08/14 15:02:49, rog wrote: > 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", On 2013/08/14 15:02:49, rog wrote: > We need a relation tag test here.
https://codereview.appspot.com/12748044/
« Back to merge proposal
Please take a look.
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) { Sprintf( "%q is not a valid relation id", relationId))
names/relation.
On 2013/08/14 15:02:49, rog wrote:
> if !IsRelation(
> panic(fmt.
> }
Done.
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.
On 2013/08/14 15:02:49, rog wrote:
> This should have a test.
Done.
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_
On 2013/08/14 15:02:49, rog wrote:
> We need a relation tag test here.
Done.
https:/ /codereview. appspot. com/12748044/