added comments and fixed docker ksqldb url

Author: javanang

.gitignore

@@ -107,6 +107,7 @@ dist
 ksqljsTest.js
 package-lock.json
 local_ignore/
+.gitignore
 
 # KSQLDB docker server settings
 ksqldb_server_config/

__tests__/integrationtests.js

@@ -1,6 +1,10 @@
 const ksqljs = require('../ksqljs/ksqlJS.js');
 
 // Pre-requisite: start a docker container
+/* To add to README: Prior to running test with 'npm test', please start the ksqlDB
+server using the command 'docker compose-up'. This will spin up a ksqlDB server on
+'http://localhost:8088'
+*/
 
 describe('--Integration Tests--', () => {
 

docker-compose.yml

@@ -43,7 +43,7 @@ services:
     ports:
       - "8088:8088"
     environment:
-      KSQL_LISTENERS: https://0.0.0.0:8088
+      KSQL_LISTENERS: http://0.0.0.0:8088
       KSQL_BOOTSTRAP_SERVERS: broker:9092
       KSQL_KSQL_LOGGING_PROCESSING_STREAM_AUTO_CREATE: "true"
       KSQL_KSQL_LOGGING_PROCESSING_TOPIC_AUTO_CREATE: "true"

ksqljs/ksqlJS.js

@@ -78,7 +78,7 @@ class ksqljs {
         this.httpsAgentHttp2 ? this.httpsAgentHttp2 : {}
       );
 
-      session.on("error", (err) => console.error(err));
+      session.on("error", (err) => reject(err));
 
       const req = session.request(
         this.secret && this.API ?