otaviolemos / thewisedev-mailing Goto Github PK

@otaviolemos, parabéns pelo código. Estou a dias o estudando e aprendendo bastante. Obrigado por compartilhar.

Eu gostaria de entender porque você persiste um UserData ao invés do User. É alguma limitação do mongo? Eu entendi sua entidade User como uma classe validadora. É isto mesmo? Poderia me explicar por favor?

Abraços.

Ao criar o teste abaixo, ele não passa, pois espaços em branco não são removidos do nome do usuário:

test('should not create user with invalid name (only blank spaces)', async () => {
    const name = '   '
    const user = User.create({ name: name, email: '[email protected]' })
    expect(user).toEqual(left(new InvalidNameError(name)))
  })

Alterar o método validate da classe Name, como segue:

static validate (name: string): boolean {
    if (!name || name.trim().length < 2 || name.trim().length > 255) {
      return false
    }
    return true
  }

Para manter a padronização com as demais classes (Name e Email) e aprimorar o encapsulamento, os atributos da classe User poderiam ser privados (private). Fiz um pull request apresentando a maneira como isso poderia ser implementado, mas você pode rejeitá-lo, se quiser. É só uma dica mesmo ;)

O projeto poderia vir com um script docker-compose.yml que subiria um mongodb+mongo express (web ui para o mongo) e um servidor smtp simulado que rodaria local (caso a API necessite de um serviço de email), e claro, no up do docker-compose poderia rodar o projeto node / a API backend em si também (dai a pessoa consegue sem ter node instalado na máquina, dependeria apenas de ter o engine do docker mesmo).

Se nao puder fazer posso enviar um PR pra ti com essa melhoria ;-)

PS: Resolvi contribuir porque devo usar seu projeto como base para uns projetos meus, e sendo assim não vejo porque não contribuir de alguma forma para melhorá-lo nem que seja só um pouco.