Skip to content

Traducción de archivo src/controllers/write/groups.js a un archivo Typescript#47

Open
amarantaVC wants to merge 4 commits intoUSB-CI3715:f24from
amarantaVC:f24
Open

Traducción de archivo src/controllers/write/groups.js a un archivo Typescript#47
amarantaVC wants to merge 4 commits intoUSB-CI3715:f24from
amarantaVC:f24

Conversation

@amarantaVC
Copy link

Este Pull Request resuelve el Issues #14 (fixes #14), en el cual se tenía como tarea traducir el archivo src/controllers/write/groups.js a Typescript.

Principales cambios: :

Tipificación de parámetros y retornos:
Se agregaron los tipos de Request y Response de Express en los parametros de todas las funciones.
Se definió el tipo de retorno de las funciones como Promise
Tipificación de variables:
Se añadió el tipo unknown con el fin de que los resultados de las funciones externas que aun no han sido traducidas a typescript no generarán error.
Se especifico el objeto groupObj de la función update como Record<string, unknown>
Comentarios de ESlint:
Se agregaron comentarios a lo largo del codigo como @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access, /* eslint-disable import/no-import-module-exports */ con el fin de evitar errores con el linter.
Cambio en la importación de los modulos:
Se hizo uso de Request y Response desde express para las importaciones para asegurar el manejo correcto con las tipificaciones necesarias.
Cambios del archivo tsconfig.json
Se modificó el archivo tsconfig.json para que el target del archivo trabajara con es2017 en vez de es6, con el fin de permitir el uso de las funciones de async y await.
Pruebas:
El test de coverage arrojó una covertura alta de 83%
Se paso el linter de manera exitosa

Amaranta Villegas
carnet: 16-11247

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11318458800

Details

  • 71 of 71 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.007%) to 82.678%

Totals Coverage Status
Change from base Build 11313630816: 0.007%
Covered Lines: 22344
Relevant Lines: 25603

💛 - Coveralls

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.

Convirtiendo src/controllers/write/groups.js de JS a TS

2 participants