Rev 6593: Minimal tests to cover using distutils.version.LooseVersion for pyrex. in file:///home/vila/src/bzr/reviews/lp1030521/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/reviews/lp1030521/

------------------------------------------------------------
revno: 6593
revision-id: v.ladeuil+lp at free.fr-20140402161244-loeovaqcu4e1dk6u
parent: a.starr.b at gmail.com-20140330175929-rd97jstcbau2j1gy
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: lp1030521
timestamp: Wed 2014-04-02 18:12:44 +0200
message:
  Minimal tests to cover using distutils.version.LooseVersion for pyrex.
-------------- next part --------------
=== modified file 'bzrlib/tests/test_setup.py'
--- a/bzrlib/tests/test_setup.py	2012-03-29 11:24:16 +0000
+++ b/bzrlib/tests/test_setup.py	2014-04-02 16:12:44 +0000
@@ -16,6 +16,7 @@
 
 """Test for setup.py build process"""
 
+from distutils import version
 import os
 import sys
 import subprocess
@@ -79,3 +80,13 @@
         self.log('stderr: %r', stderr)
         self.assertEqual(0, p.returncode,
                          'invocation of %r failed' % args)
+
+
+class TestDistutilsVersion(tests.TestCase):
+
+    def test_version_with_string(self):
+        # We really care about two pyrex specific versions and our ability to
+        # detect them
+        lv = version.LooseVersion
+        self.assertTrue(lv("0.9.4.1") < lv('0.17.beta1'))
+        self.assertTrue(lv("0.9.6.3") < lv('0.10'))