Skip to content

Conversation

@cyberbono3
Copy link

@cyberbono3 cyberbono3 commented Oct 22, 2025

This PR adds 2 integrations tests:

  • TestDefaultRPC_Integration_GetAppBlock
  • TestDefaultRPC_Integration_GetTransactionsByBlock

RPC methods have introduced in Add rpc block methods PR

@cyberbono3 cyberbono3 changed the title TestDefaultRPC_Integration_GetAppBlock Add integration tests for GetAppBlock and GetTransactionsByBlocknumber Oct 23, 2025
@cyberbono3 cyberbono3 changed the title Add integration tests for GetAppBlock and GetTransactionsByBlocknumber integration tests for GetAppBlock and GetTransactionsByBlocknumber Oct 23, 2025
@cyberbono3 cyberbono3 changed the title integration tests for GetAppBlock and GetTransactionsByBlocknumber Integration tests for GetAppBlock and GetTransactionsByBlocknumber Oct 23, 2025
@cyberbono3 cyberbono3 marked this pull request as ready for review October 23, 2025 15:21
@cyberbono3 cyberbono3 requested review from 0xaigen and JekaMas October 26, 2025 15:16
@github-actions
Copy link

Code Metrics Report

#10 (a27fc80) #10 (ad14d38) +/-
Coverage 35.3% 35.3% 0.0%
Test Execution Time 6s 2s -4s
Details
  |                     | #10 (a27fc80) | #10 (ad14d38) | +/-  |
  |---------------------|---------------|---------------|------|
  | Coverage            |         35.3% |         35.3% | 0.0% |
  |   Files             |            11 |            11 |    0 |
  |   Lines             |           368 |           368 |    0 |
  |   Covered           |           130 |           130 |    0 |
+ | Test Execution Time |            6s |            2s |  -4s |

Code coverage of files in pull request scope (34.6% → 34.6%)

Files Coverage +/- Status
application/block.go 0.0% 0.0% modified
application/genesis.go 74.0% 0.0% modified
application/state_transition.go 0.8% 0.0% modified
cmd/main.go 75.8% 0.0% modified

Reported by octocov

@0xKrishna 0xKrishna closed this Oct 31, 2025
@github-actions github-actions bot mentioned this pull request Nov 6, 2025
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.

5 participants