Skip to content

Commit

Permalink
HSEARCH-4705 Merge ES test dialects
Browse files Browse the repository at this point in the history
- create a merged dialect for all ES versions
- cleanup integration pom from dialect class property
  • Loading branch information
marko-bekhta authored and yrodiere committed Oct 13, 2022
1 parent 1428ab7 commit cd5d04e
Show file tree
Hide file tree
Showing 6 changed files with 599 additions and 33 deletions.
16 changes: 0 additions & 16 deletions parents/integrationtest/pom.xml
Expand Up @@ -491,7 +491,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-5.6}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch5TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -502,7 +501,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-6.2}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch60TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -513,7 +511,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-6.3}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch63TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -524,7 +521,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-6.6}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch64TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -535,7 +531,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-6.7}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch67TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -546,7 +541,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-6.8}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch68TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -557,7 +551,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.2}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch70TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -568,7 +561,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.6}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch73TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -579,7 +571,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.7}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch77TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -590,7 +581,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.9}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch78TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -601,7 +591,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.10}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch710TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -612,7 +601,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.11}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch710TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -623,7 +611,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-7.17}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch712TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -640,7 +627,6 @@
<test.elasticsearch.run.elastic.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.elastic.skip>
<test.elasticsearch.connection.distribution>elastic</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.elasticsearch.latest-8.4}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch712TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -651,7 +637,6 @@
<test.elasticsearch.run.opensearch.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.opensearch.skip>
<test.elasticsearch.connection.distribution>opensearch</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.opensearch.latest-1.3}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch710TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand All @@ -662,7 +647,6 @@
<test.elasticsearch.run.opensearch.skip>${test.elasticsearch.run.skip}</test.elasticsearch.run.opensearch.skip>
<test.elasticsearch.connection.distribution>opensearch</test.elasticsearch.connection.distribution>
<test.elasticsearch.connection.version>${version.org.opensearch.latest-2.0}</test.elasticsearch.connection.version>
<test.elasticsearch.testdialect>org.hibernate.search.util.impl.integrationtest.backend.elasticsearch.dialect.Elasticsearch710TestDialect</test.elasticsearch.testdialect>
</properties>
</profile>

Expand Down
3 changes: 0 additions & 3 deletions pom.xml
Expand Up @@ -625,8 +625,6 @@
-Dhibernate.search.backend.aws.credentials.access_key_id=${test.elasticsearch.connection.aws.credentials.access_key_id}
-Dhibernate.search.backend.aws.credentials.secret_access_key=${test.elasticsearch.connection.aws.credentials.secret_access_key}
</test.elasticsearch.connection.jvm.args>
<!-- Control how integration tests format the requests sent to Elasticsearch -->
<test.elasticsearch.testdialect></test.elasticsearch.testdialect>

<!-- Lucene tests properties -->

Expand Down Expand Up @@ -2222,7 +2220,6 @@
<org.hibernate.search.version>${project.version}</org.hibernate.search.version>
<org.hibernate.search.enable_performance_tests>${org.hibernate.search.enable_performance_tests}</org.hibernate.search.enable_performance_tests>
<org.hibernate.search.fail_on_unreleased_service>true</org.hibernate.search.fail_on_unreleased_service>
<org.hibernate.search.integrationtest.backend.elasticsearch.testdialect>${test.elasticsearch.testdialect}</org.hibernate.search.integrationtest.backend.elasticsearch.testdialect>
<org.hibernate.search.integrationtest.backend.elasticsearch.version>${test.elasticsearch.connection.distribution}:${test.elasticsearch.connection.version}</org.hibernate.search.integrationtest.backend.elasticsearch.version>
</systemPropertyVariables>
<jvm>${java-version.test.launcher}</jvm>
Expand Down

0 comments on commit cd5d04e

Please sign in to comment.