We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
At the moment metalctl completely lacks tests. Which makes testing big PRs(example) quite a routine.
metalctl
Intergration tests would be a great addition. Another, lighter, solution would be adding unit tests with using metal-go mock client.
metal-go
This issue seems to be quite massive to solve it in one PR. Solving it by parts(smaller PRs adding small test coverage) should be more feasible.
cc @Gerrit91
The text was updated successfully, but these errors were encountered:
All base commands now have unit test coverage (through #146).
Sorry, something went wrong.
No branches or pull requests
At the moment
metalctl
completely lacks tests. Which makes testing big PRs(example) quite a routine.Intergration tests would be a great addition. Another, lighter, solution would be adding unit tests with using
metal-go
mock client.This issue seems to be quite massive to solve it in one PR. Solving it by parts(smaller PRs adding small test coverage) should be more feasible.
cc @Gerrit91
The text was updated successfully, but these errors were encountered: