Corrections and feedback.

system_tests/vision.py

@@ -24,6 +24,7 @@
 
 from system_test_utils import unique_resource_id
 from retry import RetryErrors
+import six
 
 
 _SYS_TESTS_DIR = os.path.abspath(os.path.dirname(__file__))
@@ -60,10 +61,10 @@ def _assert_coordinate(self, coordinate):
         if coordinate is None:
             return
         self.assertIsInstance(coordinate, (int, float))
-        self.assertGreater(abs(coordinate), 0.0)
+        self.assertNotEqual(coordinate, 0.0)
 
 
-class TestVisionClientLogo(BaseVisionTestCase):
+class TestVisionClientLogo(unittest.TestCase):
     def setUp(self):
         self.to_delete_by_case = []
 
@@ -221,6 +222,10 @@ def test_detect_faces_filename(self):
 
 
 class TestVisionClientLabel(BaseVisionTestCase):
+    DESCRIPTIONS = ['car', 'vehicle', 'land vehicle', 'automotive design',
+                    'wheel', 'automobile make', 'luxury vehicle',
+                    'sports car', 'performance car', 'automotive exterior']
+
     def setUp(self):
         self.to_delete_by_case = []
 
@@ -229,12 +234,10 @@ def tearDown(self):
             value.delete()
 
     def _assert_label(self, label):
-        descriptions = ['car', 'vehicle', 'land vehicle', 'automotive design',
-                        'wheel', 'automobile make', 'luxury vehicle',
-                        'sports car', 'performance car', 'automotive exterior']
+
         self.assertIsInstance(label, EntityAnnotation)
-        self.assertIn(label.description, descriptions)
-        self.assertIsInstance(label.mid, unicode)
+        self.assertIn(label.description, self.DESCRIPTIONS)
+        self.assertIsInstance(label.mid, six.text_type)
         self.assertGreater(label.score, 0.0)
 
     def test_detect_labels_content(self):