Feature/mpt 18301 add cloud account alert tests (#364)

* Add cloud-accounts page mocks and update e2e tests

Add a comprehensive cloud-accounts-page.mocks.ts providing GraphQL mock responses (organizations, allowed actions, current employee, data sources, organization features/theme/perspectives, pools, marketplace data, etc.) and update e2e test pages and specs to use the new mocks. Adjusted base-page, cloud-accounts-page/connect pages, recommendations-page, and cloud-accounts tests, plus updated expected screenshots/PDF fixtures to match the mocked output. These changes stabilize and align end-to-end tests with deterministic mock data.

* Remove commented privacy-policy steps

Delete commented-out test.step blocks that accepted the privacy policy in e2etests/tests/invitation-flow-tests.spec.ts. Cleans up leftover/dead code in the invitation flow tests by removing two redundant commented sections.

* Add billing reimport UI and test

Add billing reimport locators and actions to CloudAccountsPage (billingReimportBtn, billingReimportSideModal, billingReimportAlert, scheduleImportBtn, importDatePickerInput, datePopup, dateSetBtn) and implement clickBillingReimportBtn and scheduleImportWithDefaultDate (opens date picker, confirms default date, clicks schedule and returns selected date). Add E2E test that opens the reimport side modal, verifies the warning alert message, schedules a billing reimport using the default date, and asserts the UpdateDataSource POST response is 200. Includes debug logs for the selected date and API response status.

* Update expected-data-expenses-chart-export.png

Author: SteveChurchillSWO

e2etests/mocks/cloud-accounts-page.mocks.ts

@@ -398,7 +398,7 @@ export abstract class BasePage {
    * @param {number} [timeout=10000] - The maximum time to wait for the loading image to disappear, in milliseconds.
    * @returns {Promise<void>} A promise that resolves when the loading image is no longer visible or exits early if the image is not present.
    */
-  async waitForLoadingPageImgToDisappear(timeout: number = 10000): Promise<void> {
+  async waitForLoadingPageImgToDisappear(timeout: number = 20000): Promise<void> {
     try {
       await this.loadingPageImg.first().waitFor({ timeout: 1000 });
     } catch (_error) {
@@ -422,7 +422,7 @@ export abstract class BasePage {
    * @param {number} [timeout=10000] - The maximum time to wait for the initialisation message to disappear, in milliseconds.
    * @returns {Promise<void>} A promise that resolves when the initialisation message is no longer visible or exits early if the message is not present.
    */
-  async waitForInitialisationToComplete(timeout: number = 10000): Promise<void> {
+  async waitForInitialisationToComplete(timeout: number = 20000): Promise<void> {
     try {
       await this.initialisationMessage.first().waitFor({ timeout: 1000 });
     } catch (_error) {

e2etests/pages/base-page.ts

@@ -37,6 +37,7 @@ export class CloudAccountsConnectPage extends BasePage {
   readonly exportNameInput: Locator;
   readonly exportAmazonS3BucketNameInput: Locator;
   readonly exportPathPrefixInput: Locator;
+  readonly alertMessage: Locator;
   //#endregion
 
   //#region Azure specific
@@ -54,6 +55,7 @@ export class CloudAccountsConnectPage extends BasePage {
 
   //#endregion
 
+
   /**
    * Initializes a new instance of the CloudAccountsConnectPage class.
    * @param {Page} page - The Playwright page object.
@@ -84,6 +86,7 @@ export class CloudAccountsConnectPage extends BasePage {
     this.exportNameInput = this.main.getByTestId('input_export_name');
     this.exportAmazonS3BucketNameInput = this.main.getByTestId('input_s3_bucket_name');
     this.exportPathPrefixInput = this.main.getByTestId('input_export_path_prefix');
+    this.alertMessage  = this.main.getByRole("alert");
     this.connectBtn = this.main.getByTestId('btn_connect_cloud_account');
     this.cancelBtn = this.main.getByTestId('btn_cancel_cloud_account');
     this.directoryTenantIDInput = this.main.getByTestId('input_tenant_id');
@@ -158,6 +161,10 @@ export class CloudAccountsConnectPage extends BasePage {
     await this.googleCloudBtn.click();
   }
 
+  async clickAccessKey(): Promise<void> {
+    await this.btnAccessKey.click();
+  }
+
   /**
    * Adds an AWS Assumed Role account by filling in the required details and connecting it.
    *

e2etests/pages/cloud-accounts-connect-page.ts

@@ -18,6 +18,22 @@ export class CloudAccountsPage extends BasePage {
   readonly table: Locator;
   readonly allCloudAccountLinks: Locator;
 
+  readonly updateSideModal: Locator;
+  readonly sideModalAccessKeyButton: Locator;
+  readonly sideModalAssumedRoleButton: Locator;
+  readonly sideModalPrimaryAlert: Locator;
+  readonly sideModalSecondaryAlert: Locator;
+  readonly updateCredentialsBtn: Locator;
+
+  readonly billingReimportBtn: Locator;
+  readonly billingReimportSideModal: Locator;
+  readonly billingReimportAlert: Locator;
+  readonly scheduleImportBtn: Locator;
+  readonly importDatePickerInput: Locator;
+  private datePopup: Locator;
+  readonly dateSetBtn: Locator;
+
+
   /**
    * Initializes a new instance of the CloudAccountsPage class.
    * @param {Page} page - The Playwright page object.
@@ -34,6 +50,20 @@ export class CloudAccountsPage extends BasePage {
     this.billingStatusCompletedIcon = this.getByAnyTestId('CheckCircleIcon', this.lastBillingImportStatus);
     this.addBtn = this.main.getByTestId('btn_add');
     this.allCloudAccountLinks = this.table.locator('xpath=(//td//a)');
+    this.updateSideModal = this.page.getByTestId('smodal_update_data_source_credentials');
+    this.sideModalAccessKeyButton = this.updateSideModal.getByTestId('btn_accessKey');
+    this.sideModalAssumedRoleButton = this.updateSideModal.getByTestId('btn_assumedRole');
+    this.sideModalPrimaryAlert = this.updateSideModal.locator('//div[@role="alert"]').first();
+    this.sideModalSecondaryAlert = this.updateSideModal.locator('//div[@role="alert"]').last();
+    this.updateCredentialsBtn = this.main.getByTestId('btn_update_data_source_credentials_modal');
+    this.billingReimportBtn = this.main.getByTestId('btn_expenses_reimport_data_source_modal');
+    this.billingReimportSideModal = this.page.getByTestId('smodal_reimport_data_source_expenses');
+    this.billingReimportAlert = this.billingReimportSideModal.locator('//div[@role="alert"]');
+    this.scheduleImportBtn = this.billingReimportSideModal.getByTestId('btn_confirm');
+    this.importDatePickerInput = this.billingReimportSideModal.getByTestId('input_importFrom');
+    this.datePopup = this.page.locator('//div[@id="simple-popover"]');
+    this.dateSetBtn = this.datePopup.getByRole('button', { name: 'Set' });
+
   }
 
   /**
@@ -44,6 +74,45 @@ export class CloudAccountsPage extends BasePage {
     await this.addBtn.click();
   }
 
+  /**
+   * Clicks the "Update Credentials" button to open the update credentials modal.
+   * This method is typically used to initiate the process of updating
+   * cloud account credentials (e.g., AWS access keys or assumed roles).
+   * @returns {Promise<void>} A promise that resolves when the button is clicked.
+   */
+  async clickUpdateCredentialsBtn(): Promise<void> {
+    await this.updateCredentialsBtn.click();
+  }
+
+  /**
+   * Clicks the "Billing Reimport" button to open the billing reimport modal.
+   * This method is used to initiate the process of re-importing billing data
+   * for a cloud account from a specified date.
+   * @returns {Promise<void>} A promise that resolves when the button is clicked.
+   */
+  async clickBillingReimportBtn(): Promise<void> {
+    await this.billingReimportBtn.click();
+  }
+
+  /**
+   * Schedules a billing data re-import using the default date in the date picker.
+   * This method performs the following steps:
+   * 1. Opens the date picker by clicking the import date input field
+   * 2. Confirms the default date by clicking the "Set" button
+   * 3. Initiates the import by clicking the "Schedule Import" button
+   * 4. Logs the scheduled date for debugging purposes
+   *
+   * @returns {Promise<string>} A promise that resolves with the text content of the selected date.
+   * @throws {Error} If the date picker input or buttons are not accessible.
+   */
+  async scheduleImportWithDefaultDate(): Promise<string> {
+    await this.importDatePickerInput.click();
+    await this.dateSetBtn.click();
+    await this.scheduleImportBtn.click();
+    debugLog(`Scheduled billing re-import with default date: ${await this.importDatePickerInput.textContent()}`);
+    return await this.importDatePickerInput.textContent();
+  }
+
   /**
    * Clicks the "Advanced" tab button on the Cloud Accounts page.
    * @returns {Promise<void>} A promise that resolves when the action is complete.
@@ -63,7 +132,6 @@ export class CloudAccountsPage extends BasePage {
     return locator.click();
   }
 
-
   /**
      * Retrieves a cloud account link by its name.
      *

e2etests/pages/cloud-accounts-page.ts

@@ -523,7 +523,7 @@ export class RecommendationsPage extends BasePage {
       { label: 'Obsolete Snapshot Chains', locator: this.obsoleteSnapshotChainsCardSavingsValue },
       { label: 'Reserved Instances Opportunities', locator: this.reservedInstancesOpportunitiesCardSavingsValue },
       { label: 'Public S3 Buckets', locator: this.publicS3BucketsCardSavingsValue },
-      { label: 'Snapshots With Non-used Images', locator: this.snapshotsWithNonUsedImagesCardSavingsValue},
+      { label: 'Snapshots With Non-used Images', locator: this.snapshotsWithNonUsedImagesCardSavingsValue },
       { label: 'Underutilized Instances', locator: this.underutilizedInstancesCardSavingsValue },
       { label: 'Underutilized RDS Instances', locator: this.underutilizedRDSInstancesCardSavingsValue },
     ];
@@ -671,6 +671,16 @@ export class RecommendationsPage extends BasePage {
     return totalSum;
   }
 
+  /**
+   * Retrieves the numeric count value from a card element.
+   *
+   * This method extracts the text content from the provided card locator
+   * and parses it as an integer. It is typically used to get count values
+   * displayed on recommendation cards (e.g., number of resources affected).
+   *
+   * @param {Locator} cardLocator - The locator for the card element containing the count value.
+   * @returns {Promise<number>} The parsed count value as an integer.
+   */
   async getCardCountValue(cardLocator: Locator): Promise<number> {
     const text = await cardLocator.textContent();
     return parseInt(text, 10);