Improve AMQP Integration Tests

- Remove old amqp tests - Enhance purge function to allow clearing queues without deleting - Fix type error in AMQP Transporter - Run the same test suite for built-in and AMQP balancing - PreferLocal = false for rpc suite.
moleculerjs · Oct 9, 2017 · 2a1886f · 2a1886f
1 parent 963e30a
commit 2a1886f
Show file tree

Hide file tree

Showing 21 changed files with 316 additions and 876 deletions.
diff --git a/package.json b/package.json
@@ -18,7 +18,7 @@
     "test": "jest --coverage --no-cache",
     "test:unit": "jest --testMatch \"**/unit/**spec.js\" --coverage --no-cache",
     "test:int": "jest --testMatch \"**/integration/**spec.js\" --coverage --no-cache",
-    "test:amqp": "jest --testMatch \"**/integration/amqp/**spc.js\""
+    "test:amqp": "jest --testMatch \"**/integration/amqp/**spc.js\" --runInBand"
   },
   "keywords": [
     "microservices",

diff --git a/src/transporters/amqp.js b/src/transporters/amqp.js
@@ -182,7 +182,7 @@ class AmqpTransporter extends Transporter {
 				.then(() => this.channel.close())
 				.then(() => this.connection.close())
 				.then(() => {
-					this.bindings = null;
+					this.bindings = [];
 					this.channel = null;
 					this.connection = null;
 				})

diff --git a/test/integration/amqp.old/events/events.spc.js b/test/integration/amqp.old/events/events.spc.js
diff --git a/test/integration/amqp.old/events/pub.js b/test/integration/amqp.old/events/pub.js
diff --git a/test/integration/amqp.old/events/sub1.js b/test/integration/amqp.old/events/sub1.js
diff --git a/test/integration/amqp.old/events/sub2.js b/test/integration/amqp.old/events/sub2.js
diff --git a/test/integration/amqp.old/events/sub3.js b/test/integration/amqp.old/events/sub3.js
diff --git a/test/integration/amqp.old/purge.js b/test/integration/amqp.old/purge.js
diff --git a/test/integration/amqp.old/rpc/fiveRequests.js b/test/integration/amqp.old/rpc/fiveRequests.js