Re: The use of AssertJ assertions

2025-07-22 Thread Olivier Lamy
AssertJ use it everywhere in one test > > class. > > > > On Mon, 21 Jul 2025 at 06:51, Giovanni van der Schelde > > wrote: > > > > > > Hi all, > > > > > > In a recent PR review, the use of AssertJ assertions was raised as a point > &g

Re: The use of AssertJ assertions

2025-07-21 Thread Łukasz Dywicki
y, 21 July 2025 at 18:40 To: dev@maven.apache.org Subject: Re: The use of AssertJ assertions Hi all, while I really like the AssertJ assertions, e.g.  for readibility and expandability (custom assertions), I'm slighty against using it in a big project like Maven (thinking of core, plugins, co

Re: The use of AssertJ assertions

2025-07-21 Thread Matthias Bünger
were adopted as an “approved” dependency? Later, Andy From: Matthias Bünger Date: Monday, 21 July 2025 at 18:40 To: dev@maven.apache.org Subject: Re: The use of AssertJ assertions Hi all, while I really like the AssertJ assertions, e.g. for readibility and expandability (custom assertions)

Re: The use of AssertJ assertions

2025-07-21 Thread Andy Law
As more of a wider question, why would this not be specified in the Parent POM if it were adopted as an “approved” dependency? Later, Andy From: Matthias Bünger Date: Monday, 21 July 2025 at 18:40 To: dev@maven.apache.org Subject: Re: The use of AssertJ assertions Hi all, while I really

Re: The use of AssertJ assertions

2025-07-21 Thread Matthias Bünger
thias Am 21.07.2025 um 06:50 schrieb Giovanni van der Schelde: Hi all, In a recent PR review, the use of AssertJ assertions was raised as a point of discussion. To avoid recurring debates and ensure the PR is reviewed for the changes it provides, I’d like to propose that we clarify the goal rega

Re: The use of AssertJ assertions

2025-07-21 Thread Elliotte Rusty Harold
On Mon, Jul 21, 2025 at 11:13 AM Delany wrote: > > If I'm not mistaken Guava's Preconditions is for runtime argument > validation and should not be used in test code. It's been a while since I used it. s/Guava/Truth I just think of them as all part of the same Google Java stack but I guess that'

Re: The use of AssertJ assertions

2025-07-21 Thread Gary Gregory
ly others I'm forgetting, it just makes the > > > code harder to read and understand. > > > > > > There are indeed test methods in AssertJ that improve error messages > > > and reporting compared to JUnit. I do wish JUnit had assertContains. > > > Howe

Re: The use of AssertJ assertions

2025-07-21 Thread Romain Manni-Bucau
ve error messages > > and reporting compared to JUnit. I do wish JUnit had assertContains. > > However, the casein question wasn't one of these. It was just a > > different way of writing a standard assertEquals. > > > > On Mon, Jul 21, 2025 at 4:53 AM Giovanni van de

Re: The use of AssertJ assertions

2025-07-21 Thread Delany
r messages > and reporting compared to JUnit. I do wish JUnit had assertContains. > However, the casein question wasn't one of these. It was just a > different way of writing a standard assertEquals. > > On Mon, Jul 21, 2025 at 4:53 AM Giovanni van der Schelde > wrote: >

Re: The use of AssertJ assertions

2025-07-21 Thread Elliotte Rusty Harold
reporting compared to JUnit. I do wish JUnit had assertContains. However, the casein question wasn't one of these. It was just a different way of writing a standard assertEquals. On Mon, Jul 21, 2025 at 4:53 AM Giovanni van der Schelde wrote: > > Hi all, > > In a recent PR review, th

Re: The use of AssertJ assertions

2025-07-20 Thread Romain Manni-Bucau
't have any problem using it. > > Project is very well maintained and has an acceptable license. > > > > One thing I would like to see is do not mix assertions framework in > > one test class - when we use AssertJ use it everywhere in one test > > class. > &

Re: The use of AssertJ assertions

2025-07-20 Thread Guillaume Nodet
1, Giovanni van der Schelde > wrote: > > > > Hi all, > > > > In a recent PR review, the use of AssertJ assertions was raised as a point > > of discussion. > > To avoid recurring debates and ensure the PR is reviewed for the changes > > it provides,

Re: The use of AssertJ assertions

2025-07-20 Thread Slawomir Jaranowski
o see is do not mix assertions framework in one test class - when we use AssertJ use it everywhere in one test class. On Mon, 21 Jul 2025 at 06:51, Giovanni van der Schelde wrote: > > Hi all, > > In a recent PR review, the use of AssertJ assertions was raised as a point > of di

The use of AssertJ assertions

2025-07-20 Thread Giovanni van der Schelde
Hi all, In a recent PR review, the use of AssertJ assertions was raised as a point of discussion. To avoid recurring debates and ensure the PR is reviewed for the changes it provides, I’d like to propose that we clarify the goal regarding this, and perhaps other, dependencies. Specifically