-
Notifications
You must be signed in to change notification settings - Fork 33
Open
Description
This issue document commit a51f7d8 from Airbnb, which didn't have a good commit message:
Merges the src/integrationTest source set into src/test for the
tenant/api, tenant/codegen, and tenant/runtime modules, removing
the separate integration-test source set from both the Gradle and Bazel
builds.
Motivation
The integrationTest source set was a historical artifact — the tests
it contained were ordinary JUnit tests that ran in the same JVM as the
unit tests and had no need for a separate source set. Consolidating into
src/test simplifies the build configuration and eliminates the
conceptual overhead of maintaining two parallel test trees.
Changes
Gradle
- Removed the
integrationTestsource set and the
integration-test.gradle.ktsconvention plugin that wired it up. - Merged
integrationTestGradle task dependencies into the standard
testtask. - Updated
build.gradle.ktsfiles fortenant/api,tenant/codegen,
andtenant/runtimeaccordingly.
Bazel
- Moved all
src/integrationTestsource files intosrc/test. - Updated all affected
BUILD.bazelfiles so each package's test
library carries the full union of deps that were previously split
across the unit-test and integration-test BUILD files. - Packages that used the
feature_app_codegenmacro
(globalid,execution) now invoke it directly from their
src/testBUILD file, preserving the same generated-GRT behaviour. - Fixed a hardcoded
src/integrationTestpath reference in
viaduct_classdiff_test_macro.bzl. - Fixed
SchemaClassFileDiffto load the default Viaduct schema from
the correct classpath path in both Gradle (default_schema.graphqls
at the classpath root) and Bazel (via a newgenrule+
java_librarytarget that renames and re-roots the generated
@@viaduct_default_schema.graphqlsfile).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels