Always honor login timeout when setting cookie expiry

Author: MarcialRosales

deps/rabbitmq_management/priv/www/js/tmpl/overview.ejs

@@ -32,7 +32,7 @@
 <% }}} %>
 </div>
 
-<div class="section">
+<div class="section" id="totals-section">
 <h2>Totals</h2>
 <div  class="hider updatable">
 <% if(!disable_stats) { %>

selenium/test/basic-auth/session-expired-with-no-refresh.js

@@ -0,0 +1,39 @@
+const { By, Key, until, Builder } = require('selenium-webdriver')
+require('chromedriver')
+const assert = require('assert')
+const { buildDriver, goToHome, captureScreensFor, teardown, delay } = require('../utils')
+
+const LoginPage = require('../pageobjects/LoginPage')
+const OverviewPage = require('../pageobjects/OverviewPage')
+
+describe('Once user is logged in', function () {
+  let driver
+  let overview
+  let captureScreen
+  this.timeout(65000) // hard-coded to 25secs because this test requires 35sec to run
+
+  before(async function () {
+    driver = buildDriver()
+    await goToHome(driver)
+    login = new LoginPage(driver)
+    overview = new OverviewPage(driver)
+    captureScreen = captureScreensFor(driver, __filename)    
+    await login.login('guest', 'guest')
+    await overview.isLoaded()    
+    await overview.selectRefreshOption("Do not refresh")
+    await overview.ensureTotalsSectionIsInvisible() // trigger a preference stored in cookie
+                                                    // which should honor login timeout     
+  })
+
+  it('it has to login after the session expires', async function () {
+    
+    await delay(60000)
+    await overview.clickOnConnectionsTab()
+    await login.isLoaded()   
+    
+  })
+
+  after(async function () {
+    await teardown(driver, this, captureScreen)
+  })
+})

selenium/test/basic-auth/session-expired.js

@@ -19,16 +19,18 @@ describe('Once user is logged in', function () {
     overview = new OverviewPage(driver)
     captureScreen = captureScreensFor(driver, __filename)
     await login.login('guest', 'guest')
-    await overview.isLoaded()
-
+    await overview.isLoaded()    
+    await overview.ensureTotalsSectionIsInvisible() // trigger a preference stored in cookie
+                                                    // which should honor login timeout 
+    
   })
 
   it('it has to login after the session expires', async function () {
 
     await delay(60000)
     await login.isLoaded() 
     await login.login('guest', 'guest')
-    await overview.isLoaded() 
+    await overview.isLoaded()
     await overview.clickOnConnectionsTab() // and we can still interact with the ui
   })
 

selenium/test/pageobjects/BasePage.js

@@ -33,14 +33,24 @@ module.exports = class BasePage {
     this.polling = parseInt(process.env.SELENIUM_POLLING) || 500 // how frequent selenium searches for an element
     this.interactionDelay = parseInt(process.env.SELENIUM_INTERACTION_DELAY) || 0 // slow down interactions (when rabbit is behind a http proxy)
   }
+ 
   async ensureSectionIsVisible(section) {
     let classes = await this.driver.findElement(section).getAttribute("class")
     if (classes.search('section-visible') < 0) {
-      return this.click(section)
+      return this.click(By.css(section.value + ' h2'))
     } else {
       return Promise.resolve(true)
     }
   }
+  async ensureSectionIsInvisible(section) {
+    let classes = await this.driver.findElement(section).getAttribute("class")
+    if (classes.search('section-invisible') < 0) {
+      return this.click(By.css(section.value + ' h2'))
+    } else {
+      
+      return Promise.resolve(true)
+    }
+  }
   async goTo(path) {
     return driver.get(d.baseUrl + path)  
   }

selenium/test/pageobjects/OverviewPage.js

@@ -12,6 +12,8 @@ const DOWNLOAD_DEFINITIONS_SECTION = By.css('div#download-definitions-section')
 const CHOOSE_BROKER_DOWNLOAD_FILE = By.css('input#download-filename')
 const DOWNLOAD_BROKER_FILE = By.css('button#upload-definitions')
 
+const TOTALS_SECTION = By.css('div#main div#totals-section')
+
 module.exports = class OverviewPage extends BasePage {
 
   async uploadBrokerDefinitions(file) {
@@ -27,4 +29,12 @@ module.exports = class OverviewPage extends BasePage {
   async downloadBrokerDefinitions(filename) {
     return this.click(DOWNLOAD_DEFINITIONS_SECTION)
   }
+
+  async ensureTotalsSectionIsVisible() {
+    return this.ensureSectionIsVisible(TOTALS_SECTION)
+  }
+  async ensureTotalsSectionIsInvisible() {
+    return this.ensureSectionIsInvisible(TOTALS_SECTION)
+  }
+  
 }