Bugfix/Renaming Conflicts Statistics #168
Conversation
There was a problem hiding this comment.
@leonardoAnjos16 tu poderia adicionar algum caso de teste (pode até ser uma versão simplificado do exemplo que tem na issue https://github.com/guilhermejccavalcanti/jFSTMerge/files/7747674/files.zip )? Um teste onde a contagem estaria errada antes desse PR, e que agora estará correta?
@guilhermejccavalcanti posso sim, vou fazer isso! |
|
@guilhermejccavalcanti, não consegui simplificar o caso da issue por ser bastante complexo e grande. Mas fabriquei um caso em que a ferramenta antes das alterações reportava 2 conflitos de renaming (mesmo o resultado do merge tendo apenas 1 conflito), enquanto a nova versão reporta corretamente 1 conflito de renaming. |
The tool wasn't counting the number of renaming conflicts properly, making it report, in some occasions, that the number of renaming conflicts was greater than the number of total conflicts. This PR fixes it by maintaining the number of renaming conflicts up to date during the execution of the renaming handler.