fix up assertEqual(None...) check to catch more cases
To finally close out bug #1280522 it turned out that our hacking check
to eliminate assertEqual(None, ...) was pretty sub optimal. The
perfect fix would actually use the tokenizer, however this is a better
regex which catches a few more missing bits in our codebase. It's
probably close enough for most of what we need.
Fix the relevant tests that would fail the updating hacking check.
Reviewed: https:/ /review. openstack. org/121582 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=5dd34c93e2c 86fdba7b71e2702 9a400777bf9424
Committed: https:/
Submitter: Jenkins
Branch: master
commit 5dd34c93e2c86fd ba7b71e27029a40 0777bf9424
Author: Sean Dague <email address hidden>
Date: Mon Sep 15 10:00:36 2014 -0400
fix up assertEqual( None... ) check to catch more cases
To finally close out bug #1280522 it turned out that our hacking check
to eliminate assertEqual(None, ...) was pretty sub optimal. The
perfect fix would actually use the tokenizer, however this is a better
regex which catches a few more missing bits in our codebase. It's
probably close enough for most of what we need.
Fix the relevant tests that would fail the updating hacking check.
Change-Id: Ic8cb1192e00140 9d827c8da55fe53 6681895944b
Closes-Bug: #1280522