From e076294ec33b555ceab58a6ae3ca6f9738177c22 Mon Sep 17 00:00:00 2001 From: Mark Hatle Date: Tue, 25 Sep 2018 11:00:55 -0400 Subject: lib/layerindexlib/tests/restapi.py: Fix test cases when BB_SKIP_NETTETS=yes Change the way the network related tests are skipped, based on how the fetch tests are handled. We introduce a 'skipIfNoNetwork()' function that will report skipped tests through the standard unit test reporting system. [YOCTO #12928] Signed-off-by: Mark Hatle --- lib/layerindexlib/tests/restapi.py | 164 ++++++++++++++++++++----------------- 1 file changed, 87 insertions(+), 77 deletions(-) diff --git a/lib/layerindexlib/tests/restapi.py b/lib/layerindexlib/tests/restapi.py index bfaac43db..58766950c 100644 --- a/lib/layerindexlib/tests/restapi.py +++ b/lib/layerindexlib/tests/restapi.py @@ -23,113 +23,122 @@ from layerindexlib.tests.common import LayersTest import logging -class LayerIndexWebRestApiTest(LayersTest): - +def skipIfNoNetwork(): if os.environ.get("BB_SKIP_NETTESTS") == "yes": - print("Unset BB_SKIP_NETTESTS to run network tests") - else: - def setUp(self): - LayersTest.setUp(self) - self.layerindex = layerindexlib.LayerIndex(self.d) - self.layerindex.load_layerindex('http://layers.openembedded.org/layerindex/api/;branch=sumo', load=['layerDependencies']) + return unittest.skip("Network tests being skipped") + return lambda f: f - def test_layerindex_is_empty(self): - self.assertFalse(self.layerindex.is_empty(), msg="Layerindex is empty") +class LayerIndexWebRestApiTest(LayersTest): - def test_layerindex_store_file(self): - self.layerindex.store_layerindex('file://%s/file.json' % self.tempdir, self.layerindex.indexes[0]) + @skipIfNoNetwork() + def setUp(self): + self.assertFalse(os.environ.get("BB_SKIP_NETTESTS") == "yes", msg="BB_SKIP_NETTESTS set, but we tried to test anyway") + LayersTest.setUp(self) + self.layerindex = layerindexlib.LayerIndex(self.d) + self.layerindex.load_layerindex('http://layers.openembedded.org/layerindex/api/;branch=sumo', load=['layerDependencies']) - self.assertTrue(os.path.isfile('%s/file.json' % self.tempdir), msg="Temporary file was not created by store_layerindex") + @skipIfNoNetwork() + def test_layerindex_is_empty(self): + self.assertFalse(self.layerindex.is_empty(), msg="Layerindex is empty") - reload = layerindexlib.LayerIndex(self.d) - reload.load_layerindex('file://%s/file.json' % self.tempdir) + @skipIfNoNetwork() + def test_layerindex_store_file(self): + self.layerindex.store_layerindex('file://%s/file.json' % self.tempdir, self.layerindex.indexes[0]) - self.assertFalse(reload.is_empty(), msg="Layerindex is empty") + self.assertTrue(os.path.isfile('%s/file.json' % self.tempdir), msg="Temporary file was not created by store_layerindex") - # Calculate layerItems in original index that should NOT be in reload - layerItemNames = [] - for itemId in self.layerindex.indexes[0].layerItems: - layerItemNames.append(self.layerindex.indexes[0].layerItems[itemId].name) + reload = layerindexlib.LayerIndex(self.d) + reload.load_layerindex('file://%s/file.json' % self.tempdir) - for layerBranchId in self.layerindex.indexes[0].layerBranches: - layerItemNames.remove(self.layerindex.indexes[0].layerBranches[layerBranchId].layer.name) + self.assertFalse(reload.is_empty(), msg="Layerindex is empty") - for itemId in reload.indexes[0].layerItems: - self.assertFalse(reload.indexes[0].layerItems[itemId].name in layerItemNames, msg="Item reloaded when it shouldn't have been") + # Calculate layerItems in original index that should NOT be in reload + layerItemNames = [] + for itemId in self.layerindex.indexes[0].layerItems: + layerItemNames.append(self.layerindex.indexes[0].layerItems[itemId].name) - # Compare the original to what we wrote... - for type in self.layerindex.indexes[0]._index: - if type == 'apilinks' or \ - type == 'layerItems' or \ - type in self.layerindex.indexes[0].config['local']: - continue - for id in getattr(self.layerindex.indexes[0], type): - self.logger.debug(1, "type %s" % (type)) + for layerBranchId in self.layerindex.indexes[0].layerBranches: + layerItemNames.remove(self.layerindex.indexes[0].layerBranches[layerBranchId].layer.name) - self.assertTrue(id in getattr(reload.indexes[0], type), msg="Id number not in reloaded index") + for itemId in reload.indexes[0].layerItems: + self.assertFalse(reload.indexes[0].layerItems[itemId].name in layerItemNames, msg="Item reloaded when it shouldn't have been") - self.logger.debug(1, "%s ? %s" % (getattr(self.layerindex.indexes[0], type)[id], getattr(reload.indexes[0], type)[id])) + # Compare the original to what we wrote... + for type in self.layerindex.indexes[0]._index: + if type == 'apilinks' or \ + type == 'layerItems' or \ + type in self.layerindex.indexes[0].config['local']: + continue + for id in getattr(self.layerindex.indexes[0], type): + self.logger.debug(1, "type %s" % (type)) - self.assertEqual(getattr(self.layerindex.indexes[0], type)[id], getattr(reload.indexes[0], type)[id], msg="Reloaded contents different") + self.assertTrue(id in getattr(reload.indexes[0], type), msg="Id number not in reloaded index") - def test_layerindex_store_split(self): - self.layerindex.store_layerindex('file://%s' % self.tempdir, self.layerindex.indexes[0]) + self.logger.debug(1, "%s ? %s" % (getattr(self.layerindex.indexes[0], type)[id], getattr(reload.indexes[0], type)[id])) - reload = layerindexlib.LayerIndex(self.d) - reload.load_layerindex('file://%s' % self.tempdir) + self.assertEqual(getattr(self.layerindex.indexes[0], type)[id], getattr(reload.indexes[0], type)[id], msg="Reloaded contents different") - self.assertFalse(reload.is_empty(), msg="Layer index is empty") + @skipIfNoNetwork() + def test_layerindex_store_split(self): + self.layerindex.store_layerindex('file://%s' % self.tempdir, self.layerindex.indexes[0]) - for type in self.layerindex.indexes[0]._index: - if type == 'apilinks' or \ - type == 'layerItems' or \ - type in self.layerindex.indexes[0].config['local']: - continue - for id in getattr(self.layerindex.indexes[0] ,type): - self.logger.debug(1, "type %s" % (type)) + reload = layerindexlib.LayerIndex(self.d) + reload.load_layerindex('file://%s' % self.tempdir) - self.assertTrue(id in getattr(reload.indexes[0], type), msg="Id number missing from reloaded data") + self.assertFalse(reload.is_empty(), msg="Layer index is empty") - self.logger.debug(1, "%s ? %s" % (getattr(self.layerindex.indexes[0] ,type)[id], getattr(reload.indexes[0], type)[id])) + for type in self.layerindex.indexes[0]._index: + if type == 'apilinks' or \ + type == 'layerItems' or \ + type in self.layerindex.indexes[0].config['local']: + continue + for id in getattr(self.layerindex.indexes[0] ,type): + self.logger.debug(1, "type %s" % (type)) - self.assertEqual(getattr(self.layerindex.indexes[0] ,type)[id], getattr(reload.indexes[0], type)[id], msg="reloaded data does not match original") + self.assertTrue(id in getattr(reload.indexes[0], type), msg="Id number missing from reloaded data") - def test_dependency_resolution(self): - # Verify depth first searching... - (dependencies, invalidnames) = self.layerindex.find_dependencies(names=['meta-python']) + self.logger.debug(1, "%s ? %s" % (getattr(self.layerindex.indexes[0] ,type)[id], getattr(reload.indexes[0], type)[id])) - first = True - for deplayerbranch in dependencies: - layerBranch = dependencies[deplayerbranch][0] - layerDeps = dependencies[deplayerbranch][1:] + self.assertEqual(getattr(self.layerindex.indexes[0] ,type)[id], getattr(reload.indexes[0], type)[id], msg="reloaded data does not match original") - if not first: - continue + @skipIfNoNetwork() + def test_dependency_resolution(self): + # Verify depth first searching... + (dependencies, invalidnames) = self.layerindex.find_dependencies(names=['meta-python']) - first = False + first = True + for deplayerbranch in dependencies: + layerBranch = dependencies[deplayerbranch][0] + layerDeps = dependencies[deplayerbranch][1:] - # Top of the deps should be openembedded-core, since everything depends on it. - self.assertEqual(layerBranch.layer.name, "openembedded-core", msg='OpenEmbedded-Core is no the first dependency') + if not first: + continue - # meta-python should cause an openembedded-core dependency, if not assert! - for dep in layerDeps: - if dep.layer.name == 'meta-python': - break - else: - self.logger.debug(1, "meta-python was not found") - self.assetTrue(False) + first = False - # Only check the first element... - break + # Top of the deps should be openembedded-core, since everything depends on it. + self.assertEqual(layerBranch.layer.name, "openembedded-core", msg='OpenEmbedded-Core is no the first dependency') + + # meta-python should cause an openembedded-core dependency, if not assert! + for dep in layerDeps: + if dep.layer.name == 'meta-python': + break else: - # Empty list, this is bad. - self.logger.debug(1, "Empty list of dependencies") - self.assertIsNotNone(first, msg="Empty list of dependencies") + self.logger.debug(1, "meta-python was not found") + self.assetTrue(False) + + # Only check the first element... + break + else: + # Empty list, this is bad. + self.logger.debug(1, "Empty list of dependencies") + self.assertIsNotNone(first, msg="Empty list of dependencies") - # Last dep should be the requested item - layerBranch = dependencies[deplayerbranch][0] - self.assertEqual(layerBranch.layer.name, "meta-python", msg="Last dependency not meta-python") + # Last dep should be the requested item + layerBranch = dependencies[deplayerbranch][0] + self.assertEqual(layerBranch.layer.name, "meta-python", msg="Last dependency not meta-python") + @skipIfNoNetwork() def test_find_collection(self): def _check(collection, expected): self.logger.debug(1, "Looking for collection %s..." % collection) @@ -149,6 +158,7 @@ class LayerIndexWebRestApiTest(LayersTest): for collection,result in tests: _check(collection, result) + @skipIfNoNetwork() def test_find_layerbranch(self): def _check(name, expected): self.logger.debug(1, "Looking for layerbranch %s..." % name) -- cgit 1.2.3-korg