Skip to content

Conversation

@denniwang
Copy link
Contributor

@denniwang denniwang commented Oct 11, 2025

todo: make create-test cli too

ℹ️ Issue

Closes #36

📝 Description

  • Implemented jest for unit testing lambdas functions

Briefly list the changes made to the code:

  1. Created jest dependency
  2. Made an example test for the /health default endpoint GET request

✔️ Verification

What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.

Provide screenshots of any new components, styling changes, or pages.

🏕️ (Optional) Future Work / Notes

  • Creating tests through cli

todo: make create-test cli too
@denniwang denniwang marked this pull request as ready for review October 14, 2025 18:19
nourshoreibah
nourshoreibah previously approved these changes Oct 15, 2025
Copy link
Collaborator

@nourshoreibah nourshoreibah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple very minor comments mostlyy for PR review guide. Awesome work yall!!!

Comment on lines 143 to 145
return `
test("test 🌞", async () => {
let res = await fetch("http://localhost:3000/${handlerName}/health")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep the emoji but maybe call it health test too so we know lol

Co-authored-by: Nour Shoreibah <168875317+nourshoreibah@users.noreply.github.com>
Copy link
Contributor

@Rayna-Yu Rayna-Yu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@nourshoreibah nourshoreibah merged commit 4dbf858 into main Oct 19, 2025
4 checks passed
@nourshoreibah nourshoreibah deleted the #36-lambda-unit-tests branch October 19, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Develop a Plan for Unit Testing Lambda Functions and Implement It

3 participants