Ledger* tests #130
Ledger* tests #130
Conversation
- More refactoring - Record Account* transactions in the storage - Do not record a transaction when importing the initial state
| @@ -1,22 +1,27 @@ | |||
| use std::{ | |||
There was a problem hiding this comment.
Let's avoid mod.rs files. There's no style guide for it really, but the Rust-lang team moved away from it and prefers having x.rs and x/y.rs instead of mod files.
cf rust-lang/rust#53125 and https://doc.rust-lang.org/book/ch07-05-separating-modules-into-different-files.html#alternate-file-paths
There was a problem hiding this comment.
https://doc.rust-lang.org/book/ch11-03-test-organization.html#submodules-in-integration-tests
To avoid having common appear in the test output, instead of creating tests/common.rs, we'll create tests/common/mod.rs. This is an alternate naming convention that Rust also understands. Naming the file this way tells Rust not to treat the common module as an integration test file. When we move the setup function code into tests/common/mod.rs and delete the tests/common.rs file, the section in the test output will no longer appear. Files in subdirectories of the tests directory don't get compiled as separate crates or have sections in the test output.
Code Coverage ReportCoverage for this commit
Coverage Report
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Integration tests for
I added support for Account* transactions recording in the storage and did some more refactoring.
I also fixed the HTML coverage output to keep only our own stats.