Added Comment Explaining Behavior

ch4nsuk3 · ch4nsuk3 · commit 0356eec98d13 · 2024-11-08T17:36:19.000-06:00
Added a comment referencing the MQTT specification for why the server may not immediately respond to an UNSUBACK
diff --git a/adafruit_minimqtt/adafruit_minimqtt.py b/adafruit_minimqtt/adafruit_minimqtt.py
@@ -865,6 +865,8 @@ def unsubscribe(  # noqa: PLR0912, Too many branches
                         self._subscribed_topics.remove(t)
                     return
                 if op != MQTT_PUBLISH:
+                    # [3.10.4] The Server may continue to deliver existing messages buffered for delivery
+                    # to the client prior to sending the UNSUBACK Packet.
                     raise MMQTTException(
                         f"invalid message received as response to UNSUBSCRIBE: {hex(op)}"
                     )

Original file line number	Diff line number	Diff line change
`@@ -865,6 +865,8 @@ def unsubscribe( # noqa: PLR0912, Too many branches`
`865`	`865`	`self._subscribed_topics.remove(t)`
`866`	`866`	`return`
`867`	`867`	`if op != MQTT_PUBLISH:`
	`868`	`+ # [3.10.4] The Server may continue to deliver existing messages buffered for delivery`
	`869`	`+ # to the client prior to sending the UNSUBACK Packet.`
`868`	`870`	`raise MMQTTException(`
`869`	`871`	`f"invalid message received as response to UNSUBSCRIBE: {hex(op)}"`
`870`	`872`	`)`