openai icon indicating copy to clipboard operation
openai copied to clipboard

Published 20 hours ago • verified publisher icon load1n9

Add tests

Open alexewerlof opened this issue 2 years ago • 3 comments

Deno has native testing support. Could we add some tests with mock data just to ensure that the parsing and encoding/decoding functionality works as expected?

alexewerlof avatar Oct 20 '23 10:10 alexewerlof

Tests would definitely be nice, especially around my SSE decoding.

lino-levan avatar Oct 20 '23 15:10 lino-levan

Yeah sounds good, a big thing for testing is testing is costly so keeping it to a minimum while testing everything would be nice

load1n9 avatar Oct 23 '23 20:10 load1n9

To reduce duplication of effort, and diverging from whatever convention and preference you may have in mind, I'd prefer if you guys add the basic tests and I'll add to it with the PRs that I'll be making. 🙏

alexewerlof avatar Oct 26 '23 13:10 alexewerlof