[cmpcodesize] Add some tests for listFunctionSizes

Add unit tests to cmpcodesize. As a basic starting point for unit
tests, add some tests for edge cases in listFunctionSizes.

Users may now run the unit tests for cmpcodesize by running
`python utils/cmpcodesize/setup.py test`.

Author: modocache

utils/cmpcodesize/cmpcodesize/compare.py

@@ -192,7 +192,7 @@ def listFunctionSizes(sizeArray):
     for pair in sorted(sizeArray, key=itemgetter(1)):
         name = pair[0]
         size = pair[1]
-        print "%8d %s" % (size, name)
+        return "%8d %s" % (size, name)
 
 
 def compareFunctionSizes(oldFiles, newFiles):
@@ -227,13 +227,13 @@ def compareFunctionSizes(oldFiles, newFiles):
 
     if onlyInFile1:
         print "Only in old file(s)"
-        listFunctionSizes(onlyInFile1)
+        print listFunctionSizes(onlyInFile1)
         print "Total size of functions only in old file: {}".format(onlyInFile1Size)
         print
 
     if onlyInFile2:
         print "Only in new files(s)"
-        listFunctionSizes(onlyInFile2)
+        print listFunctionSizes(onlyInFile2)
         print "Total size of functions only in new file: {}".format(onlyInFile2Size) 
         print
 

utils/cmpcodesize/cmpcodesize/main.py

@@ -166,7 +166,7 @@ def main():
             sizes = collections.defaultdict(int)
             for file in oldFiles:
                 readSizes(sizes, file, True, False)
-            listFunctionSizes(sizes.items())
+            print listFunctionSizes(sizes.items())
         else:
             compareFunctionSizes(oldFiles, newFiles)
     else:

utils/cmpcodesize/setup.py

@@ -20,6 +20,8 @@
     description="A tool to compare the size of Swift compiler build products.",
     keywords='compare size swift',
 
+    test_suite='tests',
+
     classifiers=[
         'Development Status :: 3 - Alpha',
         'Environment :: Console',

utils/cmpcodesize/tests/__init__.py

@@ -0,0 +1,16 @@
+import unittest
+
+from cmpcodesize.compare import listFunctionSizes
+
+
+class ListFunctionSizesTestCase(unittest.TestCase):
+    def test_when_size_array_is_none_raises(self):
+        with self.assertRaises(TypeError):
+            listFunctionSizes(None)
+
+    def test_when_size_array_is_empty_returns_none(self):
+        self.assertIsNone(listFunctionSizes([]))
+
+
+if __name__ == '__main__':
+    unittest.main()