feat: Added --no-input option to disable standard input

[serendicoder]

Fixes #1030

[coveralls]

Coverage remained the same at 100.0% when pulling 3359503 on serendicoder:web-ext-fix-1030 into 06ce2a2 on mozilla:master.

[kumar303]

Thanks! When I filed the issue, I didn't realize it was such a small change :) Hopefully I'm not forgetting something else that needs changing.

[serendicoder]

@kumar303 , since you had mentioned adding noInput parameter in #1028 and not hooking it up in yargs, I made these PR changes. I wanted to know if tests are done for this feature. I am still new to the codebase and will bring up any issue, if noticed. Thanks.

[kumar303]

I wanted to know if tests are done for this feature.

Yes, I think they are. We don't really have tests for the configuration of option parameters, unfortunately. We have a few integration tests but they are awkward and I wouldn't suggest adding one to test this feature. We need to improve web-ext in this regard -- i.e. introduce a way to test option configuration. I have some ideas like #65 .

[caitmuenster]

Thanks for your patch, @serendicoder! Your contribution has been added to our recognition wiki and your Mozillians profile has been vouched. :)

Welcome onboard! I look forward to seeing you around.

[serendicoder]

Hi @caitmuenster , I changed my profile settings so my Mozillian profile is at https://mozillians.org/en-US/u/serendicoder/
Could you please vouch the same :) Thanks again!

[caitmuenster]

Hi @serendicoder! No problem -- that profile has just been vouched for. :)