Merge pull request #8319 from woocommerce/tap-stats-chart-ui-test

[UI Tests] - Add view detailed stats chart UI test

Author: jostnes

WooCommerce/Classes/ViewRelated/Dashboard/Stats v4/StoreStatsV4PeriodViewController.swift

@@ -354,6 +354,7 @@ private extension StoreStatsV4PeriodViewController {
                                                                       comment: "VoiceOver accessibility label for the store revenue chart's Y-axis.")
         chartAccessibilityView.isAccessibilityElement = true
         chartAccessibilityView.accessibilityTraits = .image
+        chartAccessibilityView.accessibilityIdentifier = "chart-image"
         chartAccessibilityView.accessibilityLabel = NSLocalizedString("Store revenue chart",
                                                                       comment: "VoiceOver accessibility label for the store revenue chart.")
         chartAccessibilityView.accessibilityLabel = String.localizedStringWithFormat(
@@ -690,6 +691,7 @@ private extension StoreStatsV4PeriodViewController {
     func updateStatsDataToDefaultStyles() {
         revenueData.font = Constants.revenueFont
         revenueData.textColor = Constants.statsTextColor
+        revenueData.accessibilityIdentifier = "revenue-value"
     }
 }
 

WooCommerce/UITestsFoundation/Screens/MyStore/MyStoreScreen.swift

@@ -3,11 +3,6 @@ import XCTest
 
 public final class MyStoreScreen: ScreenObject {
 
-    // TODO: Remove force `try` once `ScreenObject` migration is completed
-    public let tabBar = try! TabNavComponent()
-    // TODO: Remove force `try` once `ScreenObject` migration is completed
-    public let periodStatsTable = try! PeriodStatsTable()
-
     static var isVisible: Bool {
         (try? MyStoreScreen().isLoaded) ?? false
     }
@@ -34,14 +29,17 @@ public final class MyStoreScreen: ScreenObject {
         return self
     }
 
+    @discardableResult
     public func goToThisWeekTab() -> MyStoreScreen {
         return tapTimeframeTab(timeframeId: "period-data-thisWeek-tab")
     }
 
+    @discardableResult
     public func goToThisMonthTab() -> MyStoreScreen {
         return tapTimeframeTab(timeframeId: "period-data-thisMonth-tab")
     }
 
+    @discardableResult
     public func goToThisYearTab() -> MyStoreScreen {
         return tapTimeframeTab(timeframeId: "period-data-thisYear-tab")
     }
@@ -69,4 +67,16 @@ public final class MyStoreScreen: ScreenObject {
     public func verifyThisYearStatsLoaded() -> MyStoreScreen {
         return verifyStatsForTimeframeLoaded(timeframe: "This Year")
     }
+
+    public func getRevenueValue() -> String {
+        return app.staticTexts["revenue-value"].label
+    }
+
+    public func tapChart() {
+        app.images["chart-image"].tap()
+    }
+
+    public func verifyRevenueUpdated(originalRevenue: String, updatedRevenue: String) {
+        XCTAssertNotEqual(originalRevenue, updatedRevenue, "Revenue is not updated!")
+    }
 }

WooCommerce/WooCommerceScreenshots/WooCommerceScreenshots.swift

@@ -43,11 +43,12 @@ class WooCommerceScreenshots: XCTestCase {
 
         // My Store
         .dismissTopBannerIfNeeded()
-        .then { ($0 as! MyStoreScreen).periodStatsTable.switchToMonthsTab() }
+        .then { ($0 as! MyStoreScreen).goToThisMonthTab() }
         .thenTakeScreenshot(named: "order-dashboard")
 
         // Orders
-        .tabBar.goToOrdersScreen()
+        try TabNavComponent()
+        .goToOrdersScreen()
         .startOrderCreation()
         .thenTakeScreenshot(named: "order-creation")
         .cancelOrderCreation()

WooCommerce/WooCommerceUITests/Mocks/mappings/jetpack-blogs/wc-analytics/reports_revenue_stats_hour.json

@@ -19,10 +19,10 @@
     "jsonBody": {
       "data": {
         "totals": {
-          "orders_count": 2,
+          "orders_count": 80,
           "num_items_sold": 0,
           "gross_sales": 0,
-          "total_sales": 23,
+          "total_sales": 815,
           "coupons": 0,
           "coupons_count": 0,
           "refunds": 0,

WooCommerce/WooCommerceUITests/README.md

@@ -15,7 +15,7 @@ The following flows are covered/planned to be covered by UI tests. Tests that ar
     - [ ] Log in with Google
 2. My Store
     - [x] Stats Today, This Week, This Month, This Year load
-    - [ ] Tap chart on stats
+    - [x] View detailed chart stats
 3. Orders
    - [x] Orders list loads
    - [ ] Single Order Screen:

WooCommerce/WooCommerceUITests/Tests/StatsTests.swift

@@ -24,4 +24,34 @@ final class StatsTests: XCTestCase {
             .goToThisYearTab()
             .verifyThisYearStatsLoaded()
     }
+
+    func test_view_detailed_chart_stats() throws {
+        let myStoreScreen = try MyStoreScreen()
+
+        var dailyRevenue = try TabNavComponent()
+            .goToMyStoreScreen()
+            .getRevenueValue()
+
+        myStoreScreen.tapChart()
+        let hourlyRevenue = myStoreScreen.getRevenueValue()
+        myStoreScreen.verifyRevenueUpdated(originalRevenue: dailyRevenue, updatedRevenue: hourlyRevenue)
+
+        myStoreScreen.goToThisWeekTab()
+        var weeklyRevenue = try MyStoreScreen().getRevenueValue()
+        myStoreScreen.tapChart()
+        dailyRevenue = myStoreScreen.getRevenueValue()
+        myStoreScreen.verifyRevenueUpdated(originalRevenue: weeklyRevenue, updatedRevenue: dailyRevenue)
+
+        myStoreScreen.goToThisMonthTab()
+        var monthlyRevenue = try MyStoreScreen().getRevenueValue()
+        myStoreScreen.tapChart()
+        weeklyRevenue = myStoreScreen.getRevenueValue()
+        myStoreScreen.verifyRevenueUpdated(originalRevenue: monthlyRevenue, updatedRevenue: weeklyRevenue)
+
+        myStoreScreen.goToThisYearTab()
+        let yearlyRevenue = try MyStoreScreen().getRevenueValue()
+        myStoreScreen.tapChart()
+        monthlyRevenue = myStoreScreen.getRevenueValue()
+        myStoreScreen.verifyRevenueUpdated(originalRevenue: yearlyRevenue, updatedRevenue: monthlyRevenue)
+    }
 }