Code review comment for lp:~roadmr/checkbox/1187216-check-job-format

Revision history for this message
Jeff Lane  (bladernr) wrote :

Just to be pedantic:

This job description:
plugin: user-interact
name: mediacard/mmc-foo
command: do_some_stuff.exe.bat.com
_description:
 PURPOSE:
    to flob the brats
 VERIFICATION:
    blab:

Results in this failure:
FAIL: test_verify_manual_jobs_have_required_fields (tests.test_message_files.MessageFileFormatTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/bladernr/development/daniel-job-desc-test/checkbox-old/checkbox/tests/test_message_files.py", line 122, in test_verify_manual_jobs_have_required_fields
    msg.format(message['name']))
AssertionError: False is not true : Missing verification in mediacard/mmc-foo

I am actually missing the "STEPS" part, not VERIFICATION.

This description returns the same error:
plugin: user-interact
name: mediacard/mmc-foo
command: do_some_stuff.exe.bat.com
_description:
 PURPOSE:
    to flob the brats
 STEPS:
    1: Steal Underpands!
    2: ??
    3: PROFIT!

only it IS missing the VERIFICATION part...

any chance to get it to say WHAT is missing?

review: Needs Fixing

« Back to merge proposal