-
Notifications
You must be signed in to change notification settings - Fork 8
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* 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 * Pd 217873 upgrade ios minimum deployment target (#42) * 🐛 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 * ⚡ update minimum iOS deployment target * 🔖 Release 1.11.3 * 🐛 remove escaping for unsafe parameters value (#45) * 🔖 Release 1.11.4 * ✨ add originalProductName to BVReview.swift and originalProductName success and failure test cases in BVReviewQueryTest (#48) * Pd 219419 remove hardcoded credentials (#47) * 🎨 removed hardcoded credentials * 🎨 added valid and invalid userId as json file * 🎨 handling user id json using BVTestUsers class * 🎨 applied gitguardian changes * 🔖 Release 1.12.0 --------- Co-authored-by: Abishekh S Kamath <[email protected]> Co-authored-by: chetan shanbag <[email protected]>
- Loading branch information
1 parent
259abf9
commit 4ccc4a0
Showing
11 changed files
with
162 additions
and
34 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
{ | ||
"buildRequest": "VcticiyaPKqkXxKeZbKRoq0a3ArcQqAObHunbMNdjOiDSSYElouJV7wHkb6nZaSLw5q8OtGFyRFyPyZAChej/RAYtPmVCleQuFiwuKub0ac=", | ||
"buildRequestHostedAuthFailure": "DueNHADxCKCHad5AvUHlz6Md3kxA9aVmrVEYiJ1ln3KD9E0yC+kNJ/rAjukeSdiS9h73xtnxerVFyafTPT53nSloTr6GvV5hDaelalQ+cPB8ajya2zak2AfHwBEv5R8v", | ||
"buildRequestHostedAuthSuccess": "9y8kITDlqeFpCLH634OJJr7ErGP13y3oZ8ePKsYT/jOAITIj2GQ31eD2NX4oz74SujG3c4PTgkiMHBdF2FGvRI7qDUSM2SP8sGT3maxErGudQPMh9TGx8kc/8Dgn6Ik1" | ||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
{ | ||
"validUserId": "test109", | ||
"invalidUserId": "invalid" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,48 @@ | ||
// | ||
// | ||
// BVTestKeys.swift | ||
// BVSwift | ||
// | ||
// Copyright © 2023 Bazaarvoice. All rights reserved. | ||
// | ||
|
||
import Foundation | ||
|
||
class BVTestKeys { | ||
enum idJson: String { | ||
case userIdJSON = "userIdJSON" | ||
case sessionTokenJSON = "sessionIdJSON" | ||
} | ||
|
||
enum userIdKeys: String { | ||
case validUserId = "validUserId" | ||
case invalidUserId = "invalidUserId" | ||
} | ||
|
||
enum sessionTokenKeys: String { | ||
case buildRequest = "buildRequest" | ||
case buildRequestHostedAuthFailure = "buildRequestHostedAuthFailure" | ||
case buildRequestHostedAuthSuccess = "buildRequestHostedAuthSuccess" | ||
} | ||
|
||
func loadKeyForId(fromJSON: idJson, forId: String) -> String { | ||
guard let resourceURL = | ||
Bundle( | ||
for: BVTestKeys.self) | ||
.url( | ||
forResource: fromJSON.rawValue, | ||
withExtension: ".json") else { | ||
return "" | ||
} | ||
do { | ||
let data = try Data(contentsOf: resourceURL, options: []) | ||
if let json = try JSONSerialization.jsonObject(with: data) as? [String : Any] { | ||
return json[forId] as? String ?? "" | ||
} else { | ||
return "" | ||
} | ||
} catch { | ||
return "" | ||
} | ||
} | ||
} |