Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add consensus node calling endpoints to postman tests #392

Closed
Nana-EC opened this issue Jul 29, 2022 · 0 comments · Fixed by #393
Closed

Add consensus node calling endpoints to postman tests #392

Nana-EC opened this issue Jul 29, 2022 · 0 comments · Fixed by #393
Assignees
@Nana-EC
Labels
enhancement New feature or request P2 process Build, test and deployment-process related tasks
Milestone
0.5.0

Comments

@Nana-EC
Copy link
Collaborator

Nana-EC commented Jul 29, 2022

Problem

Currently the postman tests are missing calls that query the consensus nodes.
This would be useful in quick sanity checks to verify relay operation as they do not require knowledge of accounts and blocks details

Solution

Add consensus node calling checks

  • eth_gasPrice
  • eth_feeHistory - use 3 last blocks

Fix base_url vs baseUrl inconsistency between the README.md and postman.js files

Alternatives

No response
@Nana-EC Nana-EC added enhancement New feature or request P2 process Build, test and deployment-process related tasks labels Jul 29, 2022
@Nana-EC Nana-EC added this to the 0.5.0 milestone Jul 29, 2022
@Nana-EC Nana-EC self-assigned this Jul 29, 2022
@Nana-EC Nana-EC mentioned this issue Jul 29, 2022
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Nana-EC Nana-EC

Labels
enhancement New feature or request P2 process Build, test and deployment-process related tasks
Projects
None yet
Milestone
0.5.0
Development

Successfully merging a pull request may close this issue.

Increase newman postman tests coverage hashgraph/hedera-json-rpc-relay
1 participant
@Nana-EC