ws3

Author: d-w-moore

irods/client_configuration/__init__.py

@@ -45,7 +45,6 @@ def __new__(meta, name, bases, attrs):
 
 
 class ConnectionsProperties(iRODSConfiguration, metaclass=iRODSConfigAliasMetaclass):
-
     __slots__=()
 
     @property
@@ -68,7 +67,6 @@ class ConfigurationValueError(ValueError,ConfigurationError): pass
 
 
 class Genquery1_Properties(iRODSConfiguration, metaclass=iRODSConfigAliasMetaclass):
-
     __slots__ = ()
 
     @property

irods/test/client_configuration_test.py

@@ -2,11 +2,10 @@
 
 import irods.client_configuration as cfg
 
+
 # Test assignments on the negative and positive space of the
 # client configuration.
-
 class TestClientConfigurationAttributes(unittest.TestCase):
-
     def test_configuration_writes_and_miswrites__issue_708(self):
         # For caching configuration objects
         configuration_level = {}
@@ -19,16 +18,15 @@ def test_configuration_writes_and_miswrites__issue_708(self):
                 attribute_name = name_parts[-1]
                 if isinstance(value, cfg.iRODSConfiguration):
                     configuration_level[dotted_name] = value
-                else:
-                    if is_conf:
-                        leaf_names.append(attribute_name)
+                elif is_conf:
+                    leaf_names.append(attribute_name)
 
-                        # Test the positive space, i.e. the 'hit'
-                        setattr(configuration_level[namespace], attribute_name, value)
+                    # Test the positive space, i.e. the 'hit'
+                    setattr(configuration_level[namespace], attribute_name, value)
 
-                        # Test the negative space, i.e. the 'miss'
-                        with self.assertRaises(AttributeError):
-                            setattr(configuration_level[namespace], attribute_name+'_1', value)
+                    # Test the negative space, i.e. the 'miss'
+                    with self.assertRaises(AttributeError):
+                        setattr(configuration_level[namespace], attribute_name + '_1', value)
 
         # These cases were identified as likely ones for possible failed writes via misspelling.
         self.assertIn('irods_query_limit', leaf_names)