[WIP] [Persistence] TreeStore Module Tests

[dylanlott]

Description

Adds tests for the TreeStore module

Summary generated by Reviewpad on 21 Jun 23 23:39 UTC

This pull request includes changes that affect multiple packages in the persistence module.

• A new file named transaction.go has been added to the package that defines a Coordinator struct and a Tx struct.
• Changes to the existing indexer package include the addition of atomic transaction methods to the TxIndexer interface.
• Changes to the treeStore struct and its methods include the addition of a new field and renaming of a method.

In addition, changes have been made to individual files that add new methods, implement unit tests, and remove unused variables.

Issue

Relates to #756

Type of change

Please mark the relevant option(s):

• [ ] New feature, functionality or library
• [ ] Bug fix
• [x] Code health or cleanup
• [ ] Major breaking change
• [ ] Documentation
• [ ] Other

List of changes

• Adds a WithTxIndexer option function for configuring the tree module
• Adds a test for creating a TreeStoreModule
• Adds tests for the Update function
• Lays the ground work for further detailed state hash tests

Testing

• [ ] make develop_test; if any code changes were made
• [ ] make test_e2e on k8s LocalNet; if any code changes were made
• [ ] e2e-devnet-test passes tests on DevNet; if any code was changed
• [ ] Docker Compose LocalNet; if any major functionality was changed or introduced
• [ ] k8s LocalNet; if any infrastructure or configuration changes were made

Required Checklist

• [ ] I have performed a self-review of my own code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have added, or updated, godoc format comments on touched members (see: tip.golang.org/doc/comment)
• [ ] I have tested my changes using the available tooling
• [ ] I have updated the corresponding CHANGELOG

If Applicable Checklist

• [ ] I have updated the corresponding README(s); local and/or global
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
• [ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

[codecov[bot]]

Codecov Report

Patch coverage: 70.00% and project coverage change: +1.06 :tada:

Comparison is base (113e3be) 30.80% compared to head (9a88f1b) 31.87%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## tree-store-module #836 +/- ## ===================================================== + Coverage 30.80% 31.87% +1.06% ===================================================== Files 109 109 Lines 9245 9253 +8 ===================================================== + Hits 2848 2949 +101 + Misses 6057 5932 -125 - Partials 340 372 +32 ``` | [Impacted Files](https://app.codecov.io/gh/pokt-network/pocket/pull/836?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pokt-network) | Coverage Δ | | |---|---|---| | [persistence/trees/module.go](https://app.codecov.io/gh/pokt-network/pocket/pull/836?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pokt-network#diff-cGVyc2lzdGVuY2UvdHJlZXMvbW9kdWxlLmdv) | `68.51% <66.66%> (+68.51%)` | :arrow_up: | | [persistence/trees/trees.go](https://app.codecov.io/gh/pokt-network/pocket/pull/836?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pokt-network#diff-cGVyc2lzdGVuY2UvdHJlZXMvdHJlZXMuZ28=) | `38.78% <100.00%> (+38.78%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

[dylanlott]

Superseded by #861