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

Source Zendesk Chat: correct problem with CI #5478

Closed
Tracked by #14146
marcosmarxm opened this issue Aug 17, 2021 · 0 comments · Fixed by #15879
Closed
Tracked by #14146

Source Zendesk Chat: correct problem with CI #5478

marcosmarxm opened this issue Aug 17, 2021 · 0 comments · Fixed by #15879

Comments

@marcosmarxm
Copy link
Member

Current Behavior

value = ['not', ['firedBefore'***

    def serialize(value) -> str:
        """Simplify comparison of nested dicts/lists"""
        if isinstance(value, Mapping):
            return json.dumps(***k: serialize(v) for k, v in value.items()*** sort_keys=True)
        if isinstance(value, List):
>           return sorted([serialize(v) for v in value***)
E           TypeError: '<' not supported between instances of 'list' and 'str'

/usr/local/lib/python3.7/site-packages/source_acceptance_test/utils/compare.py:78: TypeError

Expected Behavior

#5476 change the field unread to correct type. Test a connection between Zendesk to Pg and normalization is working but CI keep failing
Screenshot from 2021-08-17 19-25-33

Logs

If applicable, please upload the logs from the failing operation.
For sync jobs, you can download the full logs from the UI by going to the sync attempt page and
clicking the download logs button at the top right of the logs display window.

LOG

replace this with
your long log
output here

Steps to Reproduce

Are you willing to submit a PR?

Remove this with your answer.

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

Successfully merging a pull request may close this issue.

5 participants