search

guilhermejccavalcanti / s3m

Semistructured merge tool for java applications.
23 stars 35 forks source link

Bugfix/Renaming Conflicts Statistics #168

Open leonardoAnjos16 opened 2 years ago

leonardoAnjos16 commented 2 years ago

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.

leonardoAnjos16 commented 2 years ago

@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!

leonardoAnjos16 commented 2 years ago

@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.