Fix: tests were not properly implemented

Author: Stefan-Gabriel Muscalu

test/internal/tls.test.js

@@ -90,25 +90,28 @@ describe('trust-on-first-use', function() {
       fs.unlinkSync(knownHostsPath);
     }
 
-    fs.writeFileSync(
-      knownHostsPath,
-      'localhost:7687 abcd\n' +
-      'localhost:7687 abcd'
-    );
-
     driver = neo4j.driver("bolt://localhost", neo4j.auth.basic("neo4j", "neo4j"), {
       encrypted: true,
       trust: "TRUST_ON_FIRST_USE",
       knownHosts: knownHostsPath
     });
 
+    driver.session(); // write into the knownHost file
+
+    // duplicate the same serverId twice
+    setTimeout(function() {
+      var text = fs.readFileSync(knownHostsPath, 'utf8');
+      fs.writeFileSync(knownHostsPath, text + text);
+    }, 1000);
+
     // When
-    driver.session().run("RETURN true AS a").then( function(data) {
-      // Then we get to here.
-      // And then the known_hosts file should have correct contents
-      expect( data.records[0].get('a') ).toBe( true );
-      done();
-    });
+    setTimeout(function() {
+      driver.session().run("RETURN true AS a").then( function(data) {
+        // Then we get to here.
+        expect( data.records[0].get('a') ).toBe( true );
+        done();
+      });
+    }, 2000);
   });
 
   it('should accept previously un-seen hosts', function(done) {
@@ -164,7 +167,7 @@ describe('trust-on-first-use', function() {
     driver.session();
 
     setTimeout(function() {
-      expect( fs.readFileSync(knownHostsPath, 'utf8').split('\n').length ).toBe( 1 );
+      expect( fs.readFileSync(knownHostsPath, 'utf8').trim().split('\n').length ).toBe( 1 );
       done();
     }, 1000);
   });