Skip to content

fix: avoid stringifying entire user_properties#101

Open
LouisLau-art wants to merge 1 commit intopytest-dev:mainfrom
LouisLau-art:fix/cleanup-unserializable-nested-values-12
Open

fix: avoid stringifying entire user_properties#101
LouisLau-art wants to merge 1 commit intopytest-dev:mainfrom
LouisLau-art:fix/cleanup-unserializable-nested-values-12

Conversation

@LouisLau-art
Copy link

Fixes #12.

  • Make cleanup_unserializable sanitize nested values so a single unserializable value (e.g. a set inside user_properties) doesn't stringify the whole user_properties payload.
  • Add a regression test covering the user_properties case.
  • Skip the existing subtests integration test when pytest-subtests isn't installed.

Test:

  • python -m pytest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cleanup_unserializable results in weird handling of unserializable user_properties

1 participant