charslaw / Pando

Pando serializes and tracks the history of a state tree
MIT License
2 stars 0 forks source link

Reorganize test classes to avoid nested classes #78

Open schultzcole opened 2 years ago

schultzcole commented 2 years ago

Currently, most of our tests are organized so that for each class under test, there is a test class, and that test class has nested classes for each method or property under test.

It would be better, I think, to not use nested classes and instead use namespaces for the same purpose. The hierarchy would have the same structure, but with a namespace instead of a class. There might be some cases where we have shared data at the test class level, which would need to be moved to a helper class or something like that.

This change is beneficial because we can split the test classes for each method into separate files, and because it might bypass the issue in #45. Or maybe not