From d79daa245b413cc41c7b371c91b784b415a53df6 Mon Sep 17 00:00:00 2001
From: Dimitrie Stefanescu <didimitrie@gmail.com>
Date: Thu, 10 Jun 2021 19:21:24 +0100
Subject: [PATCH] test(server): allows for port override in startHttp to
 potentially prevent EADDRINUSE errs in test

---
 packages/server/app.js                            | 5 ++++-
 packages/server/modules/stats/tests/stats.spec.js | 4 ++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/packages/server/app.js b/packages/server/app.js
index 205a881ab..defcd632b 100644
--- a/packages/server/app.js
+++ b/packages/server/app.js
@@ -117,9 +117,12 @@ exports.init = async ( ) => {
  * @param  {[type]} app [description]
  * @return {[type]}     [description]
  */
-exports.startHttp = async ( app ) => {
+exports.startHttp = async ( app, customPortOverride ) => {
   let bindAddress = process.env.BIND_ADDRESS || '127.0.0.1'
   let port = process.env.PORT || 3000
+
+  if ( customPortOverride ) port = customPortOverride
+
   app.set( 'port', port )
 
   let frontendHost = process.env.FRONTEND_HOST || 'localhost'
diff --git a/packages/server/modules/stats/tests/stats.spec.js b/packages/server/modules/stats/tests/stats.spec.js
index 8842672f4..89ccfc1f2 100644
--- a/packages/server/modules/stats/tests/stats.spec.js
+++ b/packages/server/modules/stats/tests/stats.spec.js
@@ -95,7 +95,7 @@ describe( 'Server stats services @stats-services', function() {
 
 } )
 
-let addr = `http://localhost:${process.env.PORT || 3000}`
+let addr = `http://localhost:3333`
 
 describe( 'Server stats api @stats-api', function() {
   let testServer
@@ -132,7 +132,7 @@ describe( 'Server stats api @stats-api', function() {
     await knex.migrate.latest( )
 
     let { app } = await init( )
-    let { server } = await startHttp( app )
+    let { server } = await startHttp( app, 3333 )
     testServer = server
 
     adminUser.id = await createUser( adminUser )