add {Bun,Node}HttpServer.layerTest for testing http servers
#3409
Conversation
🦋 Changeset detectedLatest commit: 9917b61 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 packages
Not sure what this means? Click here to learn what changesets are. Click here if you"re a maintainer who wants to add another changeset to this PR |
|
@tim-smart could you please take a look and give me some feedback? I"d like to know what you think about this in general so I can decide whether to move forward with it. |
| @@ -0,0 +1,24 @@ | |||
| /** | |||
| * @since 0.41.3 | |||
There was a problem hiding this comment.
If you keep all the since tags at 1.0.0, as we are not yet api stable.
There was a problem hiding this comment.
I"ll change it. I noticed Giulio uses precise @since versions in the /schema and I think it"s beneficial for the consumers of the lib.
|
Thanks for the feedback. I removed the in-memory impl from this PR and adjusted. |
tim-smart
changed the title
{Bun,Node}HttpServer.layerTest for testing http servers
POC of http testing modules, it is introducing three new modules
NodeHttpTestandBunHttpTest- exposing layers enabling live testing of http apps, I have similar implementation in theeffect-httppackage, and one implementation was already used in the test suite for /platform-node so this is only about exposing this functionality for convenient testing in the user-landHttpTestexposing layers for "in-memory" testing of http apps, it mimics the client <> server comm using a queue, there"s an implementation of request / response classes enabling conversionsclient request -> server requestandserver response -> client responseand finally layers with the implementation ofHttpClientandHttpServerthat exchange requests / responses through the queue and map the requests / responses.UPDATE: the in-memory implementation was removed and this PR now introduces only the live testing layers.