[ecash-coinselect] Fix byte count calculation logic

Summary:
T3225

This diff fixes bytecount related logic and unit tests, including contextual comments.

Test Plan: npm test

Reviewers: Fabien, bytesofman, #bitcoin_abc

Reviewed By: Fabien, #bitcoin_abc

Differential Revision: https://reviews.bitcoinabc.org/D14380