21704: clarified comments

[arvados.git] / services / workbench2 / cypress / e2e / banner-tooltip.cy.js

diff --git a/services/workbench2/cypress/e2e/banner-tooltip.cy.js b/services/workbench2/cypress/e2e/banner-tooltip.cy.js
index 35d8bde374ca7d6687120edc47e12a4e3c90c143..7cb2bc405e77889144e42841fa39a40c079776b9 100644 (file)
--- a/services/workbench2/cypress/e2e/banner-tooltip.cy.js
+++ b/services/workbench2/cypress/e2e/banner-tooltip.cy.js
@@ -61,7 +61,11 @@ describe('Banner / tooltip tests', function () {
 
         //login here instead of in specific tests to preserve localStorage and intercept listener
         cy.loginAs(adminUser);
+
+        //must be in localstorage to have banner option in notifications menu
+        //it doesn't matter what the value is, as long as it's truthy
         window.localStorage.setItem('bannerFileData', 'foo');
+
         cy.intercept({ method: 'GET', url: '**/arvados/v1/config?nocache=*' }, (req) => {
             req.on('response', (res) => {
                 res.body.Workbench.BannerUUID = collectionUUID;
@@ -75,7 +79,7 @@ describe('Banner / tooltip tests', function () {
     });
 
     it('should re-show the banner', () => {
-        //reload instead of cy.loginas() to preserve localStorage and intercept listener
+        //reload instead of cy.loginAs() to preserve localStorage and intercept listener
         //logged in as adminUser
         cy.reload();
         cy.waitForDom();