Skip to content
New issue

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

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

Already on GitHub? # to your account

Add a test for user API #148

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

allenporter
Copy link
Collaborator

@allenporter allenporter commented Dec 4, 2024

This test currently fails

@chemelli74
Copy link
Owner

Tests are created as sync and not async so we need for now to pin aiohttp to a old version

@allenporter
Copy link
Collaborator Author

The response parsing works fine:

INFO     root:rest.py:163 response body: {"user": {"encodedId": "some-profile-id", "locale": "en_US", "fullName": "Full name", "displayName": "Display name", "displayNameSetting": "name", "firstName": "First", "lastName": "Last"}}

The issue I was explaining over discord is

response_type=None, # noqa: E501
) is setting response_type=None which means that the API client returns None
return_data = None

and as I was saying this is consistent with the response data models not being produced https://github.com/chemelli74/fitbit-web-api/tree/main/fitbit_web_api/models and the return type in the documentation being None https://github.com/chemelli74/fitbit-web-api/blob/main/docs/UserApi.md#return-type-1

This is all repeating what I said on discord, because the fitibt openapi spec itself does not include these.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants