Skip to content

WIP - test for cli args - #70

Open
lholmquist wants to merge 6 commits into
nodejs:mainfrom
lholmquist:69-test-for-cli-args
Open

WIP - test for cli args#70
lholmquist wants to merge 6 commits into
nodejs:mainfrom
lholmquist:69-test-for-cli-args

Conversation

@lholmquist

Copy link
Copy Markdown
Contributor

fixes#69

This PR is for adding in tests for the commands of the cli.

It is currently a WIP, and more commits will be added to it. Probably 1 commit per command

* use chunk instead of compiledData
* store the common cmd in a variable for use in other tests
@codecov

codecovBot commented Jul 17, 2019

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@ Coverage Diff @@## master #70 +/- ##
==========================================
+ Coverage 57.14% 60.49% +3.34% 
==========================================
Files 18 18 Lines 448 448 ==========================================
+ Hits 256 271 +15 + Misses 192 177 -15
Impacted FilesCoverage Δ
bin/cmd.js41.57% <0%> (+12.35%)⬆️
lib/utils.js70.96% <0%> (+12.9%)⬆️

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 86b9053...d8695c0. Read the comment docs.

@lholmquist

Copy link
Copy Markdown
ContributorAuthor

The changes in #73 will affect this PR a little bit, so just putting this on hold until that one is sorted out

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Tests for command line arguments

1 participant

@lholmquist