Skip to content

Commit 1288f0e

Browse files
committed
fixes more tests
1 parent 7f16a3a commit 1288f0e

File tree

4 files changed

+9
-5
lines changed

4 files changed

+9
-5
lines changed

snapquery/qlever.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,11 @@ def read_query(self) -> str:
3030
Returns:
3131
str: The SPARQL query extracted from the short URL.
3232
"""
33-
self.fetch_final_url()
33+
self.fetch_final_url(self.short_url)
3434
if self.url:
3535
try:
36-
response = requests.get(self.url)
37-
response.raise_for_status()
38-
soup = BeautifulSoup(response.content, "html.parser")
36+
text=self.get_wikitext(self.url)
37+
soup = BeautifulSoup(text, "html.parser")
3938
query_element = soup.find("textarea", {"id": "query"})
4039
if query_element and query_element.text:
4140
self.sparql = query_element.text.strip()

snapquery/sparql_analyzer.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
"""
2+
Created on 2024-05-03
3+
@author: tholzheim
4+
"""
15
import logging
26
import random
37
import re

tests/test_sparql_analyzer.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ def test_transform_with_clause_to_subquery(self):
190190
self.assertTrue(SparqlAnalyzer.has_blazegraph_with_clause(query))
191191
self.assertFalse(SparqlAnalyzer.has_blazegraph_with_clause(expected_query))
192192
fixed_query = SparqlAnalyzer.transform_with_clause_to_subquery(query)
193+
self.maxDiff=None
193194
self.assertEqual(expected_query, fixed_query)
194195
self.assertFalse(SparqlAnalyzer.is_valid(query))
195196
self.assertTrue(fixed_query)

tests/test_wd_query_parsing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class TestWikidataQueryParsing(Basetest):
1818
Wiki pages (both via Short URL resolution and direct template parsing).
1919
"""
2020

21-
def setUp(self, debug=True, profile=True):
21+
def setUp(self, debug=False, profile=True):
2222
"""
2323
Set up the test environment.
2424
Initializes a temporary NamedQueryManager and two extractor instances:

0 commit comments

Comments
 (0)