You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by "Eric Pugh (Jira)" <ji...@apache.org> on 2022/07/18 00:49:00 UTC

[jira] [Created] (SOLR-16300) Migrate from deprecated assertThat() to org.hamcrest.MatcherAssert.assertThat()

Eric Pugh created SOLR-16300:
--------------------------------

             Summary: Migrate from deprecated assertThat() to org.hamcrest.MatcherAssert.assertThat()
                 Key: SOLR-16300
                 URL: https://issues.apache.org/jira/browse/SOLR-16300
             Project: Solr
          Issue Type: Sub-task
          Components: Tests
    Affects Versions: 9.0, main (10.0)
            Reporter: Eric Pugh


The org.junit.Assert that our unit tests all extend from indicates that assertThat() is deprecated, and to use org.hamcrest.MatcherAssert.assertThat() instead.

We currently in our tests have both a mix of the assertThat() AND MatcherAssert.assertThat() methods...  

It *appears* that I can also just override the assertThat() method defined in org.junit.Assert in our base test classes to be one that calls out MatcherAssert.assertThat() which might let us change fewer lines of code..  

I'd love some direction from more experienced folks on what the right direction to go is!



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

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org