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.

Sign up for GitHub

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

Jump to bottom

IN - Implementación de test para el plan de indicación #468

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

IN - Implementación de test para el plan de indicación #468

wants to merge 1 commit into from

Conversation

MarianoCampetella
Copy link
Collaborator

@MarianoCampetella MarianoCampetella commented Mar 3, 2023 •
edited
Loading

Requerimiento

Funcionalidad desarrollada

  1. Se crearon nuevos test para el plan de indicacion.
    NOTA:
  • A medida que se mejore el plan de indicaciones es necesario agregar los otros test que hacen falta. Hasta el momento solamente se pudieron crear los dos primeros.
  • En la clase mapa-cama.js solamente se agrego 'internacion:indicaciones:*' en la linea 131. El resto es linteo nomas.

Requiere actualizaciones en la base de datos

  • Si
  • No

@MarianoCampetella MarianoCampetella requested a review from a team as a code owner March 3, 2023 18:42
@MarianoCampetella MarianoCampetella requested review from MCele and negro89 and removed request for a team March 3, 2023 18:42
@silviroa
Copy link
Contributor

silviroa commented Mar 6, 2023

USUARIO: mcampetella
BUILD NUMBER: 8535
CYPRESS RUN: 7714
TEST START: 2023-03-06T11:33:17.682Z
TOTAL: 408
SUCCESS: 404
FAIL: 2
SKIPPED: 2

@silviroa
Copy link
Contributor

silviroa commented Mar 6, 2023

USUARIO: mcampetella
BUILD NUMBER: 8536
CYPRESS RUN: 7715
TEST START: 2023-03-06T15:44:25.460Z
TOTAL: 408
SUCCESS: 404
FAIL: 2
SKIPPED: 2

@MarianoCampetella
Copy link
Collaborator Author

Se van a mejorar los test ya que se agregaron nuevas funcionalidades al plan de indicaciones, las cuales deben ser consideradas.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MCele MCele Awaiting requested review from MCele MCele is a code owner automatically assigned from andes/code-review

@negro89 negro89 Awaiting requested review from negro89 negro89 is a code owner automatically assigned from andes/code-review

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
changes request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MarianoCampetella @silviroa