Fix part of oppia#21646: Added CUJ 4.1 - See statistics and previous explorations on the creator dashboard (oppia#22042)

* Fix part of oppia#21646: Added CUJ 4.1

* Resolve comments and fixed test-duration

* fix: removed flag from save exploration function

Author: omgupta-iitk

core/templates/pages/creator-dashboard-page/creator-dashboard-page.component.html

@@ -23,7 +23,7 @@ <h1 tabindex="0" [innerHTML]="'I18N_DASHBOARD_CREATOR_DASHBOARD' | translate"></
             [hidden]="dashboardStats.numRatings || relativeChangeInTotalPlays">
             {{ dashboardStats.averageRatings || 'N/A' }}
         </h2>
-        <div [hidden]="!dashboardStats.numRatings">
+        <div class="e2e-test-oppia-total-users" [hidden]="!dashboardStats.numRatings">
           <p *ngIf="dashboardStats.numRatings === 1">
             (by {{ dashboardStats.numRatings }} user)
           </p>

core/tests/ci-test-suite-configs/acceptance.json

@@ -48,6 +48,10 @@
       "name": "exploration-editor/save-draft-publish-and-discard-the-changes",
       "module": "core/tests/puppeteer-acceptance-tests/specs/exploration-editor/save-draft-publish-and-discard-the-changes.spec.ts"
     },
+    {
+      "name": "exploration-editor/verify-statistics-and-previous-explorations",
+      "module": "core/tests/puppeteer-acceptance-tests/specs/exploration-editor/verify-statistics-and-previous-explorations.spec.ts"
+    },
     {
       "name": "logged-in-user/create-and-delete-account",
       "module": "core/tests/puppeteer-acceptance-tests/specs/logged-in-user/create-and-delete-account.spec.ts"

core/tests/puppeteer-acceptance-tests/specs/exploration-editor/verify-statistics-and-previous-explorations.spec.ts

@@ -0,0 +1,96 @@
+// Copyright 2025 The Oppia Authors. All Rights Reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS-IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+/**
+ * @fileoverview Acceptance tests for viewing statistics and past explorations on the creator dashboard.
+ * User Journey: Create explorations, have another user play and rate one, then verify stats as the creator.
+ */
+
+import {UserFactory} from '../../utilities/common/user-factory';
+import testConstants from '../../utilities/common/test-constants';
+import {LoggedOutUser} from '../../utilities/user/logged-out-user';
+import {LoggedInUser} from '../../utilities/user/logged-in-user';
+import {ExplorationEditor} from '../../utilities/user/exploration-editor';
+
+const DEFAULT_SPEC_TIMEOUT_MSECS = testConstants.DEFAULT_SPEC_TIMEOUT_MSECS;
+
+describe('Exploration Editor', function () {
+  let explorationEditor: ExplorationEditor;
+  let loggedInUser: LoggedInUser & LoggedOutUser;
+
+  beforeAll(async function () {
+    explorationEditor = await UserFactory.createNewUser(
+      'explorationEditor',
+      'exploration_editor@example.com'
+    );
+
+    await explorationEditor.createAndPublishAMinimalExplorationWithTitle(
+      'Rational Numbers'
+    );
+    await explorationEditor.createAndPublishAMinimalExplorationWithTitle(
+      'Real Numbers',
+      'Algebra',
+      false
+    );
+    await explorationEditor.createAndPublishAMinimalExplorationWithTitle(
+      'Fractions',
+      'Algebra',
+      false
+    );
+    // These explorations are not published but are saved as drafts.
+    await explorationEditor.createAndSaveAMinimalExploration();
+    await explorationEditor.createAndSaveAMinimalExploration();
+
+    loggedInUser = await UserFactory.createNewUser(
+      'loggedInUser',
+      'logged_in_user@example.com'
+    );
+  }, DEFAULT_SPEC_TIMEOUT_MSECS);
+
+  it(
+    'should display created explorations and their statistics on the creator dashboard after creating, playing, and rating as a logged-in user',
+    async function () {
+      await loggedInUser.navigateToCommunityLibraryPage();
+      await loggedInUser.searchForLessonInSearchBar('Rational Numbers');
+      await loggedInUser.playLessonFromSearchResults('Rational Numbers');
+      await loggedInUser.expectExplorationCompletionToastMessage(
+        'Congratulations for completing this lesson!'
+      );
+      await loggedInUser.rateExploration(3, 'Nice!', false);
+
+      await explorationEditor.navigateToCreatorDashboardPage();
+      await explorationEditor.expectAverageRatingAndUsersToBe(3, 1);
+      await explorationEditor.expectTotalPlaysToBe(1);
+      await explorationEditor.expectOpenFeedbacksToBe(1);
+      await explorationEditor.expectNumberOfSubscribersToBe(0);
+      await explorationEditor.expectNumberOfExplorationsToBe(5);
+      await explorationEditor.expectExplorationNameToAppearNTimes(
+        'Rational Numbers'
+      );
+      await explorationEditor.expectExplorationNameToAppearNTimes(
+        'Real Numbers'
+      );
+      await explorationEditor.expectExplorationNameToAppearNTimes('Fractions');
+      await explorationEditor.expectExplorationNameToAppearNTimes(
+        'Untitled',
+        2
+      );
+    },
+    DEFAULT_SPEC_TIMEOUT_MSECS
+  );
+
+  afterAll(async function () {
+    await UserFactory.closeAllBrowsers();
+  });
+});

core/tests/puppeteer-acceptance-tests/utilities/user/exploration-editor.ts

@@ -219,6 +219,10 @@ const stayAnonymousCheckbox = '.e2e-test-stay-anonymous-checkbox';
 const responseTextareaSelector = '.e2e-test-feedback-response-textarea';
 const sendButtonSelector = '.e2e-test-oppia-feedback-response-send-btn';
 const errorSavingExplorationModal = '.e2e-test-discard-lost-changes-button';
+const totalPlaysSelector = '.e2e-test-oppia-total-plays';
+const numberOfOpenFeedbacksSelector = '.e2e-test-oppia-open-feedback';
+const avarageRatingSelector = '.e2e-test-oppia-average-rating';
+const usersCountInRatingSelector = '.e2e-test-oppia-total-users';
 
 const LABEL_FOR_SAVE_DESTINATION_BUTTON = ' Save Destination ';
 export class ExplorationEditor extends BaseUser {
@@ -1706,14 +1710,18 @@ export class ExplorationEditor extends BaseUser {
 
   /**
    * Function for creating an exploration with only EndExploration interaction with given title.
+   * @param {boolean} flag - Determines whether to dismiss the welcome modal.
    */
   async createAndPublishAMinimalExplorationWithTitle(
     title: string,
-    category: string = 'Algebra'
+    category: string = 'Algebra',
+    flag: boolean = true
   ): Promise<string | null> {
     await this.navigateToCreatorDashboardPage();
     await this.navigateToExplorationEditorPage();
-    await this.dismissWelcomeModal();
+    if (flag) {
+      await this.dismissWelcomeModal();
+    }
     await this.createMinimalExploration(
       'Exploration intro text',
       'End Exploration'
@@ -2058,6 +2066,145 @@ export class ExplorationEditor extends BaseUser {
     await this.waitForNetworkIdle();
   }
 
+  /**
+   * Function to create and save a new untitled exploration containing only the EndExploration interaction.
+   */
+  async createAndSaveAMinimalExploration(): Promise<void> {
+    await this.navigateToCreatorDashboardPage();
+    await this.navigateToExplorationEditorPage();
+    await this.createMinimalExploration(
+      'Exploration intro text',
+      'End Exploration'
+    );
+    await this.saveExplorationDraft();
+  }
+
+  /**
+   * Function to verify the average rating and the number of users who submitted ratings.
+   * @param {number} expectedRating - The expected average rating.
+   * @param {number} expectedUsers - The expected count of users who submitted ratings.
+   */
+  async expectAverageRatingAndUsersToBe(
+    expectedRating: number,
+    expectedUsers: number
+  ): Promise<void> {
+    await this.page.waitForSelector(avarageRatingSelector, {
+      visible: true,
+    });
+    const avarageRating = await this.page.$eval(
+      avarageRatingSelector,
+      element => parseFloat((element as HTMLElement).innerText.trim())
+    );
+    if (avarageRating !== expectedRating) {
+      throw new Error(
+        `Expected average rating to be ${expectedRating}, but found ${avarageRating}.`
+      );
+    }
+    const totalUsersText = await this.page.$eval(
+      usersCountInRatingSelector,
+      el => (el as HTMLElement).innerText.trim() || ''
+    );
+    // Extract number from text (e.g., "by 3 users" → 3).
+    const totalUsersMatch = totalUsersText.match(/\d+/);
+    const totalUsers = totalUsersMatch ? parseInt(totalUsersMatch[0], 10) : 0;
+    if (totalUsers !== expectedUsers) {
+      throw new Error(
+        `Expected ${expectedUsers} users to have submitted ratings, but found only ${totalUsers} instead.`
+      );
+    }
+  }
+
+  /**
+   * Function to check the expected number of open feedback entries.
+   * @param {number} number - The expected count of open feedback entries.
+   */
+  async expectOpenFeedbacksToBe(number: number): Promise<void> {
+    await this.page.waitForSelector(numberOfOpenFeedbacksSelector, {
+      visible: true,
+    });
+    const numberOfOpenFeedbacks = await this.page.$eval(
+      numberOfOpenFeedbacksSelector,
+      el => parseInt((el as HTMLElement).innerText.trim(), 10)
+    );
+    if (numberOfOpenFeedbacks !== number) {
+      throw new Error(
+        `Expected open feedback count to be ${number}, but found ${numberOfOpenFeedbacks}.`
+      );
+    }
+  }
+
+  /**
+   * Function to check the expected total number of plays."
+   * @param {number} number - The expected total play count.
+   */
+  async expectTotalPlaysToBe(number: number): Promise<void> {
+    await this.page.waitForSelector(totalPlaysSelector, {
+      visible: true,
+    });
+    const numberOfTotalPlays = await this.page.$eval(totalPlaysSelector, el =>
+      parseInt((el as HTMLElement).innerText.trim(), 10)
+    );
+    if (numberOfTotalPlays !== number) {
+      throw new Error(
+        `Expected total plays count to be ${number}, but found ${numberOfTotalPlays}.`
+      );
+    }
+  }
+
+  /**
+   * Function to check the expected total number of explorations.
+   * @param {number} number - The expected count of total explorations.
+   */
+  async expectNumberOfExplorationsToBe(number: number): Promise<void> {
+    await this.page.waitForSelector(explorationSummaryTileTitleSelector, {
+      visible: true,
+    });
+    const titlesOnPage = await this.page.$$eval(
+      explorationSummaryTileTitleSelector,
+      elements => elements.map(el => el.textContent?.trim() || '')
+    );
+    const count = titlesOnPage.length;
+
+    if (count !== number) {
+      throw new Error(
+        `Expected ${number} explorations, but found ${count} instead.`
+      );
+    }
+  }
+
+  /**
+   * Function to check the presence and expected number of occurrences of an exploration.
+   * @param {string} explorationName - The name of the exploration.
+   * @param {number} numberOfOccurrence - The expected occurrence count of the exploration.
+   */
+  async expectExplorationNameToAppearNTimes(
+    explorationName: string,
+    numberOfOccurrence: number = 1
+  ): Promise<void> {
+    await this.page.waitForSelector(explorationSummaryTileTitleSelector, {
+      visible: true,
+    });
+
+    // Extract all exploration titles.
+    const titlesOnPage = await this.page.$$eval(
+      explorationSummaryTileTitleSelector,
+      elements => elements.map(el => el.textContent?.trim() || '')
+    );
+
+    // Count occurrences of the target exploration.
+    const count = titlesOnPage.filter(
+      title => title === explorationName
+    ).length;
+
+    if (numberOfOccurrence === 1 && count !== numberOfOccurrence) {
+      throw new Error(`Exploration "${explorationName}" not found.`);
+    } else if (count !== numberOfOccurrence) {
+      throw new Error(
+        `Exploration "${explorationName}" found ${count} times, but expected ${numberOfOccurrence} times.`
+      );
+    }
+  }
+
   /**
    * Opens an exploration in the editor.
    * @param {string} explorationName - The name of the exploration.

scripts/common.py

@@ -217,6 +217,7 @@
     'exploration-editor/load-complete-and-restart-exploration-preview',
     'exploration-editor/publish-the-exploration-with-an-interaction',
     'exploration-editor/save-draft-publish-and-discard-the-changes',
+    'exploration-editor/verify-statistics-and-previous-explorations',
     'logged-in-user/subscribe-to-creator-and-view-all-'
     'explorations-by-that-creator',
     'logged-in-user/create-and-delete-account',