datastar icon indicating copy to clipboard operation
datastar copied to clipboard

Missing sdk tests

Open Superpat opened this issue 11 months ago • 4 comments

The sdk test suite covers a lot of the possible event cases in a data based way, but they dont cover certain error cases for which we will need hand crafted tests.

We need to identify these missing tests and add them.

Two glaring omissions:

1- delays between events Something as simple as waiting a second before sending the second event

2- client aborts Abort the curl process, have it reconnect and check if the output is okay

Any other ideas ?

Superpat avatar Feb 01 '25 17:02 Superpat

The dotnet sdk was not allowing extra headers (neither was my deno sdk originally). We probably need a test that requires a cookie to be present in the response, see https://discord.com/channels/1296224603642925098/1333871441275519077/1336693953252032642

Superpat avatar Feb 05 '25 13:02 Superpat

This is over 2 weeks old. Is there any movement on it?

bencroker avatar Feb 19 '25 18:02 bencroker

I wont be doing this until #604 and #605 are done. But their completion does not preclude somebody else taking up this issue.

Superpat avatar Feb 19 '25 19:02 Superpat

@jmstevers perhaps?

bencroker avatar Feb 19 '25 19:02 bencroker