Cleaned up code as per flake8

nexus_constructor/field_utils.py

@@ -125,7 +125,7 @@ def find_field_type(item: "ValueType", ignore_names=INVALID_FIELD_NAMES) -> Call
         return update_existing_stream_field
     elif isinstance(item, Link):
         return update_existing_link_field
-    elif isinstance(item,FileWriter):
+    elif isinstance(item, FileWriter):
         return update_existing_filewriter_field
     else:
         try:

nexus_constructor/json/load_from_json.py

@@ -202,7 +202,7 @@ def _load_from_json_dict(self, json_dict: Dict) -> bool:
         self.entry_node = self._read_json_object(json_dict[CommonKeys.CHILDREN][0])
         self.model.entry.attributes = self.entry_node.attributes
         for child in self.entry_node.children:
-            if isinstance(child, (Dataset, Link, Group)):
+            if isinstance(child, (Dataset, Link, FileWriter, Group)):
                 self.model.entry[child.name] = child
             else:
                 self.model.entry.children.append(child)
@@ -277,7 +277,7 @@ def _read_json_object(self, json_object: Dict, parent_node: Group = None):
         elif CommonKeys.MODULE in json_object and NodeType.CONFIG in json_object:
             module_type = json_object[CommonKeys.MODULE]
             if (
-                ( module_type == WriterModules.DATASET.value or module_type == WriterModules.FILEWRITER.value )
+                (module_type == WriterModules.DATASET.value or module_type == WriterModules.FILEWRITER.value)
                 and json_object[NodeType.CONFIG][CommonKeys.NAME]
                 == CommonAttrs.DEPENDS_ON
             ):

nexus_constructor/validators.py

@@ -395,7 +395,6 @@ class FieldType(Enum):
     filewriter = "Filewriter"
 
 
-
 class FieldValueValidator(QValidator):
     """
     Validates the field value line edit to check that the entered string is castable to the selected numpy type.

tests/test_validators.py

@@ -172,6 +172,7 @@ def test_GIVEN_valid_string_value_WHEN_validating_field_value_THEN_returns_accep
     assert validator.validate(strvalue, 0) == QValidator.Acceptable
     validator.is_valid.emit.assert_called_once_with(True)
 
+
 def test_GIVEN_valid_name_value_WHEN_validating_filewriter_THEN_returns_acceptable_and_emits_signal_with_true():
     strvalue = "123a"
 
@@ -186,6 +187,7 @@ def test_GIVEN_valid_name_value_WHEN_validating_filewriter_THEN_returns_acceptab
     assert validator.validate(strvalue, 0) == QValidator.Acceptable
     validator.is_valid.emit.assert_called_once_with(True)
 
+
 def test_GIVEN_invalid_float_value_WHEN_validating_field_value_THEN_returns_intermediate_and_emits_signal_with_false():
     invalid_value = "sdfn"