refactor: change AppendValues -> Append TestPointValueIterator

InfluxCommunity · Feb 6, 2025 · a6c623e · a6c623e
1 parent 6626433
commit a6c623e
Showing 1 changed file with 22 additions and 22 deletions.
diff --git a/influxdb3/point_value_iterator_test.go b/influxdb3/point_value_iterator_test.go
@@ -60,30 +60,30 @@ func TestPointValueIterator(t *testing.T) {
 	rec := rb.NewRecord() // first record is empty
 	_ = writer.Write(rec)
 
-	rb.Field(0).(*array.Int64Builder).AppendValues([]int64{0}, nil)
-	rb.Field(1).(*array.Uint8Builder).AppendValues([]uint8{1}, nil)
-	rb.Field(2).(*array.Int8Builder).AppendValues([]int8{2}, nil)
-	rb.Field(3).(*array.Uint16Builder).AppendValues([]uint16{3}, nil)
-	rb.Field(4).(*array.Int16Builder).AppendValues([]int16{4}, nil)
-	rb.Field(5).(*array.Uint32Builder).AppendValues([]uint32{5}, nil)
-	rb.Field(6).(*array.BinaryBuilder).AppendStringValues([]string{"6"}, nil)
-	rb.Field(7).(*array.LargeStringBuilder).AppendValues([]string{"7"}, nil)
-	rb.Field(8).(*array.BinaryBuilder).AppendStringValues([]string{"8"}, nil)
+	rb.Field(0).(*array.Int64Builder).Append(0)
+	rb.Field(1).(*array.Uint8Builder).Append(1)
+	rb.Field(2).(*array.Int8Builder).Append(2)
+	rb.Field(3).(*array.Uint16Builder).Append(3)
+	rb.Field(4).(*array.Int16Builder).Append(4)
+	rb.Field(5).(*array.Uint32Builder).Append(5)
+	rb.Field(6).(*array.BinaryBuilder).Append([]byte{6})
+	rb.Field(7).(*array.LargeStringBuilder).Append("7")
+	rb.Field(8).(*array.BinaryBuilder).Append([]byte{8})
 	rb.Field(9).(*array.Date32Builder).Append(arrow.Date32(int32(9)))
 	rb.Field(10).(*array.Date64Builder).Append(arrow.Date64(int64(10)))
 
 	rec = rb.NewRecord()
 	_ = writer.Write(rec)
 
-	rb.Field(0).(*array.Int64Builder).AppendValues([]int64{0}, nil)
-	rb.Field(1).(*array.Uint8Builder).AppendValues([]uint8{1}, nil)
-	rb.Field(2).(*array.Int8Builder).AppendValues([]int8{2}, nil)
-	rb.Field(3).(*array.Uint16Builder).AppendValues([]uint16{3}, nil)
-	rb.Field(4).(*array.Int16Builder).AppendValues([]int16{4}, nil)
-	rb.Field(5).(*array.Uint32Builder).AppendValues([]uint32{5}, nil)
-	rb.Field(6).(*array.BinaryBuilder).AppendStringValues([]string{"6"}, nil)
-	rb.Field(7).(*array.LargeStringBuilder).AppendValues([]string{"7"}, nil)
-	rb.Field(8).(*array.BinaryBuilder).AppendStringValues([]string{"8"}, nil)
+	rb.Field(0).(*array.Int64Builder).Append(0)
+	rb.Field(1).(*array.Uint8Builder).Append(1)
+	rb.Field(2).(*array.Int8Builder).Append(2)
+	rb.Field(3).(*array.Uint16Builder).Append(3)
+	rb.Field(4).(*array.Int16Builder).Append(4)
+	rb.Field(5).(*array.Uint32Builder).Append(5)
+	rb.Field(6).(*array.BinaryBuilder).Append([]byte{6})
+	rb.Field(7).(*array.LargeStringBuilder).Append("7")
+	rb.Field(8).(*array.BinaryBuilder).Append([]byte{8})
 	rb.Field(9).(*array.Date32Builder).Append(arrow.Date32(int32(9)))
 	rb.Field(10).(*array.Date64Builder).Append(arrow.Date64(int64(10)))
 
@@ -151,14 +151,14 @@ func TestPointValueIterator(t *testing.T) {
 	assert.True(t, resultSet5[0] == uint32(5))
 	assert.True(t, resultSet5[1] == uint32(5))
 
-	assert.True(t, resultSet6[0].([]uint8)[0] == uint8(54))
-	assert.True(t, resultSet6[1].([]uint8)[0] == uint8(54))
+	assert.True(t, resultSet6[0].([]uint8)[0] == uint8(6))
+	assert.True(t, resultSet6[1].([]uint8)[0] == uint8(6))
 
 	assert.True(t, resultSet7[0] == "7")
 	assert.True(t, resultSet7[1] == "7")
 
-	assert.True(t, resultSet8[0].([]uint8)[0] == uint8(56))
-	assert.True(t, resultSet8[1].([]uint8)[0] == uint8(56))
+	assert.True(t, resultSet8[0].([]uint8)[0] == uint8(8))
+	assert.True(t, resultSet8[1].([]uint8)[0] == uint8(8))
 
 	assert.True(t, resultSet9[0] == arrow.Date32(int32(9)))
 	assert.True(t, resultSet9[1] == arrow.Date32(int32(9)))