Skip to content

Commit

Permalink
test_upgrade_common: add mock for get_package_version
Browse files Browse the repository at this point in the history
Fixes regression introduced by ceph#891

Signed-off-by: Jan Fajerski <jfajerski@suse.com>
(cherry picked from commit c84bde3)
  • Loading branch information
Jan Fajerski committed Jan 25, 2017
1 parent 14fc86e commit f3b9e8c
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion teuthology/test/task/test_install.py
Expand Up @@ -122,13 +122,15 @@ def test_upgrade_is_downgrade(self):
for t in assert_ok_vals:
assert install._upgrade_is_downgrade(t[0], t[1]) == False

@patch("teuthology.packaging.get_package_version")
@patch("teuthology.misc.get_system_type")
@patch("teuthology.task.install.verify_package_version")
@patch("teuthology.task.install.get_upgrade_version")
def test_upgrade_common(self,
m_get_upgrade_version,
m_verify_package_version,
m_get_system_type):
m_get_system_type,
m_get_package_version):
expected_system_type = 'deb'
def make_remote():
remote = Mock()
Expand Down Expand Up @@ -169,6 +171,7 @@ def only(self, role):
],
}
m_get_upgrade_version.return_value = "11.0.0"
m_get_package_version.return_value = "10.2.4"
m_get_system_type.return_value = "deb"
def upgrade(ctx, node, remote, pkgs, system_type):
assert system_type == expected_system_type
Expand Down

0 comments on commit f3b9e8c

Please sign in to comment.