From b8c443b44fcad29325ec33ac2b3932c59eb6a711 Mon Sep 17 00:00:00 2001 From: Zack Cerza Date: Wed, 16 Jul 2014 12:07:55 -0600 Subject: [PATCH] Fix get_hash() arg ordering Signed-off-by: Zack Cerza --- teuthology/suite.py | 2 +- teuthology/test/test_suite.py | 6 +++++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/teuthology/suite.py b/teuthology/suite.py index 3a28ea791b..e1672dabb6 100644 --- a/teuthology/suite.py +++ b/teuthology/suite.py @@ -331,7 +331,7 @@ def get_worker(machine_type): def get_hash(project='ceph', branch='master', flavor='basic', - distro='ubuntu', machine_type='plana'): + machine_type='plana', distro='ubuntu'): """ Find the hash representing the head of the project's repository via querying a gitbuilder repo. diff --git a/teuthology/test/test_suite.py b/teuthology/test/test_suite.py index 14869a3725..27558b6dbc 100644 --- a/teuthology/test/test_suite.py +++ b/teuthology/test/test_suite.py @@ -64,12 +64,16 @@ class TestSuiteOnline(object): if 'TEST_ONLINE' not in os.environ: skip("To run these sets, set the environment variable TEST_ONLINE") - def test_ceph_hash(self): + def test_ceph_hash_simple(self): resp = requests.get( 'https://api.github.com/repos/ceph/ceph/git/refs/heads/master') ref_hash = resp.json()['object']['sha'] assert suite.get_hash('ceph') == ref_hash + def test_kernel_hash_saya(self): + # We don't currently have these packages. + assert suite.get_hash('kernel', 'master', 'basic', 'saya') is None + def test_all_master_branches(self): # Don't attempt to send email config.results_email = None -- 2.39.5