just make check depend on bulkCdkIntegrationTest

airbytehq · Oct 1, 2024 · b8aa176 · b8aa176
1 parent 9f0b366
commit b8aa176
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.github/workflows/gradle.yml b/.github/workflows/gradle.yml
@@ -77,7 +77,7 @@ jobs:
           gradle-distribution-sha-256-sum-warning: false
           concurrent: true
           # TODO: be able to remove the skipSlowTests property
-          arguments: --scan check :airbyte-cdk:bulk:bulkCdkIntegrationTest -DskipSlowTests=true
+          arguments: --scan check -DskipSlowTests=true
 
   set-instatus-incident-on-failure:
     name: Create Instatus Incident on Failure

diff --git a/airbyte-cdk/bulk/core/load/build.gradle b/airbyte-cdk/bulk/core/load/build.gradle
@@ -1,8 +1,9 @@
+// simply declaring the source sets is sufficient to populate them with
+// src/integrationTest/java+resources + src/integrationTest/kotlin.
 sourceSets {
     integrationTest {
     }
 }
-
 kotlin {
     sourceSets {
         testIntegration {
@@ -31,8 +32,9 @@ task integrationTest(type: Test) {
     useJUnitPlatform()
     mustRunAfter tasks.check
 }
-
 configurations {
     integrationTestImplementation.extendsFrom testImplementation
     integrationTestRuntimeOnly.extendsFrom testRuntimeOnly
 }
+// These tests are lightweight enough to run on every PR.
+rootProject.check.dependsOn(integrationTest)