Skip to content

Commit

Permalink
refinar comentário
Browse files Browse the repository at this point in the history
  • Loading branch information
PauloGoncalvesBH committed Apr 27, 2024
1 parent 869e3e4 commit f8baeb0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions tests/functions/tokenIsValid.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ const { tokenIsValid } = require('../../src/authorization')
// Essas validações são de certa forma simples, porém pegam todo o comportamento possível para
// esse método.

// Embora com esse teste é possível ter certeza de como essa função lida com os dados enviados,
// não conseguimos saber se ele barraria de maneira correta as tentativas de acesso e permitiria
// apenas o que encaixa na regra dessa função.
// Embora com esse teste é possível ter certeza de como esse método lida com o token enviado,
// não conseguimos saber se ele barraria as tentativas de acesso com token inválido na função lambda
// e se permitiria apenas o que encaixa na regra desse método.

// É devido a essa incerteza do comportamento de todo o código integrado é que os testes
// no diretório /tests/handler fazem sentido
Expand Down
6 changes: 3 additions & 3 deletions tests/handler/authorization.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@ const LambdaTester = require('lambda-tester')

const { handler } = require('../../src/index')

// Esse teste permite validar que o método 'tokenIsValid' está corretamente
// Esse teste permite validar que o método 'tokenIsValid' está corretamente integrado
// com o código, bem como demonstra que a aplicação sabe lidar com a ausência do token.

// Não seria possível fazer essa mesma automação sem o pacote lambda-tester.

// Mais importante que validar que o método 'tokenIsValid' funciona (via teste unitário no arquivo 'tests/functions/tokenIsValid.spec.js'),
// é entender se ele está bem integrado com a aplicação e que ela vai saber utilizar esse método
// e lidar com qualquer tipo de dado enviado.

// Não seria possível fazer essa mesma automação sem o pacote lambda-tester, necessitando de teste manual.

describe('Authorization', () => {
it('Should return unauthorized access when token is invalid', async function () {
const unauthorizedToken = { token: '123qasfnghwuiiu' }
Expand Down
2 changes: 1 addition & 1 deletion tests/handler/catchError.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ const { handler } = require('../../src/index')
// com o erro sendo enviado para o 'catch' do arquivo src/index.js e retornando uma
// mensagem amigável para o usuário.

// Não seria possível fazer essa mesma automação sem o pacote lambda-tester.
// Não é possível fazer essa validação manualmente, podendo ser feito apenas unitariamente e com uso do pacote lambda-tester.

jest.mock('../../src/authorization')

Expand Down

0 comments on commit f8baeb0

Please sign in to comment.