The generator is a little too greedy with Iterable of Path and make tests fails
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Empieza descomprimiendo el ejemplo assertj-foo.zip enlazado y ejecutando FooTest para reproducir la aserción hasPath que falla. Rastrea cómo el generador gestiona los tipos que implementan java.lang.Iterable, incluida la distinción con java.util.Collection. Se considera terminado cuando los tipos Iterable similares a Path usan semántica de igualdad, mientras que los tipos Collection conservan sus predicados de Collection, y la prueba pasa.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
With 2.1.0 version, the generator is too greedy when encountering a class implementing an java.lang.Iterable. It should ignore it unless the type extends java.util.Collection (where that make sense) or explicitly a java.lang.Iterable (where it is harder to tell it makes sense).
Sample classes: assertj-foo.zip
The test (FooTest) will fail because the usage of hasPath is NOT the same than using an iterable: we want to test an equals rather than a collection predicate. Here the exception: java.lang.AssertionError: Expecting: <foobar\a>to contain: <[foobar\a]>but could not find: <[foobar\a]>
A working alternative is to convert the Path into a List extracting the Iterable content, but that is wrong by all means:
- first, the default semantic is not good (the default = search for subsequence in any order)
- second, as an user, I want to test the path using
equals(and that is what the generator produce if I have a class not implementingjava.lang.Iterable):
final List<Path> paths = new ArrayList<>();
test.forEach(paths::add); // says ["foobar", "A"]
assertThat(foo).hasPath(paths);
- Lenguaje dominante
- Java
- Estrellas
- 72
- Forks
- 47
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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.
Más de assertj/assertj-generator
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
assertj/assertj-generator#278 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
assertj/assertj-generator#220 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
assertj/assertj-generator#219 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
assertj/assertj-generator#204 · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
assertj/assertj-generator#197 · 7 comentarios ·
Todos los issues de assertj/assertj-generator
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
OpenAPITools/openapi-generator#25014 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openhab/openhab-core#5847 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
apache/parquet-java#3820 ·
Los mantenedores suelen responder en 1 día