jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-04-25 09:47:52 +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 Bram Kragten
parent 07acabdb36
commit 252b13e63a
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")
return self.async_abort(reason="unknown")
drive = response.json()
drive: dict = response.json()
await self.async_set_unique_id(drive["parentReference"]["driveId"])
@ -94,7 +94,10 @@ class OneDriveConfigFlow(AbstractOAuth2FlowHandler, domain=DOMAIN):
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)
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,
json={
"parentReference": {"driveId": "mock_drive_id"},
"shared": {"owner": {"user": {"displayName": "John Doe"}}},
"createdBy": {"user": {"displayName": "John Doe"}},
},
)
yield adapter