Skip to content

Commit

Permalink
fix: remove JVM integration tests module (#2976)
Browse files Browse the repository at this point in the history
This is not really needed anymore, as we have full integration tests
  • Loading branch information
stuartwdouglas authored Oct 3, 2024
1 parent c726b29 commit 666736e
Show file tree
Hide file tree
Showing 12 changed files with 6 additions and 300 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,12 @@ public boolean trigger(CodeGenContext context) throws CodeGenException {
if (!fileName.endsWith(".pb")) {
continue;
}
var module = Module.parseFrom(Files.readAllBytes(file));
Module module;
try {
module = Module.parseFrom(Files.readAllBytes(file));
} catch (Exception e) {
throw new CodeGenException("Failed to parse " + file, e);
}
for (var decl : module.getDeclsList()) {
String packageName = PACKAGE_PREFIX + module.getName();
if (decl.hasTypeAlias()) {
Expand Down
104 changes: 0 additions & 104 deletions jvm-runtime/ftl-runtime/java/integration-tests/pom.xml

This file was deleted.

Binary file not shown.
Binary file not shown.
Binary file not shown.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Empty file.

This file was deleted.

1 change: 0 additions & 1 deletion jvm-runtime/ftl-runtime/java/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
<modules>
<module>deployment</module>
<module>runtime</module>
<module>integration-tests</module>
<module>build-parent</module>
</modules>
</project>

0 comments on commit 666736e

Please sign in to comment.