fix(lwcomponent): better JSON handling components

[hazedav]

Summary

When attempting to json.Unmarshal the lwcomponent.Component object several quirks are present.

{
    "name": "abc",
    "description": "def",
    "type": "BINARY",
    "version": {},
    "artifacts": [
        {
            "os": "darwin",
            "arch": "amd64",
            "url": "",
            "signature": ""
        }
    ],
    "breadcrumbs": {},
    "command_name": ""
}

1. Some items should be omitted if empty (i.e. url and command_name)
2. version should be a string not a dict

How did you test this change?

Unit tests added

Issue

https://lacework.atlassian.net/browse/ALLY-1229

[afiune]

We need this field.

[hazedav]

Please see MarshalJSON() and UnmarshalJSON() methods for Component which properly handles this field via LatestVersionString. Also TestJSON shows this properly being marshaled and unmarshaled.

[afiune]

Maybe:

Suggested change

	LatestVersion semver.Version `json:"-"`
	LatestVersion semver.Version `json:"version,omitempty"`

[hazedav]

Please see MarshalJSON() and UnmarshalJSON() methods for Component which properly handles this field via LatestVersionString. Also TestJSON shows this properly being marshaled and unmarshaled.

[afiune]

You're way to smart!

[afiune]

Way to smart!!

[hazedav]

Thanks for the speedy review @afiune