From fa945d5875edeb3b466c4d1c01d675a8543f8f39 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakub=20Kadl=C4=8D=C3=ADk?= Date: Thu, 10 Sep 2015 13:28:21 +0200 Subject: [PATCH] Change asserted behavior after fe4c0bf --- test/functional/release_copr_tests.py | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/test/functional/release_copr_tests.py b/test/functional/release_copr_tests.py index adb4c42..5f56495 100644 --- a/test/functional/release_copr_tests.py +++ b/test/functional/release_copr_tests.py @@ -17,6 +17,7 @@ Functional Tests for the CoprReleaser. from functional.fixture import TitoGitTestFixture +import mock from tito.compat import * # NOQA from tito.release import CoprReleaser from unit import Capture @@ -72,9 +73,14 @@ class CoprReleaserTests(TitoGitTestFixture): releaser.release(dry_run=True) self.assertTrue(releaser.srpm_submitted is not None) - def test_no_remote_defined(self): + @mock.patch("tito.release.CoprReleaser._submit") + @mock.patch("tito.release.CoprReleaser._upload") + def test_no_remote_defined(self, upload, submit): self.releaser_config.remove_option("test", "remote_location") - with Capture(silent=True): - self.assertRaises(SystemExit, CoprReleaser, PKG_NAME, None, '/tmp/tito/', - self.config, {}, 'test', self.releaser_config, False, - False, False, **{'offline': True}) + releaser = CoprReleaser(PKG_NAME, None, '/tmp/tito/', + self.config, {}, 'test', self.releaser_config, False, + False, False, **{'offline': True}) + releaser.release(dry_run=True) + + self.assertFalse(upload.called) + self.assertTrue(submit.called)