path: remove redundant accountID from path string

[pinheadmz]

Fixes #484

However, as discussed with @nodar-chkuaselidze in slack, because this string gets written to the db this would break consistency with all keys saved before the change and possibly require a db migration.

This needs testing because it affects the API output, and may break downstream applications.

[nodech]

String does not get written to the database, account is.
Actually Path is being written.

[pinheadmz]

Yeah, here? https://github.com/bcoin-org/bcoin/blob/master/lib/wallet/walletdb.js#L1206-L1215

[nodech]

That is Path object, not an actual string. :)

[codecov-io]

Codecov Report

Merging #502 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #502      +/-   ##
==========================================
+ Coverage   61.93%   61.94%   +<.01%     
==========================================
  Files         156      156              
  Lines       26093    26085       -8     
==========================================
- Hits        16161    16158       -3     
+ Misses       9932     9927       -5

Impacted Files	Coverage Δ
lib/wallet/path.js	72.56% <100%> (ø)	⬆️
lib/wallet/nodeclient.js	80.39% <0%> (-0.38%)	⬇️
lib/wallet/walletdb.js	77.57% <0%> (+0.14%)	⬆️
lib/client/node.js	29.41% <0%> (+1.63%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3623d04...de05626. Read the comment docs.

[pinheadmz]

Rebased on master to double check test coverage