updated getAliasesforAddress to throw error if address endpoint returns an error response
updated error handling unit test accordingly

rebase

• updated getAliasDetails to throw error if alias endpoint returns an error response
• updated error handling unit tests accordingly

Updated unit test to use the updated isValidStoredWallet validation function from D14190

rebase

@bot cashtab-tests

mb a rebase issue?

fixed typos

Will update response handling once D14193 is propagated to alias.etokens.cash.

updated comments

Updated unit tests

updated unit tests

Upgraded ecashaddrjs

Updated chronik websocket tests to specifically differentiate state between AddedToMempool and BlockConnected events

globalized the fetch() mock override, fixed linting and unit test issues

We'll be using this in many scripts, so it should be kept in its own file. e.g. create a config.js and export this here

We'll be using this in many scripts, so it should be kept in its own file. e.g. create a config.js and export this here

removed chronik url from input args

removed chronik url from input args

updated comments

resolved merge conflicts in package-lock.json

added contextual comments

updated address decoding call, new unit test for an address with no utxos and added contextual comments

TIL chronik does this...I've been using chronik.tx(tokenId)
Is there a difference in the response?

updated chronik mock and package.json

Not worth an individual diff + unit tests on a library, will use directly in txhistory/utxo retrieval examples

wrapping ecashaddr.decode call

In D14146#316296, @teamcity wrote:

Build Bitcoin ABC Diffs / Diff Testing (app-dev-examples) failed.

switching to assert.rejects() for exception validation

Looking into why the tests passes locally but phab is complaining about a validation error which is what the unit tests were expecting...

Simplified ecashaddrjs usage and updated readme

updated error handling

The defaultCashtabCache structure is there to support the migration of legacy wallets and ensure they have the right cache attributes.
Will throw up a separate diff to address the antipattern in feedback.

• updated test file to align with naming convention
• added new test to ensure chronik errors are being thrown