flux icon indicating copy to clipboard operation
flux copied to clipboard

Ensure we have good coverage of csv and json encoders and decoders.

Open nathanielc opened this issue 6 years ago • 1 comments

Once we have confidence in our coverage of the encoders through the units tests remove the unnecessary use of the encoder/decoders in the end to end tests.

nathanielc avatar Nov 19 '18 17:11 nathanielc

ideas:

  1. Wrap the CSV encoder to take a block of text to diff, then we can possibly run the existing CSV tests to cover the encoder. These could run similarly to how we have it set up now: input.csv -> flux query -> output.csv
  2. Wrap the CSV encoder + decoder together to ensure that production/consumption between the two are precise.

aanthony1243 avatar Dec 17 '18 18:12 aanthony1243