Skip to content
This repository has been archived by the owner on Apr 3, 2023. It is now read-only.

Added ExclusionStrategy to be used when serializing command reports. #22

Closed
wants to merge 2 commits into from
Closed

Conversation

gileli121
Copy link

This exclusion strategy makes sure to skip serialization of any classes implementing the WebDriver interface.
Since this object is redundant when reporting command parameters and doesn’t pass serialization - we exclude it.
See google/gson#1540 for more details on the serialization problem.

Fixes #21

This exclusion strategy makes sure to skip serialization of any classes implementing the WebDriver interface.
Since this object is redundant when reporting command parameters and doesn’t pass serialization - we exclude it.
See google/gson#1540 for more details on the serialization problem. (edited)

Signed-off-by: Gil Eliyahu <gil.eliyahu@testproject.io>
…xecution.

Signed-off-by: Gil Eliyahu <gil.eliyahu@testproject.io>
@mstrelex
Copy link
Member

mstrelex commented Aug 7, 2020

Can't run CI on forked PRs for now, closing.

@mstrelex mstrelex closed this Aug 7, 2020
mstrelex pushed a commit that referenced this pull request Aug 7, 2020
Added ExclusionStrategy to be used when serializing command reports. #22
@gileli121 gileli121 deleted the bugs/TP-12192 branch August 13, 2020 09:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Command reporting fails and thus false-fail the test when trying to serialize Actions class
3 participants