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.

Sign up for GitHub

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

Jump to bottom

Add testing #77

Open
1 of 3 tasks
cseickel opened this issue Jan 18, 2022 · 1 comment
Open
1 of 3 tasks

Add testing #77

cseickel opened this issue Jan 18, 2022 · 1 comment
Labels
build Used for CI, linting, testing, and other dev tools.

Comments

@cseickel
Copy link
Contributor

cseickel commented Jan 18, 2022 •
edited

We really need to add a full suite of tests to this project, which can be run both locally and as a merge check. A handful of tests were created in #61 using plenary, which looks pretty good.

What needs to be tested?

  • Every documented config option should do what it says
  • Every bug fix should include a regression test (maybe just for future fixes?)
  • All variations of vim commands (:NeoTree...) should do what the documentation says.
@cseickel cseickel added the build Used for CI, linting, testing, and other dev tools. label Jan 18, 2022
@levouh
Copy link
Contributor

levouh commented Jan 29, 2022

I'll be working on #61 this weekend, and with the new source design it should be pretty trivial. Right now that one focuses on unit tests, however you can do functional tests with plenary's version of busted as well.

@levouh levouh mentioned this issue Jan 30, 2022
Merged
cseickel added a commit that referenced this issue Jan 30, 2022
@cseickel
test: add clear_all_state() method for use in test cases, part of #77
d1d24b1
@cseickel cseickel mentioned this issue Feb 3, 2022
Closed
cseickel added a commit that referenced this issue Feb 4, 2022
@cseickel
test: add command tests, add gh action to run tests on PRs, part of #77
0cb9a12
cseickel added a commit that referenced this issue Feb 4, 2022
@cseickel
test: add command tests, add gh action to run tests on PRs, part of #77
ae1ac1f
cseickel added a commit that referenced this issue Feb 4, 2022
@cseickel
test: add command tests, add gh action to run tests on PRs, part of #77…
1e1bca3 
… (#122)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
build Used for CI, linting, testing, and other dev tools.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cseickel @levouh