[JENKINS-70464] Test BuildParameterResolverExtensionTest
#286
Conversation
The defaultValue was the same as the value returned by the CustomObject class, so it was not clear which result was being returned. Changed the return value from the CustomObject class to be clearly different from the default value.
There was a problem hiding this comment.
This is a great pull request @abhishekmaity . Thanks very much! The coverage improvement is very nice.
I modified the last test in the suite with c7ca01f to make it more clear that the return value is provided by the CustomObject and not from the default value string included in the parameter definition.
Does that change make sense to you @abhishekmaity?
@MarkEWaite It brings more clarity and I'm okay with it. Thanks for improvising the PR. |
In response to #114,
Testing done
BuildParameterResolverExtensionTest.javaSubmitter checklist