Remove waitForSelector for some dashboards page

USER_GUIDE.md

@@ -45,6 +45,8 @@ You can also find this information using help command.
 opensearch-reporting-cli --help
 ```
 
+NOTE: The tenant in the url has the higher priority than tenant value provided as command option. For example, if the command is `opensearch-reporting-cli -u http://localhost:5601/goto/069af6d6f3294421ec163b07fef91e5d?security_tenant=private -t global` then tenant value *private* will be used for generating report because url contains *security_tenant=private*.
+
 ## Environment Variable File
 
 Reporting CLI also reads environment variables from .env file in the current directory.

src/arguments.js

@@ -60,6 +60,9 @@ export async function getCommandArguments() {
             .default(DEFAULT_EMAIL_SUBJECT)
             .env(ENV_VAR.EMAIL_SUBJECT))
 
+    program.addHelpText('after', `
+Note: The tenant in the url has the higher priority than tenant value provided as command option.`);
+
     program.parse(process.argv);
     const options = program.opts();
     spinner.start('Fetching the arguments values');

src/constants.js

@@ -41,9 +41,9 @@ export const AUTH = {
 }
 
 export const URL_SOURCE = {
-  DASHBOARDS: 'dashboards#',
-  VISUALIZE: "Visualize",
-  DISCOVER: "discover#",
+  DASHBOARDS: "/app/dashboards#",
+  VISUALIZE: "/app/visualize#",
+  DISCOVER: "/app/discover#",
   NOTEBOOKS: "notebooks",
 }
 

src/download-helpers.js

@@ -94,32 +94,6 @@ export async function downloadReport(url, format, width, height, filename, authT
 
     // force wait for any resize to load after the above DOM modification.
     await new Promise(resolve => setTimeout(resolve, 1000));
-
-    switch (reportSource) {
-      case REPORT_TYPE.DASHBOARD:
-        await page.waitForSelector(SELECTOR.DASHBOARD, {
-          visible: true,
-        });
-        break;
-      case REPORT_TYPE.VISUALIZATION:
-        await page.waitForSelector(SELECTOR.VISUALIZATION, {
-          visible: true,
-        });
-        break;
-      case REPORT_TYPE.NOTEBOOK:
-        await page.waitForSelector(SELECTOR.NOTEBOOK, {
-          visible: true,
-        });
-        break;
-      case REPORT_TYPE.DISCOVER:
-        await page.waitForSelector(SELECTOR.DISCOVER, {
-          visible: true,
-        });
-        break;
-      default:
-        break;
-    }
-
     await waitForDynamicContent(page);
     let buffer;
     spinner.text = `Downloading Report...`;

test/help.test.js

@@ -32,6 +32,8 @@ Options:
   --smtppassword <password>              smtp password (env: OPENSEARCH_SMTP_PASSWORD)
   --subject <subject>                    email Subject (default: "This is an email containing your opensearch dashboard report", env: OPENSEARCH_EMAIL_SUBJECT)
   -h, --help                             display help for command
+
+Note: The tenant in the url has the higher priority than tenant value provided as command option.
 `;
         let result = await cli(['-h'], '.');
         expect(result.code).toBe(0);