Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: [CI-14390]: Added new enhancements to plugins/jira #26

Merged
merged 8 commits into from
Oct 4, 2024

Conversation

vanisrikanithi
Copy link
Contributor

@vanisrikanithi vanisrikanithi commented Sep 25, 2024

This review covers all Jira issues #21,#22,#23,#24 as per jira spec.
There are some commented lines which are used for debugging. once the review is approved, will remove them.

@vanisrikanithi vanisrikanithi changed the title JEN-1 commit Jira Issues changes [JIRA] JEN-1 Jira plugin support for the Issues Sep 26, 2024
Copy link
Member

@Ompragash Ompragash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, add unit tests for the changes in plugin_test.go and util_test.go respectively

plugin/plugin.go Outdated
State: state,
UpdateSequenceNumber: args.Build.Number,
References: []struct {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a check to make sure the branch and URI values exist before adding them to the References so it'll only be added when the necessary values are available

plugin/plugin.go Outdated Show resolved Hide resolved
plugin/util.go Outdated Show resolved Hide resolved
plugin/util.go Show resolved Hide resolved
@Ompragash Ompragash changed the title [JIRA] JEN-1 Jira plugin support for the Issues feat: [CI-14390]: Added new enhancements to plugins/jira Sep 26, 2024
@vanisrikanithi
Copy link
Contributor Author

also, add unit tests for the changes in plugin_test.go and util_test.go respectively

util_test.go updated with unit tests for new functions. In plugin.go not implemented any new

also, add unit tests for the changes in plugin_test.go and util_test.go respectively
Added test cases for all the new apis that are implemented in util_test.go. There are no new APIs are implemented in plugin.go

@vanisrikanithi
Copy link
Contributor Author

Review comments addressed and updated the new patch.

plugin/plugin.go Outdated Show resolved Hide resolved
plugin/util.go Outdated Show resolved Hide resolved
@vanisrikanithi
Copy link
Contributor Author

@Ompragash @jamie-harness I have addressed the review comments. Please take a look at the latest changes.

plugin/plugin.go Outdated Show resolved Hide resolved
plugin/plugin.go Show resolved Hide resolved
plugin/plugin.go Outdated Show resolved Hide resolved
plugin/util.go Outdated Show resolved Hide resolved
plugin/util.go Outdated Show resolved Hide resolved
plugin/util.go Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants