Skip to content

Add common JSON contract checks - #171

Merged
yoshifuminakamura merged 1 commit into
developfrom
feature/common-json-contract-tests
Sep 7, 2026
Merged

yoshifuminakamura merged 1 commit into
developfrom
feature/common-json-contract-tests

Conversation

@yoshifuminakamura

@yoshifuminakamura yoshifuminakamura commented Sep 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add Result JSON contract assertions for common metadata blocks emitted by result processing
  • add a generic estimation common JSON contract test using a synthetic contract app
  • keep the change test-only; no portal routes, public/console access policy, app-specific tests, or runtime behavior changes

Tests

  • bash scripts/tests/test_process_and_send_results.sh
  • bash scripts/tests/test_estimation_common_json_contract.sh
  • bash scripts/tests/test_estimation_run_timing.sh
  • bash scripts/tests/test_genesis_gpu_mlp_estimation.sh
  • bash -n scripts/tests/test_process_and_send_results.sh scripts/tests/test_estimation_common_json_contract.sh scripts/tests/test_estimation_run_timing.sh scripts/tests/test_genesis_gpu_mlp_estimation.sh
  • shellcheck -S error scripts/tests/test_process_and_send_results.sh scripts/tests/test_estimation_common_json_contract.sh scripts/tests/test_estimation_run_timing.sh
  • python3 scripts/tests/check_text_integrity.py
  • python3 scripts/tests/check_commit_messages.py

Signed-off-by: yoshifuminakamura <nakamura@riken.jp>
@yoshifuminakamura
yoshifuminakamura force-pushed the feature/common-json-contract-tests branch from 6474adb to a98c472 Compare September 7, 2026 14:19
@yoshifuminakamura
yoshifuminakamura merged commit 5097030 into develop Sep 7, 2026
7 checks passed
@yoshifuminakamura
yoshifuminakamura deleted the feature/common-json-contract-tests branch September 7, 2026 14:32
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