Remove JUnit5BestPractices from RemoveObsoleteRunners (#476)

* Remove `JUnit5BestPractices` from `RemoveObsoleteRunners`

As that includes `TestsShouldNotBePublic`, which is an opinionated recipe that results in a lot of (optional) changes.
As reported via Slack: https://rewriteoss.slack.com/archives/C01A843MWG5/p1705404676460549

* Minimize expected changes

src/main/resources/META-INF/rewrite/spring-boot-24.yml

@@ -135,4 +135,3 @@ recipeList:
         # There is a third variant of this class in spring-cloud-commons and there is not yet a suitable migration
         # path. Once one exists, this can likely be added for removal as well.
         #- org.springframework.cloud.test.ModifiedClassPathRunner
-  - org.openrewrite.java.testing.junit5.JUnit5BestPractices

src/testWithSpringBoot_2_3/java/org/openrewrite/java/spring/boot2/SpringBoot2JUnit4to5MigrationTest.java

@@ -68,10 +68,10 @@ public void testFindAll() {
               import org.springframework.boot.test.context.SpringBootTest;
                             
               @SpringBootTest
-              class ProductionConfigurationTests {
+              public class ProductionConfigurationTests {
                             
                   @Test
-                  void findAll() {
+                  public void testFindAll() {
                   }
               }
               """
@@ -115,10 +115,10 @@ static class CustomConfiguration {
               import org.springframework.test.context.junit.jupiter.SpringJUnitConfig;
 
               @SpringJUnitConfig(classes = ProductionConfigurationTests.CustomConfiguration.class)
-              class ProductionConfigurationTests {
+              public class ProductionConfigurationTests {
 
                   @Test
-                  void findAll() {
+                  public void testFindAll() {
                   }
 
                   @Configuration