-
Couldn't load subscription status.
- Fork 22
Description
With the merging of #43, we are now in a position to implement tests for all modules to improve stability. The initial PR did not add tests for all pre-existing modules, so we will have to make a concerted effort to do so. Current modules missing tests are as follows (I have loosely assigned some modules, please feel free to complain if I have done so incorrectly):
- gtk (Assigned to me)
- alacritty (Assigned to me)
- bottom
- fuzzel programs/fuzzel: add test file #75
- gammastep programs/gammastep: add hooks option & add tests #64
- ghostty (Assigned to @nezia1)
- git (Assigned to me)
- helix
- hypridle
- hyprland (Assigned to @nezia1)
- hyprlock
- keepassxc
- lsd
- obs-studio (Assigned to me)
- spotify-player (Assigned to me)
- starship
- tofi
- vscode (Assigned to me)
- zsh (@nezia1 programs/zsh: rewrite the zsh module #53)
Why it's important to implement tests for these modules as a concerted effort rather than as they are modified: anytime the library is modified, all tests are ran due to widespread reliance. Additionally, any PR that has treewide changes would need to run many tests. Without widespread testing, things may break without us realizing due to our reliance on tests.