Tiny simplification of connection initialization

Author: lutovich

src/v1/internal/connector.js

@@ -477,10 +477,12 @@ class Connection {
    * @protected
    */
   _markInitialized(metadata) {
-    const serverVersion = metadata.server;
+    const serverVersion = metadata ? metadata.server : null;
     if (!this.server.version) {
       this.server.version = serverVersion;
-      if (ServerVersion.fromString(serverVersion).compareTo(VERSION_3_2_0) < 0) {
+
+      const version = ServerVersion.fromString(serverVersion);
+      if (version.compareTo(VERSION_3_2_0) < 0) {
         this._packer.disableByteArrays();
       }
     }
@@ -533,9 +535,7 @@ class ConnectionState {
         }
       },
       onCompleted: metaData => {
-        if (metaData && metaData.server) {
-          this._connection._markInitialized(metaData);
-        }
+        this._connection._markInitialized(metaData);
         this._initialized = true;
         if (this._resolvePromise) {
           this._resolvePromise(this._connection);

test/v1/types.test.js

@@ -143,7 +143,7 @@ describe('byte arrays', () => {
   let originalTimeout = jasmine.DEFAULT_TIMEOUT_INTERVAL;
   let serverSupportsByteArrays = false;
 
-  beforeEach(done => {
+  beforeAll(done => {
     jasmine.DEFAULT_TIMEOUT_INTERVAL = 60000;
 
     const driver = neo4j.driver('bolt://localhost', sharedNeo4j.authToken);
@@ -156,7 +156,7 @@ describe('byte arrays', () => {
     });
   });
 
-  afterEach(() => {
+  afterAll(() => {
     jasmine.DEFAULT_TIMEOUT_INTERVAL = originalTimeout;
   });