Skip to content
This repository has been archived by the owner on Dec 2, 2023. It is now read-only.

Update user #7

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Update user #7

wants to merge 5 commits into from

Conversation

francinehahn
Copy link
Contributor

Implementação do endpoint de finalização de cadastro do player

O que foi feito

  • Endpoint PATCH de finalização de cadastro
  • Testes unitários da camada controller
  • Testes unitários da camada service
  • Testes e2e

@@ -113,3 +118,194 @@ describe('PlayerController (e2e)', () => {
}
});
});

Choose a reason for hiding this comment

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

Separar este teste em um arquivo separado, devido a quantia de testes. Teremos então 2 arquivos:

  • create-player.e2e.spec.ts (este arquivo)
  • finish-player-registration.e2e.spec.ts (arquivo contendo esta parte)

}
})


describe('Finish player registration', () => {

Choose a reason for hiding this comment

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

Separar este teste em um arquivo separado, devido a quantia de testes. Teremos então 2 arquivos:

player.controller.spec.ts (este arquivo)
finish-player.controller.spec.ts (arquivo contendo esta parte)

@@ -66,4 +71,36 @@ describe('PlayerService', () => {
expect(token).toBe("validToken");
});
});

describe('finishPlayerRegistration', () => {

Choose a reason for hiding this comment

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

brunomestanza2 minutes ago
Separar este teste em um arquivo separado, devido a quantia de testes. Teremos então 2 arquivos:

player.service.spec.ts (este arquivo)
finish-player.service.spec.ts (arquivo contendo esta parte)

Copy link
Contributor

@CarolineMartins09 CarolineMartins09 left a comment

Choose a reason for hiding this comment

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

Mudanças realizada como foi observado pelo Bruno, aplicação rodando tranquilamente com testes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants