From 47643bbb29ed34fa80b0bbfe5f7a7b8d7443cf47 Mon Sep 17 00:00:00 2001
From: minozzang508 <minozzang508@naver.com>
Date: Sun, 17 Mar 2024 20:33:51 +0900
Subject: [PATCH] refactor: remove emitting error event on connack timeout

---
 src/lib/client.ts       |  1 -
 test/abstract_client.ts | 12 ------------
 2 files changed, 13 deletions(-)

diff --git a/src/lib/client.ts b/src/lib/client.ts
index ac66a1e81..b645d5e0e 100644
--- a/src/lib/client.ts
+++ b/src/lib/client.ts
@@ -852,7 +852,6 @@ 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)
 
diff --git a/test/abstract_client.ts b/test/abstract_client.ts
index c7daa1bc6..d51063548 100644
--- a/test/abstract_client.ts
+++ b/test/abstract_client.ts
@@ -3004,18 +3004,6 @@ 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',
 			{