---
# System prepended metadata

title: Mejoras en acciones

---

# Mejoras en acciones
## Jhonatan
### GENERAL
  - En los esquemas, idSchema está mal, ya que no se valida que sea uuid

### PRIORITIZE_SUBTASKS
- schemas
  - no se válida que parent.priority sea un uuid
  - no se válida que parent.subtask sea un uuid
  - no se testean todos los parámetros del payload
- docs
- errors
- dbChanges
  - los tests no están hechos en función de los ultimos estándares
- confirmAction
  - no tiene case independiente, usa C_SUBTASK

### EDIT_TITLE_SUBTASK
- schemas
  - no se testean todos los parámetros del payload
- docs
  - no se válida que la subtarea exista antes de buscar la información de la subtarea padre y de la prioridad
  - los tests no están hechos en función de los ultimos estándares
  - no se testean todas las branches
- dbChanges
  - los tests no están hechos en función de los ultimos estándares
- confirmAction
  - no tiene case independiente, usa C_SUBTASK

### ASSIGN_DURATION_SUBTASK
- schemas
  - no se testean todos los parámetros del payload
- docs
  - no se válida que la subtarea exista antes de buscar la información de las subtareas y de la prioridad
  - los tests no están hechos en función de los ultimos estándares
- errors
  - no se testean todas las branches
- confirmAction
  - no tiene case independiente, usa C_SUBTASK

### DELETE_SUBTASK
- docs
  - no se válida que la subtarea exista antes de buscar la información de la subtarea padre y de la prioridad
  - los tests no están hechos en función de los ultimos estándares
  - no se testean todas las branches
- errors
  - no se testean todas las branches
- dbChanges
  - los tests no están hechos en función de los ultimos estándares

### COMPLETE_SUBTASK
- schemas
  - se prueban pocos casos
- docs
  - no se válida que la subtarea exista antes de buscar la información de la subtarea padre y de la prioridad
  - los tests no están hechos en función de los ultimos estándares
  - no se testean todas las branches
- dbChanges
  - los tests no están hechos en función de los ultimos estándares
### UNDO_COMPLETE_SUBTASK
- docs
  - no se válida que la subtarea exista antes de buscar la información de la subtarea padre y de la prioridad
  - los tests no están hechos en función de los ultimos estándares
  - no se testean todas las branches
- dbchanges
  - los tests no están hechos en función de los ultimos estándares
  - no se testean todas las branches
- confirmAction
  - no tiene case independiente, usa C_SUBTASK
### DELETE_DRAFT_REQUEST
- docs
  - no se válida que el pedido exista para buscar la info de los receptores y del proyecto
### SEEN_REQUEST_FIELD_OWNER
- docs
  - no se válida que exista el request antes de buscar la información del emisor y el receptor
  - los tests no están hechos en función de los ultimos estándares
### DELETE_DRAFT_PRIORITY
- docs
  - no se válida que exista el usuario y el pedido para buscar información a partir de ellos
  - no se testan todas las branches
- errors
  - no se válida que sea una prioridad compartida
  - no se válida que el usuario sea el creador
  - no se válida que no sea un compromiso
- dbchanges
  - no se testean todas las branches
### EXTEND_DATE_LIMIT
- docs
  - no se válida que el pedido exista para buscar información a partir de él
- dbchanges
  - los tests no cumplen con los estándares actuales
### ACCEPT_COMMITMENT
- docs
  - no se válida que exista el compromiso antes de usar la información que se busca
- dbchanges
  - los tests están muy mal hechos, no cumplen con los estándares
### REJECT_COMMITMENT
- docs
  - no se válida que exista el compromiso para poder buscar información de él.
### ACCEPT_CORRECTION
- docs
  - no se válida que exista el compromiso en la BD antes de buscar información relacionada a él
- dbchanges
  - los tests no cumplen con los estándares actuales
### UPDATE_PROFILE
- schema
  - no tiene definido un número máximo de caracteres
### SEND_PLANIFICATION
- docs
  - no se testean todas las branches
  - lo tests no cumplen con los estándares actuales
- dbchanges
  - no se testean todas las branches
  - lo tests no cumplen con los estándares actuales
- confirmation
  - no se testean todas las branches
### UPDATE_COMMITMENT_DATE_LIMIT
- docs
  - no se válida que exista el compromiso antes de buscar información a partir de él
  - no se cumple con el estándar de usar Promise.all para las queries
- dbchanges
  - los tests no cumplen con los estándares actuales
  - no se testean todas las branches
### CREATE_USER
- docs
  - no se válida que el líder exista antes de buscar información a partir de el
  - los tests no cumplen con los estándares actuales
  - no se testean todas las branches
  - se busca información del leader2 sin necesidad
- dbchanges
  - los tests no cumplen con los estándares actuales
### SEEN_TUTORIAL_VIDEO
- schemas
  - se deberia agregar una validación para que sean solo posibles los ids que están definidos en tutorialsVideosList.json de la app
### SEND_RESPONSE_TO_REQUEST_HAP
- schema
  - response no tiene limite de caracteres
### SEND_CLARIFICATION_MEETING_PROPOSAL
- schema
  - los tests no utiliza la acción generadora
- dbchanges
  - los tests no cumplen con los ultimos estándares
### CANCEL_CLARIFICATION_MEETING_EVENT
- docs
  - no se testean todas las branches
- dbchanges
  - no se testean todas las branches
  - los tests no cumplen con los estándares actuales
### UPDATE_GOAL
- dbchanges
  - no se testean todas las ramas
### SEND_REQUEST
- docs
  - no se cumplen con los ultimos estándares
- dbchanges
  - no se cumplen con los ultimos estándares
  - no se testean todas las branches
### TO_DRAFT_REQUEST
- schema
  - no se testean varios parámetros del payload
- dbchanges
  - no se testean todas las branches
  - los tests no cumplen con los ultimos estandares
- derived
  - no se testean todas las branches
- confirmation
  - no se testean todas las branches
### SEND_DRAFT_REQUEST
- dbchanges
  - los tests no cumplen con los estándares actuales
  - no se testean todas las branches
- derived
  - no se testean todas las branches
### ACCEPT_REQUEST
- docs
  - los tests no cumplen con los estándares actuales
- errors
  - no se testean todas las branches
- dbchanges
  - los tests no cumplen con los estándares actuales
  - no se testean todas las branches
- derived
  - no se testean todas las branches
### UPDATE_REQUEST
- docs
  - No cumple con los estándares actuales
- errors
  - no se testean todas las branches
- dbchanges
  - No cumple con los estándares actuales
  - no se testean todas las branches
- derived
  - no se testean todas las branches
### TO_DRAFT_REQUEST_SENT
- errors
  - no se testean todas las branches
### ACCEPT_PLANIFICATION_COMMITMENT
- errors
  - no se testean todas las branches
- dbchanges
  - los tests no cumplen con los estandares actuales
- confirmation
  - no se testean todas las branches
### SEND_REQUEST_FROM_PLANIFICATION
- docs
  - los tests no cumplen con los estándares actuales
- errors
  - no se testean todas las branches
- dbchanges
  - los tests no cumplen con los estándares actuales
- confirmation
  - no se testean todas las branches
### DELETE_USER
- docs
  - no se testean todas las branches
  - los tests no cumplen con los estándares actuales
- error
- derived
- confirmation
  - no tiene tests
### CONFIRM_CLARIFICATION_MEETING_PROPOSAL
- docs
  - no se válida que exista la reunión antes de buscar información a partir de ella
  - los tests no cumplen con los estándares actuales
- dbchanges
  - no se testean todas las branches
  - los tests no cumplen con los estándares actuales



## Víctor
### COMPLETE_TASK
- dbchanges
  - no se testean todas las branches
  - no cumple con los ultimos estándares
- docs
  - no cumple con los ultimos estándares
### ASSIGN_DURATION_PRIORITY
- confirmation
  - no tiene case independiente, usa C_TASK
- dbchanges
  - no cumple con los ultimos estándares
- docs
  - no cumple con los ultimos estándares
### EDIT_TITLE_TASK
- confirmation
  - no tiene case independiente, usa C_TASK
- dbchanges
  - no cumple con los ultimos estándares
- docs
  - no cumple con los ultimos estándares
### SET_LINK_TASK
- confirmation
  - no tiene case independiente, usa C_TASK
- dbchanges
  - no cumple con los ultimos estándares
- docs
  - no cumple con los ultimos estándares
### ADD_SUBTASK
- confirmation
  - no tiene case independiente, usa C_SUBTASK
- docs
  - no cumple con los ultimos estándares
### SET_LINK_SUBTASK
- confirmation
  - no tiene case independiente, usa C_SUBTASK
- dbchanges
  - no cumple con los ultimos estándares
- docs
  - no cumple con los ultimos estándares
  - no se testean todas las branches
### ASK_CLARIFICATION_REQUEST
- confirmation
  - no tiene case independiente, usa C_REQUEST_IO
- docs
  - no cumple con los ultimos estándares
### VIEW_RESOURCE
- docs
  - no cumple con los ultimos estándares
### UPDATE_DRAFT_REQUEST
- dbchanges
  - no se testean todas las branches
- errors
  - se utiliza toBeTruthy y toBeFalsy
- derived
  - hay valores que se pueden definir en una constante y reuzarlos ya que se setean varias veces
### APPROVE_NEW_DATE_LIMIT
- dbchanges
  - no se testean todas las lineas
- docs
  - no se testean todas las lineas
### REJECT_REQUEST
- dbchanges
  - no se testean todas las branches
### UPDATE_REQUEST_DATE_LIMIT
- dbchanges
  - no se testean todas las branches
- docs
  - no cumple con los ultimos estándares
- errors
  - no se testean todas las funciones
### CANCEL_COMMITMENT
- dbchanges
  - no se testean todas las branches
- docs
  - no se testean todas las funciones
- schema
  - no tiene test creado
### SEEN_COMMITMENT
- dbchanges
  - no cumple con los ultimos estándares
- errors
  - no tiene test creado
### SEND_COMMITMENT
- errors
  - no se testean todas las branches
  - se usa toBeTruthy
### UNDO_SEND_COMMITMENT
- derived
  - no se testean todas las branches
### GET_PROJECT_DETAILS
- errors
  - no se testean todas las branches
### GET_ACTIVE_COMMITMENTS_AND_REQUESTS_FROM_REPORT
- docs
  - no se testean todas las branches
  - no se cumple con los ultimos estándares
### GET_DIRECT_REPORTS_DRAFTS
- docs
  - no se testean todas las branches
### GET_GOAL_CYCLE_DETAILS
- docs
  - no se testean todas las funciones
  - no se cumple con los ultimos estándares
### UPDATE_PROJECT_SECTION
- dbchanges
  - no cumple con los ultimos estándares
### UPDATE_PROJECT_SECTION
- derived
  - no tiene CASE en getDerivedAction
### NEXT_DAY
- derived
  - no tiene CASE en getDerivedAction
- errors
  - no tiene CASE en getActionError
- dbchanges
  - no se testean todas las branches
- docs
  - no se cumple con los ultimos estándares
### COMPLETE_EVENT
- dbchanges
  - no cumple con los ultimos estándares
- docs
  - no tiene test
### CONFIRM_ATTENDANCE
- derived
  - no tiene test
- docs
  - no tiene test
- errors
  - no tiene test
- dbchanges
  - no cumple con los ultimos estándares
### GET_RELATED_DELIVERIES
- errors
  - no tiene CASE en getActionError
  - no tiene test
- derived
  - no se tesean todas las funciones
### GET_APPROVED_REQUESTS
- errors
  - no tiene CASE en getActionError
  - no tiene test
### SET_DATE_LIMIT_TASK
- dbchanges
  - no cumple con los ultimos estándares
  - no se testean todas las branches
- docs
  - no cumple con los ultimos estándares
### CREATE_GOAL
- dbchanges
  - no cumple con los ultimos estándares
  - no se testean todas las branches
- docs
  - no cumple con los ultimos estándares
### DELETE_GOAL
- docs
  - no se testean todas las funciones
### REACTIVATE_GOAL
- dbchanges
  - no cumple con los ultimos estándares
  - no se testean todas las branches
- docs
  - no cumple con los ultimos estándares
  - no se testean todas las branches
### APPROVE_EXTERNAL_COMMITMENT
- dbchanges
  - no cumple con los ultimos estándares
  - no se testean todas las branches
- docs
  - no cumple con los ultimos estándares
### CREATE_EXTERNAL_COMMITMENT
- dbchanges
  - no se testean todas las branches
- docs
  - no se testean todas las branches
- errors
  - hace uso de toBeTruthy
### CREATE_PROJECT
- docs
  - no se testean todas las funciones
### DELETE_ATTACHED_FILE
- dbchanges
  - no tiene test
- derived
  - no tiene test
### DELETE_ATTACHED_FILE
- dbchanges
  - no tiene test
- derived
  - no tiene test
- schema
  - no tiene test
### DELETE_CALENDAR_EVENT
- docs
  - no cumple con los ultimos estándares
### GET_MONTHLY_INDICATORS_AVG
- docs
  - no cumple con los ultimos estándares
  - no se testean todas las branches
### GOOGLE_LOGOUT
- dbchanges
  - no se testean todos los statements
- docs
  - no cumple con los ultimos estándares
### OUTLOOK_LOGOUT
- dbchanges
  - no se testean todos los statements
- docs
  - no cumple con los ultimos estándares
### REGISTER_COMMITMENT
- confirmation
  - se agrego el test
### SET_PROJECTED_TIME
- confirmAction
  - no tiene case independiente, usa C_SUBTASK
- dbchanges
  - no cumple con los ultimos estándares
- errors
  - no se testean todos los statements
