Skip to content

Commit 519e0b6

Browse files
committed
Move some util classes to polaris-service-common as test fixtures
1 parent d8c1804 commit 519e0b6

File tree

14 files changed

+38
-14
lines changed

14 files changed

+38
-14
lines changed

quarkus/service/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ dependencies {
9090
testFixturesApi(project(":polaris-tests"))
9191

9292
testImplementation(project(":polaris-api-management-model"))
93+
testImplementation(testFixtures(project(":polaris-service-common")))
9394

9495
testImplementation("org.apache.iceberg:iceberg-api:${libs.versions.iceberg.get()}:tests")
9596
testImplementation("org.apache.iceberg:iceberg-core:${libs.versions.iceberg.get()}:tests")

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/ManagementServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
import org.apache.polaris.core.admin.model.PolarisCatalog;
3333
import org.apache.polaris.core.admin.model.StorageConfigInfo;
3434
import org.apache.polaris.core.admin.model.UpdateCatalogRequest;
35-
import org.apache.polaris.service.quarkus.TestServices;
35+
import org.apache.polaris.service.TestServices;
3636
import org.junit.jupiter.api.Test;
3737

3838
public class ManagementServiceTest {

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisAuthzTestBase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,11 @@
6767
import org.apache.polaris.core.persistence.PolarisMetaStoreSession;
6868
import org.apache.polaris.service.admin.PolarisAdminService;
6969
import org.apache.polaris.service.catalog.BasePolarisCatalog;
70+
import org.apache.polaris.service.catalog.PolarisPassthroughResolutionView;
7071
import org.apache.polaris.service.catalog.io.DefaultFileIOFactory;
7172
import org.apache.polaris.service.catalog.io.FileIOFactory;
7273
import org.apache.polaris.service.config.DefaultConfigurationStore;
7374
import org.apache.polaris.service.config.RealmEntityManagerFactory;
74-
import org.apache.polaris.service.quarkus.catalog.PolarisPassthroughResolutionView;
7575
import org.apache.polaris.service.storage.PolarisStorageIntegrationProviderImpl;
7676
import org.assertj.core.api.Assertions;
7777
import org.junit.jupiter.api.AfterEach;

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisOverlappingCatalogTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
import org.apache.polaris.core.admin.model.CatalogProperties;
3434
import org.apache.polaris.core.admin.model.CreateCatalogRequest;
3535
import org.apache.polaris.core.admin.model.StorageConfigInfo;
36-
import org.apache.polaris.service.quarkus.TestServices;
36+
import org.apache.polaris.service.TestServices;
3737
import org.junit.jupiter.params.ParameterizedTest;
3838
import org.junit.jupiter.params.provider.CsvSource;
3939

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisOverlappingTableTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
import org.apache.polaris.core.admin.model.CreateCatalogRequest;
3737
import org.apache.polaris.core.admin.model.FileStorageConfigInfo;
3838
import org.apache.polaris.core.admin.model.StorageConfigInfo;
39-
import org.apache.polaris.service.quarkus.TestServices;
39+
import org.apache.polaris.service.TestServices;
4040
import org.junit.jupiter.api.DisplayName;
4141
import org.junit.jupiter.params.ParameterizedTest;
4242
import org.junit.jupiter.params.provider.Arguments;

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/BasePolarisCatalogTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,12 @@
9595
import org.apache.polaris.core.storage.cache.StorageCredentialCache;
9696
import org.apache.polaris.service.admin.PolarisAdminService;
9797
import org.apache.polaris.service.catalog.BasePolarisCatalog;
98+
import org.apache.polaris.service.catalog.PolarisPassthroughResolutionView;
9899
import org.apache.polaris.service.catalog.io.DefaultFileIOFactory;
99100
import org.apache.polaris.service.catalog.io.FileIOFactory;
101+
import org.apache.polaris.service.catalog.io.TestFileIOFactory;
100102
import org.apache.polaris.service.config.RealmEntityManagerFactory;
101103
import org.apache.polaris.service.exception.IcebergExceptionMapper;
102-
import org.apache.polaris.service.quarkus.catalog.io.TestFileIOFactory;
103104
import org.apache.polaris.service.storage.PolarisStorageIntegrationProviderImpl;
104105
import org.apache.polaris.service.task.TableCleanupTaskHandler;
105106
import org.apache.polaris.service.task.TaskExecutor;

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/BasePolarisCatalogViewTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@
5656
import org.apache.polaris.core.persistence.PolarisMetaStoreSession;
5757
import org.apache.polaris.service.admin.PolarisAdminService;
5858
import org.apache.polaris.service.catalog.BasePolarisCatalog;
59+
import org.apache.polaris.service.catalog.PolarisPassthroughResolutionView;
5960
import org.apache.polaris.service.catalog.io.DefaultFileIOFactory;
6061
import org.apache.polaris.service.catalog.io.FileIOFactory;
6162
import org.apache.polaris.service.config.RealmEntityManagerFactory;

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/io/FileIOExceptionsTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@
3838
import org.apache.polaris.core.admin.model.FileStorageConfigInfo;
3939
import org.apache.polaris.core.admin.model.PolarisCatalog;
4040
import org.apache.polaris.core.admin.model.StorageConfigInfo;
41-
import org.apache.polaris.service.quarkus.TestServices;
41+
import org.apache.polaris.service.catalog.io.TestFileIOFactory;
42+
import org.apache.polaris.service.TestServices;
4243
import org.junit.jupiter.api.BeforeAll;
4344
import org.junit.jupiter.api.BeforeEach;
4445
import org.junit.jupiter.params.ParameterizedTest;

service/common/build.gradle.kts

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919

2020
plugins {
2121
id("polaris-server")
22+
id("java-test-fixtures")
2223
alias(libs.plugins.jandex)
2324
}
2425

@@ -91,6 +92,27 @@ dependencies {
9192
testImplementation(libs.assertj.core)
9293
testImplementation(libs.mockito.core)
9394
testRuntimeOnly("org.junit.platform:junit-platform-launcher")
95+
96+
testFixturesImplementation(project(":polaris-core"))
97+
testFixturesImplementation(project(":polaris-api-management-service"))
98+
testFixturesImplementation(project(":polaris-api-iceberg-service"))
99+
100+
testFixturesImplementation(libs.jakarta.enterprise.cdi.api)
101+
testFixturesImplementation(libs.jakarta.annotation.api)
102+
testFixturesImplementation(libs.jakarta.inject.api)
103+
testFixturesImplementation(libs.jakarta.ws.rs.api)
104+
105+
testFixturesImplementation(platform(libs.iceberg.bom))
106+
testFixturesImplementation("org.apache.iceberg:iceberg-api")
107+
testFixturesImplementation("org.apache.iceberg:iceberg-core")
108+
testFixturesImplementation("org.apache.iceberg:iceberg-aws")
109+
110+
testFixturesImplementation(platform(libs.google.cloud.storage.bom))
111+
testFixturesImplementation("com.google.cloud:google-cloud-storage")
112+
testFixturesImplementation(platform(libs.awssdk.bom))
113+
testFixturesImplementation("software.amazon.awssdk:sts")
114+
testFixturesImplementation("software.amazon.awssdk:iam-policy-builder")
115+
testFixturesImplementation("software.amazon.awssdk:s3")
94116
}
95117

96118
tasks.named("javadoc") { dependsOn("jandex") }

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/TestServices.java renamed to service/common/src/testFixtures/java/org/apache/polaris/service/TestServices.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
* specific language governing permissions and limitations
1717
* under the License.
1818
*/
19-
package org.apache.polaris.service.quarkus;
19+
package org.apache.polaris.service;
2020

2121
import com.google.auth.oauth2.AccessToken;
2222
import com.google.auth.oauth2.GoogleCredentials;
@@ -41,10 +41,10 @@
4141
import org.apache.polaris.service.catalog.IcebergCatalogAdapter;
4242
import org.apache.polaris.service.catalog.api.IcebergRestCatalogApi;
4343
import org.apache.polaris.service.catalog.api.IcebergRestCatalogApiService;
44+
import org.apache.polaris.service.catalog.io.TestFileIOFactory;
4445
import org.apache.polaris.service.config.DefaultConfigurationStore;
4546
import org.apache.polaris.service.config.RealmEntityManagerFactory;
4647
import org.apache.polaris.service.persistence.InMemoryPolarisMetaStoreManagerFactory;
47-
import org.apache.polaris.service.quarkus.catalog.io.TestFileIOFactory;
4848
import org.apache.polaris.service.storage.PolarisStorageIntegrationProviderImpl;
4949
import org.apache.polaris.service.task.TaskExecutor;
5050
import org.mockito.Mockito;

0 commit comments

Comments
 (0)