From f8baeb05f8db6e0e84e8855bb64d5868cfc4461f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Paulo=20Gon=C3=A7alves?= <29241659+PauloGoncalvesBH@users.noreply.github.com> Date: Sat, 27 Apr 2024 15:26:29 +0100 Subject: [PATCH] =?UTF-8?q?refinar=20coment=C3=A1rio?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- tests/functions/tokenIsValid.spec.js | 6 +++--- tests/handler/authorization.spec.js | 6 +++--- tests/handler/catchError.spec.js | 2 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/functions/tokenIsValid.spec.js b/tests/functions/tokenIsValid.spec.js index bd9ba8b..2ed210d 100644 --- a/tests/functions/tokenIsValid.spec.js +++ b/tests/functions/tokenIsValid.spec.js @@ -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 diff --git a/tests/handler/authorization.spec.js b/tests/handler/authorization.spec.js index 9c870e7..1b07839 100644 --- a/tests/handler/authorization.spec.js +++ b/tests/handler/authorization.spec.js @@ -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' } diff --git a/tests/handler/catchError.spec.js b/tests/handler/catchError.spec.js index c7c94e0..e39df76 100644 --- a/tests/handler/catchError.spec.js +++ b/tests/handler/catchError.spec.js @@ -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')