Added TODO for testing more invariants.

panmari · panmari · commit b9e4c6d9ee57 · 2025-07-01T21:32:05.000+02:00
diff --git a/s2/encode_fuzz_test.go b/s2/encode_fuzz_test.go
@@ -8,13 +8,13 @@ import (
 )
 
 func FuzzDecodeCellUnion(f *testing.F) {
-	cuCells := CellUnion([]CellID{
+	cu := CellUnion([]CellID{
 		CellID(0x33),
 		CellID(0x8e3748fab),
 		CellID(0x91230abcdef83427),
 	})
 	buf := new(bytes.Buffer)
-	if err := cuCells.Encode(buf); err != nil {
+	if err := cu.Encode(buf); err != nil {
 		f.Errorf("error encoding %v: ", err)
 	}
 	f.Add(buf.Bytes())
@@ -28,5 +28,6 @@ func FuzzDecodeCellUnion(f *testing.F) {
 		if got := c.ApproxArea(); got < 0 {
 			t.Errorf("ApproxArea() = %v, want >= 0. CellUnion: %v", got, c)
 		}
+		// TODO: Test more invariants that should hold for all CellUnion.
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -8,13 +8,13 @@ import (`
`8`	`8`	`)`
`9`	`9`
`10`	`10`	`func FuzzDecodeCellUnion(f *testing.F) {`
`11`		`- cuCells := CellUnion([]CellID{`
	`11`	`+ cu := CellUnion([]CellID{`
`12`	`12`	`CellID(0x33),`
`13`	`13`	`CellID(0x8e3748fab),`
`14`	`14`	`CellID(0x91230abcdef83427),`
`15`	`15`	`})`
`16`	`16`	`buf := new(bytes.Buffer)`
`17`		`- if err := cuCells.Encode(buf); err != nil {`
	`17`	`+ if err := cu.Encode(buf); err != nil {`
`18`	`18`	`f.Errorf("error encoding %v: ", err)`
`19`	`19`	`}`
`20`	`20`	`f.Add(buf.Bytes())`
`@@ -28,5 +28,6 @@ func FuzzDecodeCellUnion(f *testing.F) {`
`28`	`28`	`if got := c.ApproxArea(); got < 0 {`
`29`	`29`	`t.Errorf("ApproxArea() = %v, want >= 0. CellUnion: %v", got, c)`
`30`	`30`	`}`
	`31`	`+ // TODO: Test more invariants that should hold for all CellUnion.`
`31`	`32`	`})`
`32`	`33`	`}`