Only a few style suggestions if you care about them:
47 + '''Verify that the default jenkins_build_token value results in a
48 + token parameter of None being passed to jenkins.build_job API.'''
Test docstrings are used by some test runners under the assumption that the first line is short enough (<80 chars) and can be used to document the test purpose.
You can give a longer description when needed after an empty line.
I.e.:
'''No token sent if empty.
Verify that the default jenkins_build_token value results in a
token parameter of None being passed to jenkins.build_job API.
'''
Nice ! Cover letter + tests make reviews easy \o/
Only a few style suggestions if you care about them:
47 + '''Verify that the default jenkins_build_token value results in a
48 + token parameter of None being passed to jenkins.build_job API.'''
Test docstrings are used by some test runners under the assumption that the first line is short enough (<80 chars) and can be used to document the test purpose.
You can give a longer description when needed after an empty line.
I.e.:
'''No token sent if empty.
Verify that the default jenkins_build_token value results in a
token parameter of None being passed to jenkins.build_job API.
'''