Blockchain

Creating a new issue based on lbryio/lbry-sdk#877

• Update ensure_directory_exists() to check that the directory is writable by the current process.
• If it's not, return a useful error
• Add tests to cover the case when
  • download directory does not exist
  • download dir exists but is not writable
  • dir exists an

It's nice if testing doesn't produce unnecessary artifacts after running with success but keeps relevant info on fail for further inspection.

Pytest should have a nice way of creating fixtures that can post-process a test based on whether it failed or not https://docs.pytest.org/en/latest/example/simple.html#making-test-result-information-available-in-fixtures

If a test fails, then keep the

Abstract

After merging rotki/rotki#3443 there have been no tests written to deserialize fiat/orders and fiat/payments:
https://binance-docs.github.io/apidocs/spot/en/#get-swap-history-user_data

Relevant part of the tests code:
https://github.com/rotki/rotki/blob/2c9ed5d892c4c571b86edb701c80838b9222bff1/rotkehlchen/tests/exchanges/test_binance.py#L590-L598
https:

For example, see #878. The checks are unnecessary

When a lot of data has been added, it might be annoying to find the data you want to refund/report. It would be good to have an easy way to search for the events.

Search params can be put into the customFilter function that is passed to handleAddedData.

GetSignaturesForAddress now has an optional Commitment level parameter https://docs.solana.com/developing/clients/jsonrpc-api#getsignaturesforaddress.

Currently get_signatures_for_address supports before, until and limit (https://github.com/michaelhly/solana-py/blob/master/src/solana/rpc/api.py#L344) but it doesn't support passing in an optional Commitment level. It would be great to be able t

https://github.com/yearn/yearn-vaults/blob/a113064afd59ecac6ae4f8ed9bfe0d0d00c727a3/.github/workflows/lint.yaml#L98