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 expired assertion in Get function, refactor Upsert and fix tests #13

Merged
merged 4 commits into from
May 12, 2021

Conversation

eze8789
Copy link
Member

@eze8789 eze8789 commented May 9, 2021

Descripcion

El principal objetivo es lograr realizar un assertion de la expiración de la key antes de devolver la información al usuario.
Por otro lado se encontró algunos posibles refactor al Upsert method y fix sobre distintos tests que no estaban pasando.

Issue Relacionada

https://github.com/gophers-latam/GoKey/projects/1#card-60712576

Motivacion y Contexto

Insertar una comprobacion de expiracion antes de efectuar un get

Como fue probado

make test provisto en el proyecto

Screenshots / capturas de pantalla (si es necesario)

Tipo de cambio

  • Bug fix (non-breaking cambios que fixean una issue)
  • Nueva feature / funcionalidad (non-breaking cambio que agrega una nueva funcionalidad)
  • Breaking change (fix o feature que va a causar un cambio en una funcionalidad existente)

Checklist

  • Estas haciendo el pull request desde un *topic/feature/bugfix branch (lado derecho). Si estas haciendo un pull request desde un fork, no lo hagas desde master!.
  • Estas haciendo el pull request contra master (lado izquierdo). Tambien de que estas usando los ultimos cambios en master.
  • Mis cambios necesitan cambio de la documentacion.
  • Actualize la documentacion acordemente.
  • Modules and dependencias fueron actualizadas acordemente; correr go mod tidy && go mod vendor
  • Agregue tests para cubrir mis cambios.
  • Todos los tests existentes pasaron.
  • Checkear que el codigo que estoy subiendo esta linteado:
    • go fmt -s
    • go vet

Copy link
Member

@tomiok tomiok left a comment

Choose a reason for hiding this comment

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

normalizar los mensajes de errores a uno solo en el GET

@tomiok tomiok merged commit ff92851 into main May 12, 2021
@tomiok tomiok deleted the feat/check-key-expire branch May 12, 2021 15:38
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.

3 participants