Skip to content

ci: add java phase-2 conformance coverage#521

Merged
pradeeban merged 1 commit intoControlCore-Project:devfrom
avinxshKD:fix/java-conformance-ci-phase2
Mar 28, 2026
Merged

ci: add java phase-2 conformance coverage#521
pradeeban merged 1 commit intoControlCore-Project:devfrom
avinxshKD:fix/java-conformance-ci-phase2

Conversation

@avinxshKD
Copy link
Copy Markdown

@avinxshKD avinxshKD commented Mar 28, 2026

Adds Java to phase-2 protocol conformance and wires existing Java regression tests into CI.

Changes:

  • Added java to the phase-2 runtime set in test_protocol_conformance_phase2.py.

  • Updated phase-2 matrix to include Java results for all existing cases in cross_runtime_matrix.phase2.json.

  • Added a java-test CI job to compile and run TestLiteralEval and TestConcoredockerApi in ci.yml.

  • Documented Java phase-2 tracking/execution in PROTOCOL_FIXTURES.md

Scope: Conformance + CI only (no runtime changes).

Varified:

  1. pytest test_protocol_conformance_phase2.py -q -o addopts='' passed.
  2. Java tests passed locally:
    TestLiteralEval
    TestConcoredockerApi

closes #520

@avinxshKD
Copy link
Copy Markdown
Author

@pradeeban @Rahuljagwani @mayureshkothare pls take a look, kept this intentionally narrow to match issue scope: phase-2 conformance + CI wiring only, no runtime behavior changes.

@avinxshKD
Copy link
Copy Markdown
Author

Also wanted your guidance on, should I prioritize:

adding a Java node end-to-end example, or
extending conformance matrix coverage beyond current parser/API cases?

@pradeeban
Copy link
Copy Markdown
Member

Java end-to-end sounds good if you want us to stop calling it a toy. :D

@pradeeban pradeeban merged commit 2b37f9c into ControlCore-Project:dev Mar 28, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants