Release 1.11.2

* Add sonarqube scan worflow

* Pd 177323 cdv filter (#13)

* Implemented CDV filter for reviews.json

* Pd 177925 seccondary ratings filter (#14)

* Implemented Secondaryratings filters for reviews.json

* Implemented Additional field filter for reviews.json (#15)

* Implemented tag dimensions filter for reviews.json

* Pd 178797 add value label cdv response (#18)

Mapped "ValueLabel" response object for context data values

* Pd 178686 tag statistics (#19)

Implemented tagstats reuest param for reviews.json and products.json

* Mapped secondary averages response objects

* Implemented Secondary Rating Distribution

* Refactoring keyword filter classes to maintain the parity

* Allow dynamic array or variadic parameters to be passed to filter function

* Release 1.10.0

* Implemented Custom Sort, Relavancy Sort, Xcode 14 fix

* Update Sonarscan workflow

* Q&A statistics changes for statistics.json

* Release 1.11.0

* Fix for configuration errors with test case changes

* Release 1.11.1

* 🐛 fixed transaction monitoring issue by adding clientId in BVAnalyticsEventTransaction (#35)

* Revert ":bug: fixed transaction monitoring issue by adding clientId in BVAnalyticsEventTransaction (#35)" (#37)

This reverts commit ba52da0e2d2209afe443df00ace8230bdbe8b498.

* Pd 215869 bv pixel transaction event not recorded on monitoring tool (#38)

* 🐛 fixed transaction monitoring issue by adding clientId in BVAnalyticsEventTransaction

* 🐛 fix transaction monitoring issue by checking client key in whitelist and blacklist dictionary
✅ add separate test cases for nonPII and PII transactions event

* ⚰️ removing unused code

* Release 1.11.2

---------

Co-authored-by: Rahul Bawane <[email protected]>
Co-authored-by: Abishekh S Kamath <[email protected]>
Co-authored-by: chetan shanbag <[email protected]>

Author: 3 people

BVSwift.podspec

@@ -8,7 +8,7 @@
 
 Pod::Spec.new do |s|
   s.name              = 'BVSwift'
-  s.version           = '1.11.1'
+  s.version           = '1.11.2'
   s.summary           = 'Simple Swift based iOS SDK to interact with the Bazaarvoice platform API.'
   s.description       = 'The Bazaarvoice Software Development Kit (SDK) is a Swift iOS library that provides an easy way to generate REST calls to the Bazaarvoice Developer API. Using this SDK, mobile developers can quickly integrate Bazaarvoice content into their native iOS apps for iPhone and iPad on iOS 8.0 or newer.'
 

BVSwift/BVAnalytics/BVAnalyticsEvent.swift

@@ -202,13 +202,13 @@ extension BVAnalyticsEvent {
 
   internal var toWhitelistDict: [String: Encodable] {
     return toDict.filter {
-      BVAnalyticsEvent.whiteList.contains($0.key)
+      BVAnalyticsEvent.whiteList.contains($0.key) || $0.key == "client"
     }
   }
 
   internal var toBlacklistDict: [String: Encodable] {
     return toDict.filter {
-      !BVAnalyticsEvent.whiteList.contains($0.key)
+      !BVAnalyticsEvent.whiteList.contains($0.key) && $0.key != "client"
     }
   }
 

BVSwift/BVCommon/Configuration/BVConstants.swift

@@ -18,7 +18,7 @@ internal struct BVConstants {
     static let appVersionField: String = "_appVersion"
     static let buildNumberField: String = "_buildNumber"
     static let sdkVersionField: String = "_bvIosSwiftSdkVersion"
-    static let bvSwiftSDKVersion: String = "1.11.1"
+    static let bvSwiftSDKVersion: String = "1.11.2"
 
 }
 

BVSwift/Support/Info.plist

@@ -17,7 +17,7 @@
 	<key>CFBundlePackageType</key>
 	<string>FMWK</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.11.1</string>
+	<string>1.11.2</string>
 	<key>CFBundleVersion</key>
 	<string>1</string>
 	<key>NSPrincipalClass</key>

BVSwiftTests/BVAnalytics/BVPixelTest.swift

@@ -342,11 +342,86 @@ class BVPixelTest: XCTestCase {
         error, "Something went horribly wrong, request took too long.")
     }
   }
-  
-  func testPixelTransactionEvent() {
+
+  func testPixelTransactionEventNonPII() {
+      
+      let expectation =
+      self.expectation(description: "testPixelTransactionEventNonPII")
+      
+      let items: [BVAnalyticsTransactionItem] = [
+          BVAnalyticsTransactionItem(
+              category: "fruit",
+              imageURL: nil,
+              name: "apples",
+              price: 5.35,
+              quantity: 10,
+              sku: "$%#$%#jksfgjsdkf$554353"),
+          BVAnalyticsTransactionItem(
+              category: "fruit",
+              imageURL: nil,
+              name: "oranges",
+              price: 8.35,
+              quantity: 10,
+              sku: "$%#$%#$jfhdjfhdf"),
+          BVAnalyticsTransactionItem(
+              category: "fruit",
+              imageURL: nil,
+              name: "bananas",
+              price: 10.35,
+              quantity: 10,
+              sku: "$%#$%#djfhdjhf554353")
+      ]
+      
+      let transaction: BVAnalyticsEvent =
+          .transaction(items: items,
+                       orderId: "id-924859485",
+                       total: 20.75,
+                       city: "Austin",
+                       country: "USA",
+                       currency: "US",
+                       shipping: 7.50,
+                       state: "TX",
+                       tax: 3.25,
+                       additional: ["locale": "en_US"])
+      
+      let numberOfEvents: UInt = 1
+      
+      BVPixel.track(transaction)
+      
+      BVAnalyticsManager.sharedManager.flush
+      { (successes: UInt?, errors: [Error]?) in
+          if let errs = errors {
+              print(errs)
+              XCTFail()
+              expectation.fulfill()
+              return
+          }
+          
+          guard let wins = successes else {
+              XCTFail()
+              expectation.fulfill()
+              return
+          }
+            
+          if numberOfEvents != wins {
+              XCTFail()
+              expectation.fulfill()
+              return
+          }
+            
+          expectation.fulfill()
+      }
+        
+      self.waitForExpectations(timeout: 20) { (error) in
+          XCTAssertNil(
+              error, "Something went horribly wrong, request took too long.")
+      }
+  }
+    
+  func testPixelTransactionEventPII() {
 
     let expectation =
-      self.expectation(description: "testPixelTransactionEvent")
+      self.expectation(description: "testPixelTransactionEventPII")
 
     let items: [BVAnalyticsTransactionItem] = [
       BVAnalyticsTransactionItem(
@@ -403,7 +478,6 @@ class BVPixelTest: XCTestCase {
           return
         }
 
-        
         if numberOfEvents != wins {
           XCTFail()
           expectation.fulfill()