Add the league-v4/entries/by-puuid endpoint

README.rst

@@ -58,7 +58,7 @@ raised as HTTPError exceptions from the Requests library.
 
     # all objects are returned (by default) as a dict
     # lets see if i got diamond yet (i probably didnt)
-    my_ranked_stats = lol_watcher.league.by_summoner(my_region, me['id'])
+    my_ranked_stats = lol_watcher.league.by_puuid(my_region, me['puuid'])
     print(my_ranked_stats)
 
     # First we get the latest version of the game from data dragon

docs/index.rst

@@ -54,7 +54,7 @@ raised as HTTPError exceptions from the Requests library.
 
     # all objects are returned (by default) as a dict
     # lets see if i got diamond yet (i probably didnt)
-    my_ranked_stats = lol_watcher.league.by_summoner(my_region, me['id'])
+    my_ranked_stats = lol_watcher.league.by_puuid(my_region, me['puuid'])
     print(my_ranked_stats)
 
     # First we get the latest version of the game from data dragon

src/riotwatcher/_apis/league_of_legends/LeagueApiV4.py

@@ -95,6 +95,22 @@ def by_summoner(self, region: str, encrypted_summoner_id: str):
             encrypted_summoner_id=encrypted_summoner_id,
         )
 
+    def by_puuid(self, region: str, puuid: str):
+        """
+        Get league entries in all queues for a given puuid
+
+        :param string region:                   the region to execute this request on
+        :param string puuid:                    the puuid to query
+
+        :returns: Set[LeagueEntryDTO]
+        """
+        return self._request_endpoint(
+            self.by_puuid.__name__,
+            region,
+            LeagueApiV4Urls.by_puuid,
+            puuid=puuid,
+        )
+
     def entries(self, region: str, queue: str, tier: str, division: str, page: int = 1):
         """
         Get all the league entries

src/riotwatcher/_apis/league_of_legends/urls/LeagueApiUrls.py

@@ -13,4 +13,5 @@ class LeagueApiV4Urls:
     by_id = LeagueV4Endpoint("/leagues/{league_id}")
     master_by_queue = LeagueV4Endpoint("/masterleagues/by-queue/{queue}")
     by_summoner = LeagueV4Endpoint("/entries/by-summoner/{encrypted_summoner_id}")
+    by_puuid = LeagueV4Endpoint("/entries/by-puuid/{puuid}")
     entries = LeagueV4Endpoint("/entries/{queue}/{tier}/{division}", page=int)

tests/_apis/league_of_legends/test_LeagueApiV4.py

@@ -113,6 +113,27 @@ def test_by_summoner(self):
 
         assert ret is expected_return
 
+    def test_by_puuid(self):
+        mock_base_api = MagicMock()
+        expected_return = object()
+        mock_base_api.raw_request.return_value = expected_return
+
+        league = LeagueApiV4(mock_base_api)
+        region = "fdsga"
+        puuid = "puuid_1"
+
+        ret = league.by_puuid(region, puuid)
+
+        mock_base_api.raw_request.assert_called_once_with(
+            LeagueApiV4.__name__,
+            league.by_puuid.__name__,
+            region,
+            f"https://{region}.api.riotgames.com/lol/league/v4/entries/by-puuid/{puuid}",
+            {},
+        )
+
+        assert ret is expected_return
+
     def test_entries(self):
         mock_base_api = MagicMock()
         expected_return = object()

tests/integration/league_of_legends/test_LeagueApiV4.py

@@ -63,7 +63,10 @@ def test_by_id(self, lol_context, region, league_id):
         actual_response = lol_context.watcher.league.by_id(region, league_id)
 
         lol_context.verify_api_call(
-            region, f"/lol/league/v4/leagues/{league_id}", {}, actual_response,
+            region,
+            f"/lol/league/v4/leagues/{league_id}",
+            {},
+            actual_response,
         )
 
     @pytest.mark.parametrize(
@@ -82,6 +85,17 @@ def test_by_summoner(self, lol_context, region, encrypted_summoner_id):
             actual_response,
         )
 
+    @pytest.mark.parametrize("puuid", ["dQw4w9WgXcQ", "12093qowie"])
+    def test_by_puuid(self, lol_context, region, puuid):
+        actual_response = lol_context.watcher.league.by_puuid(region, puuid)
+
+        lol_context.verify_api_call(
+            region,
+            f"/lol/league/v4/entries/by-puuid/{puuid}",
+            {},
+            actual_response,
+        )
+
     @pytest.mark.parametrize("queue", queues)
     @pytest.mark.parametrize("tier", ["IRON", "SILVER", "DIAMOND"])
     @pytest.mark.parametrize("division", ["I", "IV"])