Update naming to reflect renamed SiteStatsRemote

Author: rachelmcr

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

@@ -483,7 +483,7 @@ private extension StoreStatsAndTopPerformersViewController {
         }
     }
 
-    func handleSiteVisitStatsStoreError(error: SiteVisitStatsStoreError) {
+    func handleSiteStatsStoreError(error: SiteStatsStoreError) {
         switch error {
         case .noPermission:
             updateSiteVisitors(mode: .hidden)
@@ -501,8 +501,8 @@ private extension StoreStatsAndTopPerformersViewController {
 
     private func handleSyncError(error: Error) {
         switch error {
-        case let siteVisitStatsStoreError as SiteVisitStatsStoreError:
-            handleSiteVisitStatsStoreError(error: siteVisitStatsStoreError)
+        case let siteStatsStoreError as SiteStatsStoreError:
+            handleSiteStatsStoreError(error: siteStatsStoreError)
         default:
             displaySyncingError()
         }

Yosemite/Yosemite/Stores/StatsStoreV4.swift

@@ -6,13 +6,13 @@ import WooFoundation
 // MARK: - StatsStoreV4
 //
 public final class StatsStoreV4: Store {
-    private let siteVisitStatsRemote: SiteStatsRemote
+    private let siteStatsRemote: SiteStatsRemote
     private let leaderboardsRemote: LeaderboardsRemote
     private let orderStatsRemote: OrderStatsRemoteV4
     private let productsRemote: ProductsRemote
 
     public override init(dispatcher: Dispatcher, storageManager: StorageManagerType, network: Network) {
-        self.siteVisitStatsRemote = SiteStatsRemote(network: network)
+        self.siteStatsRemote = SiteStatsRemote(network: network)
         self.leaderboardsRemote = LeaderboardsRemote(network: network)
         self.orderStatsRemote = OrderStatsRemoteV4(network: network)
         self.productsRemote = ProductsRemote(network: network)
@@ -164,7 +164,7 @@ private extension StatsStoreV4 {
 
         let quantity = timeRange.siteVisitStatsQuantity(date: latestDateToInclude, siteTimezone: siteTimezone)
 
-        siteVisitStatsRemote.loadSiteVisitorStats(for: siteID,
+        siteStatsRemote.loadSiteVisitorStats(for: siteID,
                                     siteTimezone: siteTimezone,
                                     unit: timeRange.siteVisitStatsGranularity,
                                     latestDateToInclude: latestDateToInclude,
@@ -174,7 +174,7 @@ private extension StatsStoreV4 {
                 self?.upsertStoredSiteVisitStats(readOnlyStats: siteVisitStats, timeRange: timeRange)
                 onCompletion(.success(()))
             case .failure(let error):
-                onCompletion(.failure(SiteVisitStatsStoreError(error: error)))
+                onCompletion(.failure(SiteStatsStoreError(error: error)))
             }
         }
     }
@@ -571,7 +571,7 @@ public enum StatsStoreV4Error: Error {
 /// - statsModuleDisabled: Jetpack site stats module is disabled for the site.
 /// - unknown: other error cases.
 ///
-public enum SiteVisitStatsStoreError: Error {
+public enum SiteStatsStoreError: Error {
     case statsModuleDisabled
     case noPermission
     case unknown

Yosemite/YosemiteTests/Stores/SiteVisitStatsStoreErrorTests.swift

@@ -2,28 +2,28 @@ import XCTest
 @testable import Yosemite
 @testable import Networking
 
-class SiteVisitStatsStoreErrorTests: XCTestCase {
+class SiteStatsStoreErrorTests: XCTestCase {
     func testNoPermissionError() {
         let remoteError = DotcomError.noStatsPermission
-        let error = SiteVisitStatsStoreError(error: remoteError)
+        let error = SiteStatsStoreError(error: remoteError)
         XCTAssertEqual(error, .noPermission)
     }
 
     func testStatsModuleDisabledError() {
         let remoteError = DotcomError.statsModuleDisabled
-        let error = SiteVisitStatsStoreError(error: remoteError)
+        let error = SiteStatsStoreError(error: remoteError)
         XCTAssertEqual(error, .statsModuleDisabled)
     }
 
     func testOtherDotcomError() {
         let remoteError = DotcomError.unknown(code: "invalid_blog", message: "This blog does not have Jetpack connected")
-        let error = SiteVisitStatsStoreError(error: remoteError)
+        let error = SiteStatsStoreError(error: remoteError)
         XCTAssertEqual(error, .unknown)
     }
 
     func testNonDotcomRemoteError() {
         let remoteError = NSError(domain: "Woo", code: 404, userInfo: nil)
-        let error = SiteVisitStatsStoreError(error: remoteError)
+        let error = SiteStatsStoreError(error: remoteError)
         XCTAssertEqual(error, .unknown)
     }
 }