Fix for GetList Unmarshalling Issue #630
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently the GetList() function for IssueLinkTypes in OnPremise and Cloud fails to return the JSON document from the API. This is because it expects a top level array of subdocuments and not a top level document containing an array of subdocuments. I have corrected this behavior and revised the tests to ensure the behavior is correct. I have also tested this behavior against my own Jira OnPremise and Cloud instances.
What type of PR is this?
What this PR does / why we need it:
As currently implemented GetList() does not return the expected list of
IssueLinkType
s. Instead, the function call returns an empty list and an error if probed deeply enough. All tests pass against the unit tests which further cause confusion. However, directly calling the endpoint and verifying the output we can discern that the function expects a different JSON document shape than that which is returned (see the test mock of the output). This PR addresses the above behavior for tests, onpremise, and Cloud. It also hopes to verify the v2 push for each of these functions - but I'd like feedback on that from a Maintainer before I call it "good".Which issue(s) this PR fixes:
NA - I didn't file an issue - apologies if that is incorrect, I'll be sure to redress that if we prefer that method.
Fixes #
OnPremise and Cloud unmarshalling for IssueLinkTypes array.
Special notes for your reviewer:
I'd like to be more involved with the project if I may. However, I'd likely need some guidance especially around the goals/nongoals specifically to do with the push for v2 and vision. I was wondering if there might be any mentorship ability or a channel/communication medium for one to leverage for ad hoc questions to assist in ramping/working on the project?
Additional documentation e.g., usage docs, etc.:
I had a previous PR up but I did something odd with git while trying to clean up my commits - apologies for that error.
Thank you!