Merge remote-tracking branch 'upstream/master'

moto/iot/models.py

@@ -28,7 +28,7 @@ class FakeThing(BaseModel):
         self.thing_name = thing_name
         self.thing_type = thing_type
         self.attributes = attributes
-        self.arn = 'arn:aws:iot:%s:1:thing/%s' % (self.region_name, thing_name)
+        self.arn = "arn:aws:iot:%s:1:thing/%s" % (self.region_name, thing_name)
         self.version = 1
         # TODO: we need to handle 'version'?
 
@@ -37,15 +37,15 @@ class FakeThing(BaseModel):
 
     def to_dict(self, include_default_client_id=False):
         obj = {
-            'thingName': self.thing_name,
-            'thingArn': self.arn,
-            'attributes': self.attributes,
-            'version': self.version
+            "thingName": self.thing_name,
+            "thingArn": self.arn,
+            "attributes": self.attributes,
+            "version": self.version,
         }
         if self.thing_type:
-            obj['thingTypeName'] = self.thing_type.thing_type_name
+            obj["thingTypeName"] = self.thing_type.thing_type_name
         if include_default_client_id:
-            obj['defaultClientId'] = self.thing_name
+            obj["defaultClientId"] = self.thing_name
         return obj
 
 
@@ -56,23 +56,27 @@ class FakeThingType(BaseModel):
         self.thing_type_properties = thing_type_properties
         self.thing_type_id = str(uuid.uuid4())  # I don't know the rule of id
         t = time.time()
-        self.metadata = {
-            'deprecated': False,
-            'creationData': int(t * 1000) / 1000.0
-        }
-        self.arn = 'arn:aws:iot:%s:1:thingtype/%s' % (self.region_name, thing_type_name)
+        self.metadata = {"deprecated": False, "creationDate": int(t * 1000) / 1000.0}
+        self.arn = "arn:aws:iot:%s:1:thingtype/%s" % (self.region_name, thing_type_name)
 
     def to_dict(self):
         return {
-            'thingTypeName': self.thing_type_name,
-            'thingTypeId': self.thing_type_id,
-            'thingTypeProperties': self.thing_type_properties,
-            'thingTypeMetadata': self.metadata
+            "thingTypeName": self.thing_type_name,
+            "thingTypeId": self.thing_type_id,
+            "thingTypeProperties": self.thing_type_properties,
+            "thingTypeMetadata": self.metadata,
         }
 
 
 class FakeThingGroup(BaseModel):
-    def __init__(self, thing_group_name, parent_group_name, thing_group_properties, region_name):
+    def __init__(
+        self,
+        thing_group_name,
+        parent_group_name,
+        thing_group_properties,
+        region_name,
+        thing_groups,
+    ):
         self.region_name = region_name
         self.thing_group_name = thing_group_name
         self.thing_group_id = str(uuid.uuid4())  # I don't know the rule of id
@@ -80,33 +84,59 @@ class FakeThingGroup(BaseModel):
         self.parent_group_name = parent_group_name
         self.thing_group_properties = thing_group_properties or {}
         t = time.time()
-        self.metadata = {
-            'creationData': int(t * 1000) / 1000.0
-        }
-        self.arn = 'arn:aws:iot:%s:1:thinggroup/%s' % (self.region_name, thing_group_name)
+        self.metadata = {"creationDate": int(t * 1000) / 1000.0}
+        if parent_group_name:
+            self.metadata["parentGroupName"] = parent_group_name
+            # initilize rootToParentThingGroups
+            if "rootToParentThingGroups" not in self.metadata:
+                self.metadata["rootToParentThingGroups"] = []
+            # search for parent arn
+            for thing_group_arn, thing_group in thing_groups.items():
+                if thing_group.thing_group_name == parent_group_name:
+                    parent_thing_group_structure = thing_group
+                    break
+            # if parent arn found (should always be found)
+            if parent_thing_group_structure:
+                # copy parent's rootToParentThingGroups
+                if "rootToParentThingGroups" in parent_thing_group_structure.metadata:
+                    self.metadata["rootToParentThingGroups"].extend(
+                        parent_thing_group_structure.metadata["rootToParentThingGroups"]
+                    )
+                self.metadata["rootToParentThingGroups"].extend(
+                    [
+                        {
+                            "groupName": parent_group_name,
+                            "groupArn": parent_thing_group_structure.arn,
+                        }
+                    ]
+                )
+        self.arn = "arn:aws:iot:%s:1:thinggroup/%s" % (
+            self.region_name,
+            thing_group_name,
+        )
         self.things = OrderedDict()
 
     def to_dict(self):
         return {
-            'thingGroupName': self.thing_group_name,
-            'thingGroupId': self.thing_group_id,
-            'version': self.version,
-            'thingGroupProperties': self.thing_group_properties,
-            'thingGroupMetadata': self.metadata
+            "thingGroupName": self.thing_group_name,
+            "thingGroupId": self.thing_group_id,
+            "version": self.version,
+            "thingGroupProperties": self.thing_group_properties,
+            "thingGroupMetadata": self.metadata,
         }
 
 
 class FakeCertificate(BaseModel):
     def __init__(self, certificate_pem, status, region_name, ca_certificate_pem=None):
         m = hashlib.sha256()
-        m.update(str(uuid.uuid4()).encode('utf-8'))
+        m.update(str(uuid.uuid4()).encode("utf-8"))
         self.certificate_id = m.hexdigest()
-        self.arn = 'arn:aws:iot:%s:1:cert/%s' % (region_name, self.certificate_id)
+        self.arn = "arn:aws:iot:%s:1:cert/%s" % (region_name, self.certificate_id)
         self.certificate_pem = certificate_pem
         self.status = status
 
         # TODO: must adjust
-        self.owner = '1'
+        self.owner = "1"
         self.transfer_data = {}
         self.creation_date = time.time()
         self.last_modified_date = self.creation_date
@@ -114,16 +144,16 @@ class FakeCertificate(BaseModel):
         self.ca_certificate_id = None
         self.ca_certificate_pem = ca_certificate_pem
         if ca_certificate_pem:
-            m.update(str(uuid.uuid4()).encode('utf-8'))
+            m.update(str(uuid.uuid4()).encode("utf-8"))
             self.ca_certificate_id = m.hexdigest()
 
     def to_dict(self):
         return {
-            'certificateArn': self.arn,
-            'certificateId': self.certificate_id,
-            'caCertificateId': self.ca_certificate_id,
-            'status': self.status,
-            'creationDate': self.creation_date
+            "certificateArn": self.arn,
+            "certificateId": self.certificate_id,
+            "caCertificateId": self.ca_certificate_id,
+            "status": self.status,
+            "creationDate": self.creation_date,
         }
 
     def to_description_dict(self):
@@ -133,14 +163,14 @@ class FakeCertificate(BaseModel):
           - previousOwnedBy
         """
         return {
-            'certificateArn': self.arn,
-            'certificateId': self.certificate_id,
-            'status': self.status,
-            'certificatePem': self.certificate_pem,
-            'ownedBy': self.owner,
-            'creationDate': self.creation_date,
-            'lastModifiedDate': self.last_modified_date,
-            'transferData': self.transfer_data
+            "certificateArn": self.arn,
+            "certificateId": self.certificate_id,
+            "status": self.status,
+            "certificatePem": self.certificate_pem,
+            "ownedBy": self.owner,
+            "creationDate": self.creation_date,
+            "lastModifiedDate": self.last_modified_date,
+            "transferData": self.transfer_data,
         }
 
 
@@ -169,10 +199,7 @@ class FakePolicy(BaseModel):
         }
 
     def to_dict(self):
-        return {
-            'policyName': self.name,
-            'policyArn': self.arn,
-        }
+        return {"policyName": self.name, "policyArn": self.arn}
 
 
 class FakePolicyVersion(object):
@@ -223,14 +250,25 @@ class FakeJob(BaseModel):
     JOB_ID_REGEX_PATTERN = "[a-zA-Z0-9_-]"
     JOB_ID_REGEX = re.compile(JOB_ID_REGEX_PATTERN)
 
-    def __init__(self, job_id, targets, document_source, document, description, presigned_url_config, target_selection,
-                 job_executions_rollout_config, document_parameters, region_name):
+    def __init__(
+        self,
+        job_id,
+        targets,
+        document_source,
+        document,
+        description,
+        presigned_url_config,
+        target_selection,
+        job_executions_rollout_config,
+        document_parameters,
+        region_name,
+    ):
         if not self._job_id_matcher(self.JOB_ID_REGEX, job_id):
             raise InvalidRequestException()
 
         self.region_name = region_name
         self.job_id = job_id
-        self.job_arn = 'arn:aws:iot:%s:1:job/%s' % (self.region_name, job_id)
+        self.job_arn = "arn:aws:iot:%s:1:job/%s" % (self.region_name, job_id)
         self.targets = targets
         self.document_source = document_source
         self.document = document
@@ -246,14 +284,14 @@ class FakeJob(BaseModel):
         self.last_updated_at = time.mktime(datetime(2015, 1, 1).timetuple())
         self.completed_at = None
         self.job_process_details = {
-            'processingTargets': targets,
-            'numberOfQueuedThings': 1,
-            'numberOfCanceledThings': 0,
-            'numberOfSucceededThings': 0,
-            'numberOfFailedThings': 0,
-            'numberOfRejectedThings': 0,
-            'numberOfInProgressThings': 0,
-            'numberOfRemovedThings': 0
+            "processingTargets": targets,
+            "numberOfQueuedThings": 1,
+            "numberOfCanceledThings": 0,
+            "numberOfSucceededThings": 0,
+            "numberOfFailedThings": 0,
+            "numberOfRejectedThings": 0,
+            "numberOfInProgressThings": 0,
+            "numberOfRemovedThings": 0,
         }
         self.document_parameters = document_parameters
 
@@ -358,16 +396,18 @@ class IoTBackend(BaseBackend):
         thing_types = self.list_thing_types()
         thing_type = None
         if thing_type_name:
-            filtered_thing_types = [_ for _ in thing_types if _.thing_type_name == thing_type_name]
+            filtered_thing_types = [
+                _ for _ in thing_types if _.thing_type_name == thing_type_name
+            ]
             if len(filtered_thing_types) == 0:
                 raise ResourceNotFoundException()
             thing_type = filtered_thing_types[0]
         if attribute_payload is None:
             attributes = {}
-        elif 'attributes' not in attribute_payload:
+        elif "attributes" not in attribute_payload:
             attributes = {}
         else:
-            attributes = attribute_payload['attributes']
+            attributes = attribute_payload["attributes"]
         thing = FakeThing(thing_name, thing_type, attributes, self.region_name)
         self.things[thing.arn] = thing
         return thing.thing_name, thing.arn
@@ -375,41 +415,68 @@ class IoTBackend(BaseBackend):
     def create_thing_type(self, thing_type_name, thing_type_properties):
         if thing_type_properties is None:
             thing_type_properties = {}
-        thing_type = FakeThingType(thing_type_name, thing_type_properties, self.region_name)
+        thing_type = FakeThingType(
+            thing_type_name, thing_type_properties, self.region_name
+        )
         self.thing_types[thing_type.arn] = thing_type
         return thing_type.thing_type_name, thing_type.arn
 
     def list_thing_types(self, thing_type_name=None):
         if thing_type_name:
             # It's weird but thing_type_name is filtered by forward match, not complete match
-            return [_ for _ in self.thing_types.values() if _.thing_type_name.startswith(thing_type_name)]
+            return [
+                _
+                for _ in self.thing_types.values()
+                if _.thing_type_name.startswith(thing_type_name)
+            ]
         return self.thing_types.values()
 
-    def list_things(self, attribute_name, attribute_value, thing_type_name, max_results, token):
+    def list_things(
+        self, attribute_name, attribute_value, thing_type_name, max_results, token
+    ):
         all_things = [_.to_dict() for _ in self.things.values()]
         if attribute_name is not None and thing_type_name is not None:
-            filtered_things = list(filter(lambda elem:
-                                          attribute_name in elem["attributes"] and
-                                          elem["attributes"][attribute_name] == attribute_value and
-                                          "thingTypeName" in elem and
-                                          elem["thingTypeName"] == thing_type_name, all_things))
+            filtered_things = list(
+                filter(
+                    lambda elem: attribute_name in elem["attributes"]
+                    and elem["attributes"][attribute_name] == attribute_value
+                    and "thingTypeName" in elem
+                    and elem["thingTypeName"] == thing_type_name,
+                    all_things,
+                )
+            )
         elif attribute_name is not None and thing_type_name is None:
-            filtered_things = list(filter(lambda elem:
-                                          attribute_name in elem["attributes"] and
-                                          elem["attributes"][attribute_name] == attribute_value, all_things))
+            filtered_things = list(
+                filter(
+                    lambda elem: attribute_name in elem["attributes"]
+                    and elem["attributes"][attribute_name] == attribute_value,
+                    all_things,
+                )
+            )
         elif attribute_name is None and thing_type_name is not None:
             filtered_things = list(
-                filter(lambda elem: "thingTypeName" in elem and elem["thingTypeName"] == thing_type_name, all_things))
+                filter(
+                    lambda elem: "thingTypeName" in elem
+                    and elem["thingTypeName"] == thing_type_name,
+                    all_things,
+                )
+            )
         else:
             filtered_things = all_things
 
         if token is None:
             things = filtered_things[0:max_results]
-            next_token = str(max_results) if len(filtered_things) > max_results else None
+            next_token = (
+                str(max_results) if len(filtered_things) > max_results else None
+            )
         else:
             token = int(token)
-            things = filtered_things[token:token + max_results]
-            next_token = str(token + max_results) if len(filtered_things) > token + max_results else None
+            things = filtered_things[token : token + max_results]
+            next_token = (
+                str(token + max_results)
+                if len(filtered_things) > token + max_results
+                else None
+            )
 
         return things, next_token
 
@@ -420,7 +487,9 @@ class IoTBackend(BaseBackend):
         return things[0]
 
     def describe_thing_type(self, thing_type_name):
-        thing_types = [_ for _ in self.thing_types.values() if _.thing_type_name == thing_type_name]
+        thing_types = [
+            _ for _ in self.thing_types.values() if _.thing_type_name == thing_type_name
+        ]
         if len(thing_types) == 0:
             raise ResourceNotFoundException()
         return thing_types[0]
@@ -430,6 +499,12 @@ class IoTBackend(BaseBackend):
 
         # can raise ResourceNotFoundError
         thing = self.describe_thing(thing_name)
+
+        # detach all principals
+        for k in list(self.principal_things.keys()):
+            if k[1] == thing_name:
+                del self.principal_things[k]
+
         del self.things[thing.arn]
 
     def delete_thing_type(self, thing_type_name):
@@ -437,7 +512,14 @@ class IoTBackend(BaseBackend):
         thing_type = self.describe_thing_type(thing_type_name)
         del self.thing_types[thing_type.arn]
 
-    def update_thing(self, thing_name, thing_type_name, attribute_payload, expected_version, remove_thing_type):
+    def update_thing(
+        self,
+        thing_name,
+        thing_type_name,
+        attribute_payload,
+        expected_version,
+        remove_thing_type,
+    ):
         # if attributes payload = {}, nothing
         thing = self.describe_thing(thing_name)
         thing_type = None
@@ -448,7 +530,9 @@ class IoTBackend(BaseBackend):
         # thing_type
         if thing_type_name:
             thing_types = self.list_thing_types()
-            filtered_thing_types = [_ for _ in thing_types if _.thing_type_name == thing_type_name]
+            filtered_thing_types = [
+                _ for _ in thing_types if _.thing_type_name == thing_type_name
+            ]
             if len(filtered_thing_types) == 0:
                 raise ResourceNotFoundException()
             thing_type = filtered_thing_types[0]
@@ -458,9 +542,9 @@ class IoTBackend(BaseBackend):
             thing.thing_type = None
 
         # attribute
-        if attribute_payload is not None and 'attributes' in attribute_payload:
-            do_merge = attribute_payload.get('merge', False)
-            attributes = attribute_payload['attributes']
+        if attribute_payload is not None and "attributes" in attribute_payload:
+            do_merge = attribute_payload.get("merge", False)
+            attributes = attribute_payload["attributes"]
             if not do_merge:
                 thing.attributes = attributes
             else:
@@ -468,46 +552,59 @@ class IoTBackend(BaseBackend):
 
     def _random_string(self):
         n = 20
-        random_str = ''.join([random.choice(string.ascii_letters + string.digits) for i in range(n)])
+        random_str = "".join(
+            [random.choice(string.ascii_letters + string.digits) for i in range(n)]
+        )
         return random_str
 
     def create_keys_and_certificate(self, set_as_active):
         # implement here
         # caCertificate can be blank
         key_pair = {
-            'PublicKey': self._random_string(),
-            'PrivateKey': self._random_string()
+            "PublicKey": self._random_string(),
+            "PrivateKey": self._random_string(),
         }
         certificate_pem = self._random_string()
-        status = 'ACTIVE' if set_as_active else 'INACTIVE'
+        status = "ACTIVE" if set_as_active else "INACTIVE"
         certificate = FakeCertificate(certificate_pem, status, self.region_name)
         self.certificates[certificate.certificate_id] = certificate
         return certificate, key_pair
 
     def delete_certificate(self, certificate_id):
         cert = self.describe_certificate(certificate_id)
-        if cert.status == 'ACTIVE':
+        if cert.status == "ACTIVE":
             raise CertificateStateException(
-                'Certificate must be deactivated (not ACTIVE) before deletion.', certificate_id)
-
-        certs = [k[0] for k, v in self.principal_things.items()
-                 if self._get_principal(k[0]).certificate_id == certificate_id]
-        if len(certs) > 0:
-            raise DeleteConflictException(
-                'Things must be detached before deletion (arn: %s)' % certs[0]
+                "Certificate must be deactivated (not ACTIVE) before deletion.",
+                certificate_id,
             )
 
-        certs = [k[0] for k, v in self.principal_policies.items()
-                 if self._get_principal(k[0]).certificate_id == certificate_id]
+        certs = [
+            k[0]
+            for k, v in self.principal_things.items()
+            if self._get_principal(k[0]).certificate_id == certificate_id
+        ]
         if len(certs) > 0:
             raise DeleteConflictException(
-                'Certificate policies must be detached before deletion (arn: %s)' % certs[0]
+                "Things must be detached before deletion (arn: %s)" % certs[0]
+            )
+
+        certs = [
+            k[0]
+            for k, v in self.principal_policies.items()
+            if self._get_principal(k[0]).certificate_id == certificate_id
+        ]
+        if len(certs) > 0:
+            raise DeleteConflictException(
+                "Certificate policies must be detached before deletion (arn: %s)"
+                % certs[0]
             )
 
         del self.certificates[certificate_id]
 
     def describe_certificate(self, certificate_id):
-        certs = [_ for _ in self.certificates.values() if _.certificate_id == certificate_id]
+        certs = [
+            _ for _ in self.certificates.values() if _.certificate_id == certificate_id
+        ]
         if len(certs) == 0:
             raise ResourceNotFoundException()
         return certs[0]
@@ -515,9 +612,15 @@ class IoTBackend(BaseBackend):
     def list_certificates(self):
         return self.certificates.values()
 
-    def register_certificate(self, certificate_pem, ca_certificate_pem, set_as_active, status):
-        certificate = FakeCertificate(certificate_pem, 'ACTIVE' if set_as_active else status,
-                                      self.region_name, ca_certificate_pem)
+    def register_certificate(
+        self, certificate_pem, ca_certificate_pem, set_as_active, status
+    ):
+        certificate = FakeCertificate(
+            certificate_pem,
+            "ACTIVE" if set_as_active else status,
+            self.region_name,
+            ca_certificate_pem,
+        )
         self.certificates[certificate.certificate_id] = certificate
         return certificate
 
@@ -565,10 +668,12 @@ class IoTBackend(BaseBackend):
 
     def delete_policy(self, policy_name):
 
-        policies = [k[1] for k, v in self.principal_policies.items() if k[1] == policy_name]
+        policies = [
+            k[1] for k, v in self.principal_policies.items() if k[1] == policy_name
+        ]
         if len(policies) > 0:
             raise DeleteConflictException(
-                'The policy cannot be deleted as the policy is attached to one or more principals (name=%s)'
+                "The policy cannot be deleted as the policy is attached to one or more principals (name=%s)"
                 % policy_name
             )
 
@@ -630,7 +735,7 @@ class IoTBackend(BaseBackend):
         """
         raise ResourceNotFoundException
         """
-        if ':cert/' in principal_arn:
+        if ":cert/" in principal_arn:
             certs = [_ for _ in self.certificates.values() if _.arn == principal_arn]
             if len(certs) == 0:
                 raise ResourceNotFoundException()
@@ -660,11 +765,15 @@ class IoTBackend(BaseBackend):
         del self.principal_policies[k]
 
     def list_principal_policies(self, principal_arn):
-        policies = [v[1] for k, v in self.principal_policies.items() if k[0] == principal_arn]
+        policies = [
+            v[1] for k, v in self.principal_policies.items() if k[0] == principal_arn
+        ]
         return policies
 
     def list_policy_principals(self, policy_name):
-        principals = [k[0] for k, v in self.principal_policies.items() if k[1] == policy_name]
+        principals = [
+            k[0] for k, v in self.principal_policies.items() if k[1] == policy_name
+        ]
         return principals
 
     def attach_thing_principal(self, thing_name, principal_arn):
@@ -686,21 +795,37 @@ class IoTBackend(BaseBackend):
         del self.principal_things[k]
 
     def list_principal_things(self, principal_arn):
-        thing_names = [k[0] for k, v in self.principal_things.items() if k[0] == principal_arn]
+        thing_names = [
+            k[0] for k, v in self.principal_things.items() if k[0] == principal_arn
+        ]
         return thing_names
 
     def list_thing_principals(self, thing_name):
-        principals = [k[0] for k, v in self.principal_things.items() if k[1] == thing_name]
+        principals = [
+            k[0] for k, v in self.principal_things.items() if k[1] == thing_name
+        ]
         return principals
 
     def describe_thing_group(self, thing_group_name):
-        thing_groups = [_ for _ in self.thing_groups.values() if _.thing_group_name == thing_group_name]
+        thing_groups = [
+            _
+            for _ in self.thing_groups.values()
+            if _.thing_group_name == thing_group_name
+        ]
         if len(thing_groups) == 0:
             raise ResourceNotFoundException()
         return thing_groups[0]
 
-    def create_thing_group(self, thing_group_name, parent_group_name, thing_group_properties):
-        thing_group = FakeThingGroup(thing_group_name, parent_group_name, thing_group_properties, self.region_name)
+    def create_thing_group(
+        self, thing_group_name, parent_group_name, thing_group_properties
+    ):
+        thing_group = FakeThingGroup(
+            thing_group_name,
+            parent_group_name,
+            thing_group_properties,
+            self.region_name,
+            self.thing_groups,
+        )
         self.thing_groups[thing_group.arn] = thing_group
         return thing_group.thing_group_name, thing_group.arn, thing_group.thing_group_id
 
@@ -712,19 +837,25 @@ class IoTBackend(BaseBackend):
         thing_groups = self.thing_groups.values()
         return thing_groups
 
-    def update_thing_group(self, thing_group_name, thing_group_properties, expected_version):
+    def update_thing_group(
+        self, thing_group_name, thing_group_properties, expected_version
+    ):
         thing_group = self.describe_thing_group(thing_group_name)
         if expected_version and expected_version != thing_group.version:
             raise VersionConflictException(thing_group_name)
-        attribute_payload = thing_group_properties.get('attributePayload', None)
-        if attribute_payload is not None and 'attributes' in attribute_payload:
-            do_merge = attribute_payload.get('merge', False)
-            attributes = attribute_payload['attributes']
+        attribute_payload = thing_group_properties.get("attributePayload", None)
+        if attribute_payload is not None and "attributes" in attribute_payload:
+            do_merge = attribute_payload.get("merge", False)
+            attributes = attribute_payload["attributes"]
             if not do_merge:
-                thing_group.thing_group_properties['attributePayload']['attributes'] = attributes
+                thing_group.thing_group_properties["attributePayload"][
+                    "attributes"
+                ] = attributes
             else:
-                thing_group.thing_group_properties['attributePayload']['attributes'].update(attributes)
-        elif attribute_payload is not None and 'attributes' not in attribute_payload:
+                thing_group.thing_group_properties["attributePayload"][
+                    "attributes"
+                ].update(attributes)
+        elif attribute_payload is not None and "attributes" not in attribute_payload:
             thing_group.attributes = {}
         thing_group.version = thing_group.version + 1
         return thing_group.version
@@ -733,13 +864,13 @@ class IoTBackend(BaseBackend):
         # identify thing group
         if thing_group_name is None and thing_group_arn is None:
             raise InvalidRequestException(
-                ' Both thingGroupArn and thingGroupName are empty. Need to specify at least one of them'
+                " Both thingGroupArn and thingGroupName are empty. Need to specify at least one of them"
             )
         if thing_group_name is not None:
             thing_group = self.describe_thing_group(thing_group_name)
             if thing_group_arn and thing_group.arn != thing_group_arn:
                 raise InvalidRequestException(
-                    'ThingGroupName thingGroupArn does not match specified thingGroupName in request'
+                    "ThingGroupName thingGroupArn does not match specified thingGroupName in request"
                 )
         elif thing_group_arn is not None:
             if thing_group_arn not in self.thing_groups:
@@ -751,13 +882,13 @@ class IoTBackend(BaseBackend):
         # identify thing
         if thing_name is None and thing_arn is None:
             raise InvalidRequestException(
-                'Both thingArn and thingName are empty. Need to specify at least one of them'
+                "Both thingArn and thingName are empty. Need to specify at least one of them"
             )
         if thing_name is not None:
             thing = self.describe_thing(thing_name)
             if thing_arn and thing.arn != thing_arn:
                 raise InvalidRequestException(
-                    'ThingName thingArn does not match specified thingName in request'
+                    "ThingName thingArn does not match specified thingName in request"
                 )
         elif thing_arn is not None:
             if thing_arn not in self.things:
@@ -765,7 +896,9 @@ class IoTBackend(BaseBackend):
             thing = self.things[thing_arn]
         return thing
 
-    def add_thing_to_thing_group(self, thing_group_name, thing_group_arn, thing_name, thing_arn):
+    def add_thing_to_thing_group(
+        self, thing_group_name, thing_group_arn, thing_name, thing_arn
+    ):
         thing_group = self._identify_thing_group(thing_group_name, thing_group_arn)
         thing = self._identify_thing(thing_name, thing_arn)
         if thing.arn in thing_group.things:
@@ -773,7 +906,9 @@ class IoTBackend(BaseBackend):
             return
         thing_group.things[thing.arn] = thing
 
-    def remove_thing_from_thing_group(self, thing_group_name, thing_group_arn, thing_name, thing_arn):
+    def remove_thing_from_thing_group(
+        self, thing_group_name, thing_group_arn, thing_name, thing_arn
+    ):
         thing_group = self._identify_thing_group(thing_group_name, thing_group_arn)
         thing = self._identify_thing(thing_name, thing_arn)
         if thing.arn not in thing_group.things:
@@ -791,31 +926,53 @@ class IoTBackend(BaseBackend):
         ret = []
         for thing_group in all_thing_groups:
             if thing.arn in thing_group.things:
-                ret.append({
-                    'groupName': thing_group.thing_group_name,
-                    'groupArn': thing_group.arn
-                })
+                ret.append(
+                    {
+                        "groupName": thing_group.thing_group_name,
+                        "groupArn": thing_group.arn,
+                    }
+                )
         return ret
 
-    def update_thing_groups_for_thing(self, thing_name, thing_groups_to_add, thing_groups_to_remove):
+    def update_thing_groups_for_thing(
+        self, thing_name, thing_groups_to_add, thing_groups_to_remove
+    ):
         thing = self.describe_thing(thing_name)
         for thing_group_name in thing_groups_to_add:
             thing_group = self.describe_thing_group(thing_group_name)
             self.add_thing_to_thing_group(
-                thing_group.thing_group_name, None,
-                thing.thing_name, None
+                thing_group.thing_group_name, None, thing.thing_name, None
             )
         for thing_group_name in thing_groups_to_remove:
             thing_group = self.describe_thing_group(thing_group_name)
             self.remove_thing_from_thing_group(
-                thing_group.thing_group_name, None,
-                thing.thing_name, None
+                thing_group.thing_group_name, None, thing.thing_name, None
             )
 
-    def create_job(self, job_id, targets, document_source, document, description, presigned_url_config,
-                   target_selection, job_executions_rollout_config, document_parameters):
-        job = FakeJob(job_id, targets, document_source, document, description, presigned_url_config, target_selection,
-                      job_executions_rollout_config, document_parameters, self.region_name)
+    def create_job(
+        self,
+        job_id,
+        targets,
+        document_source,
+        document,
+        description,
+        presigned_url_config,
+        target_selection,
+        job_executions_rollout_config,
+        document_parameters,
+    ):
+        job = FakeJob(
+            job_id,
+            targets,
+            document_source,
+            document,
+            description,
+            presigned_url_config,
+            target_selection,
+            job_executions_rollout_config,
+            document_parameters,
+            self.region_name,
+        )
         self.jobs[job_id] = job
 
         for thing_arn in targets: