Skip to content
New issue

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

Add tests for core functionality #38

Open
reibitto opened this issue Jul 29, 2020 · 1 comment
Open

Add tests for core functionality #38

reibitto opened this issue Jul 29, 2020 · 1 comment

Comments

@reibitto
Copy link
Owner

There are no tests right now and that makes it really easy to break things. "Add tests" is too broad of an issue, so for now the focus should probably be the Command.search functionality. The function doesn't require a broad set of dependencies, so it should be pretty easy to set up tests for. Just create a Vector of arbitrary commands and aliases and then pass in a search string and make sure the results are correct.

@reibitto
Copy link
Owner Author

Initial work was merged with #51. The next step is writing tests for all the other commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant