update ava dependency from 2.1.0 to 5.3.0 (#266)

* update ava dependency from 2.1.0 to 5.3.0
* update used of throws to always pass expectation object

Author: dgrove-oss

package-lock.json

@@ -5,7 +5,7 @@
   "main": "lib/main.js",
   "typings": "lib/main.d.ts",
   "engines": {
-    "node": ">=6.0.0"
+    "node": ">=18.0.0"
   },
   "directories": {
     "test": "tests"
@@ -41,7 +41,7 @@
   "devDependencies": {
     "@types/node": "^12.0.10",
     "@types/swagger-schema-official": "^2.0.17",
-    "ava": "^2.1.0",
+    "ava": "^5.3.0",
     "codecov": "^3.5.0",
     "jszip": "^3.1.3",
     "nock": "^11.7.0",

package.json

@@ -477,7 +477,7 @@ test('create a new action with concurrency setting', t => {
 
 test('create an action without providing an action body', t => {
   const actions = new Actions()
-  t.throws(() => actions.create({ name: '12345' }), /Missing mandatory action/)
+  t.throws(() => actions.create({ name: '12345' }), { message: /Missing mandatory action/ })
 })
 
 test('create a new action with version parameter', t => {
@@ -556,23 +556,23 @@ test('creating sequence action with invalid sequence parameter', t => {
 
   const actions = new Actions(client)
 
-  t.throws(() => actions.create({ name: '12345', sequence: 'string' }), /Invalid sequence parameter/)
-  t.throws(() => actions.create({ name: '12345', sequence: { foo: 'bar' } }), /Invalid sequence parameter/)
+  t.throws(() => actions.create({ name: '12345', sequence: 'string' }), { message: /Invalid sequence parameter/ })
+  t.throws(() => actions.create({ name: '12345', sequence: { foo: 'bar' } }), { message: /Invalid sequence parameter/ })
 })
 
 test('creating sequence action with empty array', t => {
   const client = {}
 
   const actions = new Actions(client)
 
-  t.throws(() => actions.create({ name: '12345', sequence: [] }), /Invalid sequence parameter/)
+  t.throws(() => actions.create({ name: '12345', sequence: [] }), { message: /Invalid sequence parameter/ })
 })
 
 test('creating action with both sequence and action parameters', t => {
   const client = {}
   const actions = new Actions(client)
 
-  t.throws(() => actions.create({ name: '12345', action: 'function main() {}', sequence: 'string' }), /Invalid options parameters/)
+  t.throws(() => actions.create({ name: '12345', action: 'function main() {}', sequence: 'string' }), { message: /Invalid options parameters/ })
 })
 
 test('should pass through requested User-Agent header', t => {

test/unit/actions.test.js

@@ -197,5 +197,5 @@ test('should throw when retrieving activation without id', t => {
   const activations = new Activations()
   return t.throws(() => {
     activations.get()
-  }, /Missing mandatory/)
+  }, { message: /Missing mandatory/ })
 })

test/unit/activations.test.js

@@ -260,11 +260,11 @@ test('should use options for parameters even if environment parameters are avail
 })
 
 test('should throw error when missing API key option.', t => {
-  t.throws(() => new Client({ api: true }), /Missing api_key parameter./)
+  t.throws(() => new Client({ api: true }), { message: /Missing api_key parameter./ })
 })
 
 test('should throw error when missing both API and API Host options.', t => {
-  t.throws(() => new Client({ api_key: true }), /Missing either api or apihost parameters/)
+  t.throws(() => new Client({ api_key: true }), { message: /Missing either api or apihost parameters/ })
 })
 
 test('should handle multiple api parameter formats', t => {
@@ -403,7 +403,7 @@ test('should return path and status code in error message', t => {
   t.throws(() => client.handleErrors({
     options: { method, url },
     statusCode
-  }), `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "Response Missing Error Message."`)
+  }), { message: `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "Response Missing Error Message."` })
 })
 
 test('should return response error string in error message', t => {
@@ -415,27 +415,27 @@ test('should return response error string in error message', t => {
     error: { error: 'hello' },
     options: { method, url },
     statusCode
-  }), `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"`)
+  }), { message: `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"` })
   t.throws(() => client.handleErrors({
     error: { response: { result: { error: 'hello' } } },
     options: { method, url },
     statusCode
-  }), `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"`)
+  }), { message: `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"` })
   t.throws(() => client.handleErrors({
     error: { response: { result: { error: { error: 'hello' } } } },
     options: { method, url },
     statusCode
-  }), `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"`)
+  }), { message: `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "hello"` })
   t.throws(() => client.handleErrors({
     error: { response: { result: { error: { statusCode: 404 } } } },
     options: { method, url },
     statusCode
-  }), `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "application error, status code: ${404}"`)
+  }), { message: `${method} ${url} Returned HTTP ${statusCode} (${http.STATUS_CODES[statusCode]}) --> "application error, status code: ${404}"` })
 })
 
 test('should throw errors for non-HTTP response failures', t => {
   const client = new Client({ api_key: true, api: true })
-  t.throws(() => client.handleErrors({ message: 'error message' }), /error message/)
+  t.throws(() => client.handleErrors({ message: 'error message' }), { message: /error message/ })
 })
 
 test('should contain x-namespace-id header when namespace in constructor options', async t => {

test/unit/client.test.js

@@ -303,11 +303,11 @@ test('should be able to update feed', t => {
 test('should throw errors without trigger parameter ', t => {
   const client = { options: {} }
   const feeds = new Feeds(client)
-  t.throws(() => feeds.feed('', { feedName: 'myFeed' }), /trigger/)
+  t.throws(() => feeds.feed('', { feedName: 'myFeed' }), { message: /trigger/ })
 })
 
 test('should throw errors without id parameter', t => {
   const client = { options: {} }
   const feeds = new Feeds(client)
-  t.throws(() => feeds.feed('', { trigger: 'myFeed' }), /feedName/)
+  t.throws(() => feeds.feed('', { trigger: 'myFeed' }), { message: /feedName/ })
 })

test/unit/feeds.test.js

@@ -51,16 +51,16 @@ test('should parse namespace from resource with explicit ns and package but miss
 })
 
 test('should throw error parsing namespace with only namespace', t => {
-  t.throws(() => names.parseNamespace('/ns'), /Invalid resource identifier/)
+  t.throws(() => names.parseNamespace('/ns'), { message: /Invalid resource identifier/ })
 })
 
 test('should throw error parsing namespace with only extra paths', t => {
-  t.throws(() => names.parseNamespace('/ns/pkg/action/extra'), /Invalid resource identifier/)
-  t.throws(() => names.parseNamespace('ns/pkg/action/extra'), /Invalid resource identifier/)
+  t.throws(() => names.parseNamespace('/ns/pkg/action/extra'), { message: /Invalid resource identifier/ })
+  t.throws(() => names.parseNamespace('ns/pkg/action/extra'), { message: /Invalid resource identifier/ })
 })
 
 test('should throw error parsing namespace with missing parts', t => {
-  t.throws(() => names.parseNamespace('/'), /Invalid resource identifier/)
+  t.throws(() => names.parseNamespace('/'), { message: /Invalid resource identifier/ })
 })
 
 test('should parse id from resource without explicit ns', t => {
@@ -84,14 +84,14 @@ test('should parse id from resource with explicit ns and package but missing lea
 })
 
 test('should throw error parsing id with only namespace', t => {
-  t.throws(() => names.parseId('/ns'), /Invalid resource identifier/)
+  t.throws(() => names.parseId('/ns'), { message: /Invalid resource identifier/ })
 })
 
 test('should throw error parsing id with only extra paths', t => {
-  t.throws(() => names.parseId('/ns/pkg/action/extra'), /Invalid resource identifier/)
-  t.throws(() => names.parseId('ns/pkg/action/extra'), /Invalid resource identifier/)
+  t.throws(() => names.parseId('/ns/pkg/action/extra'), { message: /Invalid resource identifier/ })
+  t.throws(() => names.parseId('ns/pkg/action/extra'), { message: /Invalid resource identifier/ })
 })
 
 test('should throw error parsing id with missing parts', t => {
-  t.throws(() => names.parseId('/'), /Invalid resource identifier/)
+  t.throws(() => names.parseId('/'), { message: /Invalid resource identifier/ })
 })

test/unit/names.test.js

@@ -135,7 +135,7 @@ test('should delete package from identifier', t => {
 
 test('should throw error trying to invoke package', t => {
   const packages = new Packages()
-  return t.throws(() => packages.invoke(), /Operation \(invoke\) not supported/)
+  return t.throws(() => packages.invoke(), { message: /Operation \(invoke\) not supported/ })
 })
 
 test('should create a new package using string id', t => {

test/unit/packages.test.js

@@ -172,11 +172,11 @@ test('should send multiple requests for array parameters', t => {
 test('should throw errors when missing resource identifier', t => {
   const resources = new Resources()
 
-  t.throws(() => resources.get(), /Missing resource identifier from parameters, supported parameter names: name/)
-  t.throws(() => resources.invoke(), /Missing resource identifier from parameters, supported parameter names: name/)
-  t.throws(() => resources.create(), /Missing resource identifier from parameters, supported parameter names: name/)
-  t.throws(() => resources.update({}), /Missing resource identifier from parameters, supported parameter names: name/)
-  t.throws(() => resources.delete(), /Missing resource identifier from parameters, supported parameter names: name/)
+  t.throws(() => resources.get(), { message: /Missing resource identifier from parameters, supported parameter names: name/ })
+  t.throws(() => resources.invoke(), { message: /Missing resource identifier from parameters, supported parameter names: name/ })
+  t.throws(() => resources.create(), { message: /Missing resource identifier from parameters, supported parameter names: name/ })
+  t.throws(() => resources.update({}), { message: /Missing resource identifier from parameters, supported parameter names: name/ })
+  t.throws(() => resources.delete(), { message: /Missing resource identifier from parameters, supported parameter names: name/ })
 })
 
 test('should parse action name from identifier', t => {
@@ -188,7 +188,7 @@ test('should parse action name from identifier', t => {
   t.is(resources.parseId({ name: id }), id)
   t.is(resources.parseId({ name: nsId }), id)
   t.is(resources.parseId({ name: nsPackageId }), `package/12345`)
-  t.throws(() => resources.parseId({ name: '/ns' }), /Invalid resource/)
+  t.throws(() => resources.parseId({ name: '/ns' }), { message: /Invalid resource/ })
 })
 
 test('should parse namespace from identifier and options', t => {

test/unit/resources.test.js

@@ -84,9 +84,9 @@ test('should retrieve routes with apigwToken and basepath', t => {
 
 test('get routes with incorrect parameters', t => {
   const routes = new Routes({ options: {} })
-  t.throws(() => { routes.get() }, /Missing mandatory parameters: basepath or name/)
-  t.throws(() => { routes.get({}) }, /Missing mandatory parameters: basepath or name/)
-  t.throws(() => { routes.get({ basepath: 'id', name: 'id' }) }, /Invalid parameters: use basepath or name, not both/)
+  t.throws(() => { routes.get() }, { message: /Missing mandatory parameters: basepath or name/ })
+  t.throws(() => { routes.get({}) }, { message: /Missing mandatory parameters: basepath or name/ })
+  t.throws(() => { routes.get({ basepath: 'id', name: 'id' }) }, { message: /Invalid parameters: use basepath or name, not both/ })
 })
 
 // OVERRIDE gateway package
@@ -172,7 +172,7 @@ test('list routes providing basepath and name', t => {
       basepath: 'bp',
       name: 'nm'
     })
-  }, /Invalid parameters: use basepath or name, not both/)
+  }, { message: /Invalid parameters: use basepath or name, not both/ })
 })
 
 test('should delete a route with basepath', t => {
@@ -242,7 +242,7 @@ test('should delete a route with parameters', t => {
 test('delete routes without providing basepath or name', t => {
   const client = { options: {} }
   const routes = new Routes(client)
-  return t.throws(() => { routes.delete() }, /Missing mandatory parameters: basepath or name/)
+  return t.throws(() => { routes.delete() }, { message: /Missing mandatory parameters: basepath or name/ })
 })
 
 test('delete routes providing basepath and name', t => {
@@ -253,7 +253,7 @@ test('delete routes providing basepath and name', t => {
       basepath: 'bp',
       name: 'nm'
     })
-  }, /Invalid parameters: use basepath or name, not both/)
+  }, { message: /Invalid parameters: use basepath or name, not both/ })
 })
 
 test('should create a route', t => {
@@ -763,8 +763,8 @@ test('should parse path parameters', t => {
 
 test('create routes missing mandatory parameters', t => {
   const routes = new Routes()
-  t.throws(() => { routes.create() }, /Missing mandatory parameters: relpath, operation, action/)
-  t.throws(() => { routes.create({ relpath: true, operation: true }) }, /Missing mandatory parameters: action/)
-  t.throws(() => { routes.create({ action: true, operation: true }) }, /Missing mandatory parameters: relpath/)
-  t.throws(() => { routes.create({ relpath: true, action: true }) }, /Missing mandatory parameters: operation/)
+  t.throws(() => { routes.create() }, { message: /Missing mandatory parameters: relpath, operation, action/ })
+  t.throws(() => { routes.create({ relpath: true, operation: true }) }, { message: /Missing mandatory parameters: action/ })
+  t.throws(() => { routes.create({ action: true, operation: true }) }, { message: /Missing mandatory parameters: relpath/ })
+  t.throws(() => { routes.create({ relpath: true, action: true }) }, { message: /Missing mandatory parameters: operation/ })
 })

test/unit/routes.test.js

@@ -135,7 +135,7 @@ test('should delete rule from string identifier', t => {
 
 test('should throw error trying to invoke rule', t => {
   const rules = new Rules()
-  return t.throws(() => rules.invoke(), /Operation \(invoke\) not supported/)
+  return t.throws(() => rules.invoke(), { message: /Operation \(invoke\) not supported/ })
 })
 
 test('create a new rule', t => {
@@ -211,21 +211,21 @@ test('create a rule without providing a rule name', t => {
   const rules = new Rules(client)
   return t.throws(() => {
     rules.create({ action: '', trigger: '' })
-  }, /name, ruleName/)
+  }, { message: /name, ruleName/ })
 })
 
 test('create a rule without providing an action name', t => {
   const rules = new Rules()
   return t.throws(() => {
     rules.create({ name: '', trigger: '' })
-  }, /Missing mandatory action parameter/)
+  }, { message: /Missing mandatory action parameter/ })
 })
 
 test('create a rule without providing a trigger name', t => {
   const rules = new Rules()
   return t.throws(() => {
     rules.create({ name: '', action: '' })
-  }, /Missing mandatory trigger parameter/)
+  }, { message: /Missing mandatory trigger parameter/ })
 })
 
 test('update existing rule', t => {