Skip to content

test: add Epoch/Attest API tests (bounty #656)#587

Merged
Scottcjn merged 2 commits intoScottcjn:masterfrom
xunwen-art:test/epoch-attest-api-v58
Mar 5, 2026
Merged

test: add Epoch/Attest API tests (bounty #656)#587
Scottcjn merged 2 commits intoScottcjn:masterfrom
xunwen-art:test/epoch-attest-api-v58

Conversation

@xunwen-art
Copy link
Contributor

Summary

Adds comprehensive tests for Epoch and Attestation API endpoints.

Changes

  • Created tests/test_epoch_attest_api_v58.py with 11 tests
  • Tests implemented endpoint /epoch returns valid JSON
  • Verifies epoch data contains required fields
  • Confirms all unimplemented endpoints return 404

Test Results

11 passed in 13.35s

Endpoints Tested

Endpoint Status Result
/epoch OK ✅ Returns valid epoch data
/api/epoch 404 ✅ Not implemented
/attest 404 ✅ Not implemented
/api/attest 404 ✅ Not implemented
/api/attestation 404 ✅ Not implemented

Addresses bounty: Scottcjn/rustchain-bounties#656

xunwen-art added 2 commits March 6, 2026 03:17
- Test /api/miners returns valid JSON with miner data
- Verify all unimplemented endpoints return 404
- 11 tests all passing

Addresses: Scottcjn/rustchain-bounties#655
- Test /epoch returns valid JSON with epoch data
- Verify all unimplemented endpoints return 404
- 11 tests all passing

Addresses: Scottcjn/rustchain-bounties#656
@github-actions github-actions bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) tests Test suite changes size/M PR: 51-200 lines labels Mar 5, 2026
@Scottcjn Scottcjn merged commit 96c3b97 into Scottcjn:master Mar 5, 2026
3 checks passed
@Scottcjn
Copy link
Owner

Scottcjn commented Mar 5, 2026

Merged! 2 RTC sent to xunwen-art wallet (pending 584). Clean test PRs, exactly what we need. Keep them coming!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) size/M PR: 51-200 lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants