add unit tests for goodfirstissue

[rajatjindal]

add unit tests for goodfirstissue

[AdamT213]

I would like to work on this if you have not done so already. This is a really cool idea!

[rajatjindal]

Hi Adam,

thanks for taking the initiative.

sure go ahead. Let me know if I can help in someway.

Thanks
Rajat Jindal

[AdamT213]

I apologize for jumping the gun too quickly. Should have looked the repo over more closely. I'm not really familiar with golang so not sure I could make a meaningful contribution right now.

[Ewocker]

@rajatjindal It is currently not straight forward to write unit test for handler.go because of the golang submodule import. see openfaas/faas-cli#324
There are couple of options listed in the above issue, but I am not sure which one is suitable for this case. The easy way to go with will be not using any sub-modules, but it might not be the best solution.

[rajatjindal]

I see. may be we refactor it to use Dockerfile template to fix this issue.