Use zio.Copy instead of manual loop

Co-authored-by: Noah Treuhaft <[email protected]>
brimdata · Nov 3, 2023 · de4ba84 · de4ba84
1 parent 5160fef
commit de4ba84
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/vng/vng_test.go b/vng/vng_test.go
@@ -58,22 +58,14 @@ func FuzzVngRoundtripBytes(f *testing.F) {
 		context := zed.NewContext()
 		reader := zngio.NewReader(context, bytesReader)
 		defer reader.Close()
-		values := make([]zed.Value, 0)
-		for {
-			value, err := reader.Read()
-			if err != nil {
-				return
-			}
-			if value == nil {
-				break
-			}
-			values = append(values, *(value.Copy()))
+		var a zbuf.Array
+		if zio.Copy(&a, reader) != nil {
+			return
 		}
-		writerOpts := vngio.WriterOpts{
+		roundtrip(t, a.Values(), vngio.WriterOpts{
 			ColumnThresh: units.Bytes(vngio.DefaultColumnThresh),
 			SkewThresh:   units.Bytes(vngio.DefaultSkewThresh),
-		}
-		roundtrip(t, values, writerOpts)
+		})
 	})
 }