grafana / sqlds

A package that assists writing SQL-driven datasources
Apache License 2.0
17 stars 12 forks source link

test: fix Next() method #120

Closed gabor closed 3 months ago

gabor commented 3 months ago

in the current codebase, in the test-driver, when Next() is called, it goes through every row, and copies the row to dest..

for example, let's say these are the rows:

Next() will first copy north, 23 into dest, then will overwrite it with east, 33, then will overwrite it with south, 41. so when it all ends, the dataframe will look like this: fields: [ [south, south, south], [41, 41, 41] ]

the PR adjusts the code so it only copies the row at the right index.