9642c1171c
Additionally ostree_ref (if parameter is given) should be placed into treefile. Relates: https://pagure.io/pungi/issue/777 Signed-off-by: Ondrej Nosek <onosek@redhat.com>
392 lines
17 KiB
Python
392 lines
17 KiB
Python
#!/usr/bin/env python
|
|
# -*- coding: utf-8 -*-
|
|
|
|
|
|
import mock
|
|
|
|
import os
|
|
import json
|
|
import sys
|
|
import datetime
|
|
|
|
sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..'))
|
|
sys.path.insert(0, os.path.join(os.path.dirname(__file__), 'bin'))
|
|
|
|
from tests import helpers
|
|
from pungi import ostree
|
|
|
|
|
|
class OstreeTreeScriptTest(helpers.PungiTestCase):
|
|
|
|
def _make_dummy_config_dir(self, path):
|
|
helpers.touch(os.path.join(path, 'fedora-atomic-docker-host.json'),
|
|
json.dumps({'ref': 'fedora-atomic/25/x86_64',
|
|
'repos': ['fedora-rawhide', 'fedora-24', 'fedora-23']}))
|
|
helpers.touch(os.path.join(path, 'fedora-rawhide.repo'),
|
|
'[fedora-rawhide]\nmirrorlist=mirror-mirror-on-the-wall')
|
|
helpers.touch(os.path.join(path, 'fedora-24.repo'),
|
|
'[fedora-24]\nmetalink=who-is-the-fairest-of-them-all')
|
|
helpers.touch(os.path.join(path, 'fedora-23.repo'),
|
|
'[fedora-23]\nbaseurl=why-not-zoidberg?')
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_full_run(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s/fedora-atomic-docker-host.json' % self.topdir,
|
|
])
|
|
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(
|
|
run.call_args_list,
|
|
[mock.call(['rpm-ostree', 'compose', 'tree', '--repo=%s' % repo,
|
|
'--write-commitid-to=%s' % (self.topdir + '/logs/Atomic/commitid.log'),
|
|
self.topdir + '/fedora-atomic-docker-host.json'],
|
|
logfile=self.topdir + '/logs/Atomic/create-ostree-repo.log', show_cmd=True, stdout=True)])
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_run_on_existing_empty_dir(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
os.mkdir(repo)
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s/fedora-atomic-docker-host.json' % self.topdir,
|
|
])
|
|
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(
|
|
run.call_args_list,
|
|
[mock.call(['rpm-ostree', 'compose', 'tree', '--repo=%s' % repo,
|
|
'--write-commitid-to=%s' % (self.topdir + '/logs/Atomic/commitid.log'),
|
|
self.topdir + '/fedora-atomic-docker-host.json'],
|
|
logfile=self.topdir + '/logs/Atomic/create-ostree-repo.log', show_cmd=True, stdout=True)])
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_run_on_initialized_repo(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
helpers.touch(os.path.join(repo, 'initialized'))
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s/fedora-atomic-docker-host.json' % self.topdir,
|
|
])
|
|
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(
|
|
run.call_args_list,
|
|
[mock.call(['rpm-ostree', 'compose', 'tree', '--repo=%s' % repo,
|
|
'--write-commitid-to=%s' % (self.topdir + '/logs/Atomic/commitid.log'),
|
|
self.topdir + '/fedora-atomic-docker-host.json'],
|
|
logfile=self.topdir + '/logs/Atomic/create-ostree-repo.log', show_cmd=True, stdout=True)])
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_update_summary(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s/fedora-atomic-docker-host.json' % self.topdir,
|
|
'--update-summary',
|
|
])
|
|
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(
|
|
run.call_args_list,
|
|
[mock.call(['rpm-ostree', 'compose', 'tree', '--repo=%s' % repo,
|
|
'--write-commitid-to=%s' % (self.topdir + '/logs/Atomic/commitid.log'),
|
|
self.topdir + '/fedora-atomic-docker-host.json'],
|
|
logfile=self.topdir + '/logs/Atomic/create-ostree-repo.log', show_cmd=True, stdout=True),
|
|
mock.call(['ostree', 'summary', '-u', '--repo=%s' % repo],
|
|
logfile=self.topdir + '/logs/Atomic/ostree-summary.log', show_cmd=True, stdout=True)]),
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_versioning_metadata(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s/fedora-atomic-docker-host.json' % self.topdir,
|
|
'--version=24',
|
|
])
|
|
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(
|
|
run.call_args_list,
|
|
[mock.call(['rpm-ostree', 'compose', 'tree', '--repo=%s' % repo,
|
|
'--write-commitid-to=%s' % (self.topdir + '/logs/Atomic/commitid.log'),
|
|
'--add-metadata-string=version=24',
|
|
self.topdir + '/fedora-atomic-docker-host.json'],
|
|
logfile=self.topdir + '/logs/Atomic/create-ostree-repo.log', show_cmd=True, stdout=True)])
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_ostree_ref(self, run):
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
self._make_dummy_config_dir(self.topdir)
|
|
treefile = os.path.join(self.topdir, 'fedora-atomic-docker-host.json')
|
|
|
|
with open(treefile, 'r') as f:
|
|
treefile_content = json.load(f)
|
|
original_repos = treefile_content['repos']
|
|
original_ref = treefile_content['ref']
|
|
replacing_ref = original_ref + '-changed'
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s' % treefile,
|
|
'--ostree-ref=%s' % replacing_ref,
|
|
])
|
|
|
|
with open(treefile, 'r') as f:
|
|
treefile_content = json.load(f)
|
|
new_repos = treefile_content['repos']
|
|
new_ref = treefile_content['ref']
|
|
|
|
# ref value in treefile should be overrided with new ref
|
|
self.assertEqual(replacing_ref, new_ref)
|
|
# repos should stay unchanged
|
|
self.assertEqual(original_repos, new_repos)
|
|
|
|
@mock.patch('pungi.ostree.utils.datetime')
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_extra_config_with_extra_repos(self, run, time):
|
|
time.datetime.now.return_value = datetime.datetime(2016, 1, 1, 1, 1)
|
|
timestamp = time.datetime.now().strftime("%Y%m%d%H%M%S")
|
|
|
|
configdir = os.path.join(self.topdir, 'config')
|
|
self._make_dummy_config_dir(configdir)
|
|
treefile = os.path.join(configdir, 'fedora-atomic-docker-host.json')
|
|
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
extra_config_file = os.path.join(self.topdir, 'extra_config.json')
|
|
extra_config = {
|
|
"repo": [
|
|
{
|
|
"name": "server",
|
|
"baseurl": "http://www.example.com/Server/repo",
|
|
},
|
|
{
|
|
"name": "optional",
|
|
"baseurl": "http://example.com/repo/x86_64/optional",
|
|
"exclude": "systemd-container",
|
|
"gpgcheck": False
|
|
},
|
|
{
|
|
"name": "extra",
|
|
"baseurl": "http://example.com/repo/x86_64/extra",
|
|
}
|
|
]
|
|
}
|
|
helpers.touch(extra_config_file, json.dumps(extra_config))
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s' % treefile,
|
|
'--extra-config=%s' % extra_config_file,
|
|
])
|
|
|
|
server_repo_name = "server-%s" % timestamp
|
|
server_repo = os.path.join(configdir, "%s.repo" % server_repo_name)
|
|
self.assertTrue(os.path.isfile(server_repo))
|
|
with open(server_repo, 'r') as f:
|
|
content = f.read()
|
|
self.assertIn("[%s]" % server_repo_name, content)
|
|
self.assertIn("name=%s" % server_repo_name, content)
|
|
self.assertIn("baseurl=http://www.example.com/Server/repo", content)
|
|
self.assertIn("gpgcheck=0", content)
|
|
|
|
optional_repo_name = "optional-%s" % timestamp
|
|
optional_repo = os.path.join(configdir, "%s.repo" % optional_repo_name)
|
|
self.assertTrue(os.path.isfile(optional_repo))
|
|
with open(optional_repo, 'r') as f:
|
|
content = f.read()
|
|
self.assertIn("[%s]" % optional_repo_name, content)
|
|
self.assertIn("name=%s" % optional_repo_name, content)
|
|
self.assertIn("baseurl=http://example.com/repo/x86_64/optional", content)
|
|
self.assertIn("gpgcheck=0", content)
|
|
|
|
extra_repo_name = "extra-%s" % timestamp
|
|
extra_repo = os.path.join(configdir, "%s.repo" % extra_repo_name)
|
|
self.assertTrue(os.path.isfile(extra_repo))
|
|
with open(extra_repo, 'r') as f:
|
|
content = f.read()
|
|
self.assertIn("[%s]" % extra_repo_name, content)
|
|
self.assertIn("name=%s" % extra_repo_name, content)
|
|
self.assertIn("baseurl=http://example.com/repo/x86_64/extra", content)
|
|
self.assertIn("gpgcheck=0", content)
|
|
|
|
treeconf = json.load(open(treefile, 'r'))
|
|
repos = treeconf['repos']
|
|
self.assertEqual(len(repos), 3)
|
|
for name in [server_repo_name, optional_repo_name, extra_repo_name]:
|
|
self.assertIn(name, repos)
|
|
|
|
@mock.patch('pungi.ostree.utils.datetime')
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_extra_config_with_keep_original_sources(self, run, time):
|
|
time.datetime.now.return_value = datetime.datetime(2016, 1, 1, 1, 1)
|
|
timestamp = time.datetime.now().strftime("%Y%m%d%H%M%S")
|
|
|
|
configdir = os.path.join(self.topdir, 'config')
|
|
self._make_dummy_config_dir(configdir)
|
|
treefile = os.path.join(configdir, 'fedora-atomic-docker-host.json')
|
|
|
|
repo = os.path.join(self.topdir, 'atomic')
|
|
|
|
extra_config_file = os.path.join(self.topdir, 'extra_config.json')
|
|
extra_config = {
|
|
"repo": [
|
|
{
|
|
"name": "server",
|
|
"baseurl": "http://www.example.com/Server/repo",
|
|
},
|
|
{
|
|
"name": "optional",
|
|
"baseurl": "http://example.com/repo/x86_64/optional",
|
|
"exclude": "systemd-container",
|
|
"gpgcheck": False
|
|
},
|
|
{
|
|
"name": "extra",
|
|
"baseurl": "http://example.com/repo/x86_64/extra",
|
|
}
|
|
],
|
|
"keep_original_sources": True
|
|
}
|
|
helpers.touch(extra_config_file, json.dumps(extra_config))
|
|
|
|
ostree.main([
|
|
'tree',
|
|
'--repo=%s' % repo,
|
|
'--log-dir=%s' % os.path.join(self.topdir, 'logs', 'Atomic'),
|
|
'--treefile=%s' % treefile,
|
|
'--extra-config=%s' % extra_config_file,
|
|
])
|
|
|
|
server_repo_name = "server-%s" % timestamp
|
|
optional_repo_name = "optional-%s" % timestamp
|
|
extra_repo_name = "extra-%s" % timestamp
|
|
|
|
treeconf = json.load(open(treefile, 'r'))
|
|
repos = treeconf['repos']
|
|
self.assertEqual(len(repos), 6)
|
|
for name in ['fedora-rawhide', 'fedora-24', 'fedora-23',
|
|
server_repo_name, optional_repo_name, extra_repo_name]:
|
|
self.assertIn(name, repos)
|
|
|
|
|
|
class OstreeInstallerScriptTest(helpers.PungiTestCase):
|
|
def setUp(self):
|
|
super(OstreeInstallerScriptTest, self).setUp()
|
|
self.product = "dummyproduct"
|
|
self.version = "1.0"
|
|
self.release = "20160101.t.0"
|
|
self.output = os.path.join(self.topdir, 'output')
|
|
self.logdir = os.path.join(self.topdir, 'logs')
|
|
self.volid = '%s-%s' % (self.product, self.version)
|
|
self.variant = 'dummy'
|
|
self.rootfs_size = None
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_run_with_args(self, run):
|
|
args = ['installer',
|
|
'--product=%s' % self.product,
|
|
'--version=%s' % self.version,
|
|
'--release=%s' % self.release,
|
|
'--output=%s' % self.output,
|
|
'--variant=%s' % self.variant,
|
|
'--rootfs-size=%s' % self.rootfs_size,
|
|
'--nomacboot',
|
|
'--isfinal']
|
|
args.append('--source=%s' % 'http://www.example.com/dummy/repo')
|
|
args.append('--installpkgs=dummy-foo')
|
|
args.append('--installpkgs=dummy-bar')
|
|
args.append('--add-template=/path/to/lorax.tmpl')
|
|
args.append('--add-template-var=ostree_osname=dummy')
|
|
args.append('--add-arch-template=/path/to/lorax-embed.tmpl')
|
|
args.append('--add-arch-template-var=ostree_repo=http://www.example.com/ostree')
|
|
ostree.main(args)
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(run.mock_calls,
|
|
[mock.call(['lorax',
|
|
'--product=dummyproduct',
|
|
'--version=1.0',
|
|
'--release=20160101.t.0',
|
|
'--source=http://www.example.com/dummy/repo',
|
|
'--variant=dummy',
|
|
'--nomacboot',
|
|
'--isfinal',
|
|
'--installpkgs=dummy-foo',
|
|
'--installpkgs=dummy-bar',
|
|
'--add-template=/path/to/lorax.tmpl',
|
|
'--add-arch-template=/path/to/lorax-embed.tmpl',
|
|
'--add-template-var=ostree_osname=dummy',
|
|
'--add-arch-template-var=ostree_repo=http://www.example.com/ostree',
|
|
'--rootfs-size=None',
|
|
self.output])])
|
|
|
|
@mock.patch('kobo.shortcuts.run')
|
|
def test_run_with_extra_config_file(self, run):
|
|
extra_config_file = os.path.join(self.topdir, 'extra_config.json')
|
|
helpers.touch(extra_config_file,
|
|
json.dumps({'repo': 'http://www.example.com/another/repo',
|
|
'installpkgs': ['dummy-foo', 'dummy-bar'],
|
|
'add_template': ['/path/to/lorax.tmpl'],
|
|
'add_template_var': ['ostree_osname=dummy-atomic',
|
|
'ostree_ref=dummy/x86_64/docker'],
|
|
'add_arch_template': ['/path/to/lorax-embed.tmpl'],
|
|
'add_arch_template_var': ['ostree_osname=dummy-atomic',
|
|
'ostree_repo=http://www.example.com/ostree']}))
|
|
args = ['installer',
|
|
'--product=%s' % self.product,
|
|
'--version=%s' % self.version,
|
|
'--release=%s' % self.release,
|
|
'--output=%s' % self.output,
|
|
'--variant=%s' % self.variant,
|
|
'--rootfs-size=%s' % self.rootfs_size,
|
|
'--nomacboot',
|
|
'--isfinal']
|
|
args.append('--source=%s' % 'http://www.example.com/dummy/repo')
|
|
args.append('--extra-config=%s' % extra_config_file)
|
|
ostree.main(args)
|
|
self.maxDiff = None
|
|
self.assertItemsEqual(run.mock_calls,
|
|
[mock.call(['lorax',
|
|
'--product=dummyproduct',
|
|
'--version=1.0',
|
|
'--release=20160101.t.0',
|
|
'--source=http://www.example.com/dummy/repo',
|
|
'--variant=dummy',
|
|
'--nomacboot',
|
|
'--isfinal',
|
|
'--installpkgs=dummy-foo',
|
|
'--installpkgs=dummy-bar',
|
|
'--add-template=/path/to/lorax.tmpl',
|
|
'--add-arch-template=/path/to/lorax-embed.tmpl',
|
|
'--add-template-var=ostree_osname=dummy-atomic',
|
|
'--add-template-var=ostree_ref=dummy/x86_64/docker',
|
|
'--add-arch-template-var=ostree_osname=dummy-atomic',
|
|
'--add-arch-template-var=ostree_repo=http://www.example.com/ostree',
|
|
'--rootfs-size=None',
|
|
self.output])])
|