[Python-checkins] r69387 - in python/branches/py3k: Lib/distutils/cmd.py Lib/distutils/tests/test_cmd.py

[tarek.ziade]

Author: tarek.ziade
Date: Sat Feb  7 01:10:48 2009
New Revision: 69387

Log:
Merged revisions 69385 via svnmerge from 
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r69385 | tarek.ziade | 2009-02-07 01:05:39 +0100 (Sat, 07 Feb 2009) | 1 line
  
  #3986 replacing string and types call (like in the Py3k branch), and put exec_msg call at the right place
........


Modified:
   python/branches/py3k/   (props changed)
   python/branches/py3k/Lib/distutils/cmd.py
   python/branches/py3k/Lib/distutils/tests/test_cmd.py

Modified: python/branches/py3k/Lib/distutils/cmd.py
==============================================================================
--- python/branches/py3k/Lib/distutils/cmd.py	(original)
+++ python/branches/py3k/Lib/distutils/cmd.py	Sat Feb  7 01:10:48 2009
@@ -155,15 +155,15 @@
         from distutils.fancy_getopt import longopt_xlate
         if header is None:
             header = "command options for '%s':" % self.get_command_name()
-        print(indent + header)
+        self.announce(indent + header, level=log.INFO)
         indent = indent + "  "
         for (option, _, _) in self.user_options:
             option = longopt_xlate(option)
             if option[-1] == "=":
                 option = option[:-1]
             value = getattr(self, option)
-            print(indent + "%s = %s" % (option, value))
-
+            self.announce(indent + "%s = %s" % (option, value),
+                          level=log.INFO)
 
     def run(self):
         """A command's raison d'etre: carry out the action it exists to
@@ -383,12 +383,9 @@
         and it is true, then the command is unconditionally run -- does no
         timestamp checks.
         """
-        if exec_msg is None:
-            exec_msg = "generating %s from %s" % (outfile, ', '.join(infiles))
         if skip_msg is None:
             skip_msg = "skipping %s (inputs unchanged)" % outfile
 
-
         # Allow 'infiles' to be a single string
         if isinstance(infiles, str):
             infiles = (infiles,)
@@ -396,10 +393,13 @@
             raise TypeError(
                   "'infiles' must be a string, or a list or tuple of strings")
 
+        if exec_msg is None:
+            exec_msg = "generating %s from %s" % (outfile, ', '.join(infiles))
+
         # If 'outfile' must be regenerated (either because it doesn't
         # exist, is out-of-date, or the 'force' flag is true) then
         # perform the action that presumably regenerates it
-        if self.force or dep_util.newer_group (infiles, outfile):
+        if self.force or dep_util.newer_group(infiles, outfile):
             self.execute(func, args, exec_msg, level)
         # Otherwise, print the "skip" message
         else:

Modified: python/branches/py3k/Lib/distutils/tests/test_cmd.py
==============================================================================
--- python/branches/py3k/Lib/distutils/tests/test_cmd.py	(original)
+++ python/branches/py3k/Lib/distutils/tests/test_cmd.py	Sat Feb  7 01:10:48 2009
@@ -5,23 +5,23 @@
 from distutils.dist import Distribution
 from distutils.errors import DistutilsOptionError
 
-class CommandTestCase(unittest.TestCase):
-
-    def test_ensure_string_list(self):
+class MyCmd(Command):
+    def initialize_options(self):
+        pass
 
-        class MyCmd(Command):
-
-            def initialize_options(self):
-                pass
+class CommandTestCase(unittest.TestCase):
 
+    def setUp(self):
         dist = Distribution()
-        cmd = MyCmd(dist)
+        self.cmd = MyCmd(dist)
 
+    def test_ensure_string_list(self):
+
+        cmd = self.cmd
         cmd.not_string_list = ['one', 2, 'three']
         cmd.yes_string_list = ['one', 'two', 'three']
         cmd.not_string_list2 = object()
         cmd.yes_string_list2 = 'ok'
-
         cmd.ensure_string_list('yes_string_list')
         cmd.ensure_string_list('yes_string_list2')
 
@@ -31,6 +31,37 @@
         self.assertRaises(DistutilsOptionError,
                           cmd.ensure_string_list, 'not_string_list2')
 
+    def test_make_file(self):
+
+        cmd = self.cmd
+
+        # making sure it raises when infiles is not a string or a list/tuple
+        self.assertRaises(TypeError, cmd.make_file,
+                          infiles=1, outfile='', func='func', args=())
+
+        # making sure execute gets called properly
+        def _execute(func, args, exec_msg, level):
+            self.assertEquals(exec_msg, 'generating out from in')
+        cmd.force = True
+        cmd.execute = _execute
+        cmd.make_file(infiles='in', outfile='out', func='func', args=())
+
+    def test_dump_options(self):
+
+        msgs = []
+        def _announce(msg, level):
+            msgs.append(msg)
+        cmd = self.cmd
+        cmd.announce = _announce
+        cmd.option1 = 1
+        cmd.option2 = 1
+        cmd.user_options = [('option1', '', ''), ('option2', '', '')]
+        cmd.dump_options()
+
+        wanted = ["command options for 'MyCmd':", '  option1 = 1',
+                  '  option2 = 1']
+        self.assertEquals(msgs, wanted)
+
 def test_suite():
     return unittest.makeSuite(CommandTestCase)