Fixes multiple issues

- Fix tox.ini to use supported python versions.
- Fix contraints regex using subscript on matches (py36).
- Fix facade return type documentation.
- Fix schema objects with array values.
- Add definition test.

Author: hpidcock

juju/client/_client1.py

@@ -701,7 +701,7 @@ async def AddRelation(self, endpoints=None, via_cidrs=None):
         '''
         endpoints : typing.Sequence[str]
         via_cidrs : typing.Sequence[str]
-        Returns -> typing.Mapping[str, ~CharmRelation]
+        Returns -> AddRelationResults
         '''
         if endpoints is not None and not isinstance(endpoints, (bytes, str, list)):
             raise Exception("Expected endpoints to be a Sequence, received: {}".format(type(endpoints)))
@@ -730,7 +730,7 @@ async def AddUnits(self, application=None, attach_storage=None, num_units=None,
         num_units : int
         placement : typing.Sequence[~Placement]
         policy : str
-        Returns -> typing.Sequence[str]
+        Returns -> AddApplicationUnitsResults
         '''
         if application is not None and not isinstance(application, (bytes, str)):
             raise Exception("Expected application to be a str, received: {}".format(type(application)))
@@ -767,7 +767,7 @@ async def AddUnits(self, application=None, attach_storage=None, num_units=None,
     async def ApplicationsInfo(self, entities=None):
         '''
         entities : typing.Sequence[~Entity]
-        Returns -> typing.Sequence[~ApplicationInfoResult]
+        Returns -> ApplicationInfoResults
         '''
         if entities is not None and not isinstance(entities, (bytes, str, list)):
             raise Exception("Expected entities to be a Sequence, received: {}".format(type(entities)))
@@ -788,7 +788,7 @@ async def ApplicationsInfo(self, entities=None):
     async def CharmConfig(self, args=None):
         '''
         args : typing.Sequence[~ApplicationGet]
-        Returns -> typing.Sequence[~ConfigResult]
+        Returns -> ApplicationGetConfigResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -809,7 +809,7 @@ async def CharmConfig(self, args=None):
     async def CharmRelations(self, application=None):
         '''
         application : str
-        Returns -> typing.Sequence[str]
+        Returns -> ApplicationCharmRelationsResults
         '''
         if application is not None and not isinstance(application, (bytes, str)):
             raise Exception("Expected application to be a str, received: {}".format(type(application)))
@@ -830,7 +830,7 @@ async def CharmRelations(self, application=None):
     async def Consume(self, args=None):
         '''
         args : typing.Sequence[~ConsumeApplicationArg]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -851,7 +851,7 @@ async def Consume(self, args=None):
     async def Deploy(self, applications=None):
         '''
         applications : typing.Sequence[~ApplicationDeploy]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if applications is not None and not isinstance(applications, (bytes, str, list)):
             raise Exception("Expected applications to be a Sequence, received: {}".format(type(applications)))
@@ -893,7 +893,7 @@ async def Destroy(self, application=None):
     async def DestroyApplication(self, applications=None):
         '''
         applications : typing.Sequence[~DestroyApplicationParams]
-        Returns -> typing.Sequence[~DestroyApplicationResult]
+        Returns -> DestroyApplicationResults
         '''
         if applications is not None and not isinstance(applications, (bytes, str, list)):
             raise Exception("Expected applications to be a Sequence, received: {}".format(type(applications)))
@@ -914,7 +914,7 @@ async def DestroyApplication(self, applications=None):
     async def DestroyConsumedApplications(self, applications=None):
         '''
         applications : typing.Sequence[~DestroyConsumedApplicationParams]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if applications is not None and not isinstance(applications, (bytes, str, list)):
             raise Exception("Expected applications to be a Sequence, received: {}".format(type(applications)))
@@ -971,7 +971,7 @@ async def DestroyRelation(self, endpoints=None, force=None, max_wait=None, relat
     async def DestroyUnit(self, units=None):
         '''
         units : typing.Sequence[~DestroyUnitParams]
-        Returns -> typing.Sequence[~DestroyUnitResult]
+        Returns -> DestroyUnitResults
         '''
         if units is not None and not isinstance(units, (bytes, str, list)):
             raise Exception("Expected units to be a Sequence, received: {}".format(type(units)))
@@ -1035,7 +1035,7 @@ async def Get(self, application=None, branch=None):
         '''
         application : str
         branch : str
-        Returns -> typing.Union[str, typing.Mapping[str, typing.Any], ForwardRef('Value'), typing.Mapping[str, str]]
+        Returns -> ApplicationGetResults
         '''
         if application is not None and not isinstance(application, (bytes, str)):
             raise Exception("Expected application to be a str, received: {}".format(type(application)))
@@ -1061,7 +1061,7 @@ async def GetCharmURL(self, application=None, branch=None):
         '''
         application : str
         branch : str
-        Returns -> typing.Union[ForwardRef('Error'), str]
+        Returns -> StringResult
         '''
         if application is not None and not isinstance(application, (bytes, str)):
             raise Exception("Expected application to be a str, received: {}".format(type(application)))
@@ -1086,7 +1086,7 @@ async def GetCharmURL(self, application=None, branch=None):
     async def GetConfig(self, entities=None):
         '''
         entities : typing.Sequence[~Entity]
-        Returns -> typing.Sequence[~ConfigResult]
+        Returns -> ApplicationGetConfigResults
         '''
         if entities is not None and not isinstance(entities, (bytes, str, list)):
             raise Exception("Expected entities to be a Sequence, received: {}".format(type(entities)))
@@ -1107,7 +1107,7 @@ async def GetConfig(self, entities=None):
     async def GetConstraints(self, entities=None):
         '''
         entities : typing.Sequence[~Entity]
-        Returns -> typing.Sequence[~ApplicationConstraint]
+        Returns -> ApplicationGetConstraintsResults
         '''
         if entities is not None and not isinstance(entities, (bytes, str, list)):
             raise Exception("Expected entities to be a Sequence, received: {}".format(type(entities)))
@@ -1128,7 +1128,7 @@ async def GetConstraints(self, entities=None):
     async def MergeBindings(self, args=None):
         '''
         args : typing.Sequence[~ApplicationMergeBindings]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -1151,7 +1151,7 @@ async def ResolveUnitErrors(self, all_=None, retry=None, tags=None):
         all_ : bool
         retry : bool
         tags : Entities
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if all_ is not None and not isinstance(all_, bool):
             raise Exception("Expected all_ to be a bool, received: {}".format(type(all_)))
@@ -1180,7 +1180,7 @@ async def ResolveUnitErrors(self, all_=None, retry=None, tags=None):
     async def ScaleApplications(self, applications=None):
         '''
         applications : typing.Sequence[~ScaleApplicationParams]
-        Returns -> typing.Sequence[~ScaleApplicationResult]
+        Returns -> ScaleApplicationResults
         '''
         if applications is not None and not isinstance(applications, (bytes, str, list)):
             raise Exception("Expected applications to be a Sequence, received: {}".format(type(applications)))
@@ -1232,7 +1232,7 @@ async def Set(self, application=None, branch=None, options=None):
     async def SetApplicationsConfig(self, args=None):
         '''
         args : typing.Sequence[~ApplicationConfigSet]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -1355,7 +1355,7 @@ async def SetConstraints(self, application=None, constraints=None):
     async def SetMetricCredentials(self, creds=None):
         '''
         creds : typing.Sequence[~ApplicationMetricCredential]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if creds is not None and not isinstance(creds, (bytes, str, list)):
             raise Exception("Expected creds to be a Sequence, received: {}".format(type(creds)))
@@ -1376,7 +1376,7 @@ async def SetMetricCredentials(self, creds=None):
     async def SetRelationsSuspended(self, args=None):
         '''
         args : typing.Sequence[~RelationSuspendedArg]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -1449,7 +1449,7 @@ async def Unset(self, application=None, branch=None, options=None):
     async def UnsetApplicationsConfig(self, args=None):
         '''
         args : typing.Sequence[~ApplicationUnset]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))
@@ -1536,7 +1536,7 @@ async def Update(self, application=None, charm_url=None, constraints=None, force
     async def UpdateApplicationSeries(self, args=None):
         '''
         args : typing.Sequence[~UpdateSeriesArg]
-        Returns -> typing.Sequence[~ErrorResult]
+        Returns -> ErrorResults
         '''
         if args is not None and not isinstance(args, (bytes, str, list)):
             raise Exception("Expected args to be a Sequence, received: {}".format(type(args)))