New version of flake8 found some errors

insights/client/utilities.py

@@ -473,7 +473,7 @@ def largest_spec_in_archive(archive_file):
                 abs_fname = os.path.join('.', data_top, fname)
                 # get the archives from inside data directory
                 data_file = tar_file.getmember(abs_fname)
-                if(data_file.size > largest_fsize):
+                if (data_file.size > largest_fsize):
                     largest_fsize = data_file.size
                     largest_file_name = fname
                     largest_spec = specs_metadata["name"]

insights/parsers/__init__.py

@@ -284,7 +284,7 @@ def calc_offset(lines, target, invert_search=False, require_all=False):
                         return offset
                 else:
                     return offset
-            elif invert_search and not(line == '' or found_any):
+            elif invert_search and not (line == '' or found_any):
                 return offset
 
         # If we get here then we didn't find any of the targets

insights/parsers/passenger_status.py

@@ -84,7 +84,7 @@ def parse_content(self, content):
                         group_list['p_list'].append(p_dict)
                     l = line.lstrip('* ')
                     # parse each key: value pair
-                    while(':' in l and l.index(':') != l.rindex(':')):
+                    while (':' in l and l.index(':') != l.rindex(':')):
                         k, l = [i.strip() for i in l.split(':', 1)]
                         v, l = [i.strip() for i in l.split(None, 1)]
                         p_dict[k] = v

insights/parsers/sealert.py

@@ -29,7 +29,7 @@ def lines_stripped(self):
         lines = self.lines[:]
         for index in range(len(self.lines) - 1, -1, -1):
             if lines[index] == '':
-                del(lines[index])
+                del (lines[index])
             else:
                 break
         return lines

insights/parsers/uname.py

@@ -445,7 +445,7 @@ def __ne__(self, other):
 
         See the `__eq__` operator for a detailed description.
         """
-        return not(self == other)
+        return not (self == other)
 
     def __lt__(self, other):
         """

insights/tests/parsers/test_satellite_postgresql_query.py

@@ -221,12 +221,12 @@ def test_basic_output_with_satellite_admin_setting():
 
 def test_satellite_admin_settings():
     settings = satellite_postgresql_query.SatelliteAdminSettings(context_wrap(SATELLITE_SETTINGS_2))
-    assert(len(settings)) == 2
+    assert (len(settings)) == 2
     assert not settings.get_setting('unregister_delete_host')
     assert settings.get_setting('destroy_vm_on_host_delete')
 
     settings = satellite_postgresql_query.SatelliteAdminSettings(context_wrap(SATELLITE_SETTINGS_3))
-    assert(len(settings)) == 2
+    assert (len(settings)) == 2
     assert not settings.get_setting('unregister_delete_host')
     assert not settings.get_setting('destroy_vm_on_host_delete')
     assert settings.get_setting('non_exist_column') is None