Resolve #2021 "04 16 branched from master ini file device"

Completed ini example. Don't merge before !23 (merged). Small unrelated change slipped in to this branch, will need to fix that first.

devices/clients/ini_client.py

@@ -58,7 +58,7 @@ class ini_client(CommClient):
         self.config_file = open(self.filename, "r")
 
         self.connected = True  # set connected to true
-        return True  # if succesfull, return true. otherwise return false
+        return True  # if successful, return true. otherwise return false
 
     def disconnect(self):
         self.connected = False  # always force a reconnect, regardless of a successful disconnect
@@ -73,16 +73,21 @@ class ini_client(CommClient):
         example annotation may include:
         - a file path and file line/location
         - COM object path
+
+        Annotations:
+            name: Required, the name of the ini variable
+            section: Required,  the section of the ini variable
+
         """
 
         # as this is an example, just print the annotation
         self.streams.debug_stream("annotation: {}".format(annotation))
         name = annotation.get('name')
         if name is None:
-            AssertionError("ini client requires a variable name to set/get")
+            ValueError("ini client requires a variable `name` in the annotation to set/get")
         section = annotation.get('section')
         if section is None:
-            AssertionError("requires a section to open")
+            ValueError("requires a `section` specified in the annotation to open")
 
         return section, name
 
@@ -120,13 +125,8 @@ class ini_client(CommClient):
         def write_function(value):
 
             if type(value) is list:
-                write_value = ""
-
-                for i in value:
-                    write_value = write_value + str(value) + ", "
+                write_value = ", ".join([str(v) for v in value])
 
-                # strip the extra ", " at the end
-                write_value = write_value[:-2]
             else:
                 write_value = str(value)