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.

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

Add tests #1

Open
3 tasks
bnb opened this issue Nov 5, 2018 · 18 comments · May be fixed by #205
Open
3 tasks

Add tests #1

bnb opened this issue Nov 5, 2018 · 18 comments · May be fixed by #205
Labels
dep: jest enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed tests About tests for the module
Milestone

Comments

@bnb
Copy link
Member

bnb commented Nov 5, 2018

The module is currently lacking tests. This is something that is 100% needed but also something I'm entirely unfamiliar with. Would 💚 some help on this!

Tasks:

  • good-first-issue -o exits appropriately
  • log returns a properly formatted output without any undefined (the repo name is especially tricky!)
  • `require('good-first-issue') operates and returns an object with the correct shape, as expected

Contributors: Feel free to add items to this list!

@bnb
Copy link
Member Author

bnb commented Nov 17, 2018

We're now using Jest!

Some needed tests:

  • good-first-issue <project> -o exits appropriately
  • log returns a properly formatted output without any undefined (the repo name is especially tricky!)
  • `require('good-first-issue') operates and returns an object with the correct shape, as expected

@bnb bnb added this to the 1.0.0 milestone Nov 17, 2018
@bnb bnb added tests About tests for the module enhancement New feature or request help wanted Extra attention is needed labels Nov 17, 2018
@hemal7735
Copy link
Collaborator

@bnb can we add task lists?
I've few comments over the first task. I doubt if it can be tested.
I've raised a question on that repo: sindresorhus/open#109

@bnb
Copy link
Member Author

bnb commented Nov 22, 2018

@hemal7735 I believe it may be on our end rather than opn's end – I less care about checking that the issue actually opened in a browser and more care that when passing the -o flag, the CLI actually exits.

Context: For a while, we weren't exiting and the CLI would continue running without actually doing anything, forcing the user to CMD + C to get out of it.

@bnb
Copy link
Member Author

bnb commented Nov 22, 2018

@hemal7735 and yes, feel free to add tasks!

@hemal7735
Copy link
Collaborator

hemal7735 commented Nov 23, 2018

@bnb yup, checking the process - exit code is doable. Thanks for the clarification.
Tricky part is to reproduce the issue.

@bnb
Copy link
Member Author

bnb commented Nov 23, 2018

I believe it was solved with 3cfb8e6 but I'd always like to maintain a check that it's exiting correctly 😬

@bnb bnb added the good first issue Good for newcomers label Dec 15, 2018
@good-first-pin good-first-pin bot pinned this issue Dec 15, 2018
@bnb bnb added the dep: jest label Dec 15, 2018
@JasonEtco JasonEtco mentioned this issue Dec 16, 2018
bnb pushed a commit that referenced this issue Feb 26, 2019
@maddhruv maddhruv unpinned this issue Aug 22, 2019
@engFelipeMonteiro
Copy link

I would like to contribute to this issue.

@alexanderbrown21
Copy link

anything happened so far with this issue? has this been resolved?

@iamtabrezkhan
Copy link

Hey @bnb

Is this still open? I would like to contribute. :)

@bnb
Copy link
Member Author

bnb commented Jun 12, 2020

@iamtabrezkhan feel free - there may have been some things done but improvements/enhancements always welcome.

@carlosserretquerol
Copy link

Hi, Im new at github and my experience qith code is from the last 2 years with some months of innactivity in between.

Is there any task I could do or somethings I need to learn if I only have a small experience in Js, Ruby, Java, C++, Python?

@bnb
Copy link
Member Author

bnb commented Oct 12, 2020

@carlosdeque-s feel free to poke around and try things out. Use the project, if you have an idea about how it could be better create an issue and I'll share feedback on it. If it's an affirmative, go ahead and start working on a PR - I can at the very least give you pointers in the right direction.

@zootechdrum
Copy link

hey @bnb Is this still a on going project ? The project seems really interesting.

@bnb
Copy link
Member Author

bnb commented Jan 12, 2021

@zootechdrum I don't actively work on it at this point for no reason other than I have other things I focus on. I could work on it; also happy to work with others to enable them to get improvements and additions shipped and eventually become a maintainer.

@p4nu p4nu linked a pull request May 29, 2022 that will close this issue
@ka1994ka
Copy link

هاي

@yabhinavy
Copy link

!assign

@adityashibu
Copy link

Hey everyone, I was wondering to get started with open source contribution. I'm an amateur in Python, OCaml, Javascript, HTML and CSS. Can anyone please explain how I can start working on this??

@AravindKVishwanath
Copy link

Hii everyone, I want to start open-source contribution. My skills are React, react native and I can also build basic backend for the applications. How can I start and continue open-source contribution?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dep: jest enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed tests About tests for the module
Projects
None yet
Development

Successfully merging a pull request may close this issue.