🚑 Fixes remove url API call in AdGuard 0.96

Signed-off-by: Franck Nijhof <frenck@addons.community>
frenck · Jun 11, 2019 · aa06bf0 · aa06bf0
1 parent a70e555
commit aa06bf0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/adguardhome/filtering.py b/adguardhome/filtering.py
@@ -56,7 +56,7 @@ async def add_url(self, name: str, url: str) -> bool:
     async def remove_url(self, url: str) -> bool:
         """Remove a new filter subscription from AdGuard Home."""
         response = await self._adguard._request(
-            "filtering/remove_url", method="POST", data="url={}".format(url)
+            "filtering/remove_url", method="POST", json_data={"url": url}
         )
         if response.rstrip() != "OK":
             raise AdGuardHomeError(

diff --git a/tests/test_filtering.py b/tests/test_filtering.py
@@ -149,8 +149,8 @@ async def test_remove_url(event_loop, aresponses):
     """Test remove filter subscription from AdGuard Home filtering."""
     # Handle to run asserts on request in
     async def response_handler(request):
-        data = await request.text()
-        assert data == "url=https://example.com/1.txt"
+        data = await request.json()
+        assert data == {"url": "https://example.com/1.txt"}
         return aresponses.Response(status=200, text="OK")
 
     aresponses.add(