pitt-crc / wrappers

User focused command line wrappers around Slurm
https://crc-pages.pitt.edu/wrappers/
GNU General Public License v3.0
1 stars 1 forks source link

Introduces utils module to support new architecture #175

Closed djperrefort closed 1 year ago

djperrefort commented 1 year ago

Version 0.4.x of the wrappers were originally developed while 0.3.x were still in production. To ensure compatibility the design and implementation was intentionally kept almost identical to version 0.3.x with the majority of improvements focused on packaging and general organization. Now that 0.4.x is the production version, we have more freedom to improve application architectures.

Issue #111 highlights the need for improved application test coverage. This will involve re-architecting some functionality into smaller/different functions that are more easily testable.

This PR introduces the utils module as a first step in refactoring the architecture. The shared command line functionality has been refactored for better SRP adherance and tests have been written for better coverage.

Migrating applications to this new module will happen over the course of several upcoming PRs.