Merge branch 'builder-rpmbuild-options' of https://github.com/domcleal/tito into domcleal-builder-rpmbuild-options

This commit is contained in:
Devan Goodwin 2015-05-12 15:35:39 -03:00
commit 6b4f6ad1f8
2 changed files with 20 additions and 4 deletions

View file

@ -70,10 +70,8 @@ class BuilderBase(object):
self.scl = self._get_optional_arg(args, 'scl', None) or \
self._get_optional_arg(kwargs, 'scl', '')
self.rpmbuild_options = self._get_optional_arg(kwargs,
'rpmbuild_options', None)
if not self.rpmbuild_options:
self.rpmbuild_options = ''
self.rpmbuild_options = self._get_optional_arg(args, 'rpmbuild_options', None) or \
self._get_optional_arg(kwargs, 'rpmbuild_options', '')
self.test = self._get_optional_arg(kwargs, 'test', False)
# Allow a builder arg to override the test setting passed in, used by

View file

@ -44,6 +44,24 @@ class BuilderTests(TitoGitTestFixture):
self.config, {}, {}, **{'offline': True, 'scl': 'ruby193'})
self.assertEqual('ruby193', builder.scl)
def test_rpmbuild_options_from_options(self):
self.create_project(PKG_NAME)
builder = Builder(PKG_NAME, None, self.output_dir,
self.config, {}, {'rpmbuild_options': '--define "foo bar"'}, **{'offline': True})
self.assertEqual('--define "foo bar"', builder.rpmbuild_options)
def test_rpmbuild_options_from_kwargs(self):
self.create_project(PKG_NAME)
builder = Builder(PKG_NAME, None, self.output_dir,
self.config, {}, {}, **{'offline': True, 'rpmbuild_options': '--define "foo bar"'})
self.assertEqual('--define "foo bar"', builder.rpmbuild_options)
def test_rpmbuild_options_missing(self):
self.create_project(PKG_NAME)
builder = Builder(PKG_NAME, None, self.output_dir,
self.config, {}, {}, **{'offline': True})
self.assertEqual('', builder.rpmbuild_options)
def test_untagged_test_version(self):
self.create_project(PKG_NAME, tag=False)
self.assertEqual("", run_command("git tag -l").strip())