summaryrefslogtreecommitdiffstats
path: root/meta/lib/oeqa
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@intel.com>2016-06-15 12:01:23 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-06-23 14:26:14 +0100
commit675843d540f30058269789b2c6808a206ab04a04 (patch)
tree85383a91e26ddebb9dc3bcd8b4c63d47dc3bf5c5 /meta/lib/oeqa
parentc65cf8ede2116ef1fa3ddeb617e7c39caacff941 (diff)
downloadpoky-675843d540f30058269789b2c6808a206ab04a04.tar.gz
oeqa: fix hasPackage, add hasPackageMatch
hasPackage() was looking for the string provided as an RE substring in the manifest, which resulted in a large number of false positives (i.e. libgtkfoo would match "gtk+"). Rewrite the manifest loader to parse the files into a proper data structure, change hasPackage to do full string matches, and add hasPackageMatch which does RE substring matches. (From OE-Core rev: b9409863af71899e02275439949e3f4cdfaf2d0f) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa')
-rw-r--r--meta/lib/oeqa/oetest.py28
-rw-r--r--meta/lib/oeqa/runtime/_ptest.py4
-rw-r--r--meta/lib/oeqa/runtime/python.py2
-rw-r--r--meta/lib/oeqa/runtime/smart.py2
-rw-r--r--meta/lib/oeqa/sdk/buildsudoku.py2
5 files changed, 26 insertions, 12 deletions
diff --git a/meta/lib/oeqa/oetest.py b/meta/lib/oeqa/oetest.py
index feb0f71eb4..4a740fb53f 100644
--- a/meta/lib/oeqa/oetest.py
+++ b/meta/lib/oeqa/oetest.py
@@ -61,14 +61,24 @@ class oeTest(unittest.TestCase):
61 61
62 @classmethod 62 @classmethod
63 def hasPackage(self, pkg): 63 def hasPackage(self, pkg):
64 for item in oeTest.tc.pkgmanifest.split('\n'): 64 """
65 if re.match(pkg, item): 65 True if the full package name exists in the manifest, False otherwise.
66 """
67 return pkg in oeTest.tc.pkgmanifest
68
69 @classmethod
70 def hasPackageMatch(self, match):
71 """
72 True if match exists in the manifest as a regular expression substring,
73 False otherwise.
74 """
75 for s in oeTest.tc.pkgmanifest:
76 if re.match(match, s):
66 return True 77 return True
67 return False 78 return False
68 79
69 @classmethod 80 @classmethod
70 def hasFeature(self,feature): 81 def hasFeature(self,feature):
71
72 if feature in oeTest.tc.imagefeatures or \ 82 if feature in oeTest.tc.imagefeatures or \
73 feature in oeTest.tc.distrofeatures: 83 feature in oeTest.tc.distrofeatures:
74 return True 84 return True
@@ -391,17 +401,18 @@ class RuntimeTestContext(TestContext):
391 401
392 self.target = target 402 self.target = target
393 403
404 self.pkgmanifest = {}
394 manifest = os.path.join(d.getVar("DEPLOY_DIR_IMAGE", True), 405 manifest = os.path.join(d.getVar("DEPLOY_DIR_IMAGE", True),
395 d.getVar("IMAGE_LINK_NAME", True) + ".manifest") 406 d.getVar("IMAGE_LINK_NAME", True) + ".manifest")
396 nomanifest = d.getVar("IMAGE_NO_MANIFEST", True) 407 nomanifest = d.getVar("IMAGE_NO_MANIFEST", True)
397 if nomanifest is None or nomanifest != "1": 408 if nomanifest is None or nomanifest != "1":
398 try: 409 try:
399 with open(manifest) as f: 410 with open(manifest) as f:
400 self.pkgmanifest = f.read() 411 for line in f:
412 (pkg, arch, version) = line.strip().split()
413 self.pkgmanifest[pkg] = (version, arch)
401 except IOError as e: 414 except IOError as e:
402 bb.fatal("No package manifest file found. Did you build the image?\n%s" % e) 415 bb.fatal("No package manifest file found. Did you build the image?\n%s" % e)
403 else:
404 self.pkgmanifest = ""
405 416
406 def _get_test_namespace(self): 417 def _get_test_namespace(self):
407 return "runtime" 418 return "runtime"
@@ -626,8 +637,11 @@ class SDKTestContext(TestContext):
626 if not hasattr(self, 'target_manifest'): 637 if not hasattr(self, 'target_manifest'):
627 self.target_manifest = d.getVar("SDK_TARGET_MANIFEST", True) 638 self.target_manifest = d.getVar("SDK_TARGET_MANIFEST", True)
628 try: 639 try:
640 self.pkgmanifest = {}
629 with open(self.target_manifest) as f: 641 with open(self.target_manifest) as f:
630 self.pkgmanifest = f.read() 642 for line in f:
643 (pkg, arch, version) = line.strip().split()
644 self.pkgmanifest[pkg] = (version, arch)
631 except IOError as e: 645 except IOError as e:
632 bb.fatal("No package manifest file found. Did you build the sdk image?\n%s" % e) 646 bb.fatal("No package manifest file found. Did you build the sdk image?\n%s" % e)
633 647
diff --git a/meta/lib/oeqa/runtime/_ptest.py b/meta/lib/oeqa/runtime/_ptest.py
index 0621028b86..71324d3da2 100644
--- a/meta/lib/oeqa/runtime/_ptest.py
+++ b/meta/lib/oeqa/runtime/_ptest.py
@@ -11,7 +11,7 @@ import subprocess
11def setUpModule(): 11def setUpModule():
12 if not oeRuntimeTest.hasFeature("package-management"): 12 if not oeRuntimeTest.hasFeature("package-management"):
13 skipModule("Image doesn't have package management feature") 13 skipModule("Image doesn't have package management feature")
14 if not oeRuntimeTest.hasPackage("smart"): 14 if not oeRuntimeTest.hasPackage("smartpm"):
15 skipModule("Image doesn't have smart installed") 15 skipModule("Image doesn't have smart installed")
16 if "package_rpm" != oeRuntimeTest.tc.d.getVar("PACKAGE_CLASSES", True).split()[0]: 16 if "package_rpm" != oeRuntimeTest.tc.d.getVar("PACKAGE_CLASSES", True).split()[0]:
17 skipModule("Rpm is not the primary package manager") 17 skipModule("Rpm is not the primary package manager")
@@ -105,7 +105,7 @@ class PtestRunnerTest(oeRuntimeTest):
105 def test_ptestrunner(self): 105 def test_ptestrunner(self):
106 self.add_smart_channel() 106 self.add_smart_channel()
107 (runnerstatus, result) = self.target.run('which ptest-runner', 0) 107 (runnerstatus, result) = self.target.run('which ptest-runner', 0)
108 cond = oeRuntimeTest.hasPackage("ptest-runner") and oeRuntimeTest.hasFeature("ptest") and oeRuntimeTest.hasPackage("-ptest") and (runnerstatus != 0) 108 cond = oeRuntimeTest.hasPackage("ptest-runner") and oeRuntimeTest.hasFeature("ptest") and oeRuntimeTest.hasPackageMatch("-ptest") and (runnerstatus != 0)
109 if cond: 109 if cond:
110 self.install_packages(self.install_complementary("*-ptest")) 110 self.install_packages(self.install_complementary("*-ptest"))
111 self.install_packages(['ptest-runner']) 111 self.install_packages(['ptest-runner'])
diff --git a/meta/lib/oeqa/runtime/python.py b/meta/lib/oeqa/runtime/python.py
index 26edb7a9b8..29a231c7c3 100644
--- a/meta/lib/oeqa/runtime/python.py
+++ b/meta/lib/oeqa/runtime/python.py
@@ -4,7 +4,7 @@ from oeqa.oetest import oeRuntimeTest, skipModule
4from oeqa.utils.decorators import * 4from oeqa.utils.decorators import *
5 5
6def setUpModule(): 6def setUpModule():
7 if not oeRuntimeTest.hasPackage("python"): 7 if not oeRuntimeTest.hasPackage("python-core"):
8 skipModule("No python package in the image") 8 skipModule("No python package in the image")
9 9
10 10
diff --git a/meta/lib/oeqa/runtime/smart.py b/meta/lib/oeqa/runtime/smart.py
index 126d614638..c7a5753991 100644
--- a/meta/lib/oeqa/runtime/smart.py
+++ b/meta/lib/oeqa/runtime/smart.py
@@ -7,7 +7,7 @@ from oeqa.utils.httpserver import HTTPService
7def setUpModule(): 7def setUpModule():
8 if not oeRuntimeTest.hasFeature("package-management"): 8 if not oeRuntimeTest.hasFeature("package-management"):
9 skipModule("Image doesn't have package management feature") 9 skipModule("Image doesn't have package management feature")
10 if not oeRuntimeTest.hasPackage("smart"): 10 if not oeRuntimeTest.hasPackage("smartpm"):
11 skipModule("Image doesn't have smart installed") 11 skipModule("Image doesn't have smart installed")
12 if "package_rpm" != oeRuntimeTest.tc.d.getVar("PACKAGE_CLASSES", True).split()[0]: 12 if "package_rpm" != oeRuntimeTest.tc.d.getVar("PACKAGE_CLASSES", True).split()[0]:
13 skipModule("Rpm is not the primary package manager") 13 skipModule("Rpm is not the primary package manager")
diff --git a/meta/lib/oeqa/sdk/buildsudoku.py b/meta/lib/oeqa/sdk/buildsudoku.py
index dea77c6599..5abbbb867f 100644
--- a/meta/lib/oeqa/sdk/buildsudoku.py
+++ b/meta/lib/oeqa/sdk/buildsudoku.py
@@ -3,7 +3,7 @@ from oeqa.utils.decorators import *
3from oeqa.utils.targetbuild import SDKBuildProject 3from oeqa.utils.targetbuild import SDKBuildProject
4 4
5def setUpModule(): 5def setUpModule():
6 if not oeSDKTest.hasPackage("gtk\+"): 6 if not oeSDKTest.hasPackage("gtk+"):
7 skipModule("Image doesn't have gtk+ in manifest") 7 skipModule("Image doesn't have gtk+ in manifest")
8 8
9class SudokuTest(oeSDKTest): 9class SudokuTest(oeSDKTest):