awslabs / llrt

LLRT (Low Latency Runtime) is an experimental, lightweight JavaScript runtime designed to address the growing demand for fast and efficient Serverless applications.
Apache License 2.0
7.76k stars 342 forks source link

chore: use expect(..) instead of assert(..) #287

Closed fredbonin closed 3 months ago

fredbonin commented 3 months ago

Issue #240

Description of changes

Used expect instead of assert for all unit tests

Checklist

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

fredbonin commented 3 months ago

Fantastic! Can we now remove deps on assert and update docs? Readme references assert module

Definitely, but with all the changes I think it would be better to do it in a second PR. What do you think ?