comq 0.17.0 → 0.17.1

package/features/events.routed.feature

@@ -17,3 +17,9 @@ Feature: Routed events
     When a message is routed to the `records` exchange under the `placed` key
     Then `orders` receives the event
     And `customers` receives nothing
+
+  Scenario: Routing over a sharded connection
+    Given an active sharded connection
+    And that `orders` is bound to the `records` exchange under the `placed` key
+    When a message is routed to the `records` exchange under the `placed` key
+    Then `orders` receives the event

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "comq",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "ComQ",
   "author": {
     "name": "Artem Gurtovoi",

package/readme.md

@@ -181,6 +181,9 @@ The queue is named rather than derived from a Consumer group, because what ident
 the Key it is bound under rather than the exchange it belongs to. It is also durable, so what is
 published while nothing is consuming is held rather than dropped.
 
+Over a [sharded connection](#sharded-connection) they behave as the rest does: `route` publishes
+to one shard, and `subscribe` consumes the queue on every one of them.
+
 ### Example
 
 ```javascript

package/source/shards/channel.js

@@ -64,6 +64,10 @@ class Channel {
     await this.#every((channel) => channel.subscribe(queue, group, consumer))
   }
 
+  async bound (exchange, queue, key, consumer) {
+    await this.#every((channel) => channel.bound(exchange, queue, key, consumer))
+  }
+
   async send (queue, buffer, options) {
     await this.#one((channel) => channel.send(queue, buffer, options))
   }
@@ -72,6 +76,10 @@ class Channel {
     await this.#one((channel) => channel.publish(exchange, buffer, options))
   }
 
+  async route (exchange, key, buffer, options) {
+    await this.#one((channel) => channel.route(exchange, key, buffer, options))
+  }
+
   async fire (queue, buffer, options) {
     // noinspection  JSValidateTypes
     return await this.#one((channel) => channel.fire(queue, buffer, options))

package/test/shards/channel.test.js

@@ -718,3 +718,36 @@ async function getCreatedChannels () {
 
   return await Promise.all(promises)
 }
+
+describe('routed', () => {
+  const exchange = generate()
+  const queue = generate()
+  const key = generate()
+  const buffer = randomBytes(8)
+  const consumer = jest.fn()
+
+  /** @type {jest.MockedObject<comq.Channel>[]} */
+  let channels
+
+  beforeEach(async () => {
+    channel = await create(connections, type)
+    channels = await getCreatedChannels()
+  })
+
+  it('should bind using all channels', async () => {
+    await channel.bound(exchange, queue, key, consumer)
+
+    for (const chan of channels) {
+      expect(chan.bound).toHaveBeenCalledWith(exchange, queue, key, consumer)
+    }
+  })
+
+  it('should route to a single channel', async () => {
+    await channel.route(exchange, key, buffer)
+
+    const used = channels.filter((chan) => chan.route.mock.calls.length > 0)
+
+    expect(used).toHaveLength(1)
+    expect(used[0].route).toHaveBeenCalledWith(exchange, key, buffer, undefined)
+  })
+})