ISSUE-3790: Remove unnecessary public modifiers in tests

catalog-rest-service/src/test/java/org/openmetadata/catalog/airflow/AirflowRESTClientIntegrationTest.java

@@ -32,7 +32,7 @@
 import org.openmetadata.catalog.exception.PipelineServiceClientException;
 
 @ExtendWith(MockitoExtension.class)
-public class AirflowRESTClientIntegrationTest {
+class AirflowRESTClientIntegrationTest {
 
   private static final String DAG_NAME = "test_dag";
   private static final String URI_TO_HANDLE_REQUEST = "/";
@@ -57,7 +57,7 @@ void setUp() {
   }
 
   @Test
-  public void testLastIngestionLogsAreRetrievedWhenStatusCodesAre200() {
+  void testLastIngestionLogsAreRetrievedWhenStatusCodesAre200() {
     Map<String, String> expectedMap = Map.of("key1", "value1", "key2", "value2");
 
     registerMockedEndpoints(200, 200);
@@ -66,7 +66,7 @@ public void testLastIngestionLogsAreRetrievedWhenStatusCodesAre200() {
   }
 
   @Test
-  public void testLastIngestionLogsExceptionWhenLoginFails() {
+  void testLastIngestionLogsExceptionWhenLoginFails() {
     registerMockedEndpoints(404, 200);
 
     Exception exception =
@@ -80,7 +80,7 @@ public void testLastIngestionLogsExceptionWhenLoginFails() {
   }
 
   @Test
-  public void testLastIngestionLogsExceptionWhenStatusCode404() {
+  void testLastIngestionLogsExceptionWhenStatusCode404() {
     registerMockedEndpoints(200, 404);
 
     Exception exception =

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/bots/BotResourceTest.java

@@ -20,7 +20,7 @@
 import org.openmetadata.catalog.resources.teams.UserResourceTest;
 import org.openmetadata.catalog.type.EntityReference;
 
-public class BotResourceTest extends EntityResourceTest<Bot, CreateBot> {
+class BotResourceTest extends EntityResourceTest<Bot, CreateBot> {
   public static User botUser;
   public static EntityReference botUserRef;
 

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/databases/DatabaseSchemaResourceTest.java

@@ -42,7 +42,7 @@
 
 @Slf4j
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-public class DatabaseSchemaResourceTest extends EntityResourceTest<DatabaseSchema, CreateDatabaseSchema> {
+class DatabaseSchemaResourceTest extends EntityResourceTest<DatabaseSchema, CreateDatabaseSchema> {
   public DatabaseSchemaResourceTest() {
     super(
         Entity.DATABASE_SCHEMA,

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/feeds/MessageParserTest.java

@@ -20,7 +20,7 @@
 import org.junit.jupiter.api.Test;
 import org.openmetadata.catalog.resources.feeds.MessageParser.EntityLink;
 
-public class MessageParserTest {
+class MessageParserTest {
   @Test
   void parseMessage() {
     String s =

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/metadata/TypeResourceTest.java

@@ -83,7 +83,7 @@ public void post_entityCreateWithInvalidName_400() {
   }
 
   @Test
-  public void put_customProperty_200() throws HttpResponseException {
+  void put_customProperty_200() throws HttpResponseException {
     Type tableEntity = getEntityByName("table", "customProperties", ADMIN_AUTH_HEADERS);
     assertTrue(listOrEmpty(tableEntity.getCustomProperties()).isEmpty());
 
@@ -108,7 +108,7 @@ public void put_customProperty_200() throws HttpResponseException {
   }
 
   @Test
-  public void put_customPropertyToPropertyType_4xx() {
+  void put_customPropertyToPropertyType_4xx() {
     // Adding a custom property to a property type is not allowed (only entity type is allowed)
     CustomProperty field =
         new CustomProperty().withName("intA").withDescription("intA").withPropertyType(INT_TYPE.getEntityReference());

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/mlmodels/MlModelResourceTest.java

@@ -63,7 +63,7 @@
 
 @Slf4j
 @TestMethodOrder(MethodOrderer.OrderAnnotation.class)
-public class MlModelResourceTest extends EntityResourceTest<MlModel, CreateMlModel> {
+class MlModelResourceTest extends EntityResourceTest<MlModel, CreateMlModel> {
 
   public static final String ALGORITHM = "regression";
   public static Dashboard DASHBOARD;

catalog-rest-service/src/test/java/org/openmetadata/catalog/resources/policies/PolicyResourceTest.java

@@ -187,7 +187,7 @@ void patch_PolicyAttributes_200_ok(TestInfo test) throws IOException {
   }
 
   @Test
-  public void get_policyResources() throws HttpResponseException {
+  void get_policyResources() throws HttpResponseException {
     // Get list of policy resources and make sure it has all the entities and other resources
     List<String> resources = getPolicyResources(ADMIN_AUTH_HEADERS);
     List<String> entities = Entity.listEntities();

catalog-rest-service/src/test/java/org/openmetadata/catalog/security/JWTTokenGeneratorTest.java

@@ -27,7 +27,7 @@
 
 @Slf4j
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-public class JWTTokenGeneratorTest {
+class JWTTokenGeneratorTest {
   protected static final String rsaPrivateKeyPath = ResourceHelpers.resourceFilePath("private_key.der");
   protected static final String rsaPublicKeyPath = ResourceHelpers.resourceFilePath("public_key.der");
   protected JWTTokenConfiguration jwtTokenConfiguration;

openmetadata-core/src/test/java/org/openmetadata/core/util/TypeUtilTest.java

@@ -11,7 +11,7 @@
 import java.util.Set;
 import org.junit.jupiter.api.Test;
 
-public class TypeUtilTest {
+class TypeUtilTest {
   private static final String customAttributes;
   private static final ObjectMapper mapper = new ObjectMapper();
 
@@ -24,7 +24,7 @@ public class TypeUtilTest {
   }
 
   @Test
-  public void testTypeValue() throws IOException {
+  void testTypeValue() throws IOException {
     JsonSchema intSchema = JsonUtils.getJsonSchema("{ \"type\" : \"integer\", \"minimum\": 10}");
     JsonSchema stringSchema = JsonUtils.getJsonSchema("{ \"type\" : \"string\"}");
     JsonNode json = mapper.readTree(customAttributes);