From: Éric Araujo <merwok@netwok.org>
Date: Thu, 6 Oct 2011 03:15:09 +0000 (+0200)
Subject: Add test that was promised in a comment but not actually written
X-Git-Tag: v3.3.0a1~1290^2~2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=fa23cc842c22b945f1fd1126810991daddbd7314;p=thirdparty%2FPython%2Fcpython.git

Add test that was promised in a comment but not actually written
---

diff --git a/Lib/packaging/tests/test_uninstall.py b/Lib/packaging/tests/test_uninstall.py
index 614b1877fad2..7603a40cd746 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]))