Minor changes

lib/client-options.js

@@ -53,15 +53,11 @@ const errors = require("./errors.js");
  * @property {String} [applicationName] An optional setting identifying the name of the application using
  * the {@link Client} instance.
  *
- * This value is passed to DSE and is useful as metadata for describing a client connection on the server side.
- *
- * [TODO: Add support for this field]
+ * This value is passed to database and is useful as metadata for describing a client connection on the server side.
  * @property {String} [applicationVersion] An optional setting identifying the version of the application using
  * the {@link Client} instance.
  *
- * This value is passed to DSE and is useful as metadata for describing a client connection on the server side.
- *
- * [TODO: Add support for this field]
+ * This value is passed to database and is useful as metadata for describing a client connection on the server side.
  * @property {Object} [monitorReporting] Options for reporting mechanism from the client to the DSE server, for
  * versions that support it.
  * [TODO: Add support for this field]
@@ -228,17 +224,14 @@ const errors = require("./errors.js");
  *
  * [TODO: Add support for this field]
  * @property {Boolean} [encoding.useUndefinedAsUnset] Valid for Cassandra 2.2 and above. Determines that, if a parameter
- * is set to
- * `undefined` it should be encoded as `unset`.
+ * is set to `undefined` it should be encoded as `unset`.
  *
  * By default, ECMAScript `undefined` is encoded as `null` in the driver. Cassandra 2.2
  * introduced the concept of unset.
  * At driver level, you can set a parameter to unset using the field `types.unset`. Setting this flag to
  * true allows you to use ECMAScript undefined as Cassandra `unset`.
  *
  * Default: true.
- *
- * [TODO: Add support for this field]
  * @property {Boolean} [encoding.useBigIntAsLong] Use [BigInt ECMAScript type](https://tc39.github.io/proposal-bigint/)
  * to represent CQL bigint and counter data types.
  * [TODO: Add support for this field]

lib/client.js

@@ -109,7 +109,7 @@ class Client extends events.EventEmitter {
      * @type {string | undefined}
      */
     get keyspace() {
-        return this.rust_client.getKeyspace();
+        return this.rustClient.getKeyspace();
     }
 
     set keyspace(_) {

test/integration/supported/client-each-row-tests.js

@@ -846,7 +846,7 @@ describe("Client", function () {
             );
         }); */
         // TODO: Fix this test
-        // No support for warnings in ResultSet
+        // Rust driver throws an error, instead of returning a warning
         /* vit(
             "2.2",
             "should include the warning in the ResultSet",

test/integration/supported/client-execute-prepared-tests.js

@@ -1054,8 +1054,8 @@ describe("Client @SERVER_API", function () {
             );
         });
 
-        // No support for warnings in result set
         // TODO: fix this test
+        // Rust driver throws an error, instead of returning a warning
         /* vit(
             "2.2",
             "should include the warning in the ResultSet",

test/integration/supported/client-execute-tests.js

@@ -60,6 +60,15 @@ describe("Client @SERVER_API", function () {
             });
         });
 
+        it("should retrieve keyspace name", function (done) {
+            const client = setupInfo.client;
+            client.connect(function (err) {
+                assert.ifError(err);
+                assert.equal(client.keyspace, setupInfo.keyspace);
+                done();
+            });
+        });
+
         // Would require error throwing refactor
         // TODO: Fix this test
         context("with incorrect query parameters", () => {
@@ -1079,9 +1088,7 @@ describe("Client @SERVER_API", function () {
             );
         });
 
-        // No support for info field
-        // TODO: Fix this test
-        /* vit(
+        vit(
             "2.2",
             "should include the warning in the ResultSet",
             function (done) {
@@ -1115,14 +1122,14 @@ describe("Client @SERVER_API", function () {
                     assert.strictEqual(result.info.warnings.length, 1);
                     helper.assertContains(result.info.warnings[0], "batch");
                     helper.assertContains(result.info.warnings[0], "exceeding");
-                    assert.ok(loggedMessage);
+                    // TODO: Fix this test
+                    // Client does not expose log events.
+                    /* assert.ok(loggedMessage); */
                     done();
                 });
             },
-        ); */
+        );
 
-        // No support for buffer as input to any data type
-        // TODO: Fix this test
         it("should support buffer as input for any data type", () => {
             const buffer4 = utils.allocBufferFromArray([0, 0, 0, 1]);
             const buffer8 = utils.allocBuffer(8);