jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-17 10:17:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Pick onedrive owner from a more reliable source (#136929)

Browse Source 
* Pick onedrive owner from a more reliable source

* fix
This commit is contained in:
Josef Zweck 2025-01-30 17:08:35 +01:00 committed by GitHub
parent 6dd2d46328
commit 63af407f8f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
homeassistant/components/onedrive/config_flow.py
View File

@ -78,7 +78,7 @@ class OneDriveConfigFlow(AbstractOAuth2FlowHandler, domain=DOMAIN):
self.logger.exception("Unknown error") self.logger.exception("Unknown error")
return self.async_abort(reason="unknown") return self.async_abort(reason="unknown")
drive = response.json() drive: dict = response.json()
await self.async_set_unique_id(drive["parentReference"]["driveId"]) await self.async_set_unique_id(drive["parentReference"]["driveId"])
@ -94,7 +94,10 @@ class OneDriveConfigFlow(AbstractOAuth2FlowHandler, domain=DOMAIN):
self._abort_if_unique_id_configured() self._abort_if_unique_id_configured()
title = f"{drive['shared']['owner']['user']['displayName']}'s OneDrive" user = drive.get("createdBy", {}).get("user", {}).get("displayName")
title = f"{user}'s OneDrive" if user else "OneDrive"
return self.async_create_entry(title=title, data=data) return self.async_create_entry(title=title, data=data)
async def async_step_reauth( async def async_step_reauth(

2
tests/components/onedrive/conftest.py
View File

@ -88,7 +88,7 @@ def mock_adapter() -> Generator[MagicMock]:
status_code=200, status_code=200,
json={ json={
"parentReference": {"driveId": "mock_drive_id"}, "parentReference": {"driveId": "mock_drive_id"},
"shared": {"owner": {"user": {"displayName": "John Doe"}}}, "createdBy": {"user": {"displayName": "John Doe"}},
}, },
) )
yield adapter yield adapter