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

context: stop catching RepoLookupError from namespace.singlenode()...

context: stop catching RepoLookupError from namespace.singlenode() As pointed out by Yuya, the RepoLookupError was there for catching errors from repo.branchtip(). However, since (localrepo: add ignoremissing parameter to branchtip, 2014-10-16), that should no longer happen. I think it should now be an error if a namespace raises a RepoLookupError, so we propagate the exception up and and make it easy to fix, rather than trying to interpret the changeid as nodeid prefix and raise a general "unknown revision '...'" error. I also don't think we should catch FilteredLookupError and LookupError from the changelog.rev() call, for the same reason as above: If a namespace returns a node that doesn't exist, we should provide a more helpful exception than "unknown revision '...'". Differential Revision: https://phab.mercurial-scm.org/D3145

Pulkit Goyal - - Load All Authors

File last commit:

r36346:8d0b0b53 default


                r37409:3198d5a2

default

Download file

             test-mdiff.py
        
                    24 lines
            
             | 668 B
            
                | text/x-python
            
             |
                PythonLexer
            
             / tests / test-mdiff.py
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      from __future__ import absolute_import

      from __future__ import print_function

      import unittest

      from mercurial import (

          mdiff,

      )

      class splitnewlinesTests(unittest.TestCase):

          def test_splitnewlines(self):

              cases = {b'a\nb\nc\n': [b'a\n', b'b\n', b'c\n'],

                       b'a\nb\nc': [b'a\n', b'b\n', b'c'],

                       b'a\nb\nc\n\n': [b'a\n', b'b\n', b'c\n', b'\n'],

                       b'': [],

                       b'abcabc': [b'abcabc'],

                       }

              for inp, want in cases.items():

                  self.assertEqual(mdiff.splitnewlines(inp), want)

      if __name__ == '__main__':

          import silenttestrunner

          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

				from __future__ import absolute_import
				from __future__ import print_function

				import unittest

				from mercurial import (
				mdiff,
				)

				class splitnewlinesTests(unittest.TestCase):

				def test_splitnewlines(self):
				cases = {b'a\nb\nc\n': [b'a\n', b'b\n', b'c\n'],
				b'a\nb\nc': [b'a\n', b'b\n', b'c'],
				b'a\nb\nc\n\n': [b'a\n', b'b\n', b'c\n', b'\n'],
				b'': [],
				b'abcabc': [b'abcabc'],
				}
				for inp, want in cases.items():
				self.assertEqual(mdiff.splitnewlines(inp), want)

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