fix #131 and fix #132: correctly handle default config files

* while I was there, I also decided to fix #4, so now we have no warnings if you define but don't use the ldap or csv connectors.

Author: adobeDan

user_sync/config.py

@@ -127,6 +127,10 @@ def get_directory_connector_configs(self):
         directory_config = self.main_config.get_dict_config('directory_users', True)
         if directory_config != None:
             connectors_config = directory_config.get_dict_config('connectors', True)
+        # make sure neither ldap nor csv connectors get reported as unused
+        if connectors_config:
+            connectors_config.get_list('ldap', True)
+            connectors_config.get_list('csv', True)
         return connectors_config
 
     def get_directory_connector_options(self, connector_name):
@@ -171,8 +175,8 @@ def get_directory_groups(self):
             if groups == None:
                 adobe_groups_by_directory_group[directory_group] = groups = []
 
-            adobe_groups_config = item.get_list_config('adobe_groups')
-            for adobe_group in adobe_groups_config.iter_values(types.StringTypes):
+            adobe_groups = item.get_list('adobe_groups', True)
+            for adobe_group in adobe_groups or []:
                 group = user_sync.rules.AdobeGroup.create(adobe_group)
                 if group is None:
                     validation_message = 'Bad adobe group: "%s" in directory group: "%s"' % (adobe_group, directory_group)
@@ -278,7 +282,7 @@ def get_rule_options(self):
         if umapi_config:
             exclude_identity_type_names = umapi_config.get_list('exclude_identity_types', True) or []
             exclude_users_regexps = umapi_config.get_list('exclude_users', True) or []
-            exclude_group_names = umapi_config.get_list('exclude_groups', True) or []
+            exclude_group_names = umapi_config.get_list('exclude_adobe_groups', True) or []
         for name in exclude_identity_type_names:
             message_format = 'Illegal value in exclude_identity_types: %s'
             identity_type = user_sync.identity_type.parse_identity_type(name, message_format)