Add views attribute to SiteVisitStatsItem in storage

Author: rachelmcr

Networking/NetworkingTests/Responses/site-visits-alt.json

@@ -13,7 +13,7 @@
     "data": [
              [
               "2014-01-01",
-              12821,
+              12831,
               1140,
               1094,
               0,
@@ -22,7 +22,7 @@
               ],
              [
               "2015-01-01",
-              14808,
+              14818,
               1634,
               1492,
               0,

Storage/Storage/Model/MIGRATIONS.md

@@ -2,6 +2,10 @@
 
 This file documents changes in the WCiOS Storage data model. Please explain any changes to the data model as well as any custom migrations.
 
+## Model 79 (Release 11.7.0.0)
+- @rachelmcr 2022-12-12
+    - Added `views` attribute to `SiteVisitStatsItem` entity.
+
 ## Model 78 (Release 11.4.0.0)
 - @rachelmcr 2022-11-18
     - Added `averageOrderValue` attribute to `OrderStatsV4Totals` entity.

Storage/Storage/Model/SiteVisitStatsItem+CoreDataProperties.swift

@@ -10,5 +10,6 @@ extension SiteVisitStatsItem {
 
     @NSManaged public var period: String?
     @NSManaged public var visitors: Int64
+    @NSManaged public var views: Int64
     @NSManaged public var stats: SiteVisitStats?
 }

Storage/Storage/Model/WooCommerce.xcdatamodeld/Model 79.xcdatamodel/contents

@@ -756,6 +756,7 @@
     </entity>
     <entity name="SiteVisitStatsItem" representedClassName="SiteVisitStatsItem" syncable="YES">
         <attribute name="period" optional="YES" attributeType="String"/>
+        <attribute name="views" optional="YES" attributeType="Integer 64" defaultValueString="0" usesScalarValueType="YES"/>
         <attribute name="visitors" optional="YES" attributeType="Integer 64" defaultValueString="0" usesScalarValueType="YES"/>
         <relationship name="stats" optional="YES" maxCount="1" deletionRule="Nullify" destinationEntity="SiteVisitStats" inverseName="items" inverseEntity="SiteVisitStats"/>
     </entity>

Yosemite/Yosemite/Model/Storage/SiteVisitStatsItem+ReadOnlyConvertible.swift

@@ -11,13 +11,14 @@ extension Storage.SiteVisitStatsItem: ReadOnlyConvertible {
     public func update(with statsItem: Yosemite.SiteVisitStatsItem) {
         period = statsItem.period
         visitors = Int64(statsItem.visitors)
+        views = Int64(statsItem.views)
     }
 
     /// Returns a ReadOnly version of the receiver.
     ///
     public func toReadOnly() -> Yosemite.SiteVisitStatsItem {
         return SiteVisitStatsItem(period: period ?? "",
                                   visitors: Int(visitors),
-                                  views: 0) // TODO-8363: Update Storage with views count
+                                  views: Int(views))
     }
 }

Yosemite/YosemiteTests/Stores/StatsStoreV4Tests.swift

@@ -603,11 +603,11 @@ private extension StatsStoreV4Tests {
 
 
     func sampleSiteVisitStatsItem1() -> Networking.SiteVisitStatsItem {
-        return SiteVisitStatsItem(period: "2014-01-01", visitors: 1135, views: 0)
+        return SiteVisitStatsItem(period: "2014-01-01", visitors: 1135, views: 12821)
     }
 
     func sampleSiteVisitStatsItem2() -> Networking.SiteVisitStatsItem {
-        return SiteVisitStatsItem(period: "2015-01-01", visitors: 1629, views: 0)
+        return SiteVisitStatsItem(period: "2015-01-01", visitors: 1629, views: 14808)
     }
 
     func sampleSiteVisitStatsMutated() -> Networking.SiteVisitStats {
@@ -619,11 +619,11 @@ private extension StatsStoreV4Tests {
 
 
     func sampleSiteVisitStatsItem1Mutated() -> Networking.SiteVisitStatsItem {
-        return SiteVisitStatsItem(period: "2014-01-01", visitors: 1140, views: 0)
+        return SiteVisitStatsItem(period: "2014-01-01", visitors: 1140, views: 12831)
     }
 
     func sampleSiteVisitStatsItem2Mutated() -> Networking.SiteVisitStatsItem {
-        return SiteVisitStatsItem(period: "2015-01-01", visitors: 1634, views: 0)
+        return SiteVisitStatsItem(period: "2015-01-01", visitors: 1634, views: 14818)
     }
 
     // MARK: - Top Earner Stats Sample