Precarious test coverage

[velo]

Managed to instrument our jar so we can extract test coverage from FlinkMainIT

As is, this is unmaintainable, FlinkMainIT depends on TouchTest running to create a accessible jacoco.exec seed

And the configuration is spread across all the files on this PR.

Bettet than nothing, but very fragile.

[velo]

Coverage before this change:

After:

81% coverage, not bad, not bad at all

[Copilot]

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• flink-sql-runner/pom.xml: Language not supported

Comments suppressed due to low confidence (2)

flink-sql-runner/src/test/java/com/datasqrl/FlinkMainIT.java:254

• Returning 0 when the destination file exists may skip necessary re-instrumentation, potentially leading to outdated coverage data. Consider re-instrumenting or validating whether the source jar has changed before skipping.

if (dest.exists()) {

flink-sql-runner/src/main/docker/docker-compose.yml:69

• The docker-compose file mounts './jacoco.job.exec' while TouchTest creates the file in the 'target' directory; please confirm that the mounted path accurately reflects the location of the generated file.

- ./jacoco.job.exec:/opt/flink/jacoco.exec