upstream/mercurial-mirror Files · tests/test-atomictempfile.py

revset: make descendants() lazier...

revset: make descendants() lazier Previously descendants() would force the provided subset to become a set. In the case of revsets like '(%ld::) - (%ld)' (as used by histedit) this would force the '- (%ld)' set to be evaluated, which produced a set containing every commit in the repo (except %ld). This takes 0.6s on large repos. This changes descendants to trust the subset to implement __contains__ efficiently, which improves the above revset to 0.16s. Shaving 0.4 seconds off of histedit. revset : (20000::) - (20000) 0) obsolete feature not enabled but 54243 markers found! ! wall 0.023640 comb 0.020000 user 0.020000 sys 0.000000 (best of 100) 1) obsolete feature not enabled but 54243 markers found! ! wall 0.019589 comb 0.020000 user 0.020000 sys 0.000000 (best of 100) This commit removes the final revset related perf hotspot from histedit. Combined with the previous two patches, they shave a little over 3 seconds off histedit on large repos.

Idan Kamara - - Load All Authors

File last commit:

r18666:fb9d1c28 default


                r22449:da05fe01

default

Download file

             test-atomictempfile.py
        
                    42 lines
            
             | 1.3 KiB
            
                | text/x-python
            
             |
                PythonLexer
            
             / tests / test-atomictempfile.py
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      import os

      import glob

      import unittest

      import silenttestrunner

      from mercurial.util import atomictempfile

      class testatomictempfile(unittest.TestCase):

          def test1_simple(self):

              if os.path.exists('foo'):

                  os.remove('foo')

              file = atomictempfile('foo')

              (dir, basename) = os.path.split(file._tempname)

              self.assertFalse(os.path.isfile('foo'))

              self.assertTrue(basename in glob.glob('.foo-*'))

              file.write('argh\n')

              file.close()

              self.assertTrue(os.path.isfile('foo'))

              self.assertTrue(basename not in glob.glob('.foo-*'))

          # discard() removes the temp file without making the write permanent

          def test2_discard(self):

              if os.path.exists('foo'):

                  os.remove('foo')

              file = atomictempfile('foo')

              (dir, basename) = os.path.split(file._tempname)

              file.write('yo\n')

              file.discard()

              self.assertFalse(os.path.isfile('foo'))

              self.assertTrue(basename not in os.listdir('.'))

          # if a programmer screws up and passes bad args to atomictempfile, they

          # get a plain ordinary TypeError, not infinite recursion

          def test3_oops(self):

              self.assertRaises(TypeError, atomictempfile)

      if __name__ == '__main__':

          silenttestrunner.main(__name__)

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				import os
				import glob
				import unittest
				import silenttestrunner

				from mercurial.util import atomictempfile

				class testatomictempfile(unittest.TestCase):
				def test1_simple(self):
				if os.path.exists('foo'):
				os.remove('foo')
				file = atomictempfile('foo')
				(dir, basename) = os.path.split(file._tempname)
				self.assertFalse(os.path.isfile('foo'))
				self.assertTrue(basename in glob.glob('.foo-*'))

				file.write('argh\n')
				file.close()

				self.assertTrue(os.path.isfile('foo'))
				self.assertTrue(basename not in glob.glob('.foo-*'))

				# discard() removes the temp file without making the write permanent
				def test2_discard(self):
				if os.path.exists('foo'):
				os.remove('foo')
				file = atomictempfile('foo')
				(dir, basename) = os.path.split(file._tempname)

				file.write('yo\n')
				file.discard()

				self.assertFalse(os.path.isfile('foo'))
				self.assertTrue(basename not in os.listdir('.'))

				# if a programmer screws up and passes bad args to atomictempfile, they
				# get a plain ordinary TypeError, not infinite recursion
				def test3_oops(self):
				self.assertRaises(TypeError, atomictempfile)

				if __name__ == '__main__':
				silenttestrunner.main(__name__)