[oe-commits] [openembedded-core] 05/06: oe-selftest: devtool: fix error message in _test_recipe_contents()

[git at git.openembedded.org]

rpurdie pushed a commit to branch master
in repository openembedded-core.

commit 0dafcb158003fb13f82c266f607d9967fca321db
Author: Paul Eggleton <paul.eggleton at linux.intel.com>
AuthorDate: Mon Oct 31 16:59:48 2016 +1300

    oe-selftest: devtool: fix error message in _test_recipe_contents()
    
    If a variable is being set in the recipe when we've explicitly passed
    None as the value to _test_recipe_contents() indicating that it
    shouldn't be set at all, then we should be printing out the variable
    name in the assertion message but it seems like I forgot to do a
    substitution. Also include the value for informational purposes.
    
    Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>
    Signed-off-by: Ross Burton <ross.burton at intel.com>
---
 meta/lib/oeqa/selftest/devtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/devtool.py b/meta/lib/oeqa/selftest/devtool.py
index 713f2c5..71d205c 100644
--- a/meta/lib/oeqa/selftest/devtool.py
+++ b/meta/lib/oeqa/selftest/devtool.py
@@ -44,7 +44,7 @@ class DevtoolBase(oeSelfTest):
                 if var and var in checkvars:
                     needvalue = checkvars.pop(var)
                     if needvalue is None:
-                        self.fail('Variable %s should not appear in recipe')
+                        self.fail('Variable %s should not appear in recipe, but value is being set to "%s"' % (var, value))
                     if isinstance(needvalue, set):
                         if var == 'LICENSE':
                             value = set(value.split(' & '))

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.