Merge "Enable some off-by-default checks"

Author: Jenkins

openstackclient/tests/functional/volume/v1/test_service.py

@@ -64,10 +64,7 @@ def test_volume_service_set(self):
             'enabled',
             cmd_output[0]['Status']
         )
-        self.assertEqual(
-            None,
-            cmd_output[0]['Disabled Reason']
-        )
+        self.assertIsNone(cmd_output[0]['Disabled Reason'])
 
         # Test volume service set --disable and --disable-reason
         disable_reason = 'disable_reason'

openstackclient/tests/functional/volume/v2/test_service.py

@@ -64,8 +64,7 @@ def test_volume_service_set(self):
             'enabled',
             cmd_output[0]['Status']
         )
-        self.assertEqual(
-            None,
+        self.assertIsNone(
             cmd_output[0]['Disabled Reason']
         )
 

tox.ini

@@ -85,6 +85,8 @@ commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasen
 
 [flake8]
 show-source = True
+# H203: Use assertIs(Not)None to check for None
+enable-extensions = H203
 exclude = .git,.tox,dist,doc,*lib/python*,*egg,build,tools
 # If 'ignore' is not set there are default errors and warnings that are set
 # Doc: http://flake8.readthedocs.org/en/latest/config.html#default