Skip to content

Conversation

@tleonhardt
Copy link
Member

Added a couple unit tests for the tokens_for_completion() function in argcomplete_bridge.py.

This partially addresses #268.

This is a first pass at increasing code coverage of the relatively new and difficult to test argcomplete_bridge.py module within cmd2.

Added a couple unit tests for tokens_for_completion() function.
@tleonhardt tleonhardt added this to the 0.9.2 milestone May 31, 2018
@tleonhardt tleonhardt self-assigned this May 31, 2018
@tleonhardt tleonhardt requested a review from anselor May 31, 2018 05:30
@codecov
Copy link

codecov bot commented May 31, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #425      +/-   ##
==========================================
+ Coverage   88.97%   89.87%   +0.89%     
==========================================
  Files           9        9              
  Lines        2676     2676              
==========================================
+ Hits         2381     2405      +24     
+ Misses        295      271      -24
Impacted Files Coverage Δ
cmd2/argcomplete_bridge.py 44.21% <0%> (+25.26%) ⬆️

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 26c9219...8ca52ba. Read the comment docs.

@tleonhardt tleonhardt merged commit 9852270 into master Jun 1, 2018
@tleonhardt tleonhardt deleted the unit_tests branch June 1, 2018 03:09
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.

3 participants