Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Equipe 6 pull request atividade 7 #343

Open
wants to merge 101 commits into
base: master
Choose a base branch
from

Conversation

mucapaz
Copy link

@mucapaz mucapaz commented Feb 7, 2015

Tomer Simis
Criação de uma classe que contém métodos que realizam operações comuns entre testes :
8537135
1fc255f
Substituição de código duplicado pelos métodos criados:
88085db
05c5202
38ba9d6

Marlon Reghert Alves dos Santos (mras):
Através da feature Code Duplicates Detection, realizei a correção de códigos duplicados nas seguintes classes e os respectivos métodos criados ( as duplicações ocorriam na mesma classe ) :
- MemberController: Criação dos métodos: isMemberInstance, redirectNotFound e renderCreateMember.
- ResearchLineController: Criação do método: renderEdit.
- FunderController: Criação do Método: getFunderInstance.

Samuel Paz Mendes (spm) :
1)No commit a98dee1 foi removido um método que, além de duplicado, não estava sendo usado pelo sistema.

2)No commit 4243da8 foi retirado código duplicado de quatro lugares diferentes através de Extract Method.

tomersimis and others added 30 commits October 27, 2014 07:48
tomersimis and others added 30 commits January 20, 2015 13:58
Adicionando metodo complementar para eliminar a duplicação existente
…um objeto do banco e um de teste

Esse mesmo trecho de código era utilizado em diversos locais, alterando apenas o valor das variáveis mock e object.
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.