libgit2 / libgit2/libgit2sharp
NormalMerge should call callback on conflicts
Abierto
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 3.5k
- Forks
- 925
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el test NormalMergeShouldCallCallbackOnConflicts y la llamada a repo.Merge usando MergeOptions.OnCheckoutNotify, CheckoutNotifyFlags.Conflict y NoFastForward. Reproduce el fallo, sigue el manejo de conflictos y confirma que el callback se ejecuta una vez mientras CheckoutConflictException sigue lanzándose.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp, git
- Área
- backend, testing
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 45/100