selftest/devtool: Compare sets instead of arrays on AssertEqual
Sets are safer when comparing internal elements and positions are not important. This commit avoid errors observed on python3 builds as reported on the below bugzilla entry. [YOCTO #9661] (From OE-Core rev: f6df164d09a4d4cf58977bf6cc0bc4f4bc71183a) Signed-off-by: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
3b39971748
commit
89d40a1f93
|
@ -818,28 +818,28 @@ class DevtoolTests(DevtoolBase):
|
|||
|
||||
# Check bbappend contents
|
||||
result = runCmd('git rev-parse HEAD', cwd=tempsrcdir)
|
||||
expectedlines = ['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n']
|
||||
expectedlines = set(['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n'])
|
||||
with open(bbappendfile, 'r') as f:
|
||||
self.assertEqual(expectedlines, f.readlines())
|
||||
self.assertEqual(expectedlines, set(f.readlines()))
|
||||
|
||||
# Check we can run it again and bbappend isn't modified
|
||||
result = runCmd('devtool update-recipe -m srcrev %s -a %s' % (testrecipe, templayerdir))
|
||||
with open(bbappendfile, 'r') as f:
|
||||
self.assertEqual(expectedlines, f.readlines())
|
||||
self.assertEqual(expectedlines, set(f.readlines()))
|
||||
# Drop new commit and check SRCREV changes
|
||||
result = runCmd('git reset HEAD^', cwd=tempsrcdir)
|
||||
result = runCmd('devtool update-recipe -m srcrev %s -a %s' % (testrecipe, templayerdir))
|
||||
self.assertFalse(os.path.exists(os.path.join(appenddir, testrecipe)), 'Patch directory should not be created')
|
||||
result = runCmd('git rev-parse HEAD', cwd=tempsrcdir)
|
||||
expectedlines = ['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n']
|
||||
expectedlines = set(['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n'])
|
||||
with open(bbappendfile, 'r') as f:
|
||||
self.assertEqual(expectedlines, f.readlines())
|
||||
self.assertEqual(expectedlines, set(f.readlines()))
|
||||
# Put commit back and check we can run it if layer isn't in bblayers.conf
|
||||
os.remove(bbappendfile)
|
||||
result = runCmd('git commit -a -m "Change the Makefile"', cwd=tempsrcdir)
|
||||
|
@ -848,12 +848,12 @@ class DevtoolTests(DevtoolBase):
|
|||
self.assertIn('WARNING: Specified layer is not currently enabled in bblayers.conf', result.output)
|
||||
self.assertFalse(os.path.exists(os.path.join(appenddir, testrecipe)), 'Patch directory should not be created')
|
||||
result = runCmd('git rev-parse HEAD', cwd=tempsrcdir)
|
||||
expectedlines = ['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n']
|
||||
expectedlines = set(['SRCREV = "%s"\n' % result.output,
|
||||
'\n',
|
||||
'SRC_URI = "%s"\n' % git_uri,
|
||||
'\n'])
|
||||
with open(bbappendfile, 'r') as f:
|
||||
self.assertEqual(expectedlines, f.readlines())
|
||||
self.assertEqual(expectedlines, set(f.readlines()))
|
||||
# Deleting isn't expected to work under these circumstances
|
||||
|
||||
@testcase(1370)
|
||||
|
|
Loading…
Reference in New Issue