[Python-checkins] cpython: ../bug-fixes/http_error_interface/.hg/last-message.txt

[Senthil Kumaran]

Looks like I used hg commit -m /path/to/.hg/last-message.txt instead
of hg commit -l /path/to/.hg/last-message.txt
I have amended it, merged it and pushed it again.

On Tue, Mar 19, 2013 at 12:04 PM, senthil.kumaran
<python-checkins at python.org> wrote:
> http://hg.python.org/cpython/rev/4f2080e9eee2
> changeset:   82765:4f2080e9eee2
> parent:      82763:4c6463b96a2c
> user:        Senthil Kumaran <senthil at uthcode.com>
> date:        Tue Mar 19 12:07:43 2013 -0700
> summary:
>   ../bug-fixes/http_error_interface/.hg/last-message.txt
>
> files:
>   Lib/test/test_urllib2.py |  39 +++++++++++++--------------
>   1 files changed, 19 insertions(+), 20 deletions(-)
>
>
> diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py
> --- a/Lib/test/test_urllib2.py
> +++ b/Lib/test/test_urllib2.py
> @@ -1387,6 +1387,10 @@
>
>  class MiscTests(unittest.TestCase):
>
> +    def opener_has_handler(self, opener, handler_class):
> +        self.assertTrue(any(h.__class__ == handler_class
> +                            for h in opener.handlers))
> +
>      def test_build_opener(self):
>          class MyHTTPHandler(urllib.request.HTTPHandler): pass
>          class FooHandler(urllib.request.BaseHandler):
> @@ -1439,10 +1443,22 @@
>          self.assertEqual(b"1234567890", request.data)
>          self.assertEqual("10", request.get_header("Content-length"))
>
> +    def test_HTTPError_interface(self):
> +        """
> +        Issue 13211 reveals that HTTPError didn't implement the URLError
> +        interface even though HTTPError is a subclass of URLError.
>
> -    def opener_has_handler(self, opener, handler_class):
> -        self.assertTrue(any(h.__class__ == handler_class
> -                            for h in opener.handlers))
> +        >>> msg = 'something bad happened'
> +        >>> url = code = fp = None
> +        >>> hdrs = 'Content-Length: 42'
> +        >>> err = urllib.error.HTTPError(url, code, msg, hdrs, fp)
> +        >>> assert hasattr(err, 'reason')
> +        >>> err.reason
> +        'something bad happened'
> +        >>> assert hasattr(err, 'headers')
> +        >>> err.headers
> +        'Content-Length: 42'
> +        """
>
>  class RequestTests(unittest.TestCase):
>
> @@ -1514,23 +1530,6 @@
>          req = Request(url)
>          self.assertEqual(req.get_full_url(), url)
>
> -def test_HTTPError_interface():
> -    """
> -    Issue 13211 reveals that HTTPError didn't implement the URLError
> -    interface even though HTTPError is a subclass of URLError.
> -
> -    >>> msg = 'something bad happened'
> -    >>> url = code = fp = None
> -    >>> hdrs = 'Content-Length: 42'
> -    >>> err = urllib.error.HTTPError(url, code, msg, hdrs, fp)
> -    >>> assert hasattr(err, 'reason')
> -    >>> err.reason
> -    'something bad happened'
> -    >>> assert hasattr(err, 'headers')
> -    >>> err.headers
> -    'Content-Length: 42'
> -    """
> -
>  def test_main(verbose=None):
>      from test import test_urllib2
>      support.run_doctest(test_urllib2, verbose)
>
> --
> Repository URL: http://hg.python.org/cpython
>
> _______________________________________________
> Python-checkins mailing list
> Python-checkins at python.org
> http://mail.python.org/mailman/listinfo/python-checkins
>