Message193364
Anyway, I found another issue on line 393 on the same file:
def test_del_nonexistent_param(self):
msg = Message()
msg.add_header('Content-Type', 'text/plain', charset='utf-8')
existing_header = msg['Content-Type']
msg.del_param('foobar', header='Content-Type')
self.assertEqual(msg['Content-Type'], 'text/plain; charset="utf-8"')
The variable existing_header is never used. Either we can remove it or change it to:
def test_del_nonexistent_param(self):
msg = Message()
msg.add_header('Content-Type', 'text/plain', charset='utf-8')
existing_header = msg['Content-Type']
msg.del_param('foobar', header='Content-Type')
self.assertEqual(msg['Content-Type'], existing_header)
At first, I wanted to create a ticket for this. But then, I thought why do not we combine these menial problems together? |
|
Date |
User |
Action |
Args |
2013-07-19 16:06:37 | vajrasky | set | recipients:
+ vajrasky, r.david.murray |
2013-07-19 16:06:37 | vajrasky | set | messageid: <1374249997.56.0.452353924918.issue18503@psf.upfronthosting.co.za> |
2013-07-19 16:06:37 | vajrasky | link | issue18503 messages |
2013-07-19 16:06:37 | vajrasky | create | |
|