add tests for resource JSON marshalling

[willnorris]

see 5f1c20a for more details

[palash25]

Hi @willnorris I would like to take this up. I will start with projects_test.go will make a PR soon.

[palash25]

This can be labeled a good-first-issue since beginners can work on writing test for different resources at the same time.

[jamesgeorge007]

@palash25 @gmlewis I would like to work on what is left further. Also, I'm not familiar with the code-base.
It would be great if you could guide me on resolving this issue 👍

[gmlewis]

Hi @jamesgeorge007! Thank you for your interest in helping out.

First, you will want to read our CONTRIBUTING.md guide.

Second, you will want to get familiar with the code by reading through it and trying it out.

Then, I suggest following the link to the original issue where it is explained what is needed in more detail.

Thanks again, and I look forward to seeing PRs from you.

[palash25]

@jamesgeorge007 hi 👋 just take a look at my commit referenced here and you will get the idea how to write the JSON test for different resources. Once it is clear just pick a resource that hasn't been covered by the tests and write a unit test for it 😄

[jamesgeorge007]

@palash25 As mentioned above I'm not familiar with the code-base. It would be great if you could suggest on what test case is to be written specifically.
Thank you

[palash25]

As mentioned above I'm not familiar with the code-base.

I have mentioned how to get familiar with the codebase. It would be great if you could re-read my previous comment, and the referenced issues and the PRs. 😄 and then follow the steps accordingly

[vaibhavsingh97]

@jamesgeorge007 Are you working on this issue? I would like to work on it 😅
I would love to start with pulls_comments_test.go

[palash25]

@vaibhavsingh97 I think there are multiple resources that need the JSON marhsalling test so if you start with pulls_comments_test.go I don't think there will be any problem since other contributors can pick up the left out resources. 👍