I'd somehow missed the existence of fail() when I wrote these. It's
preferable here so you don't get the somewhat useless "false is not
true" message in the case of failure.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
bbappendfile = bbappend
break
else:
- self.assertTrue(False, 'bbappend for recipe %s does not seem to be created in test layer' % testrecipe)
+ self.fail('bbappend for recipe %s does not seem to be created in test layer' % testrecipe)
return bbappendfile
def _create_temp_layer(self, templayerdir, addlayer, templayername, priority=999, recipepathspec='recipes-*/*'):
self.assertIn('add-file.patch', line, 'Unexpected warning found in output:\n%s' % line)
break
else:
- self.assertTrue(False, 'Patch warning not found in output:\n%s' % output)
+ self.fail('Patch warning not found in output:\n%s' % output)
@testcase(1188)
def test_recipetool_appendfile_script(self):
bbappendfile = f
break
if not bbappendfile:
- self.assertTrue(False, 'No bbappend file created')
+ self.fail('No bbappend file created')
runCmd('rm -rf %s/recipes-*' % self.templayerdir)
return bbappendfile