don't set state to disconnected multiple times

lhoward · lhoward · commit 5f55d01cf68d · 2024-11-27T10:11:30.000+11:00
diff --git a/Sources/SwiftOCA/OCP.1/Ocp1Connection+Connect.swift b/Sources/SwiftOCA/OCP.1/Ocp1Connection+Connect.swift
@@ -177,8 +177,10 @@ extension Ocp1Connection {
         .trace(
           "failed to send message: error \(error), new connection state \(connectionState); disconnecting"
         )
-      _updateConnectionState(connectionState)
-      try await _disconnectDeviceAfterConnectionFailure()
+      if isConnected {
+        _updateConnectionState(connectionState)
+        try await _disconnectDeviceAfterConnectionFailure()
+      }
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -177,8 +177,10 @@ extension Ocp1Connection {`
`177`	`177`	`.trace(`
`178`	`178`	`"failed to send message: error \(error), new connection state \(connectionState); disconnecting"`
`179`	`179`	`)`
`180`		`- _updateConnectionState(connectionState)`
`181`		`- try await _disconnectDeviceAfterConnectionFailure()`
	`180`	`+ if isConnected {`
	`181`	`+ _updateConnectionState(connectionState)`
	`182`	`+ try await _disconnectDeviceAfterConnectionFailure()`
	`183`	`+ }`
`182`	`184`	`}`
`183`	`185`	`}`
`184`	`186`