Fix some flake8 and isort errors

Author: peternewman

data/rdm/PidDataTest.py

@@ -51,5 +51,6 @@ def testLoad(self):
     self.assertEqual(store.ManufacturerIdToName(32767),
                      "RESERVED FOR PROTOTYPING/EXPERIMENTAL USE ONLY")
 
+
 if __name__ == '__main__':
   unittest.main()

tools/rdm/TestDefinitions.py

@@ -27,8 +27,9 @@
                               RDM_MANUFACTURER_PID_MIN,
                               RDM_MANUFACTURER_SD_MAX, RDM_MANUFACTURER_SD_MIN,
                               RDM_MAX_DOMAIN_NAME_LENGTH,
-                              RDM_MAX_HOSTNAME_LENGTH, RDM_MIN_HOSTNAME_LENGTH,
+                              RDM_MAX_HOSTNAME_LENGTH,
                               RDM_MAX_SERIAL_NUMBER_LENGTH,
+                              RDM_MIN_HOSTNAME_LENGTH,
                               RDM_ZERO_FOOTPRINT_DMX_ADDRESS)
 from ola.StringUtils import StringEscape
 from ola.testing.rdm import TestMixins

tools/rdm/TestMixins.py

@@ -201,7 +201,8 @@ def VerifyResult(self, response, fields):
 
     if self.MAX_LENGTH and len(url_field) > self.MAX_LENGTH:
       self.AddAdvisory(
-          '%s field in %s was %d, will ACK Overflow after %d, could you make a shorter URL?' %
+          '%s field in %s was %d, will ACK Overflow after %d, could you make '
+          'a shorter URL?' %
           (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name,
            len(url_field), self.MAX_LENGTH))
 
@@ -214,7 +215,8 @@ def VerifyResult(self, response, fields):
             (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name, url_field))
       else:
         # Advisory if not HTTP(S) and maybe FTP for firmware...
-        if self.ALLOWED_SCHEMAS and url_result.scheme not in self.ALLOWED_SCHEMAS:
+        if (self.ALLOWED_SCHEMAS and
+            (url_result.scheme not in self.ALLOWED_SCHEMAS)):
           self.AddAdvisory(
               '%s field in %s had schema %s, expected one of %s' %
               (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name,
@@ -227,13 +229,16 @@ def VerifyResult(self, response, fields):
               (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name))
         # else:
           # TODO(Peter): Check for a dot in the netloc
-          # TODO(Peter): Check the netloc domain isn't a prohibited internal only one
+          # TODO(Peter): Check the netloc domain isn't a prohibited internal
+          # only one
 
-        # TODO(Peter): Optionally expect at least one other section (product/firmware)
+        # TODO(Peter): Optionally expect at least one other section
+        # (product/firmware)
     except ValueError as err:
       self.SetFailed(
           '%s field in %s didn\'t parse as a URL due to %s, was %s' %
-          (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name, str(err), url_field))
+          (self.EXPECTED_FIELDS[0].capitalize(), self.pid.name, str(err),
+           url_field))
 
 
 class GetRequiredMixin(ResponderTestFixture):