(polaris) branch main updated: Rename test classes of metastore manager + fix typo (#203)

[ta...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

takidau pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 2d572d2  Rename test classes of metastore manager + fix typo (#203)
2d572d2 is described below

commit 2d572d27a3f3d34473c57b6d7278f0b6ae9f2197
Author: Yuya Ebihara <eb...@gmail.com>
AuthorDate: Tue Aug 27 02:29:32 2024 +0900

    Rename test classes of metastore manager + fix typo (#203)
    
    * Rename test classes of metastore manager
    
    * Fix typo
---
 ...taStoreTest.java => PolarisEclipseLinkMetaStoreManagerTest.java} | 4 ++--
 .../core/persistence/PolarisTreeMapMetaStoreManagerTest.java        | 2 +-
 ...taStoreManagerTest.java => BasePolarisMetaStoreManagerTest.java} | 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreTest.java b/extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreManagerTest.java
similarity index 95%
rename from extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreTest.java
rename to extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreManagerTest.java
index 93a68ea..ea28f8c 100644
--- a/extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreTest.java
+++ b/extension/persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreManagerTest.java
@@ -28,8 +28,8 @@ import org.apache.polaris.core.PolarisCallContext;
 import org.apache.polaris.core.PolarisConfigurationStore;
 import org.apache.polaris.core.PolarisDefaultDiagServiceImpl;
 import org.apache.polaris.core.PolarisDiagnostics;
+import org.apache.polaris.core.persistence.BasePolarisMetaStoreManagerTest;
 import org.apache.polaris.core.persistence.PolarisMetaStoreManagerImpl;
-import org.apache.polaris.core.persistence.PolarisMetaStoreManagerTest;
 import org.apache.polaris.core.persistence.PolarisTestMetaStoreManager;
 import org.junit.jupiter.api.extension.ExtensionContext;
 import org.junit.jupiter.params.ParameterizedTest;
@@ -43,7 +43,7 @@ import org.mockito.Mockito;
  *
  * @author aixu
  */
-public class PolarisEclipseLinkMetaStoreTest extends PolarisMetaStoreManagerTest {
+public class PolarisEclipseLinkMetaStoreManagerTest extends BasePolarisMetaStoreManagerTest {
 
   @Override
   protected PolarisTestMetaStoreManager createPolarisTestMetaStoreManager() {
diff --git a/polaris-core/src/test/java/org/apache/polaris/core/persistence/PolarisTreeMapMetaStoreManagerTest.java b/polaris-core/src/test/java/org/apache/polaris/core/persistence/PolarisTreeMapMetaStoreManagerTest.java
index 2364585..6317c70 100644
--- a/polaris-core/src/test/java/org/apache/polaris/core/persistence/PolarisTreeMapMetaStoreManagerTest.java
+++ b/polaris-core/src/test/java/org/apache/polaris/core/persistence/PolarisTreeMapMetaStoreManagerTest.java
@@ -25,7 +25,7 @@ import org.apache.polaris.core.PolarisDefaultDiagServiceImpl;
 import org.apache.polaris.core.PolarisDiagnostics;
 import org.mockito.Mockito;
 
-public class PolarisTreeMapMetaStoreManagerTest extends PolarisMetaStoreManagerTest {
+public class PolarisTreeMapMetaStoreManagerTest extends BasePolarisMetaStoreManagerTest {
   @Override
   public PolarisTestMetaStoreManager createPolarisTestMetaStoreManager() {
     PolarisDiagnostics diagServices = new PolarisDefaultDiagServiceImpl();
diff --git a/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/PolarisMetaStoreManagerTest.java b/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java
similarity index 98%
rename from polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/PolarisMetaStoreManagerTest.java
rename to polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java
index ab26345..bf97e41 100644
--- a/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/PolarisMetaStoreManagerTest.java
+++ b/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java
@@ -61,7 +61,7 @@ import org.threeten.extra.MutableClock;
  *
  * @author bdagevil
  */
-public abstract class PolarisMetaStoreManagerTest {
+public abstract class BasePolarisMetaStoreManagerTest {
 
   protected final MutableClock timeSource = MutableClock.of(Instant.now(), ZoneOffset.UTC);
 
@@ -322,7 +322,7 @@ public abstract class PolarisMetaStoreManagerTest {
         Stream.concat(firstTasks.stream(), newTaskList.stream().map(PolarisBaseEntity::getName))
             .collect(Collectors.toSet());
 
-    // only 10 tasks are unnassigned. Requesting 20, we should only receive those 10
+    // only 10 tasks are unassigned. Requesting 20, we should only receive those 10
     List<PolarisBaseEntity> lastTen =
         metaStoreManager.loadTasks(callCtx, executorId, 20).getEntities();
 
@@ -344,7 +344,7 @@ public abstract class PolarisMetaStoreManagerTest {
 
     timeSource.add(Duration.ofMinutes(10));
 
-    // all the tasks are unnassigned. Fetch them all
+    // all the tasks are unassigned. Fetch them all
     List<PolarisBaseEntity> allTasks =
         metaStoreManager.loadTasks(callCtx, executorId, 20).getEntities();