Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REF] Fix serialized membership field handling in RelationshipTest #17469

Merged
merged 1 commit into from
Jun 2, 2020

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jun 2, 2020

Overview

Test cleanup - fix api input used in unit test.

Technical Details

These fields were being unnecessarily and incorrectly serialized by the test.
Passing an array is preferred and it will handle the serialization correctly.

Comments

This should be merged on pass as it's only test cleanup.

These fields were being unnecessarily and incorrectly serialized by the test.
Passing an array is preferred and it will handle the serialization correctly.
@civibot
Copy link

civibot bot commented Jun 2, 2020

(Standard links)

@eileenmcnaughton eileenmcnaughton merged commit 5751186 into civicrm:master Jun 2, 2020
@eileenmcnaughton eileenmcnaughton deleted the memberTypeTest branch June 2, 2020 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants