Ticket #5785: versions-5785.patch

File versions-5785.patch, 4.4 KB (added by thijs, 2 years ago)
  • twisted/python/test/test_versions.py

     
    9292        vb = Version("whatever", 1, 0, 0) 
    9393        self.assertTrue(va < vb) 
    9494        self.assertFalse(va > vb) 
    95         self.assertNotEquals(vb, va) 
     95        self.assertNotEqual(vb, va) 
    9696 
    9797 
    9898    def test_comparingPrereleases(self): 
     
    115115        self.assertEqual(_inf, _inf) 
    116116 
    117117 
    118     def testDontAllowBuggyComparisons(self): 
     118    def test_incomparableVersions(self): 
     119        """ 
     120        Comparing versions raises L{IncomparableVersions} when the package 
     121        names differ. 
     122        """ 
    119123        self.assertRaises(IncomparableVersions, 
    120124                          cmp, 
    121125                          Version("dummy", 1, 0, 0), 
    122126                          Version("dumym", 1, 0, 0)) 
    123127 
    124128 
     129    def test_dontAllowOtherObjects(self): 
     130        """ 
     131        Comparing versions when the other object is not a L{Version}. 
     132        """ 
     133        self.assertEqual(cmp(Version("dummy", 1, 0, 0), "foo"), 1) 
     134 
     135 
    125136    def test_repr(self): 
    126137        """ 
    127138        Calling C{repr} on a version returns a human-readable string 
     
    139150        self.assertEqual(repr(Version("dummy", 1, 2, 3, prerelease=4)), 
    140151                          "Version('dummy', 1, 2, 3, prerelease=4)") 
    141152 
    142  
    143153    def test_str(self): 
    144154        """ 
    145155        Calling C{str} on a version returns a human-readable string 
     
    157167                          "[dummy, version 1.0.0pre1]") 
    158168 
    159169 
    160     def testShort(self): 
     170    def test_shortVersion(self): 
     171        """ 
     172        Calling C{short()} on a version returns a short x.x.x representation of 
     173        the version. 
     174        """ 
    161175        self.assertEqual(Version('dummy', 1, 2, 3).short(), '1.2.3') 
    162176 
    163177 
     
    293307        parsed. 
    294308        """ 
    295309        self.checkSVNFormat("8", VERSION_8_ENTRIES, '22715') 
    296          
    297      
     310 
     311 
    298312    def test_detectVersion9(self): 
    299313        """ 
    300314        Verify that version 9 format files will be properly detected and 
    301315        parsed. 
    302316        """ 
    303317        self.checkSVNFormat("9", VERSION_9_ENTRIES, '22715') 
    304          
    305          
     318 
     319 
    306320    def test_detectVersion10(self): 
    307321        """ 
    308322        Verify that version 10 format files will be properly detected and 
     
    316330        self.assertEqual(self.getVersion()._getSVNVersion(), '22715') 
    317331 
    318332 
     333    def test_reprRevisionNumber(self): 
     334        """ 
     335        Calling C{repr} on a version with an SVN version returns a human-readable 
     336        string representation of the version including the revision number. 
     337        """ 
     338        self.svnEntries.child("entries").setContent(VERSION_10_ENTRIES) 
     339        self.assertEqual(repr(self.getVersion()), 
     340            "Version('twisted_python_versions_package', 1, 0, 0)  # (SVN r22715)") 
     341 
     342 
    319343    def test_detectUnknownVersion(self): 
    320344        """ 
    321         Verify that a new version of SVN will result in the revision 'Unknown'. 
     345        Verify that a new and unknown version of SVN will result in the 
     346        revision 'Unknown'. 
     347        """ 
     348        self.checkSVNFormat("some-random-new-version", "ooga booga!", 
     349            "Unknown") 
     350 
     351 
     352    def test_missingEntriesFile(self): 
     353        """ 
     354        Verify that a version 4 format file with invalid entries will result 
     355        in the revision 'Unknown'. 
    322356        """ 
    323         self.checkSVNFormat("some-random-new-version", "ooga booga!", 'Unknown') 
     357        self.checkSVNFormat("4", "foo", "Unknown") 
     358 
  • twisted/python/versions.py

     
    209209            formatFile = os.path.join(svn, 'format') 
    210210            if os.path.exists(formatFile): 
    211211                # It looks like a less-than-version-10 working copy. 
    212                 format = file(formatFile).read().strip() 
     212                format = open(formatFile).read().strip() 
    213213                parser = getattr(self, '_parseSVNEntries_' + format, None) 
    214214            else: 
    215215                # It looks like a version-10-or-greater working copy, which 
     
    220220                return 'Unknown' 
    221221 
    222222            entriesFile = os.path.join(svn, 'entries') 
    223             entries = file(entriesFile) 
     223            entries = open(entriesFile) 
    224224            try: 
    225225                try: 
    226226                    return parser(entries)