Usuario busca perfis por nome #20
Conversation
- apenas usuários logados podem pesquisar perfis na plataforma Co-authored-by: Eliseu Ramos <[email protected]>
- inclui form de busca de perfis na header da aplicação - oculta form de busca para users não autenticados - adicionado Warden::Test::Helpers para usar login_as nos testes Co-authored-by: Eliseu Ramos <[email protected]> Co-authored-by: Caique Arruda <[email protected]>
- cria método da classe user para pesquisa por nome
- adiciona i18n na aplicação
Co-authored-by: Eliseu Ramos <[email protected]>
Co-authored-by: Caique Arruda <[email protected]>
|
|
- usuário faz login para acessar o form de busca pelo nome
- cria seeds para criação de users
Co-authored-by: Eliseu Ramos <[email protected]>
DanSmaR
changed the title
…11-portfoliorrr into feat/usuario-busca-usuarios
…at/usuario-busca-usuarios Co-authored-by: Rodrigo Gyodai <[email protected]>
|
Fiz uma revisão, as o PR está marcado como draft. Quando quiserem pedir uma revisão, mudem para open, ok? |
|
Reparei que este PR está marcado com 4 assignees. é isso mesmo? |
- Corrigido testes de acordo com sugestões do @akaninja Co-authored-by: Rodrigo Gyodai <[email protected]>
Co-authored-by: Rodrigo Gyodai <[email protected]>
Eh que o GyodaiDDA estava resolvendo os problemas pessoais dele e os outros dois me ajudaram a fazer essa issue. |
Boa! Arrumei lá |
There was a problem hiding this comment.
Parece ok, vou deixar aprovado com alguns comentários para vocês avaliarem.
Um comentário geral: normalmente o primeiro teste de um arquivo é interessante que seja um de sucesso, o caminho feliz mais comum. Desse jeito qualquer outra pessoa do time, conseque abrir o arquivo e logo saber o fluxo principal da funcionalidade, como ela deve funcionar de forma completa. E na sequência desse teste de sucesso, vem as variações, que seriam os testes de falha e variações de sucesso. Isso vai ajudar na hora de entender e revisar o código. Provavelmente vai ajudar vocês na hora da implementação também. Não costumamos fazer as implementações com TDD começando pelas falhas, mas sim pelo caminho de sucesso. :)
só pode ver página de resultados se estiver logado
Como já tínhamos o expect(result.all.count).to eq 0 removi expect(result.first).to eq nil
@akaninja e @gabdemiranda
resolve #10