Use 9091+ ports for tests and remove debug output

Turned out that port 9090 is occupied on TeamCity build agents and thus can't
be used for boltkit tests. This commit makes tests use ports starting from
9091 and removes usages of port 13001. It also removes debug printouts used
for test failures investigation.

Author: lutovich

test/resources/boltkit/routing_table_with_zero_ttl.script

@@ -5,7 +5,7 @@
 C: RUN "CALL dbms.cluster.routing.getServers" {}
    PULL_ALL
 S: SUCCESS {"fields": ["ttl", "servers"]}
-   RECORD [0, [{"addresses": ["127.0.0.1:13001","127.0.0.1:9091","127.0.0.1:9092","127.0.0.1:9999"],"role": "ROUTE"}, {"addresses": ["127.0.0.1:9999"], "role": "READ"},{"addresses": ["127.0.0.1:9999"], "role": "WRITE"}]]
+   RECORD [0, [{"addresses": ["127.0.0.1:9091","127.0.0.1:9092","127.0.0.1:9093","127.0.0.1:9999"],"role": "ROUTE"}, {"addresses": ["127.0.0.1:9999"], "role": "READ"},{"addresses": ["127.0.0.1:9999"], "role": "WRITE"}]]
    SUCCESS {}
 C: RUN "MATCH (n) RETURN n" {}
    PULL_ALL

test/v1/boltkit.js

@@ -23,7 +23,7 @@ var fs = require('fs');
 var tmp = require('tmp');
 
 var BoltKit = function (verbose) {
-  this.verbose = verbose || true;
+  this.verbose = verbose || false;
 };
 
 BoltKit.prototype.startWithTemplate = function (scriptTemplate, parameters, port) {
@@ -37,17 +37,16 @@ BoltKit.prototype.startWithTemplate = function (scriptTemplate, parameters, port
 BoltKit.prototype.start = function(script, port) {
   var spawn = childProcess.spawn, server, code = -1;
 
-  console.log('+++ Starting boltstub on port ' + port);
   server = spawn('/usr/local/bin/boltstub', ['-v', port, script]);
   if (this.verbose) {
     server.stdout.on('data', (data) => {
-      console.log(`+++ Boltstup on port ${port}:\n${data}`);
+      console.log(`${data}`);
     });
     server.stderr.on('data', (data) => {
-      console.log(`+++ Boltstup on port ${port}:\n${data}`);
+      console.log(`${data}`);
     });
     server.on('end', function (data) {
-      console.log(`+++ Boltstup on port ${port}:\n${data}`);
+      console.log(data);
     });
   }
 
@@ -56,7 +55,7 @@ BoltKit.prototype.start = function(script, port) {
   });
 
   server.on('error', function (err) {
-    console.log('Failed to start child process on port ' + port + ' error: ' + err);
+    console.log('Failed to start child process:' + err);
   });
 
   var Server = function(){};

test/v1/routing.driver.boltkit.it.js

@@ -21,18 +21,12 @@ import neo4j from "../../src/v1";
 import boltkit from "./boltkit";
 import RoutingTable from "../../src/v1/internal/routing-table";
 
-jasmine.getEnv().addReporter({
-  specStarted: result => {
-    console.log('+++ Running test: ' + result.fullName);
-  }
-});
-
 describe('routing driver', () => {
   let originalTimeout;
 
   beforeAll(() => {
     originalTimeout = jasmine.DEFAULT_TIMEOUT_INTERVAL;
-    jasmine.DEFAULT_TIMEOUT_INTERVAL = 60000;
+    jasmine.DEFAULT_TIMEOUT_INTERVAL = 20000;
   });
 
   afterAll(() => {
@@ -660,9 +654,9 @@ describe('routing driver', () => {
 
     const kit = new boltkit.BoltKit();
     const server1 = kit.start('./test/resources/boltkit/routing_table_with_zero_ttl.script', 9999);
-    const server2 = kit.start('./test/resources/boltkit/no_writers.script', 13001);
-    const server3 = kit.start('./test/resources/boltkit/no_writers.script', 9091);
-    const server4 = kit.start('./test/resources/boltkit/no_writers.script', 9092);
+    const server2 = kit.start('./test/resources/boltkit/no_writers.script', 9091);
+    const server3 = kit.start('./test/resources/boltkit/no_writers.script', 9092);
+    const server4 = kit.start('./test/resources/boltkit/no_writers.script', 9093);
 
     kit.run(() => {
       const driver = newDriver('bolt+routing://127.0.0.1:9999');
@@ -672,7 +666,7 @@ describe('routing driver', () => {
         expect(result1.summary.server.address).toEqual('127.0.0.1:9999');
         session1.close();
 
-        assertHasRouters(driver, ['127.0.0.1:13001', '127.0.0.1:9091', '127.0.0.1:9092', '127.0.0.1:9999']);
+        assertHasRouters(driver, ['127.0.0.1:9091', '127.0.0.1:9092', '127.0.0.1:9093', '127.0.0.1:9999']);
         const memorizingRoutingTable = setUpMemorizingRoutingTable(driver);
 
         const session2 = driver.session();
@@ -860,15 +854,15 @@ describe('routing driver', () => {
         expect(result1.summary.server.address).toEqual('127.0.0.1:9999');
         session1.close();
 
-        assertHasRouters(driver, ['127.0.0.1:13001', '127.0.0.1:9091', '127.0.0.1:9092', '127.0.0.1:9999']);
+        assertHasRouters(driver, ['127.0.0.1:9091', '127.0.0.1:9092', '127.0.0.1:9093', '127.0.0.1:9999']);
         const memorizingRoutingTable = setUpMemorizingRoutingTable(driver);
 
         const session2 = driver.session();
         session2.run('MATCH (n) RETURN n').then(result2 => {
           expect(result2.summary.server.address).toEqual('127.0.0.1:9999');
           session2.close();
 
-          memorizingRoutingTable.assertForgotRouters(['127.0.0.1:13001', '127.0.0.1:9091', '127.0.0.1:9092']);
+          memorizingRoutingTable.assertForgotRouters(['127.0.0.1:9091', '127.0.0.1:9092', '127.0.0.1:9093']);
           assertHasRouters(driver, ['127.0.0.1:9999']);
           driver.close();
 
@@ -953,8 +947,8 @@ describe('routing driver', () => {
   it('should accept routing table with 1 router, 1 reader and 1 writer', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001'],
-        readers: ['127.0.0.1:9091'],
+        routers: ['127.0.0.1:9091'],
+        readers: ['127.0.0.1:9092'],
         writers: ['127.0.0.1:9999']
       },
       9999, done);
@@ -963,8 +957,8 @@ describe('routing driver', () => {
   it('should accept routing table with 2 routers, 1 reader and 1 writer', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001', '127.0.0.1:9091'],
-        readers: ['127.0.0.1:9091'],
+        routers: ['127.0.0.1:9091', '127.0.0.1:9092'],
+        readers: ['127.0.0.1:9092'],
         writers: ['127.0.0.1:9999']
       },
       9999, done);
@@ -973,8 +967,8 @@ describe('routing driver', () => {
   it('should accept routing table with 1 router, 2 readers and 1 writer', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001'],
-        readers: ['127.0.0.1:9091', '127.0.0.1:9092'],
+        routers: ['127.0.0.1:9091'],
+        readers: ['127.0.0.1:9092', '127.0.0.1:9093'],
         writers: ['127.0.0.1:9999']
       },
       9999, done);
@@ -983,8 +977,8 @@ describe('routing driver', () => {
   it('should accept routing table with 2 routers, 2 readers and 1 writer', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001', '127.0.0.1:9091'],
-        readers: ['127.0.0.1:9092', '127.0.0.1:9093'],
+        routers: ['127.0.0.1:9091', '127.0.0.1:9092'],
+        readers: ['127.0.0.1:9093', '127.0.0.1:9094'],
         writers: ['127.0.0.1:9999']
       },
       9999, done);
@@ -993,39 +987,39 @@ describe('routing driver', () => {
   it('should accept routing table with 1 router, 1 reader and 2 writers', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001'],
-        readers: ['127.0.0.1:9091'],
-        writers: ['127.0.0.1:9999', '127.0.0.1:9092']
+        routers: ['127.0.0.1:9091'],
+        readers: ['127.0.0.1:9092'],
+        writers: ['127.0.0.1:9999', '127.0.0.1:9093']
       },
       9999, done);
   });
 
   it('should accept routing table with 2 routers, 1 reader and 2 writers', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001', '127.0.0.1:9091'],
-        readers: ['127.0.0.1:9092'],
-        writers: ['127.0.0.1:9999', '127.0.0.1:9093']
+        routers: ['127.0.0.1:9091', '127.0.0.1:9092'],
+        readers: ['127.0.0.1:9093'],
+        writers: ['127.0.0.1:9999', '127.0.0.1:9094']
       },
       9999, done);
   });
 
   it('should accept routing table with 1 router, 2 readers and 2 writers', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001'],
-        readers: ['127.0.0.1:9091', '127.0.0.1:9092'],
-        writers: ['127.0.0.1:9999', '127.0.0.1:9093']
+        routers: ['127.0.0.1:9091'],
+        readers: ['127.0.0.1:9092', '127.0.0.1:9093'],
+        writers: ['127.0.0.1:9999', '127.0.0.1:9094']
       },
       9999, done);
   });
 
   it('should accept routing table with 2 routers, 2 readers and 2 writers', done => {
     testRoutingTableAcceptance(
       {
-        routers: ['127.0.0.1:13001', '127.0.0.1:9091'],
-        readers: ['127.0.0.1:9092', '127.0.0.1:9093'],
-        writers: ['127.0.0.1:9999', '127.0.0.1:9094']
+        routers: ['127.0.0.1:9091', '127.0.0.1:9092'],
+        readers: ['127.0.0.1:9093', '127.0.0.1:9094'],
+        writers: ['127.0.0.1:9999', '127.0.0.1:9095']
       },
       9999, done);
   });