seatsio 72.2.0 → 72.4.0

package/README.md

@@ -256,6 +256,18 @@ let event = await client.events.create(chart.key)
 console.log(`Created a chart with key ${chart.key} and an event with key: ${event.key}`)
 ```
 
+### Listing categories
+
+```js
+import { SeatsioClient, Region } from 'seatsio'
+
+let client = new SeatsioClient(Region.EU(), <COMPANY ADMIN KEY>, <WORKSPACE PUBLIC KEY>)
+let categories = await client.charts.listCategories("the chart key")
+for (const category of categoryList) {
+    console.log(category.label)
+}
+```
+
 ## Error Handling
 When an API call results in an error, a rejected promise is returned with a value that looks like
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "seatsio",
-  "version": "72.2.0",
+  "version": "72.4.0",
   "main": "index.js",
   "description": "Official JavaScript and Node.JS client library for the Seats.io REST API",
   "license": "MIT",
@@ -13,16 +13,16 @@
     "url": "https://github.com/seatsio/seatsio-js"
   },
   "dependencies": {
-    "axios": "1.2.2"
+    "axios": "1.3.4"
   },
   "devDependencies": {
-    "eslint": "8.31.0",
+    "eslint": "8.35.0",
     "eslint-config-standard": "17.0.0",
-    "eslint-plugin-import": "2.26.0",
-    "eslint-plugin-n": "15.6.0",
+    "eslint-plugin-import": "2.27.5",
+    "eslint-plugin-n": "15.6.1",
     "eslint-plugin-promise": "6.1.1",
-    "jest": "29.3.1",
-    "jest-cli": "29.3.1",
+    "jest": "29.4.3",
+    "jest-cli": "29.4.3",
     "uuid": "9.0.0"
   }
 }

package/src/Charts/Charts.js

@@ -68,6 +68,11 @@ class Charts {
         return this.client.delete(`/charts/${chartKey}/categories/${categoryKey}`)
     }
 
+    listCategories (key) {
+        return this.client.get(`/charts/${key}/categories`)
+            .then((res) => res.data.categories)
+    }
+
     /**
      * @param {string} key
      * @returns {object}

package/src/Events/Channels.js

@@ -16,6 +16,10 @@ class Channels {
         })
     }
 
+    addMultiple (eventKey, channelCreationParams) {
+        return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels`, channelCreationParams)
+    }
+
     remove (eventKey, channelKey) {
         return this.client.delete(`/events/${encodeURIComponent(eventKey)}/channels/${encodeURIComponent(channelKey)}`)
     }

package/tests/charts/manageCategories.test.js

@@ -30,3 +30,15 @@ test('should remove a category', async () => {
     const retrievedChart = await client.charts.retrievePublishedVersion(chart.key)
     expect(retrievedChart.categories.list).toEqual([{ key: 1, label: 'Category 1', color: '#aaaaaa', accessible: false }])
 })
+
+test('should retrieve the categories of a chart', async () => {
+    const { client } = await testUtils.createTestUserAndClient()
+    const categories = [
+        { key: 1, label: 'Category 1', color: '#aaaaaa', accessible: false },
+        { key: 'cat2', label: 'Category 2', color: '#bbbbbb', accessible: true }
+    ]
+    const chart = await client.charts.create('aChart', null, categories)
+
+    const categoryList = await client.charts.listCategories(chart.key)
+    expect(categoryList).toEqual(categories)
+})

package/tests/events/channels/addChannel.test.js

@@ -31,6 +31,42 @@ test('can add a channel', async () => {
     ])
 })
 
+test('can add multiple channels', async () => {
+    const {
+        client,
+        user
+    } = await testUtils.createTestUserAndClient()
+    const chartKey = testUtils.getChartKey()
+    await testUtils.createTestChart(chartKey, user.secretKey)
+    const event = await client.events.create(chartKey)
+
+    await client.events.channels.addMultiple(
+        event.key,
+        [
+            { key: 'channelKey1', name: 'channel 1', color: '#FFFF98', index: 1, objects: ['A-1', 'A-2'] },
+            { key: 'channelKey2', name: 'channel 2', color: '#FFFF99', index: 2, objects: ['A-3'] }
+        ]
+    )
+
+    const retrievedEvent = await client.events.retrieve(event.key)
+    expect(retrievedEvent.channels).toEqual([
+        new Channel({
+            key: 'channelKey1',
+            name: 'channel 1',
+            color: '#FFFF98',
+            index: 1,
+            objects: ['A-1', 'A-2']
+        }),
+        new Channel({
+            key: 'channelKey2',
+            name: 'channel 2',
+            color: '#FFFF99',
+            index: 2,
+            objects: ['A-3']
+        })
+    ])
+})
+
 test('index is optional', async () => {
     const {
         client,

package/tests/exponentialBackoff.test.js

@@ -2,13 +2,13 @@ const Region = require('../src/Region')
 const { SeatsioClient } = require('../index.js')
 
 test('aborts eventually if server keeps returning 429', async () => {
-    const client = new SeatsioClient(new Region('https://httpbin.org'))
+    const client = new SeatsioClient(new Region('https://httpbin.seatsio.net'))
     const start = new Date()
     try {
         await client.client.get('/status/429')
         throw new Error('Should have failed')
     } catch (e) {
-        expect(e).toBe('get /status/429 resulted in 429 TOO MANY REQUESTS error')
+        expect(e).toBe('get /status/429 resulted in 429 Too Many Requests error')
         const waitTime = new Date().getTime() - start.getTime()
         expect(waitTime).toBeGreaterThan(10000)
         expect(waitTime).toBeLessThan(20000)
@@ -16,33 +16,33 @@ test('aborts eventually if server keeps returning 429', async () => {
 })
 
 test('aborts directly if server returns error other than 429', async () => {
-    const client = new SeatsioClient(new Region('https://httpbin.org'))
+    const client = new SeatsioClient(new Region('https://httpbin.seatsio.net'))
     const start = new Date()
     try {
         await client.client.get('/status/400')
         throw new Error('Should have failed')
     } catch (e) {
-        expect(e).toBe('get /status/400 resulted in 400 BAD REQUEST error')
+        expect(e).toBe('get /status/400 resulted in 400 Bad Request error')
         const waitTime = new Date().getTime() - start.getTime()
         expect(waitTime).toBeLessThan(2000)
     }
 })
 
 test('aborts directly if server returns 429 but max retries 0', async () => {
-    const client = new SeatsioClient(new Region('https://httpbin.org')).setMaxRetries(0)
+    const client = new SeatsioClient(new Region('https://httpbin.seatsio.net')).setMaxRetries(0)
     const start = new Date()
     try {
         await client.client.get('/status/429')
         throw new Error('Should have failed')
     } catch (e) {
-        expect(e).toBe('get /status/429 resulted in 429 TOO MANY REQUESTS error')
+        expect(e).toBe('get /status/429 resulted in 429 Too Many Requests error')
         const waitTime = new Date().getTime() - start.getTime()
         expect(waitTime).toBeLessThan(2000)
     }
 })
 
 test('returns successfully when the server sends a 429 first, but then a successful response', async () => {
-    const client = new SeatsioClient(new Region('https://httpbin.org'))
+    const client = new SeatsioClient(new Region('https://httpbin.seatsio.net'))
     for (let i = 0; i < 20; ++i) {
         const response = await client.client.get('/status/429:0.25,204:0.75')
         expect(response.status).toBe(204)

package/tests/subaccounts/filterSubaccounts.test.js

@@ -21,13 +21,13 @@ test('should filter subaccounts ', async () => {
 test('should filter subaccounts with special characters', async () => {
     const { client } = await testUtils.createTestUserAndClient()
     let i = 0
-    await testUtils.createArray(20, () => client.subaccounts.create('test-/@/' + i++))
+    await testUtils.createArray(10, () => client.subaccounts.create('test-/@/' + i++))
     const retrievedSubaccountKeys = []
     for await (const subaccount of client.subaccounts.listAll('test-/@/1')) {
         retrievedSubaccountKeys.push(subaccount.secretKey)
     }
 
-    expect(retrievedSubaccountKeys.length).toEqual(11)
+    expect(retrievedSubaccountKeys.length).toEqual(1)
 })
 
 test('should filter with no results ', async () => {

package/tests/testUtils.js

@@ -81,14 +81,14 @@ module.exports = {
         return uuidv4() + '@mailinator.com'
     },
 
-    createArray (length, fn) {
+    async createArray (length, fn) {
         const array = []
 
         for (let i = 0; i < length; ++i) {
-            array.push(fn())
+            array.push(await fn())
         }
 
-        return Promise.all(array)
+        return array
     },
 
     deferred () {