diff options
| author | Ola x Nilsson <ola.x.nilsson@axis.com> | 2016-10-25 13:03:33 +0200 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-12-13 22:55:20 +0000 |
| commit | 276523149914a75a939278b2e577383a2fdf3813 (patch) | |
| tree | 3c321784a0902b290980562deeb93a1bf91a5d8e | |
| parent | 9170a88cbddd510e2cf93049af8825f20cb61426 (diff) | |
| download | poky-276523149914a75a939278b2e577383a2fdf3813.tar.gz | |
recipetool: selftest: Add test for recipetool plugin loading
Test that recipetool plugins are loaded in a well defined order.
(From OE-Core rev: 044de8424a454a7057906e44eb56e2134ebb17e4)
Signed-off-by: Ola x Nilsson <ola.x.nilsson@axis.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta-selftest/lib/recipetool/bbpath.py | 41 | ||||
| -rw-r--r-- | meta/lib/oeqa/selftest/recipetool.py | 44 |
2 files changed, 85 insertions, 0 deletions
diff --git a/meta-selftest/lib/recipetool/bbpath.py b/meta-selftest/lib/recipetool/bbpath.py new file mode 100644 index 0000000000..783b2dc769 --- /dev/null +++ b/meta-selftest/lib/recipetool/bbpath.py | |||
| @@ -0,0 +1,41 @@ | |||
| 1 | import argparse | ||
| 2 | |||
| 3 | already_loaded = False | ||
| 4 | register_count = 0 | ||
| 5 | |||
| 6 | def plugin_name(filename): | ||
| 7 | return os.path.splitext(os.path.basename(filename))[0] | ||
| 8 | |||
| 9 | def plugin_init(plugins): | ||
| 10 | global already_loaded | ||
| 11 | already_loaded = plugin_name(__file__) in (plugin_name(p.__name__) for p in plugins) | ||
| 12 | |||
| 13 | def print_name(opts): | ||
| 14 | print (__file__) | ||
| 15 | |||
| 16 | def print_bbdir(opts): | ||
| 17 | print (__file__.replace('/lib/recipetool/bbpath.py','')) | ||
| 18 | |||
| 19 | def print_registered(opts): | ||
| 20 | #global kept_context | ||
| 21 | #print(kept_context.loaded) | ||
| 22 | print ("1") | ||
| 23 | |||
| 24 | def multiloaded(opts): | ||
| 25 | global already_loaded | ||
| 26 | print("yes" if already_loaded else "no") | ||
| 27 | |||
| 28 | def register_commands(subparsers): | ||
| 29 | global register_count | ||
| 30 | register_count += 1 | ||
| 31 | |||
| 32 | def addparser(name, helptxt, func): | ||
| 33 | parser = subparsers.add_parser(name, help=helptxt, | ||
| 34 | formatter_class=argparse.ArgumentDefaultsHelpFormatter) | ||
| 35 | parser.set_defaults(func=func) | ||
| 36 | return parser | ||
| 37 | |||
| 38 | addparser('pluginfile', 'Print the filename of this plugin', print_name) | ||
| 39 | addparser('bbdir', 'Print the BBPATH directory of this plugin', print_bbdir) | ||
| 40 | addparser('count', 'How many times have this plugin been registered.', print_registered) | ||
| 41 | addparser('multiloaded', 'How many times have this plugin been initialized', multiloaded) | ||
diff --git a/meta/lib/oeqa/selftest/recipetool.py b/meta/lib/oeqa/selftest/recipetool.py index db1f8deeb0..bcc2b46188 100644 --- a/meta/lib/oeqa/selftest/recipetool.py +++ b/meta/lib/oeqa/selftest/recipetool.py | |||
| @@ -1,5 +1,6 @@ | |||
| 1 | import os | 1 | import os |
| 2 | import logging | 2 | import logging |
| 3 | import shutil | ||
| 3 | import tempfile | 4 | import tempfile |
| 4 | import urllib.parse | 5 | import urllib.parse |
| 5 | 6 | ||
| @@ -485,6 +486,47 @@ class RecipetoolTests(RecipetoolBase): | |||
| 485 | inherits = ['pkgconfig', 'autotools'] | 486 | inherits = ['pkgconfig', 'autotools'] |
| 486 | self._test_recipe_contents(recipefile, checkvars, inherits) | 487 | self._test_recipe_contents(recipefile, checkvars, inherits) |
| 487 | 488 | ||
| 489 | def _copy_file_with_cleanup(self, srcfile, basedstdir, *paths): | ||
| 490 | dstdir = basedstdir | ||
| 491 | self.assertTrue(os.path.exists(dstdir)) | ||
| 492 | for p in paths: | ||
| 493 | dstdir = os.path.join(dstdir, p) | ||
| 494 | if not os.path.exists(dstdir): | ||
| 495 | os.makedirs(dstdir) | ||
| 496 | self.track_for_cleanup(dstdir) | ||
| 497 | dstfile = os.path.join(dstdir, os.path.basename(srcfile)) | ||
| 498 | if srcfile != dstfile: | ||
| 499 | shutil.copy(srcfile, dstfile) | ||
| 500 | self.track_for_cleanup(dstfile) | ||
| 501 | |||
| 502 | def test_recipetool_load_plugin(self): | ||
| 503 | """Test that recipetool loads only the first found plugin in BBPATH.""" | ||
| 504 | |||
| 505 | recipetool = runCmd("which recipetool") | ||
| 506 | fromname = runCmd("recipetool --quiet pluginfile") | ||
| 507 | srcfile = fromname.output | ||
| 508 | bbpath = get_bb_var('BBPATH') | ||
| 509 | searchpath = bbpath.split(':') + [os.path.dirname(recipetool.output)] | ||
| 510 | plugincontent = [] | ||
| 511 | with open(srcfile) as fh: | ||
| 512 | plugincontent = fh.readlines() | ||
| 513 | try: | ||
| 514 | self.assertIn('meta-selftest', srcfile, 'wrong bbpath plugin found') | ||
| 515 | for path in searchpath: | ||
| 516 | self._copy_file_with_cleanup(srcfile, path, 'lib', 'recipetool') | ||
| 517 | result = runCmd("recipetool --quiet count") | ||
| 518 | self.assertEqual(result.output, '1') | ||
| 519 | result = runCmd("recipetool --quiet multiloaded") | ||
| 520 | self.assertEqual(result.output, "no") | ||
| 521 | for path in searchpath: | ||
| 522 | result = runCmd("recipetool --quiet bbdir") | ||
| 523 | self.assertEqual(result.output, path) | ||
| 524 | os.unlink(os.path.join(result.output, 'lib', 'recipetool', 'bbpath.py')) | ||
| 525 | finally: | ||
| 526 | with open(srcfile, 'w') as fh: | ||
| 527 | fh.writelines(plugincontent) | ||
| 528 | |||
| 529 | |||
| 488 | class RecipetoolAppendsrcBase(RecipetoolBase): | 530 | class RecipetoolAppendsrcBase(RecipetoolBase): |
| 489 | def _try_recipetool_appendsrcfile(self, testrecipe, newfile, destfile, options, expectedlines, expectedfiles): | 531 | def _try_recipetool_appendsrcfile(self, testrecipe, newfile, destfile, options, expectedlines, expectedfiles): |
| 490 | cmd = 'recipetool appendsrcfile %s %s %s %s %s' % (options, self.templayerdir, testrecipe, newfile, destfile) | 532 | cmd = 'recipetool appendsrcfile %s %s %s %s %s' % (options, self.templayerdir, testrecipe, newfile, destfile) |
| @@ -574,6 +616,8 @@ class RecipetoolAppendsrcBase(RecipetoolBase): | |||
| 574 | self.assertIn(filesdir, filesextrapaths) | 616 | self.assertIn(filesdir, filesextrapaths) |
| 575 | 617 | ||
| 576 | 618 | ||
| 619 | |||
| 620 | |||
| 577 | class RecipetoolAppendsrcTests(RecipetoolAppendsrcBase): | 621 | class RecipetoolAppendsrcTests(RecipetoolAppendsrcBase): |
| 578 | 622 | ||
| 579 | @testcase(1273) | 623 | @testcase(1273) |
