fix naver.py with newer version API (#152)

* fix naver.py with newer version API

remove naver.py print debugs.

* add naver login mockup data, fix if picture isn't set, set None. before ""

* fix naver test cases... Sorry!

remove debugging print functions...

* code quality fix!!

Author: dalbodeule

fastapi_sso/sso/naver.py

@@ -23,4 +23,9 @@ async def get_discovery_document(self) -> DiscoveryDocument:
         }
 
     async def openid_from_response(self, response: dict, session: Optional["httpx.AsyncClient"] = None) -> OpenID:
-        return OpenID(display_name=response["properties"]["nickname"], provider=self.provider)
+        return OpenID(
+            id=response["response"]["id"],
+            display_name=response["response"]["nickname"],
+            picture=response["response"]["profile_image"] or None,
+            provider=self.provider,
+        )

tests/test_openid_responses.py

@@ -28,7 +28,11 @@
             id="test", provider="spotify", display_name="testuser", email="[email protected]", picture="https://myimage"
         ),
     ),
-    (NaverSSO, {"properties": {"nickname": "test"}}, OpenID(display_name="test", provider="naver")),
+    (
+        NaverSSO,
+        {"response": {"nickname": "test", "profile_image": "https://myimage", "id": "test"}},
+        OpenID(id="test", display_name="test", provider="naver", picture="https://myimage"),
+    ),
     (
         MicrosoftSSO,
         {"mail": "[email protected]", "displayName": "Test User", "id": "test", "givenName": "Test", "surname": "User"},