I think FakeTransaction in index.ts should be completely exposed as FakeTransaction and not just a shortened Fake. This name is not descriptive enough - this could point to various things - and especially in the context of tx instantiation we should be rather more precise. This is also confusing when looking at the documentation and seeing the fully written class name.
I thought twice about renaming the file names in src and test as well but then concluded this to be unnecessary and decided against. Can nevertheless update this as well if considered to be more consistent.
I think
FakeTransactioninindex.tsshould be completely exposed asFakeTransactionand not just a shortenedFake. This name is not descriptive enough - this could point to various things - and especially in the context of tx instantiation we should be rather more precise. This is also confusing when looking at the documentation and seeing the fully written class name.I thought twice about renaming the file names in
srcandtestas well but then concluded this to be unnecessary and decided against. Can nevertheless update this as well if considered to be more consistent.