mirror of
https://github.com/privatevoid-net/nix-super.git
synced 2024-11-22 22:16:16 +02:00
33ca905cdb
pararameterisation is not actually needed the way things are currently set up, and it confused me when trying to understand what the code does. all but one test sources vars-and-functions.sh, which nominally only defines variables, but in practice is always coupled with the actual initialisation. while the cleaner way of making this more legible would be to source variables and initialisation separately, this would produce a huge diff. the change requires a few small fixes to keep the tests working: - only create test home directory during initialisation that vars-and-functions.sh wrote to the file system seems not write - fix creation of the test directory due to statefulness, the test home directory was implicitly creating the test root, too. decoupling that made it apparent that this was probably not intentional, and certainly confusing. - only source vars-and-functions.sh if init.sh is not needed there is one test case that only needs a helper function but no initialisation side effects - remove some unnecessary cleanups and split parts of re-used test code there were confusing bits in how initialisation code was repurposed, which break if trying to refactor the outer layers naively... |
||
|---|---|---|
| .. | ||
| advanced-topics | ||
| architecture | ||
| command-ref | ||
| contributing | ||
| figures | ||
| installation | ||
| language | ||
| package-management | ||
| protocols | ||
| release-notes | ||
| store | ||
| _redirects | ||
| c-api.md | ||
| favicon.png | ||
| favicon.svg | ||
| glossary.md | ||
| introduction.md | ||
| quick-start.md | ||
| SUMMARY.md.in | ||
