From 95cc7bd7ff8511a218251dab4b588eff74b96b25 Mon Sep 17 00:00:00 2001 From: Diane Trout Date: Fri, 4 May 2012 12:05:29 -0700 Subject: [PATCH] 100% test coverage for alphanum.py --- htsworkflow/util/alphanum.py | 13 ++----------- htsworkflow/util/test/test_alphanum.py | 13 +++++++++++++ 2 files changed, 15 insertions(+), 11 deletions(-) diff --git a/htsworkflow/util/alphanum.py b/htsworkflow/util/alphanum.py index 5e25606..c9d6649 100644 --- a/htsworkflow/util/alphanum.py +++ b/htsworkflow/util/alphanum.py @@ -35,10 +35,10 @@ def chunkify(str): return a list of numbers and non-numeric substrings of +str+ the numeric substrings are converted to integer, non-numeric are left as is """ - if type(str) in types.StringTypes: + if type(str) in types.StringTypes: chunks = re.findall("(\d+|\D+)",str) #convert numeric strings to numbers - chunks = [re.match('\d',x) and int(x) or x for x in chunks] + chunks = [re.match('\d',x) and int(x) or x for x in chunks] return chunks elif type(str) in [types.IntType, types.LongType, types.FloatType]: return [str] @@ -58,12 +58,3 @@ def alphanum(a,b): bChunks = chunkify(b) return cmp(aChunks,bChunks) #built in comparison works once data is prepared - - - -if __name__ == "__main__": - unsorted = ["1000X Radonius Maximus","10X Radonius","200X Radonius","20X Radonius","20X Radonius Prime","30X Radonius","40X Radonius","Allegia 50 Clasteron","Allegia 500 Clasteron","Allegia 51 Clasteron","Allegia 51B Clasteron","Allegia 52 Clasteron","Allegia 60 Clasteron","Alpha 100","Alpha 2","Alpha 200","Alpha 2A","Alpha 2A-8000","Alpha 2A-900","Callisto Morphamax","Callisto Morphamax 500","Callisto Morphamax 5000","Callisto Morphamax 600","Callisto Morphamax 700","Callisto Morphamax 7000","Callisto Morphamax 7000 SE","Callisto Morphamax 7000 SE2","QRS-60 Intrinsia Machine","QRS-60F Intrinsia Machine","QRS-62 Intrinsia Machine","QRS-62F Intrinsia Machine","Xiph Xlater 10000","Xiph Xlater 2000","Xiph Xlater 300","Xiph Xlater 40","Xiph Xlater 5","Xiph Xlater 50","Xiph Xlater 500","Xiph Xlater 5000","Xiph Xlater 58"] - sorted = unsorted[:] - sorted.sort(alphanum) - print '+++++Sorted...++++' - print '\n'.join(sorted) diff --git a/htsworkflow/util/test/test_alphanum.py b/htsworkflow/util/test/test_alphanum.py index f7b488d..3fd6025 100644 --- a/htsworkflow/util/test/test_alphanum.py +++ b/htsworkflow/util/test/test_alphanum.py @@ -27,6 +27,19 @@ class testAlphanum(unittest.TestCase): for i in xrange(len(scratch)): self.failUnlessEqual(scratch[i], sorted[i]) + def test_long_names(self): + unsorted = ["1000X Radonius Maximus","10X Radonius","200X Radonius","20X Radonius","20X Radonius Prime","30X Radonius","40X Radonius","Allegia 50 Clasteron","Allegia 500 Clasteron","Allegia 51 Clasteron","Allegia 51B Clasteron","Allegia 52 Clasteron","Allegia 60 Clasteron","Alpha 100","Alpha 2","Alpha 200","Alpha 2A","Alpha 2A-8000","Alpha 2A-900","Callisto Morphamax","Callisto Morphamax 500","Callisto Morphamax 5000","Callisto Morphamax 600","Callisto Morphamax 700","Callisto Morphamax 7000","Callisto Morphamax 7000 SE","Callisto Morphamax 7000 SE2","QRS-60 Intrinsia Machine","QRS-60F Intrinsia Machine","QRS-62 Intrinsia Machine","QRS-62F Intrinsia Machine","Xiph Xlater 10000","Xiph Xlater 2000","Xiph Xlater 300","Xiph Xlater 40","Xiph Xlater 5","Xiph Xlater 50","Xiph Xlater 500","Xiph Xlater 5000","Xiph Xlater 58"] + expected = ['10X Radonius', '20X Radonius', '20X Radonius Prime', '30X Radonius', '40X Radonius', '200X Radonius', '1000X Radonius Maximus', 'Allegia 50 Clasteron', 'Allegia 51 Clasteron', 'Allegia 51B Clasteron', 'Allegia 52 Clasteron', 'Allegia 60 Clasteron', 'Allegia 500 Clasteron', 'Alpha 2', 'Alpha 2A', 'Alpha 2A-900', 'Alpha 2A-8000', 'Alpha 100', 'Alpha 200', 'Callisto Morphamax', 'Callisto Morphamax 500', 'Callisto Morphamax 600', 'Callisto Morphamax 700', 'Callisto Morphamax 5000', 'Callisto Morphamax 7000', 'Callisto Morphamax 7000 SE', 'Callisto Morphamax 7000 SE2', 'QRS-60 Intrinsia Machine', 'QRS-60F Intrinsia Machine', 'QRS-62 Intrinsia Machine', 'QRS-62F Intrinsia Machine', 'Xiph Xlater 5', 'Xiph Xlater 40', 'Xiph Xlater 50', 'Xiph Xlater 58', 'Xiph Xlater 300', 'Xiph Xlater 500', 'Xiph Xlater 2000', 'Xiph Xlater 5000', 'Xiph Xlater 10000'] + + s = unsorted[:] + s.sort(alphanum) + self.failUnlessEqual(s, expected) + + def test_bad_input(self): + unsorted = [object(), (1,3j)] + s = unsorted[:] + self.failUnlessRaises(ValueError, s.sort, alphanum) + def suite(): return unittest.makeSuite(testAlphanum, 'test') -- 2.30.2