Skip to content

Commit

Permalink
better debugging messages
Browse files Browse the repository at this point in the history
  • Loading branch information
Dustin J. Mitchell committed Jun 13, 2010
1 parent 9be91f5 commit 29f8105
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions slave/buildslave/test/fake/runprocess.py
Expand Up @@ -91,16 +91,17 @@ def __init__(self, builder, command, workdir, **kwargs):
if key in default_values:
if default_values[key] == kwargs[key]:
continue # default values are expected
msg.append('%s: expected default (%r), got %r' %
msg.append('%s: expected default (%r),\n got %r' %
(key, default_values[key], kwargs[key]))
else:
msg.append('%s: unexpected arg, value = %r' % (key, kwargs[key]))
elif key not in kwargs:
msg.append('%s: did not get expected arg' % (key,))
elif exp.kwargs[key] != kwargs[key]:
msg.append('%s: expected %r, got %r' % (key, exp.kwargs[key], kwargs[key]))
msg.append('%s: expected %r,\n got %r' % (key, exp.kwargs[key], kwargs[key]))
if msg:
msg.insert(0, 'did not get expected __init__ arguments:')
msg.insert(0, 'did not get expected __init__ arguments for\n '
+ " ".join(kwargs.get('command', ['unknown command'])))
self._expectations[:] = [] # don't expect any more instances, since we're failing
raise AssertionError("\n".join(msg))

Expand Down

0 comments on commit 29f8105

Please sign in to comment.