The idea here is to exercise the DEFAULT_METHOD_WHITELIST. I've renamed 
the test to make this clear.

On 1/29/14, 0:26, Cory Benfield wrote:
>
> In test/with_dummyserver/test_connectionpool.py:
>
> > +        headers = {'test-name': 'test_read_total_retries'}
> > +        retry = Retry(read=0, connect=0, total=3, codes_whitelist=[418])
> > +        resp = self.pool.request('GET', '/successful_retry',
> > +                                 headers=headers, retries=retry)
> > +        self.assertEqual(resp.status, 200)
> > +
> > +    def test_retries_wrong_whitelist(self):
> > +        retry = Retry(read=3, connect=0, codes_whitelist=[202])
> > +        resp = self.pool.request('GET', '/successful_retry',
> > +                                 headers={'test-name': 'test_wrong_whitelist'},
> > +                                 retries=retry)
> > +        self.assertEqual(resp.status, 418)
> > +
> > +    def test_retries_odd_whitelist(self):
> > +        retry = Retry(read=3, connect=0, codes_whitelist=[418])
> > +        resp = self.pool.request('OPTIONS', '/successful_retry',
>
> Is there any particular reason this is an |OPTIONS| request? It leaps 
> out at me as being different but I can't see why it is, it seems 
> irrelevant for the test at hand.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/shazow/urllib3/pull/326/files#r9258858>.
>