Skip to content

Commit

Permalink
Updated several (transitive) dependencies (OFBIZ-13123) (#819)
Browse files Browse the repository at this point in the history
ServicePurgeTest.groovy and SimpleTests.groovy are no longer integration tests.
So they should be put in sourceSets block test part.

Thjs also means to delete now useledd basetests.xml and to remove
service-purge-test test case from service/testdef/servicetests.xml

This does it to allow integration tests to pass and to add
ServicePurgeTest.groovy and SimpleTests.groovy in Junit tests
  • Loading branch information
JacquesLeRoux committed Oct 14, 2024
1 parent 3e9fe46 commit f8ad232
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 32 deletions.
2 changes: 2 additions & 0 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -267,6 +267,8 @@ sourceSets {
include 'org/apache/ofbiz/service/ModelServiceTest.groovy'
include 'org/apache/ofbiz/test/TestServices.groovy'
include 'org/apache/ofbiz/base/util/FileUtilTests.groovy'
include 'org/apache/ofbiz/service/test/ServicePurgeTest.groovy'
include 'org.apache.ofbiz.base.test.SimpleTests.groovy'
}
resources {
srcDirs = getDirectoryInActiveComponentsIfExists('src/test/resources')
Expand Down
27 changes: 0 additions & 27 deletions framework/base/testdef/basetests.xml

This file was deleted.

5 changes: 0 additions & 5 deletions framework/service/testdef/servicetests.xml
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,4 @@ under the License.
<test-case case-name="service-permission-tests">
<junit-test-suite class-name="org.apache.ofbiz.service.test.ServicePermissionTests"/>
</test-case>

<test-case case-name="service-purge-test">
<junit-test-suite class-name="org.apache.ofbiz.service.test.ServicePurgeTest"/>
</test-case>

</test-suite>

0 comments on commit f8ad232

Please sign in to comment.