Use testtools.SkipIf instead so that secrets mocking isn't broken for python 3.5.

Author: camporter

tests/CLI/modules/user_tests.py

@@ -6,6 +6,7 @@
 """
 import json
 import sys
+import unittest
 
 import mock
 
@@ -167,12 +168,10 @@ def test_create_user_no_confirm(self, confirm_mock):
         result = self.run_command(['user', 'create', 'test', '-e', '[email protected]', '-p', 'testword'])
         self.assertEqual(result.exit_code, 2)
 
+    @unittest.SkipIf(sys.version_info < (3, 6), "Secrets module only exists in version 3.6+")
     @mock.patch('secrets.choice')
     @mock.patch('SoftLayer.CLI.formatting.confirm')
     def test_create_user_generate_password_36(self, confirm_mock, secrets):
-        if sys.version_info < (3, 6):
-            self.skipTest("Secrets module only exists in version 3.6+")
-
         secrets.return_value = 'Q'
         confirm_mock.return_value = True
         result = self.run_command(['user', 'create', 'test', '-e', '[email protected]', '-p', 'generate'])