Skip to content

Develop#5

Open
manoelsrs wants to merge 73 commits intojjfernandes87:masterfrom
manoelsrs:develop
Open

Develop#5
manoelsrs wants to merge 73 commits intojjfernandes87:masterfrom
manoelsrs:develop

Conversation

@manoelsrs
Copy link
Copy Markdown

XP Marvel Challenge

Boa tarde pessoal, tudo bom??

Segue o PR do teste solicitado por vocês para a vaga de android nativo.

O teste foi iniciado no domingo passado, durante a semana, na maioria das vezes, eu progredi nele sempre após o horário comercial, quando eu chegava em casa..

Sobre o Marvel Challenge app

-> O teste pode ser dividido em três partes, splash, home com fragments de personagens e favoritos e tela de detalhes do personagem, com a opção de clicar na estrela (que fica em cima, ao lado direito da tela) para favoritar ou deletar um personagem.

-> Sobre a arquitetura, a parte de home eu fiz usando viewmodel e android paging library e a parte de detalhes do personagem eu fiz no MVP que eu estou acostumado. Ambas as listas possuem paginação.

-> As keys para request na api da Marvel eu optei por deixá-las no gradle.properties.

-> Foram feitos testes usando mockito, junit e robolectric.

Como meu atual emprego está me consumindo demais, na minha opinião daria para melhorar mais este teste e por exemplo, para entrar em produção eu colocaria muito mais testes unitários e faria teste de interface com Espresso (que não foi feito nesse aqui).

Acredito que da forma que está, é possível analisar bem os meus conhecimentos.

Esse final de semana eu devo trabalhar no meu projeto atual, para não prejudicar minha sprint nem meus companheiros de equipe. Caso tenha um tempo a mais, criarei uma nova branch aqui para melhorias e mando para vocês na semana que vem.

Qualquer dúvida, sugestão ou o que for preciso, contem comigo.

Grande abraço!
Manoel Soares Neto

Manoel Soares Neto and others added 30 commits September 13, 2019 16:50
…mmon-architecture

Feature/add splash screen and common architecture
Manoel Soares Neto and others added 30 commits September 19, 2019 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant