[ 
https://issues.apache.org/jira/browse/LUCENE-10662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17573181#comment-17573181
 ] 

Dawid Weiss commented on LUCENE-10662:
--------------------------------------

> Why is it necessary to break the inheritance in order to achieve what is 
> wanted here? 

I think it's because static imports won't be resolved properly in a subclass if 
there's an "assertThat" method in a superclass, which would require the kind of 
delegation trickery you mentioned, Mike. 

I'm a bit torn on this one, actually. I like aspectj but it does seem like 
changing LuceneTestCase's inheritance may be too invasive for both Lucene and 
existing downstream projects that rely on it.

> Make LuceneTestCase to not extend from org.junit.Assert
> -------------------------------------------------------
>
>                 Key: LUCENE-10662
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10662
>             Project: Lucene - Core
>          Issue Type: Test
>          Components: general/test
>            Reporter: Marios Trivyzas
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Since *LuceneTestCase* is a very useful abstract class that can be extended 
> and used by many projects, having it extending *org.junit.Assert* limits all 
> users to exclusively use the static methods of {*}org.junit.Assert{*}. In our 
> project we want to use [https://joel-costigliola.github.io/assertj] where the 
> main method to call is *org.assertj.core.api.Assertions.assertThat* which 
> conflicts with the deprecated {*}org.junit.Assert.assertThat{*}, recognized 
> by default by the compiler. So one can only use assertj if on every call uses 
> fully qualified name for the *assertThat* method, i.e.
>  
> {code:java}
> org.assertj.core.api.Assertions.assertThat(myObj.name()).isEqualTo(expectedName)
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to