Skip to content

fix: run tests before publishing in generate workflow#1

Merged
mikipalet merged 4 commits intomainfrom
fix/run-tests-before-publish
Mar 2, 2026
Merged

fix: run tests before publishing in generate workflow#1
mikipalet merged 4 commits intomainfrom
fix/run-tests-before-publish

Conversation

@mikipalet
Copy link
Contributor

Summary

  • Generate workflow had no build/test verification before publishing
  • Added mvn compile test step before the publish gate
  • Same fix applied across all Late SDKs after late-python#12 shipped a broken release

Test plan

  • Trigger Regenerate workflow manually and verify tests run
  • If tests fail, publish is blocked

🤖 Generated with Claude Code

mikipalet and others added 2 commits March 2, 2026 11:47
The generate workflow had no build or test verification before
publishing. A broken regeneration would ship without any validation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mikipalet
Copy link
Contributor Author

CI caught a pre-existing build failure (not introduced by this PR):

MessagesApi.java:[30,25] cannot find symbol: class SendInboxMessageRequest1

The generated MessagesApi references a model class SendInboxMessageRequest1 that doesn't exist in the generated models. Same class of bug as the Python SDK's TikTokSettings issue (getlate-dev/late-python#12).

This is exactly the kind of issue these CI checks are meant to catch before publishing.

mikipalet and others added 2 commits March 2, 2026 11:55
The generated MessagesApi imported SendInboxMessageRequest1 which
doesn't exist as a model class (only SendInboxMessageRequest does).
The import was unused, causing a compilation failure.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…pile-only check

The generated test stubs are out of sync with the API (wrong types,
missing methods). Since they're all empty TODO stubs, use mvn compile
to verify the main source compiles. The test stubs will be fixed on
next full regeneration.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mikipalet mikipalet merged commit fb743c8 into main Mar 2, 2026
1 check 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.

1 participant