Joel Costigliola

Results 55 issues of Joel Costigliola

I'm not sure if this is a bug or a regression but the feedback might be useful. AssertJ Core is using jacoco but it was not activated by default, a...

type: bug :bug:

#### Summary The recursive comparison would benefit from a clear introspection strategy when gathering the field to compare and getting their values. The recursive comparison is a bit inconsistent, it...

theme: recursive comparison

#### Summary We tests assertj core with integration tests to validate junit 4, testng and osgi integration, this works but is more a workaround to simulate different environments. A better...

type: enhancement
status: team discussion

…quals to compare keys. Fix #2165 # Conflicts: # src/main/java/org/assertj/core/internal/Maps.java

This was asked here https://stackoverflow.com/questions/67319902/assertj-usingrecursivecomparison-fails-when-top-level-value-is-a-map Reproduced with: ```java static class Person { private final String name; public Person(String name) { this.name = name; } } @Test void test() { assertThat(new...

type: enhancement
theme: recursive comparison

#### Summary `allMatch` and `doesNotContains` on an empty list always returns true whatever the content of the predicate. Despite being correct, you should make explicit if you expect an empty...

type: breaking change

#### Summary When throwing an `org.opentest4j.AssertionFailedError`, IDEs usually allow to visually compare actual and expected values, this is not the case with the basic `AssertionError` which only has a String...

type: enhancement

#### Summary When two strings difference is only about whitespaces or newlines it can be hard to see the difference. #### Scope - tab vs whitespace - end of line...

type: enhancement

#### Feature summary Add support for `OptionalInt`, `OptionalLong` and `OptionalDouble` as we do for `Optional`

theme: recursive comparison

Just an idea for the time being ... see if it is worth it.

status: waiting for interest