You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I had a brief look into extending AssertJ. Every custom Assertion class needs to extend AbstractAssert<$Classname, $CtElement>. But we don't want to repeat very basic assertions like CtElement#isImplicit. If we have multiple assertion classes like assertThat(CtElement) and assertThat(CtMethod) we need ugly casts to call the basic CtElement assertions. A possible solution is we use some code generation to copy the basic assertions to the more advanced methods. Almost like we do with our visitor codegen.
Currently, we have assertions/testutils in
src/main
instead ofsrc/test
see https://github.com/INRIA/spoon/tree/master/src/main/java/spoon/testing. Given the java 17 release will be an already breaking major release, I propose to either remove or move these classes. A good replacement is either the default junit methods or we create our on SpoonAssertions extending AssertJ , see https://joel-costigliola.github.io/assertj/assertj-core-custom-assertions.html.The text was updated successfully, but these errors were encountered: