libgit2 / libgit2/libgit2sharp
NormalMerge should call callback on conflicts
Aberta
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- C#
- Estrelas
- 3.5k
- Forks
- 925
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
Reproduction steps
[Fact]
public void NormalMergeShouldCallCallbackOnConflicts()
{
string committishToMerge = "fast_forward";
using (var repo = new Repository(SandboxMergeTestRepo()))
{
Touch(repo.Info.WorkingDirectory, "b.txt", "this is an alternate change");
var conflicts = 0;
var options = new MergeOptions
{
OnCheckoutNotify = (path, flags) =>
{
conflicts++;
return true;
},
CheckoutNotifyFlags = CheckoutNotifyFlags.Conflict,
FastForwardStrategy = FastForwardStrategy.NoFastForward,
};
Assert.Throws<CheckoutConflictException>(() => repo.Merge(committishToMerge, Constants.Signature, options));
Assert.Equal(1, conflicts) ;
}
}
Expected behavior
Test shouldn't fail
Actual behavior
Test fails
Version of LibGit2Sharp (release number or SHA1)
df3b22a754ef56da8d7e3c330ce2d783c2b7982e
Operating system(s) tested; .NET runtime tested
Windows netcore 2.1
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece pelo teste NormalMergeShouldCallCallbackOnConflicts e pela chamada a repo.Merge usando MergeOptions.OnCheckoutNotify, CheckoutNotifyFlags.Conflict e NoFastForward. Reproduza a falha, acompanhe o tratamento de conflitos e confirme que o callback é executado uma vez enquanto CheckoutConflictException continua sendo lançada.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- csharp, git
- Domínio
- backend, testing
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Claramente especificada
- Facilidade para iniciantes
- 45/100