Separate exclusion list for source and binary types

Separate list of excluded images for source and binary unit
tests.

Change-Id: I4b6032db25c898272491c44c7a06f7cddf7a2298
Closes-Bug: #1487252
This commit is contained in:
Michal Rostecki 2015-10-13 06:57:21 +02:00 committed by Sam Yaple
parent c15df14737
commit dcbfbcd2c8
1 changed files with 32 additions and 27 deletions

View File

@ -10,24 +10,27 @@
# License for the specific language governing permissions and limitations
# under the License.
import abc
import os
import sys
from mock import patch
from os import path
from oslo_log import fixture as log_fixture
from oslo_log import log as logging
from oslotest import base
import six
import testtools
import sys
sys.path.append(path.abspath(path.join(path.dirname(__file__), '../tools')))
sys.path.append(
os.path.abspath(os.path.join(os.path.dirname(__file__), '../tools')))
from kolla.cmd import build
LOG = logging.getLogger(__name__)
class BuildTest(base.BaseTestCase):
@six.add_metaclass(abc.ABCMeta)
class BuildTest(object):
excluded_images = abc.abstractproperty()
def setUp(self):
super(BuildTest, self).setUp()
@ -42,16 +45,9 @@ class BuildTest(base.BaseTestCase):
LOG.info("Running with args %s", self.build_args)
bad_results, good_results, unmatched_results = build.main()
# these are images that are known to not build properly
excluded_images = ["gnocchi-base",
"murano-base",
"ironic-pxe",
"ironic-discoverd",
"mistral-base"]
failures = 0
for image, result in six.iteritems(bad_results):
if image in excluded_images:
if image in self.excluded_images:
if result is 'error':
continue
failures = failures + 1
@ -70,46 +66,55 @@ class BuildTest(base.BaseTestCase):
self.assertEqual(failures, 0, "%d failure(s) occurred" % failures)
class BuildTestCentosBinary(BuildTest):
class BuildTestCentosBinary(BuildTest, base.BaseTestCase):
excluded_images = ["gnocchi-base",
"murano-base",
"ironic-pxe",
"ironic-discoverd",
"mistral-base",
"murano-base"]
def setUp(self):
super(BuildTestCentosBinary, self).setUp()
self.build_args.extend(["--base", "centos",
"--type", "binary"])
class BuildTestCentosSource(BuildTest):
class BuildTestCentosSource(BuildTest, base.BaseTestCase):
excluded_images = ["gnocchi-base",
"murano-base",
"ironic-pxe",
"ironic-discoverd",
"mistral-base"]
def setUp(self):
super(BuildTestCentosSource, self).setUp()
self.build_args.extend(["--base", "centos",
"--type", "source"])
class BuildTestUbuntuSource(BuildTest):
class BuildTestUbuntuSource(BuildTest, base.BaseTestCase):
excluded_images = []
def setUp(self):
super(BuildTestUbuntuSource, self).setUp()
self.build_args.extend(["--base", "ubuntu",
"--type", "source"])
class DeployTestCentosBinary(BuildTest):
class DeployTestCentosBinary(BuildTestCentosBinary):
def setUp(self):
super(DeployTestCentosBinary, self).setUp()
self.build_args.extend(["--base", "centos",
"--type", "binary",
"--profile", "gate"])
self.build_args.extend(["--profile", "gate"])
class DeployTestCentosSource(BuildTest):
class DeployTestCentosSource(BuildTestCentosSource):
def setUp(self):
super(DeployTestCentosSource, self).setUp()
self.build_args.extend(["--base", "centos",
"--type", "source",
"--profile", "gate"])
self.build_args.extend(["--profile", "gate"])
class DeployTestUbuntuSource(BuildTest):
class DeployTestUbuntuSource(BuildTestUbuntuSource):
def setUp(self):
super(DeployTestUbuntuSource, self).setUp()
self.build_args.extend(["--base", "ubuntu",
"--type", "source",
"--profile", "gate"])
self.build_args.extend(["--profile", "gate"])