aerospike 5.9.0 → 5.11.0

package/README.md

@@ -18,7 +18,7 @@ It supports the following operating systems:
 - Amazon Linux 2023
 - Ubuntu 20.04/22.04 (Focal Fossa, Jammy Jellyfish)
 - Many Linux distributions compatible with one of the above OS releases.
-- macOS 12/13
+- macOS 12/13/14
 
 The client is compatible with arm64, aarch64, and x86_64 architectures.
 
@@ -71,7 +71,7 @@ record using the Aerospike database.
 ```js
 const Aerospike = require('aerospike')
 
-// INSERT HOSTNAME AND PORT NUMBER OF AEROPSIKE SERVER NODE HERE!
+// INSERT HOSTNAME AND PORT NUMBER OF AEROSPIKE SERVER NODE HERE!
 const config = {
   hosts: '192.168.33.10:3000',
 }
@@ -188,6 +188,10 @@ To install library prerequisites using `apt`:
 sudo apt install g++ libssl libssl-dev zlib1g-dev
 ```
 
+### Windows
+
+See our [Windows README.md](https://github.com/aerospike/aerospike-client-nodejs/blob/master/README_WINDOWS.md) for details on how to build and install on windows.
+
 ### macOS
 
 Before starting with the Aerospike Nodejs Client, verify that you have the following prerequisites:

package/examples/typescript.ts

@@ -0,0 +1,23 @@
+import * as Aerospike from 'aerospike';
+
+(async function () {
+  let client = null;
+  try {
+    client = await Aerospike.connect();
+    const key = new Aerospike.Key('test', 'test', 'abcd');
+    const bins = {
+      name: 'Norma',
+      age: 31
+    };
+
+    await client.put(key, bins);
+    const record = await client.get(key);
+    console.info('Record:', record);
+    await client.remove(key);
+  } catch (error) {
+    console.error('Error:', error);
+    process.exit(1);
+  } finally {
+    if (client) client.close();
+  }
+})();

package/lib/client.js

@@ -1471,7 +1471,7 @@ Client.prototype.batchWrite = function (records, policy, callback) {
  * const Aerospike = require('aerospike')
  * var path = require('path');
  *
- * // INSERT HOSTNAME AND PORT NUMBER OF AEROPSIKE SERVER NODE HERE!
+ * // INSERT HOSTNAME AND PORT NUMBER OF AEROSPIKE SERVER NODE HERE!
  * const config = {
  *   hosts: '192.168.33.10:3000',
  *   // Timeouts disabled, latency dependent on server location. Configure as needed.

package/lib/lists.js

@@ -1079,6 +1079,61 @@ exports.clear = function (bin) {
   return new ListOperation(opcodes.LIST_CLEAR, bin)
 }
 
+/**
+ * @summary Creates list create operation.
+ *
+ *
+ * @param {string} bin - bin name.
+ * @param {number} order - list order.
+ * @parm {boolean} pad - If true, the context is allowed to be beyond list boundaries. In that case, nil
+ * list entries will be inserted to satisfy the context position.
+ * @param {boolean} persistIndex - If true, persist list index. A list index improves lookup performance,
+ * but requires more storage. A list index can be created for a top-level ordered list only. Nested and
+ * unordered list indexes are not supported.
+ * @param {number} ctx - optional path to nested list. If not defined, the top-level list is used.
+ *
+ * @returns {Object} Operation that can be passed to the {@link Client#operate} command.
+ *
+ * @example
+ *
+ * const Aerospike = require('aerospike')
+ * const lists = Aerospike.lists
+ * const key = new Aerospike.Key('test', 'demo', 'listKey')
+ *
+ * // INSERT HOSTNAME AND PORT NUMBER OF AEROSPIKE SERVER NODE HERE!
+ * var config = {
+ *   hosts: '192.168.33.10:3000',
+ *   // Timeouts disabled, latency dependent on server location. Configure as needed.
+ *   policies: {
+ *     operate : new Aerospike.OperatePolicy({socketTimeout : 0, totalTimeout : 0})
+ *   }
+ * }
+ *
+ * Aerospike.connect(config).then(async client => {
+ *   let ops = [
+ *     lists.create('list', lists.order.ORDERED, false, true)
+ *   ]
+ *   let result = await client.operate(key, ops)
+ *   console.log(result.bins)                    // => { list: null }
+ *   let record = await client.get(key)
+ *   console.log(record.bins)                    // => { list: [] }
+ *
+ *   await client.remove(key)
+ *   client.close()
+ * })
+ */
+exports.create = function (bin, order, pad = false, persistIndex = false, ctx) {
+  const op = new ListOperation(opcodes.LIST_CREATE, bin)
+  op.order = order
+  op.pad = pad
+  op.persistIndex = persistIndex
+  if (ctx === undefined) {
+    return op
+  }
+
+  return op.withContext(ctx)
+}
+
 /**
  * @summary Sets the list element at the specified index to a new value.
  * @description This operation returns no result.

package/lib/policies/batch_write_policy.js

@@ -36,6 +36,14 @@ class BatchWritePolicy {
      *
      */
     this.filterExpression = props.filterExpression
+
+    /**
+     * The time-to-live (expiration) of the record in seconds.
+     *
+     * @type number
+     */
+    this.ttl = props.ttl
+
     /**
      * Specifies the behavior for the key.
      *

package/lib/policy.js

@@ -60,7 +60,7 @@ const AdminPolicy = require('./policies/admin_policy')
  * * {@link WritePolicy} - Applies to {@link Client#put}.
  * * {@link BatchPolicy} - Applies to {@link Client#batchRead} as well as the
  *   deprecated {@link Client#batchExists}, {@link Client#batchGet}, and {@link
- *   Client#batchSelect} operations.
+ *   Client#batchSelect} operations. Also used when providing batchParentWrite policy to a client configuration.
  * * {@link BatchApplyPolicy} - Applies to {@link Client#batchApply}.
  * * {@link BatchReadPolicy} - Applies to {@link Client#batchRead}.
  * * {@link BatchRemovePolicy} - Applies to {@link Client#batchRemove}.
@@ -443,6 +443,7 @@ function policyClass (type) {
     case 'batchRead': return BatchReadPolicy
     case 'batchRemove': return BatchRemovePolicy
     case 'batchWrite': return BatchWritePolicy
+    case 'batchParentWrite': return BatchPolicy
     case 'batchApply': return BatchApplyPolicy
     case 'commandQueue': return CommandQueuePolicy
     case 'hll': return HLLPolicy

package/lib/status.js

@@ -24,6 +24,37 @@ const as = require('bindings')('aerospike.node')
  * @description Database operation error codes.
  */
 
+/**
+ * One or more keys failed in a batch.
+ * @const {number}
+ */
+exports.BATCH_FAILED = exports.AEROSPIKE_BATCH_FAILED = as.status.AEROSPIKE_BATCH_FAILED
+
+/**
+ * No response received from server.
+ * @const {number}
+ */
+exports.NO_RESPONSE = exports.AEROSPIKE_NO_RESPONSE = as.status.AEROSPIKE_NO_RESPONSE
+
+/**
+ * Max errors limit reached.
+ * @const {number}
+ */
+exports.MAX_ERROR_RATE = exports.AEROSPIKE_MAX_ERROR_RATE = as.status.AEROSPIKE_MAX_ERROR_RATE
+
+/**
+ * Abort split batch retry and use normal node retry instead.
+ * Used internally and should not be returned to user.
+ * @const {number}
+ */
+exports.USE_NORMAL_RETRY = exports.AEROSPIKE_USE_NORMAL_RETRY = as.status.AEROSPIKE_USE_NORMAL_RETRY
+
+/**
+ * Max retries limit reached.
+ * @const {number}
+ */
+exports.ERR_MAX_RETRIES_EXCEEDED = exports.AEROSPIKE_ERR_MAX_RETRIES_EXCEEDED = as.status.AEROSPIKE_ERR_MAX_RETRIES_EXCEEDED
+
 /**
  * Async command delay queue is full.
  * @const {number}

package/lib/typedefs.js

@@ -229,6 +229,8 @@
  *
  * @property {ApplyPolicy} apply - Default apply policy
  * @property {BatchPolicy} batch - Default batch policy
+ * @property {BatchPolicy} batchWrite - Default write policy used in batch operate commands. Contains a {@link BatchWritePolicy}.
+ * @property {BatchPolicy} batchParentWrite - Default parent policy used in batch write commands. Contains a {@link BatchPolicy}.
  * @property {InfoPolicy} info - Default info policy
  * @property {OperatePolicy} operate - Default operate policy
  * @property {ReadPolicy} read - Default read policy

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aerospike",
-  "version": "5.9.0",
+  "version": "5.11.0",
   "description": "Aerospike Client Library",
   "keywords": [
     "aerospike",
@@ -42,6 +42,7 @@
     "install": "npm-run-all removeExtraBinaries build",
     "build": "node-pre-gyp install --fallback-to-build",
     "test": "mocha",
+    "test-dry-run": "mocha --dry-run",
     "test-noserver": "GLOBAL_CLIENT=false mocha -g '#noserver'",
     "lint": "standard",
     "posttest": "npm run lint",
@@ -58,26 +59,25 @@
     "@mapbox/node-pre-gyp": "^1.0.11",
     "bindings": "^1.5.0",
     "minimatch": "^3.1.2",
-    "nan": "^2.17.0",
-    "node-gyp": "^8.4.1"
+    "nan": "^2.18.0",
+    "node-gyp": "^10.0.1",
+    "npm-run-all": "^4.1.5"
   },
   "devDependencies": {
-    "@types/node": "^17.0.45",
+    "@types/node": "^20.11.18",
     "chai": "^4.3.7",
     "choma": "^1.2.1",
     "codecov": "^3.8.3",
     "deep-eql": "^4.1.3",
-    "dirty-chai": "^2.0.1",
-    "husky": "^7.0.4",
-    "mocha": "^9.2.2",
+    "husky": "^9.0.11",
+    "mocha": "^10.3.0",
     "mocha-clean": "^1.0.0",
-    "npm-run-all": "^4.1.5",
     "nyc": "^15.1.0",
     "p-throttle": "^3.1.0",
-    "semver": "^7.5.4",
-    "standard": "^16.0.4",
-    "tmp": "^0.1.0",
-    "yargs": "^15.4.1"
+    "semver": "^7.6.0",
+    "standard": "^17.1.0",
+    "tmp": "^0.2.1",
+    "yargs": "^17.7.2"
   },
   "standard": {
     "ignore": [

package/scripts/build-package.ps1

@@ -36,7 +36,23 @@ function build_nodejs_client {
 
 build_nodejs_client v18
 Copy-Item -Recurse build\release\* lib\binding\node-v108-win32-x64
+Remove-Item .\lib\binding\node-v108-win32-x64\obj
+Remove-Item .\lib\binding\node-v108-win32-x64\aerospike.pdb
+Remove-Item .\lib\binding\node-v108-win32-x64\aerospike.ipdb
+Remove-Item .\lib\binding\node-v108-win32-x64\aerospike.iobj
+
 build_nodejs_client v20
 Copy-Item -Recurse build\release\* lib\binding\node-v115-win32-x64
+Remove-Item -Recurse .\lib\binding\node-v115-win32-x64\obj
+Remove-Item .\lib\binding\node-v115-win32-x64\aerospike.pdb
+Remove-Item .\lib\binding\node-v115-win32-x64\aerospike.ipdb
+Remove-Item .\lib\binding\node-v115-win32-x64\aerospike.iobj
+
+build_nodejs_client v21
+Copy-Item -Recurse build\release\* lib\binding\node-v120-win32-x64
+Remove-Item -Recurse .\lib\binding\node-v120-win32-x64\obj
+Remove-Item .\lib\binding\node-v120-win32-x64\aerospike.pdb
+Remove-Item .\lib\binding\node-v120-win32-x64\aerospike.ipdb
+Remove-Item .\lib\binding\node-v120-win32-x64\aerospike.iobj
 
 nvm use v18.12.1

package/scripts/build-package.sh

@@ -56,6 +56,7 @@ rm -rf ${AEROSPIKE_NODEJS_RELEASE_HOME}/node-*-${OS_FLAVOR}-*
 
 build_nodejs_client v18
 build_nodejs_client v20
+build_nodejs_client v21
 
 nvm use v20
 

package/scripts/prebuiltBinding.js

@@ -11,40 +11,56 @@ const rm = util.promisify(fs.rm)
   const arch = await os.arch()
 
   if (platform === 'darwin') {
+    rm('lib/binding/node-v120-win32-x64', { recursive: true, force: true })
     rm('lib/binding/node-v115-win32-x64', { recursive: true, force: true })
     rm('lib/binding/node-v108-win32-x64', { recursive: true, force: true })
     if (arch === 'x64') {
+      rm('lib/binding/node-v120-darwin-arm64', { recursive: true, force: true })
       rm('lib/binding/node-v115-darwin-arm64', { recursive: true, force: true })
       rm('lib/binding/node-v108-darwin-arm64', { recursive: true, force: true })
     } else {
+      rm('lib/binding/node-v120-darwin-x64', { recursive: true, force: true })
       rm('lib/binding/node-v115-darwin-x64', { recursive: true, force: true })
       rm('lib/binding/node-v108-darwin-x64', { recursive: true, force: true })
     }
     await rm('lib/binding/openssl@3', { recursive: true, force: true })
     await rm('lib/binding/openssl@1', { recursive: true, force: true })
   } else if (platform === 'win32') {
+    rm('lib/binding/node-v120-darwin-arm64', { recursive: true, force: true })
     rm('lib/binding/node-v115-darwin-arm64', { recursive: true, force: true })
     rm('lib/binding/node-v108-darwin-arm64', { recursive: true, force: true })
+    rm('lib/binding/node-v120-darwin-x64', { recursive: true, force: true })
     rm('lib/binding/node-v115-darwin-x64', { recursive: true, force: true })
     rm('lib/binding/node-v108-darwin-x64', { recursive: true, force: true })
     await rm('lib/binding/openssl@3', { recursive: true, force: true })
     await rm('lib/binding/openssl@1', { recursive: true, force: true })
   } else {
+    rm('lib/binding/node-v120-win32-x64', { recursive: true, force: true })
     rm('lib/binding/node-v115-win32-x64', { recursive: true, force: true })
     rm('lib/binding/node-v108-win32-x64', { recursive: true, force: true })
+    rm('lib/binding/node-v120-darwin-arm64', { recursive: true, force: true })
     rm('lib/binding/node-v115-darwin-arm64', { recursive: true, force: true })
     rm('lib/binding/node-v108-darwin-arm64', { recursive: true, force: true })
+    rm('lib/binding/node-v120-darwin-x64', { recursive: true, force: true })
     rm('lib/binding/node-v115-darwin-x64', { recursive: true, force: true })
     rm('lib/binding/node-v108-darwin-x64', { recursive: true, force: true })
 
-    const output = await exec('openssl version')
-    const version = output.stdout
-    const openssl = version.split(' ')[1].slice(0, 1)
+    const output = await exec("ldd --version | awk 'NR==1{print $NF}'")
+    const version = Number(output.stdout)
+
+    let openssl
+    if (version < 2.33) {
+      openssl = '1'
+    } else {
+      openssl = '3'
+    }
 
     if (arch === 'x64') {
+      await rename('lib/binding/openssl@' + openssl + '/node-v120-linux-x64', 'lib/binding/node-v120-linux-x64')
       await rename('lib/binding/openssl@' + openssl + '/node-v115-linux-x64', 'lib/binding/node-v115-linux-x64')
       await rename('lib/binding/openssl@' + openssl + '/node-v108-linux-x64', 'lib/binding/node-v108-linux-x64')
     } else {
+      await rename('lib/binding/openssl@' + openssl + '/node-v120-linux-arm64', 'lib/binding/node-v120-linux-arm64')
       await rename('lib/binding/openssl@' + openssl + '/node-v115-linux-arm64', 'lib/binding/node-v115-linux-arm64')
       await rename('lib/binding/openssl@' + openssl + '/node-v108-linux-arm64', 'lib/binding/node-v108-linux-arm64')
     }

package/src/include/enums.h

@@ -21,6 +21,7 @@
 #define TTL_NAMESPACE_DEFAULT 0
 #define TTL_NEVER_EXPIRE -1
 #define TTL_DONT_UPDATE -2
+#define TTL_CLIENT_DEFAULT -3
 
 v8::Local<v8::Object> auth_mode_enum_values();
 v8::Local<v8::Object> bitwise_enum_values();

package/src/main/client.cc

@@ -108,7 +108,7 @@ NAN_METHOD(AerospikeClient::Connect)
 }
 
 /**
- *  Close the connections to the Aeropsike cluster.
+ *  Close the connections to the Aerospike cluster.
  */
 NAN_METHOD(AerospikeClient::Close)
 {

package/src/main/config.cc

@@ -257,6 +257,26 @@ int config_from_jsobject(as_config *config, Local<Object> configObj,
 			}
 		}
 
+		policy_val = Nan::Get(policies_obj, Nan::New("batchWrite").ToLocalChecked())
+						 .ToLocalChecked();
+		if (policy_val->IsObject()) {
+			if ((rc = batchwrite_policy_from_jsobject(&policies->batch_write,
+												policy_val.As<Object>(),
+												log)) != AS_NODE_PARAM_OK) {
+				goto Cleanup;
+			}
+		}
+
+		policy_val = Nan::Get(policies_obj, Nan::New("batchParentWrite").ToLocalChecked())
+						 .ToLocalChecked();
+		if (policy_val->IsObject()) {
+			if ((rc = batchpolicy_from_jsobject(&policies->batch_parent_write,
+												policy_val.As<Object>(),
+												log)) != AS_NODE_PARAM_OK) {
+				goto Cleanup;
+			}
+		}
+
 		policy_val = Nan::Get(policies_obj, Nan::New("info").ToLocalChecked())
 						 .ToLocalChecked();
 		if (policy_val->IsObject()) {

package/src/main/enums/exp_enum.cc

@@ -128,6 +128,7 @@ Local<Object> exp_opcode_values()
 
 	Local<Object> exp_type = Nan::New<Object>();
 	set(exp_type, "NIL", as_exp_type::AS_EXP_TYPE_NIL);
+	set(exp_type, "BOOL", as_exp_type::AS_EXP_TYPE_BOOL);
 	set(exp_type, "INT", as_exp_type::AS_EXP_TYPE_INT);
 	set(exp_type, "STR", as_exp_type::AS_EXP_TYPE_STR);
 	set(exp_type, "LIST", as_exp_type::AS_EXP_TYPE_LIST);

package/src/main/enums/ttl.cc

@@ -31,5 +31,6 @@ Local<Object> ttl_enum_values()
 	set(obj, "NAMESPACE_DEFAULT", TTL_NAMESPACE_DEFAULT);
 	set(obj, "NEVER_EXPIRE", TTL_NEVER_EXPIRE);
 	set(obj, "DONT_UPDATE", TTL_DONT_UPDATE);
+	set(obj, "CLIENT_DEFAULT", TTL_CLIENT_DEFAULT);
 	return scope.Escape(obj);
 }

package/src/main/list_operations.cc

@@ -1032,6 +1032,29 @@ bool add_list_increment_op(as_operations *ops, const char *bin,
 	return true;
 }
 
+bool add_list_create_op(as_operations *ops, const char *bin,
+						   as_cdt_ctx *context, Local<Object> op, LogInfo *log)
+{
+	as_list_order order;
+	if (get_uint32_property((uint32_t*)&order, op, "order", log) != AS_NODE_PARAM_OK) {
+		return false;
+	}
+
+	bool pad;
+	if (get_bool_property(&pad, op, "pad", log) != AS_NODE_PARAM_OK) {
+		return false;
+	}
+
+	bool persist_index;
+	if (get_bool_property(&persist_index, op, "persistIndex", log) != AS_NODE_PARAM_OK) {
+		return false;
+	}
+
+	as_v8_debug(log, "order=%i, pad=%s, persist_index=%s", order, pad ? "true" : "false", persist_index ? "true" : "false");
+	as_operations_list_create_all(ops, bin, context, order, pad, persist_index);
+	return true;
+}
+
 bool add_list_size_op(as_operations *ops, const char *bin, as_cdt_ctx *context,
 					  Local<Object> obj, LogInfo *log)
 {
@@ -1084,7 +1107,8 @@ const ops_table_entry ops_table[] = {
 	{"LIST_GET_BY_RANK", add_list_get_by_rank_op},
 	{"LIST_GET_BY_RANK_RANGE", add_list_get_by_rank_range_op},
 	{"LIST_INCREMENT", add_list_increment_op},
-	{"LIST_SIZE", add_list_size_op}};
+	{"LIST_SIZE", add_list_size_op},
+	{"LIST_CREATE", add_list_create_op}};
 
 int add_list_op(as_operations *ops, uint32_t opcode, Local<Object> op,
 				LogInfo *log)

package/src/main/policy.cc

@@ -436,6 +436,11 @@ int batchwrite_policy_from_jsobject(as_policy_batch_write *policy,
 		AS_NODE_PARAM_OK) {
 		return rc;
 	}
+	if ((rc = get_optional_uint32_property((uint32_t *)&policy->ttl,
+										   NULL, obj, "ttl", log)) !=
+		AS_NODE_PARAM_OK) {
+		return rc;
+	}
 	if ((rc = get_optional_uint32_property((uint32_t *)&policy->gen, NULL, obj,
 										   "gen", log)) != AS_NODE_PARAM_OK) {
 		return rc;
@@ -445,6 +450,11 @@ int batchwrite_policy_from_jsobject(as_policy_batch_write *policy,
 		AS_NODE_PARAM_OK) {
 		return rc;
 	}
+	if ((rc = get_optional_uint32_property((uint32_t *)&policy->ttl, NULL, obj,
+										 "ttl", log)) !=
+		AS_NODE_PARAM_OK) {
+		return rc;
+	}
 	if ((rc = get_optional_bool_property(&policy->durable_delete, NULL, obj,
 										 "durableDelete", log)) !=
 		AS_NODE_PARAM_OK) {

package/src/main/util/conversions_batch.cc

@@ -204,6 +204,7 @@ int batch_write_record_from_jsobject(as_batch_records *batch_records,
 		Local<Array> ops = maybeOps.As<Array>();
 		as_v8_debug(log, "Adding operations to batch write record");
 		record->ops = as_operations_new(ops->Length());
+		record->ops->ttl = AS_RECORD_CLIENT_DEFAULT_TTL;
 		if (operations_from_jsarray(record->ops, ops, (LogInfo *)log) !=
 			AS_NODE_PARAM_OK) {
 			as_v8_error(

package/test/admin.js

@@ -20,7 +20,6 @@
 
 const Aerospike = require('../lib/aerospike')
 const helper = require('./test_helper')
-const options = require('./util/options')
 
 function getRandomInt (max) {
   return Math.floor(Math.random() * max)
@@ -440,7 +439,7 @@ context('admin commands', async function () {
         // Should fail, assert failure if error is not returned.
         expect(1).to.equal(2)
       } catch (error) {
-        expect(error).to.exist().and.have.property('code', Aerospike.status.INVALID_ROLE)
+        expect(error).to.exist.and.have.property('code', Aerospike.status.INVALID_ROLE)
       }
     })
 
@@ -452,7 +451,7 @@ context('admin commands', async function () {
         // Should fail, assert failure if error is not returned.
         expect(1).to.equal(2)
       } catch (error) {
-        expect(error).to.exist().and.have.property('code', Aerospike.status.INVALID_ROLE)
+        expect(error).to.exist.and.have.property('code', Aerospike.status.INVALID_ROLE)
       }
     })
   })
@@ -460,24 +459,28 @@ context('admin commands', async function () {
   describe('Client#changePassword()', function () {
     it('Changes password for user', async function () {
       client.changePassword(username1, 'password350', null)
+      await wait(waitMs + 30000)
       const config = {
-        hosts: options.host + ':' + options.port,
+        hosts: helper.config.hosts,
         user: username1,
         password: 'password350'
       }
       const dummyClient = await Aerospike.connect(config)
-      dummyClient.close()
+      return dummyClient.close()
     })
 
     it('With policy', async function () {
       client.changePassword(username2, 'password250', policy)
+      await wait(waitMs + 3000)
+
       const config = {
-        hosts: options.host + ':' + options.port,
+        hosts: helper.config.hosts,
         user: username2,
         password: 'password250'
       }
+
       const dummyClient = await Aerospike.connect(config)
-      dummyClient.close()
+      return dummyClient.close()
     })
   })
 
@@ -490,7 +493,7 @@ context('admin commands', async function () {
         // Should fail, assert failure if error is not returned.
         expect(1).to.equal(2)
       } catch (error) {
-        expect(error).to.exist().and.have.property('code', Aerospike.status.INVALID_USER)
+        expect(error).to.exist.and.have.property('code', Aerospike.status.INVALID_USER)
       }
     })
     it('With policy', async function () {
@@ -501,7 +504,7 @@ context('admin commands', async function () {
         // Should fail, assert failure if error is not returned.
         expect(1).to.equal(2)
       } catch (error) {
-        expect(error).to.exist().and.have.property('code', Aerospike.status.INVALID_USER)
+        expect(error).to.exist.and.have.property('code', Aerospike.status.INVALID_USER)
       }
     })
   })