Integration tests errors: master against 208

Description

We made a fix (https://github.com/Sage-Bionetworks/synapsePythonClient/pull/497)
to handle failures in the existing client against staging (208).
Re-ran from staging and got a different set of errors:

Environment

None

Activity

Show:
Bruce Hoff
January 25, 2018, 12:08 AM

This was addressed by reverting the change to the back-end json parser.

Bruce Hoff
January 24, 2018, 2:06 PM

I updated the fix-version to remind us to close this out with the 1.7.4 release. Is this done?

Ziming Dong
December 10, 2017, 8:34 AM

Oh yeah.... once we get a rowset JSON from Synapse backend we try to convert the row values into based on the column type.

Since the backend is back to being permissive about row values for update, I think we should just revert the change for casting to strings in the python client. Enforcing conversion to string makes it unintuitive for users since they would have to manually cast every values from string if they wanted to use the values.

Fixed

Assignee

Ziming Dong

Reporter

Xavier Schildwachter

Labels

None

Validator

Xavier Schildwachter

Development Area

None

Release Version History

None

Slack Channel

None

Fix versions

Affects versions

Priority

Major