Skip to content

Commit

Permalink
pr remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
wolf4ood committed May 22, 2023
1 parent 1fe84c0 commit 9a5c76b
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 53 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/verify.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -126,9 +126,9 @@ jobs:
- uses: ./.github/actions/setup-java

- name: Run E2E tests
run: ./gradlew :edc-tests:runtime:build test -DincludeTags="EndToEndTest"
run: ./gradlew test -DincludeTags="EndToEndTest"

postgres-end-to-end-tests:
postgres-tests:
runs-on: ubuntu-latest
needs: [ verify-formatting ]

Expand All @@ -145,4 +145,4 @@ jobs:
- uses: ./.github/actions/setup-java

- name: Run Postgresql E2E tests
run: ./gradlew :edc-tests:runtime:build test -DincludeTags="PostgresqlIntegrationTest"
run: ./gradlew test -DincludeTags="PostgresqlIntegrationTest"
4 changes: 4 additions & 0 deletions edc-tests/e2e-tests/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,10 @@ dependencies {
testImplementation(libs.edc.ext.jsonld)
testImplementation(libs.edc.dsp)
testImplementation(testFixtures(libs.edc.sql.core))

testCompileOnly(project(":edc-tests:runtime:extensions"))
testCompileOnly(project(":edc-tests:runtime:runtime-memory"))
testCompileOnly(project(":edc-tests:runtime:runtime-postgresql"))
}

// do not publish
Expand Down
32 changes: 0 additions & 32 deletions edc-tests/runtime/build.gradle.kts

This file was deleted.

10 changes: 2 additions & 8 deletions edc-tests/runtime/runtime-memory/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
plugins {
`java-library`
id("application")
id("com.github.johnrengelman.shadow") version "8.1.1"
}


Expand All @@ -28,13 +27,13 @@ dependencies {
exclude(module = "data-encryption")
}


implementation(project(":edc-tests:runtime:extensions"))

// use basic (all in-mem) data plane
runtimeOnly(project(":edc-dataplane:edc-dataplane-base")) {
exclude("org.eclipse.edc", "api-observability")
}

runtimeOnly(project(":edc-tests:runtime:extensions"))

implementation(libs.edc.core.controlplane)
// for the controller
Expand All @@ -45,11 +44,6 @@ application {
mainClass.set("org.eclipse.edc.boot.system.runtime.BaseRuntime")
}

tasks.withType<com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar> {
mergeServiceFiles()
archiveFileName.set("app.jar")
}

// do not publish
edcBuild {
publish.set(false)
Expand Down
4 changes: 2 additions & 2 deletions edc-tests/runtime/runtime-postgresql/README.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
# In-Memory Runtime for Testing Purposes
# PostgreSQL Runtime for Testing Purposes

This module provides a very small, purely in-mem runtime to execute tests against. Not intended for anything other than testing!
This module provides a very small,runtime using PostgreSQL as persistence backend to execute tests against. Not intended for anything other than testing!
10 changes: 2 additions & 8 deletions edc-tests/runtime/runtime-postgresql/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
plugins {
`java-library`
id("application")
id("com.github.johnrengelman.shadow") version "8.1.1"
}


Expand All @@ -29,13 +28,13 @@ dependencies {
exclude(module = "hashicorp-vault")
}


implementation(project(":edc-tests:runtime:extensions"))

// use basic (all in-mem) data plane
runtimeOnly(project(":edc-dataplane:edc-dataplane-base")) {
exclude("org.eclipse.edc", "api-observability")
}

runtimeOnly(project(":edc-tests:runtime:extensions"))

implementation(libs.edc.core.controlplane)
// for the controller
Expand All @@ -46,11 +45,6 @@ application {
mainClass.set("org.eclipse.edc.boot.system.runtime.BaseRuntime")
}

tasks.withType<com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar> {
mergeServiceFiles()
archiveFileName.set("app.jar")
}

// do not publish
edcBuild {
publish.set(false)
Expand Down

0 comments on commit 9a5c76b

Please sign in to comment.