From fa23cc842c22b945f1fd1126810991daddbd7314 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89ric=20Araujo?= Date: Thu, 6 Oct 2011 05:15:09 +0200 Subject: [PATCH] Add test that was promised in a comment but not actually written --- Lib/packaging/tests/test_uninstall.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Lib/packaging/tests/test_uninstall.py b/Lib/packaging/tests/test_uninstall.py index 614b1877fad..7603a40cd74 100644 --- a/Lib/packaging/tests/test_uninstall.py +++ b/Lib/packaging/tests/test_uninstall.py @@ -1,6 +1,7 @@ """Tests for the uninstall command.""" import os import sys +import logging from io import StringIO import stat import packaging.util @@ -105,14 +106,14 @@ class UninstallTestCase(support.TempdirManager, def test_remove_issue(self): # makes sure if there are OSErrors (like permission denied) - # remove() stops and display a clean error + # remove() stops and displays a clean error dist, site_packages = self.install_dist('Meh') # breaking os.rename old = os.rename def _rename(source, target): - raise OSError + raise OSError(42, 'impossible operation') os.rename = _rename try: @@ -120,6 +121,10 @@ class UninstallTestCase(support.TempdirManager, finally: os.rename = old + logs = [log for log in self.get_logs(logging.INFO) + if log.startswith('Error:')] + self.assertEqual(logs, ['Error: [Errno 42] impossible operation']) + self.assertTrue(remove('Meh', paths=[site_packages]))