Merge pull request #13868 from rabbitmq/fix-selenium-issue-starting-others

Fix selenium issues when running two rabbitmq

Author: michaelklishin

selenium/bin/components/rabbitmq

@@ -194,6 +194,6 @@ start_docker_rabbitmq() {
     -v ${TEST_DIR}:/config \
     ${RABBITMQ_DOCKER_IMAGE}
 
-  wait_for_message rabbitmq "Server startup complete"
+  wait_for_message rabbitmq "Server startup complete"  
   end "RabbitMQ ready"
 }

selenium/bin/suite_template

@@ -594,8 +594,8 @@ determine_required_components_including_rabbitmq() {
 }
 determine_required_components_excluding_rabbitmq() {
   for (( i=1; i<=$#; i++)) {
-    if [[ $i != "rabbitmq" ]]; then
-      eval val='$'$i
+    eval val='$'$i
+    if [[ "$val" != "rabbitmq" ]] && [[ "$val" != "other_rabbitmq" ]]; then
       REQUIRED_COMPONENTS+=( "$val" )
     fi
   }

selenium/fakeportal/proxy.js

@@ -1,6 +1,6 @@
 var http = require('http'),
     httpProxy = require('http-proxy');
-const {log, error} = require('./utils.js')    
+
 const XMLHttpRequest = require('xmlhttprequest').XMLHttpRequest
 
 const rabbitmq_url = process.env.RABBITMQ_URL || 'http://0.0.0.0:15672/';
@@ -35,6 +35,13 @@ log("fakeproxy listening on port " + port + ".  RABBITMQ_URL=" + rabbitmq_url)
 server.listen(port);
 
 
+function log(message) {
+  console.log(new Date() + " " + message)
+}
+function error(message) {
+  console.error(new Date() + " " + message)
+}
+
 function default_if_blank(value, defaultValue) {
   if (typeof value === "undefined" || value === null || value == "") {
     return defaultValue;

selenium/suites/mgt/mgt-only-exchanges.sh

@@ -3,7 +3,8 @@
 SCRIPT="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 
 TEST_CASES_PATH=/exchanges
-TEST_CONFIG_PATH=/mgt-only
+TEST_CONFIG_PATH=/basic-auth
+PROFILES="disable-metrics"
 
 source $SCRIPT/../../bin/suite_template $@
 run

selenium/test/basic-auth/env.disable-metrics

@@ -0,0 +1 @@
+export DISABLE_METRICS=true

selenium/test/exchanges/management.js

@@ -8,6 +8,8 @@ const OverviewPage = require('../pageobjects/OverviewPage')
 const ExchangesPage = require('../pageobjects/ExchangesPage')
 const ExchangePage = require('../pageobjects/ExchangePage')
 
+const DISABLE_METRICS = process.env.DISABLE_METRICS || false
+
 describe('Exchange management', function () {
   let login
   let exchanges
@@ -76,7 +78,6 @@ describe('Exchange management', function () {
     await exchanges.clickOnSelectTableColumns()
     let table = await exchanges.getSelectableTableColumns()
 
-    assert.equal(2, table.length)
     let overviewGroup = { 
         "name" : "Overview:",
         "columns": [
@@ -88,14 +89,18 @@ describe('Exchange management', function () {
     }    
     assert.equal(JSON.stringify(table[0]), JSON.stringify(overviewGroup))
 
-    let messageRatesGroup = { 
-      "name" : "Message rates:",
-      "columns": [
-        {"name:":"rate in","id":"checkbox-exchanges-rate-in"},
-        {"name:":"rate out","id":"checkbox-exchanges-rate-out"}
-      ]
+    if (!DISABLE_METRICS) {
+      assert.equal(table.length, 2)
+
+      let messageRatesGroup = { 
+        "name" : "Message rates:",
+        "columns": [
+          {"name:":"rate in","id":"checkbox-exchanges-rate-in"},
+          {"name:":"rate out","id":"checkbox-exchanges-rate-out"}
+        ]
+      }
+      assert.equal(JSON.stringify(table[1]), JSON.stringify(messageRatesGroup))
     }
-    assert.equal(JSON.stringify(table[1]), JSON.stringify(messageRatesGroup))
 
   })