Skip to content

Added more tests for path completion and shell command completion#160

Merged
tleonhardt merged 3 commits intomasterfrom
unit_tests
Jul 1, 2017
Merged

Added more tests for path completion and shell command completion#160
tleonhardt merged 3 commits intomasterfrom
unit_tests

Conversation

@tleonhardt
Copy link
Copy Markdown
Member

Also:

  • Added a test for do_pause
  • Cleaned up a bunch of magic numbers from existing test for shell and path command completion

This closes #109

Also:

- Added a test for do_pause
- Cleaned up a bunch of magic numbers from existing test for shell and path command completion
Windows doesn't have an "ls" command by default ...
It passes on my Windows 10 VM, but fails on AppVeyor.  But manual testing reveals the feature is working.
@tleonhardt tleonhardt added this to the 0.7.4 milestone Jul 1, 2017
@tleonhardt tleonhardt self-assigned this Jul 1, 2017
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 1, 2017

Codecov Report

Merging #160 into master will increase coverage by 0.69%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
+ Coverage   90.23%   90.93%   +0.69%     
==========================================
  Files           1        1              
  Lines        1147     1147              
==========================================
+ Hits         1035     1043       +8     
+ Misses        112      104       -8
Impacted Files Coverage Δ
cmd2.py 90.93% <0%> (+0.69%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4ded9f...e968c41. Read the comment docs.

@tleonhardt tleonhardt merged commit 79d5b67 into master Jul 1, 2017
@tleonhardt tleonhardt deleted the unit_tests branch July 1, 2017 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add some unit tests related to tab completion

1 participant