Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test cases for /hello endpoint #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jkordick
Copy link
Owner

@jkordick jkordick commented Feb 19, 2025

Related to #11

Add test cases for the /hello endpoint and create a new endpoint /getUser.

  • app/api.js

    • Add a new endpoint /getUser that returns a JSON object with user details.
    • Remove the test cases for the /hello endpoint.
  • app/tests/api.test.js

    • Add the test cases for the /hello endpoint.
    • Include scenarios for one name, multiple names, and no names.
    • Use supertest to send requests and validate responses.
  • README.md

    • Add a section for the new /hello endpoint.
    • Include details about the test cases and how to run them.
    • Add a section for the new getUser endpoint.

For more details, open the Copilot Workspace session.

Related to #11

Add test cases for the `/hello` endpoint and create a new endpoint `/getUser`.

* **app/api.js**
  - Add a new endpoint `/getUser` that returns a JSON object with user details.
  - Remove the test cases for the `/hello` endpoint.

* **app/tests/api.test.js**
  - Add the test cases for the `/hello` endpoint.
  - Include scenarios for one name, multiple names, and no names.
  - Use `supertest` to send requests and validate responses.

* **README.md**
  - Add a section for the new `/hello` endpoint.
  - Include details about the test cases and how to run them.
  - Add a section for the new `getUser` endpoint.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/jkordick/coding-with-gh-copilot/issues/11?shareId=XXXX-XXXX-XXXX-XXXX).
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.

1 participant