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

Implement testing suite #66

Open
jwu910 opened this issue Apr 2, 2018 · 8 comments
Open

Implement testing suite #66

jwu910 opened this issue Apr 2, 2018 · 8 comments
Labels
enhancement New feature or request pinned Pinned issue question Further information is requested

Comments

@jwu910
Copy link
Owner

jwu910 commented Apr 2, 2018

Need find and implement a testing suite to test utility functions.

I have used Jest for web app projects, but not for CLI tools.
@jwu910 jwu910 added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Apr 2, 2018
@jwu910
Copy link
Owner Author

jwu910 commented Apr 2, 2018

Maybe https://mochajs.org/

@jwu910 jwu910 removed the help wanted Extra attention is needed label Jun 29, 2018
@jwu910 jwu910 added this to the Release 1.0.0 milestone Jul 18, 2018
@jwu910 jwu910 added the help wanted Extra attention is needed label Jul 18, 2018
@jwu910 jwu910 removed this from the Release 1.0.0 milestone Jul 19, 2018
@stale
Copy link

stale bot commented Sep 17, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Sep 17, 2018
@jwu910 jwu910 added pinned Pinned issue and removed wontfix This will not be worked on labels Sep 17, 2018
@jwu910 jwu910 removed the help wanted Extra attention is needed label Oct 1, 2018
@dhuang612
Copy link

dhuang612 commented Oct 28, 2019 •
edited
Loading

I can take this on, and add Mocha tests to this project

@tawfiek
Copy link

tawfiek commented Dec 26, 2020

I am willing to work on this issue if no one already dose.

@jwu910
Copy link
Owner Author

jwu910 commented Dec 27, 2020

@tawfiek sounds good, been a stale issue for a long time, haven't had time to look into it as I haven't had time to work on much lately.

Thanks!

@tawfiek
Copy link

tawfiek commented Dec 27, 2020 •
edited
Loading

Never mind,
I think it's a great tool and need to be well maintained to take a better chance in the community.
I'll have a deeper look into the code base over the weekend (FRI, SAT) to start write some tests.

Is there any certain technologies you prefer to work with for test assertion, runner and coverage ?!

@jwu910

@jwu910
Copy link
Owner Author

jwu910 commented Jan 4, 2021

No preference, I have seen some CLI tools tested with Ava, but I have not had time to look into that. Be happy to help discuss any findings

tawfiek added a commit to tawfiek/check-it-out that referenced this issue Feb 11, 2021
@tawfiek
test: config test and adding structure
c99e037 
configure mocha for testing and adding NYC as test coverage tool adding the file structure to start
writing tests

fix jwu910#66
@tawfiek
Copy link

tawfiek commented Feb 14, 2021

@jwu910 action needed

tawfiek added a commit to tawfiek/check-it-out that referenced this issue Feb 18, 2021
@tawfiek
ci(Travis): add lint command into travis scripts
682a493 
jwu910#66
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request pinned Pinned issue question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jwu910 @tawfiek @dhuang612