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) |