upstream/mercurial-mirror Files · tests/test-ui-color.py

run-tests: mechanism to report exceptions during test execution...

run-tests: mechanism to report exceptions during test execution Sometimes when running tests you introduce a ton of exceptions. The most extreme example of this is running Mercurial with Python 3, which currently spews thousands of exceptions when running the test harness. This commit adds an opt-in feature to run-tests.py to aggregate exceptions encountered by `hg` when running tests. When --exceptions is used, the test harness enables the "logexceptions" extension in the test environment. This extension wraps the Mercurial function to handle exceptions and writes information about the exception to a random filename in a directory defined by the test harness via an environment variable. At the end of the test harness, these files are parsed, aggregated, and a list of all unique Mercurial frames triggering exceptions is printed in order of frequency. This feature is intended to aid Python 3 development. I've only really tested it on Python 3. There is no shortage of improvements that could be made. e.g. we could write a separate file containing the exception report - maybe even an HTML report. We also don't capture which tests demonstrate the exceptions, so there's no turnkey way to test whether a code change made an exception disappear. Perfect is the enemy of good. I think the current patch is useful enough to land. Whoever uses it can send patches to imprve its usefulness. Differential Revision: https://phab.mercurial-scm.org/D1477

Pierre-Yves David - - Load All Authors

File last commit:

r31095:b4cb86ab default


                r35191:bd8875b6

default

Download file

             test-ui-color.py
        
                    38 lines
            
             | 921 B
            
                | text/x-python
            
             |
                PythonLexer
            
             / tests / test-ui-color.py
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        Pulkit Goyal
    
tests: make test-ui-color use absolute_import

              r28915
            
      from __future__ import absolute_import, print_function

        Simon Heimberg
    
cleanup: remove unused imports...

              r19322
            
      import os

        Pulkit Goyal
    
tests: make test-ui-color use absolute_import

              r28915
            
      from mercurial import (

          dispatch,

          ui as uimod,

      )

        Brodie Rao
    
color: call correct superclass method in write_err...

              r11732
            
      # ensure errors aren't buffered

        Pierre-Yves David
    
color: drop the 'colorui' class...

              r31095
            
      testui = uimod.ui()

        Brodie Rao
    
color: call correct superclass method in write_err...

              r11732
            
      testui.pushbuffer()

        Matt Mackall
    
i18n: wrap false positives for translation detection

              r17956
            
      testui.write(('buffered\n'))

      testui.warn(('warning\n'))

        Brodie Rao
    
color: call correct superclass method in write_err...

              r11732
            
      testui.write_err('error\n')

        Pulkit Goyal
    
py3: make test-ui-color use print_function

              r28682
            
      print(repr(testui.popbuffer()))

        Idan Kamara
    
color: check if ui is already a subclass of colorui before wrapping it...

              r14516
            
      # test dispatch.dispatch with the same ui object

      hgrc = open(os.environ["HGRCPATH"], 'w')

      hgrc.write('[extensions]\n')

      hgrc.write('color=\n')

      hgrc.close()

        Yuya Nishihara
    
ui: factor out ui.load() to create a ui without loading configs (API)...

              r30559
            
      ui_ = uimod.ui.load()

        Idan Kamara
    
color: check if ui is already a subclass of colorui before wrapping it...

              r14516
            
      ui_.setconfig('ui', 'formatted', 'True')

        Idan Kamara
    
ui: use I/O descriptors internally...

              r14614
            
      # we're not interested in the output, so write that to devnull

      ui_.fout = open(os.devnull, 'w')

        Idan Kamara
    
color: check if ui is already a subclass of colorui before wrapping it...

              r14516
            
      # call some arbitrary command just so we go through

      # color's wrapped _runcommand twice.

      def runcmd():

          dispatch.dispatch(dispatch.request(['version', '-q'], ui_))

      runcmd()

        Pierre-Yves David
    
color: drop the 'colorui' class...

              r31095
            
      print("colored? %s" % (ui_._colormode is not None))

        Idan Kamara
    
color: check if ui is already a subclass of colorui before wrapping it...

              r14516
            
      runcmd()

        Pierre-Yves David
    
color: drop the 'colorui' class...

              r31095
            
      print("colored? %s" % (ui_._colormode is not None))

	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

Pulkit Goyal tests: make test-ui-color use absolute_import	r28915	from __future__ import absolute_import, print_function

Simon Heimberg cleanup: remove unused imports...	r19322	import os
Pulkit Goyal tests: make test-ui-color use absolute_import	r28915	from mercurial import (
		dispatch,
		ui as uimod,
		)
Brodie Rao color: call correct superclass method in write_err...	r11732
		# ensure errors aren't buffered
Pierre-Yves David color: drop the 'colorui' class...	r31095	testui = uimod.ui()
Brodie Rao color: call correct superclass method in write_err...	r11732	testui.pushbuffer()
Matt Mackall i18n: wrap false positives for translation detection	r17956	testui.write(('buffered\n'))
		testui.warn(('warning\n'))
Brodie Rao color: call correct superclass method in write_err...	r11732	testui.write_err('error\n')
Pulkit Goyal py3: make test-ui-color use print_function	r28682	print(repr(testui.popbuffer()))
Idan Kamara color: check if ui is already a subclass of colorui before wrapping it...	r14516
		# test dispatch.dispatch with the same ui object
		hgrc = open(os.environ["HGRCPATH"], 'w')
		hgrc.write('[extensions]\n')
		hgrc.write('color=\n')
		hgrc.close()

Yuya Nishihara ui: factor out ui.load() to create a ui without loading configs (API)...	r30559	ui_ = uimod.ui.load()
Idan Kamara color: check if ui is already a subclass of colorui before wrapping it...	r14516	ui_.setconfig('ui', 'formatted', 'True')

Idan Kamara ui: use I/O descriptors internally...	r14614	# we're not interested in the output, so write that to devnull
		ui_.fout = open(os.devnull, 'w')

Idan Kamara color: check if ui is already a subclass of colorui before wrapping it...	r14516	# call some arbitrary command just so we go through
		# color's wrapped _runcommand twice.
		def runcmd():
		dispatch.dispatch(dispatch.request(['version', '-q'], ui_))

		runcmd()
Pierre-Yves David color: drop the 'colorui' class...	r31095	print("colored? %s" % (ui_._colormode is not None))
Idan Kamara color: check if ui is already a subclass of colorui before wrapping it...	r14516	runcmd()
Pierre-Yves David color: drop the 'colorui' class...	r31095	print("colored? %s" % (ui_._colormode is not None))