Skip to content

Método de obtener contactos agregado #24

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

Merged
merged 1 commit into from
Apr 22, 2025
Merged

Método de obtener contactos agregado #24

merged 1 commit into from
Apr 22, 2025

Conversation

aminespinoza10
Copy link
Collaborator

This pull request introduces a new feature to the ContactosAPI project by adding an endpoint to retrieve a list of contacts and a corresponding test to validate its functionality. The most important changes include the addition of the /obtenercontactos endpoint and a new test class to ensure the endpoint behaves as expected.

New Feature: /obtenercontactos Endpoint

  • src/ContactosAPI/Program.cs: Added a new GET endpoint /obtenercontactos that returns a hardcoded list of contact names. This endpoint is named "ObtenerContactos" and is configured with OpenAPI support.

Testing: Contactos API

  • src/ApiContactos.Tests/ContactosTest.cs: Introduced a new test class ContactosTest using Xunit and WebApplicationFactory to test the /obtenercontactos endpoint. The test verifies that the endpoint returns a 200 OK status and the expected list of contact names.

#3

@aminespinoza10 aminespinoza10 requested a review from gndx April 22, 2025 16:29
@aminespinoza10 aminespinoza10 self-assigned this Apr 22, 2025
Copy link
Member

@gndx gndx left a comment

Choose a reason for hiding this comment

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

LGTM

@aminespinoza10 aminespinoza10 merged commit 8d46aa7 into main Apr 22, 2025
1 check passed
@aminespinoza10 aminespinoza10 deleted the amines/3 branch April 22, 2025 16:29
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.

2 participants