Angstrom review / refactor

[EndymionJkb]

Description

Built on the original PR #1, this adds a ton of documentation (had to deep dive to understand and be able to review it properly), and also does a fair bit of refactoring. Specifically:

1. Adds a BaseTest layer and helpers (e.g., to make an account a node), saving a lot of duplicate code
2. Renames internal variables to be more specific about what they mean
3. Retains error names (even though I really don't like them!) for compatibility
4. Separates things that were combined together; e.g., using modifiers for simple checks, encapsulating the block setting in unlockAngstrom, and taming all the various unlock functions for clarity, mostly meaning renaming them semantically.
5. Providing a single external manual unlock
6. Adding getters for internal state
7. No semantic changes to tests - just incorporating the base test and helpers

Type of change

• Bug fix
• New feature
• Breaking change
• Dependency changes
• Code refactor / cleanup
• Optimization: [ ] gas / [ ] bytecode
• Documentation or wording changes
• Other

Checklist:

• The diff is legible and has no extraneous changes
• Complex code has been commented, including external interfaces
• Tests have 100% code coverage
• The base branch is either main, or there's a description of how to merge

Issue Resolution

[joaobrunoah]

Nice! For sure it's better organized.

[joaobrunoah]

Hmm, nice!