Add optional page_type parameter to ParselyMetadata class in iOS SDK

Demo/ParselyDemo/FirstViewController.swift

@@ -7,7 +7,7 @@ class FirstViewController: UIViewController {
 
     @IBAction func didTouchButton(_ sender: Any) {
         os_log("didTouchButton", log: OSLog.default, type: .debug)
-        let demoMetas = ParselyMetadata(authors: ["Yogi Berr"])
+        let demoMetas = ParselyMetadata(authors: ["Yogi Berr"], page_type: "post")
         delegate.parsely.trackPageView(url: "http://parsely.com/path/cool-blog-post/1?qsarg=nawp&anotherone=yup", metadata: demoMetas, extraData: ["product-id": "12345"], siteId: "subdomain.parsely-test.com")
     }
 

Sources/Metadata.swift

@@ -9,6 +9,7 @@ public class ParselyMetadata {
     var section: String?
     var tags: Array<String>?
     var duration: TimeInterval?
+    var page_type: String?
 
     /**
      A class to manage and re-use metadata. Metadata contained in an instance of this
@@ -22,6 +23,7 @@ public class ParselyMetadata {
      - Parameter section: Same as section for website integration.
      - Parameter tags: Up to 20 tags on an event are allowed.
      - Parameter duration: Durations passed explicitly to trackVideoStart take precedence over any in metadata.
+     - Parameter page_type: The type of page being tracked
     */
     public init(canonical_url: String? = nil,
          pub_date: Date? = nil,
@@ -30,7 +32,8 @@ public class ParselyMetadata {
          image_url: String? = nil,
          section: String? = nil,
          tags: Array<String>? = nil,
-         duration: TimeInterval? = nil) {
+         duration: TimeInterval? = nil,
+         page_type: String? = nil) {
         self.canonical_url = canonical_url
         self.pub_date = pub_date
         self.title = title
@@ -39,6 +42,7 @@ public class ParselyMetadata {
         self.section = section
         self.tags = tags
         self.duration = duration
+        self.page_type = page_type
     }
 
     func toDict() -> Dictionary<String, Any> {
@@ -68,6 +72,9 @@ public class ParselyMetadata {
         if let duration {
             metas["duration"] = duration
         }
+        if let page_type {
+            metas["page_type"] = page_type
+        }
 
         return metas
     }

Tests/MetadataTests.swift

@@ -11,7 +11,8 @@ class MetadataTests: XCTestCase {
         "image_url": "http://parsely-test.com/image2",
         "section": "Things my mother says",
         "tags": ["tag1", "tag2"],
-        "duration": TimeInterval(100)
+        "duration": TimeInterval(100),
+        "page_type": "post"
     ]
 
     func testToDictEmpty() {
@@ -37,7 +38,8 @@ class MetadataTests: XCTestCase {
             image_url: expected["image_url"] as? String,
             section: expected["section"] as? String,
             tags: expected["tags"] as? Array<String>,
-            duration: expected["duration"] as? TimeInterval
+            duration: expected["duration"] as? TimeInterval,
+            page_type: expected["page_type"] as? String
         )
         let actual: Dictionary<String, Any> = metasUnderTest.toDict()
         let pubDateUnix: String = String(format:"%i", (expected["pub_date"]! as! Date).millisecondsSince1970)
@@ -67,6 +69,9 @@ class MetadataTests: XCTestCase {
         XCTAssertEqual(actual["duration"]! as! TimeInterval, expected["duration"]! as! TimeInterval,
                        "The duration field in the result of ParselyMetadata.toDict should match the duration argument " +
                        "used at initialization")
+        XCTAssertEqual(actual["page_type"]! as! String, expected["page_type"]! as! String,
+                       "The page_type field in the result of ParselyMetadata.toDict should match the page_type argument " +
+                       "used at initialization")                       
     }
 
     func testMetadata() {
@@ -78,7 +83,8 @@ class MetadataTests: XCTestCase {
             image_url: expected["image_url"] as? String,
             section: expected["section"] as? String,
             tags: expected["tags"] as? Array<String>,
-            duration: expected["duration"] as? TimeInterval
+            duration: expected["duration"] as? TimeInterval,
+            page_type: expected["page_type"] as? String
         )
         XCTAssertEqual(metasUnderTest.canonical_url, expected["canonical_url"]! as? String,
                        "The canonical_url field on ParselyMetadata should match the canonical_url argument " +
@@ -104,5 +110,8 @@ class MetadataTests: XCTestCase {
         XCTAssertEqual(metasUnderTest.duration, expected["duration"]! as? TimeInterval,
                        "The duration field on ParselyMetadata should match the duration argument " +
                        "used at initialization")
+        XCTAssertEqual(metasUnderTest.page_type, expected["page_type"]! as? String,
+                       "The page_type field on ParselyMetadata should match the page_type argument " +
+                       "used at initialization")                       
     }
 }

Tests/RequestBuilderTests.swift

@@ -13,7 +13,8 @@ class RequestBuilderTests: XCTestCase {
             image_url: "http://parsely-test.com/image2",
             section: "Things my mother says",
             tags: ["tag1", "tag2"],
-            duration: TimeInterval(100)
+            duration: TimeInterval(100),
+            page_type: "post"
         )
         return [Event(
             "pageview",

Tests/VideoTests.swift

@@ -78,7 +78,8 @@ class VideoTests: ParselyTestCase {
             image_url: nil,
             section: testSectionFirst,
             tags: nil,
-            duration: nil
+            duration: nil,
+            page_type: nil
         )
 
         videoManager.trackPlay(
@@ -104,7 +105,8 @@ class VideoTests: ParselyTestCase {
             image_url: nil,
             section: testSectionSecond,
             tags: nil,
-            duration: nil
+            duration: nil,
+            page_type: nil
         )
 
         videoManager.trackPlay(