Fixes #39411 - Display locked field as yes/no in job template info#65
Conversation
lhellebr
force-pushed
the
39411-fix-locked-field-boolean
branch
from
09c1840 to
9a8eb08
Compare
| out, _err = capture_io do | ||
| cmd.run(['--id=1']) | ||
| end | ||
| _(out).must_match(/Locked:\s+(yes|no)/) |
There was a problem hiding this comment.
Don't we know which option it will actually be?
There was a problem hiding this comment.
Yes, no :)
I wasn't sure whether to code this so the template can change, or hardcode it, and it seems you prefer hardcoding. I'll update it.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
lhellebr
force-pushed
the
39411-fix-locked-field-boolean
branch
from
9a8eb08 to
24afbbe
Compare
looks like copr is having a moment |
adamruzicka
left a comment
adamruzicka
left a comment
There was a problem hiding this comment.
This is good, let's wait for robottelo to bless this.
|
/packit build |
|
robobless: SatelliteQE/robottelo#21786 (comment) |
ofedoren
left a comment
ofedoren
left a comment
There was a problem hiding this comment.
Thanks, @lhellebr and @adamruzicka !
3 participants
The
lockedfield injob-template infowas missing theFields::Booleantype annotation, causing it to display rawtrue/falsevalues instead ofyes/nolike other template types (e.g. partition tables).Redmine: https://projects.theforeman.org/issues/39411