Skip to content

Commit

Permalink
Merge pull request #1754 from DataDog/jmoskovich/fix-resourceid-e2e
Browse files Browse the repository at this point in the history
Remove resourceId field from e2e payloads
  • Loading branch information
jonathanmos authored Dec 6, 2023
2 parents f38f6be + 2b2d3d6 commit bece8a1
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ internal abstract class BaseSessionReplayTest<R : Activity> {
// will be executed in Bitrise and currently Bitrise does not own a specific model for the
// API 33. They only have a standard emulator for this API with the required screen size and
// X,Y positions are different from the ones we have in our local emulator.
// Also the base64 encoded images values are inconsistent from one run to another so will
// Also the base64 and resourceId encoded images values are inconsistent from one run to another so will
// be removed from the payload.

return this.asJsonObject.apply {
Expand All @@ -254,6 +254,7 @@ internal abstract class BaseSessionReplayTest<R : Activity> {
wireframeJson.remove("x")
wireframeJson.remove("y")
wireframeJson.remove("base64")
wireframeJson.remove("resourceId")
wireframeJson
}?.fold(JsonArray()) { acc, jsonObject ->
acc.add(jsonObject)
Expand Down

0 comments on commit bece8a1

Please sign in to comment.