Fix tests:

- Fix test due to using deprecated rdflib ConjunctiveGraph instead of Dataset
- Comment out 3 tests that were targeting endpoints that are not available anymore. Apart for http://zbw.eu/beta/sparql/stw/query that errors on POST queries but works on GET, might need to be investigated
- Fix CLI tests failing due to slightly different result in response, now checking the length of results bindings, this will be more reliable in the future and is a good marker for a successful query
- Fix tests for error message by checking if the error is in the message (instead of checking only the end of the message)
- Added pytest and using it to run the tests. It is a more modern and reliable solution that works with the current code. unittest had a bug with the 3 tests where warnings are caught (when running each test alone, they work, but when running all with unittest catching the warnings stops working, but it works with pytest). I fixed the bug (so now all tests works with pytest and unittest, but the trust in unittest is lost). Fixes #192
- Added python 3.11 and 3.12 to the github actions tests

Author: vemonet

.github/workflows/test.yml

@@ -13,11 +13,11 @@ jobs:
         fail-fast: false
         max-parallel: 1
         matrix:
-            python-version: ["3.7", "3.8", "3.9", "3.10"]
+            python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"]
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v5
       with:
         python-version: ${{ matrix.python-version }}
     - name: Install dependencies
@@ -26,7 +26,7 @@ jobs:
         pip install '.[dev]'
     - name: Mypy
       run: |
-        mypy --show-error-context --show-error-codes
+        mypy --show-error-context
     - name: Test [wrapper]
       if: ${{ always() }}
       run: |

.gitignore

@@ -11,3 +11,4 @@ doc
 .settings/
 .venv/
 .tox/
+.coverage

README.rst

@@ -35,13 +35,13 @@ You can install SPARQLWrapper from GitHub::
 You can install SPARQLWrapper from Debian::
 
    $ sudo apt-get install python-sparqlwrapper
-   
+
 .. note::
 
    Be aware that there could be a gap between the latest version of SPARQLWrapper
    and the version available as Debian package.
 
-Also, the source code of the package can be downloaded 
+Also, the source code of the package can be downloaded
 in ``.zip`` and ``.tar.gz`` formats from `GitHub SPARQLWrapper releases <https://github.com/RDFLib/sparqlwrapper/releases>`_.
 Documentation is included in the distribution.
 
@@ -125,9 +125,9 @@ This query gets a boolean response from DBPedia's SPARQL endpoint:
 
    sparql = SPARQLWrapper("http://dbpedia.org/sparql")
    sparql.setQuery("""
-       ASK WHERE { 
+       ASK WHERE {
            <http://dbpedia.org/resource/Asturias> rdfs:label "Asturias"@es
-       }    
+       }
    """)
    sparql.setReturnFormat(XML)
    results = sparql.query().convert()
@@ -307,7 +307,7 @@ Return formats
 
 The expected return formats differs per query type (``SELECT``, ``ASK``, ``CONSTRUCT``, ``DESCRIBE``...).
 
-.. note:: From the `SPARQL specification <https://www.w3.org/TR/sparql11-protocol/#query-success>`_, 
+.. note:: From the `SPARQL specification <https://www.w3.org/TR/sparql11-protocol/#query-success>`_,
   *The response body of a successful query operation with a 2XX response is either:*
 
   * ``SELECT`` and ``ASK``: a SPARQL Results Document in XML, JSON, or CSV/TSV format.
@@ -430,12 +430,12 @@ in the return value). This features becomes particularly useful when the ``OPTIO
 GET or POST
 ^^^^^^^^^^^
 
-By default, all SPARQL services are invoked using HTTP **GET** verb. However, 
+By default, all SPARQL services are invoked using HTTP **GET** verb. However,
 **POST** might be useful if the size of the query
 extends a reasonable size; this can be set in the query instance.
 
 Note that some combinations may not work yet with all SPARQL processors
-(e.g., there are implementations where **POST + JSON return** does not work). 
+(e.g., there are implementations where **POST + JSON return** does not work).
 Hopefully, this problem will eventually disappear.
 
 
@@ -914,8 +914,7 @@ Organizations involved:
 * `Salzburg Research <http://www.salzburgresearch.at>`_
 * `Foundation CTIC <http://www.fundacionctic.org/>`_
 
-
-.. |Build Status| image:: https://secure.travis-ci.org/RDFLib/sparqlwrapper.svg?branch=master
-   :target: https://travis-ci.org/RDFLib/sparqlwrapper
+.. |Build Status| image:: https://github.com/RDFLib/sparqlwrapper/actions/workflows/test.yml/badge.svg
+   :target: https://github.com/RDFLib/sparqlwrapper/actions/workflows/test.yml
 .. |PyPi version| image:: https://badge.fury.io/py/SPARQLWrapper.svg
    :target: https://pypi.python.org/pypi/SPARQLWrapper

SPARQLWrapper/Wrapper.py

@@ -39,7 +39,7 @@
 from SPARQLWrapper import __agent__
 
 if TYPE_CHECKING:
-    from rdflib import Graph
+    from rdflib import Dataset, Graph
 
 
 
@@ -649,7 +649,7 @@ def setUseKeepAlive(self) -> None:
         :raises ImportError: when could not be imported ``keepalive.HTTPHandler``.
         """
         try:
-            from keepalive import HTTPHandler  # type: ignore[import]
+            from keepalive import HTTPHandler  # type: ignore[import-not-found]
 
             if urllib.request._opener and any(  # type: ignore[attr-defined]
                 isinstance(h, HTTPHandler) for h in urllib.request._opener.handlers  # type: ignore[attr-defined]
@@ -1071,21 +1071,21 @@ def _convertXML(self) -> Document:
         :return: converted result.
         :rtype: :class:`xml.dom.minidom.Document`
         """
-        doc = parse(self.response)
+        doc: Any = parse(self.response)
         rdoc = cast(Document, doc)
         return rdoc
 
-    def _convertRDF(self) -> "Graph":
+    def _convertRDF(self) -> "Dataset":
         """
         Convert a RDF/XML result into an RDFLib Graph. This method can be overwritten
         in a subclass for a different conversion method.
 
         :return: converted result.
         :rtype: :class:`rdflib.graph.Graph`
         """
-        from rdflib import ConjunctiveGraph
-        retval = ConjunctiveGraph()
-        retval.parse(self.response, format="xml")  # type: ignore[no-untyped-call]
+        from rdflib import Dataset
+        retval = Dataset()
+        retval.parse(self.response, format="xml")
         return retval
 
     def _convertN3(self) -> bytes:
@@ -1118,18 +1118,18 @@ def _convertTSV(self) -> bytes:
         """
         return self.response.read()
 
-    def _convertJSONLD(self) -> "Graph":
+    def _convertJSONLD(self) -> "Dataset":
         """
         Convert a RDF JSON-LD result into an RDFLib Graph. This method can be overwritten
         in a subclass for a different conversion method.
 
         :return: converted result
         :rtype: :class:`rdflib.graph.Graph`
         """
-        from rdflib import ConjunctiveGraph
+        from rdflib import Dataset
 
-        retval = ConjunctiveGraph()
-        retval.parse(self.response, format="json-ld")  # type: ignore[no-untyped-call]
+        retval = Dataset()
+        retval.parse(self.response, format="json-ld")
         return retval
 
     def convert(self) -> ConvertResult:

SPARQLWrapper/main.py

@@ -145,7 +145,7 @@ def main(test: Optional[List[str]] = None) -> None:
     elif isinstance(results, bytes):
         # "csv", "tsv", "turtle", "n3"
         print(results.decode("utf-8"))
-    elif isinstance(results, rdflib.graph.ConjunctiveGraph):
+    elif isinstance(results, rdflib.graph.Dataset):
         # "rdf"
         print(results.serialize())
     else:

SPARQLWrapper/sparql_dataframe.py

@@ -58,7 +58,7 @@ def get_sparql_typed_dict(
         row = {}
         for k in x:
             v = x[k]
-            vv = rdflib.term.Literal(v.value, datatype=v.datatype).toPython()  # type: ignore[no-untyped-call]
+            vv = rdflib.term.Literal(v.value, datatype=v.datatype).toPython()
             row[k] = vv
         d.append(row)
     return d

pyproject.toml

@@ -1,3 +1,10 @@
 [build-system]
 requires = ["wheel", "setuptools"]
 build-backend = "setuptools.build_meta"
+
+[tool.pytest.ini_options]
+addopts = [
+    "--cov=SPARQLWrapper",
+    "--cov-report=term-missing",
+    "--durations=10", # Show 10 slowest tests durations
+]

setup.cfg

@@ -49,6 +49,8 @@ dev =
     mypy>=0.931
     pandas>=1.3.5
     pandas-stubs>=1.2.0.48
+    pytest>=8.3.5
+    pytest-cov>=6.1.1
 
 pandas =
     pandas>=1.3.5

test/test.rq

@@ -5,5 +5,5 @@ WHERE {
   ?pl
     a dbo:ProgrammingLanguage;
     rdfs:label ?pllabel.
-  FILTER(SUBSTR(STR(?pllabel), 1, 1) = "P")
+  FILTER(SUBSTR(STR(?pllabel), 1, 5) = "PARLO")
 } ORDER BY ?pllabel LIMIT 1