feat: added getInflationReward method

[kadern0]

Fixes #316

Signed-off-by: Pablo Caderno kaderno@gmail.com

[codecov]

Codecov Report

Merging #317 (899cbe4) into master (32119e6) will decrease coverage by 0.16%.
The diff coverage is 28.57%.

❗ Current head 899cbe4 differs from pull request most recent head 4b70121. Consider uploading reports for the commit 4b70121 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
- Coverage   90.37%   90.20%   -0.17%     
==========================================
  Files          34       34              
  Lines        2555     2562       +7     
==========================================
+ Hits         2309     2311       +2     
- Misses        246      251       +5     

[kevinheavey]

Can you add tests to tests/integration/test_async_http_client.py and tests/integration/test_http_client.py please? Imitating an existing test should suffice

[kadern0]

@kevinheavey, I've added the test code but I'm unable to make it pass. Could you have a look?

[kevinheavey]

@kadern0 maybe try it with Processed commitment?

[kadern0]

@kevinheavey, using Processed I get:

resp = InvalidParamsMessage {
    message: "Method does not support commitment below `confirmed`",
}

And with Confirmed or Finalized I get:

raw = '{"jsonrpc":"2.0","error":{"code":-32004,"message":"Block not available for slot 8192"},"id":0}\n'
parser = <class 'solders.rpc.responses.GetInflationRewardResp'>

[kevinheavey]

@kadern0 does it work if you sleep for a few seconds? I think the local validator perhaps hasn't confirmed any blocks by the time we call the method https://docs.rs/solana-rpc/latest/src/solana_rpc/rpc.rs.html#551

[michaelhly]

superseded by #413