fix: emit error event on connack timeout (#1781)

* fix: emit `error` event on connack timeout

* fix: force close client

Author: robertsLando

src/lib/client.ts

@@ -850,6 +850,7 @@ export default class MqttClient extends TypedEventEmitter<MqttClientEventCallbac
 			this.log(
 				'!!connectTimeout hit!! Calling _cleanUp with force `true`',
 			)
+			this.emit('error', new Error('connack timeout'))
 			this._cleanUp(true)
 		}, this.options.connectTimeout)
 

test/abstract_client.ts

@@ -3004,6 +3004,18 @@ export default function abstractTest(server, config, ports) {
 			}, 50)
 		})
 
+		it('should emit connack timeout error', function _test(t, done) {
+			const client = connect({
+				connectTimeout: 0,
+				reconnectPeriod: 5000,
+			})
+
+			client.on('error', (err) => {
+				assert.equal(err.message, 'connack timeout')
+				client.end(true, done)
+			})
+		})
+
 		it(
 			'should resend in-flight QoS 1 publish messages from the client',
 			{