Add unit tests for move2kube-api

[ashokponkumar]

Use the go's unit test framework similar to https://github.com/konveyor/move2kube/blob/master/types/info/versioninfo_test.go and write unit-tests for move2kube-api.

[IsabellaDev]

I'd like to try on this, but I'm new to Golang, could you please give me some hint like which functions need a test function? I checked your sample and noticed that not all functions have a test one. Thanks.

[ashokponkumar]

@HarikrishnanBalagopal Can you give some high level guidance on how we could test this package?

[nasirhm]

I would like to work on this @ashokponkumar

[ashokponkumar]

I would like to work on this @ashokponkumar

Sure @nasirhm .

[nasirhm]

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

[ashokponkumar]

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

@HarikrishnanBalagopal Can you please look into @nasirhm 's question.

[HarikrishnanBalagopal]

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

Hi @nasirhm , Thanks for looking into this issue.
konveyor/move2kube#881
You can look at the above for some tutorials and steps on how to get started writing tests.

The types package you mentioned is a good place to start since it has small functions which you can test individually.
Please let us know if you get stuck.

[satyazzz123]

@ashokponkumar @HarikrishnanBalagopal can I work on this issue?

[satyazzz123]

@HarikrishnanBalagopal @ashokponkumar i have raised a PR. Please have a look