Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: two issues with the N3 serializer #1858

Merged

Commits on May 12, 2022

  1. fix: two issues with the N3 serializer

    This patch fixes two issues with the N3 serializer:
    - The N3 serializer incorrectly considered a subject as already
      serialized if it has been serialized inside a quoted graph.
    - The N3 serializer does not consider that the predicate of
      a triple can also be a graph.
    
    Other changes included in this patch:
    - Changed `test.testutils.GraphHelper` to support nested/quoted graphs.
    - Moved the tests from `test/test_n3_formula.py` into
      `test/test_serializers/test_serializer_n3.py`.
    - Include positive syntax tests from the N3 test suite that is smaller
      than 1024KB and that is not using new N3 syntax into round trip tests.
      This is mainly to check that there is no regressions after the changes
      made.
    
    Fixes:
    - RDFLib#1807
    - RDFLib#1701
    aucampia committed May 12, 2022
    Configuration menu
    Copy the full SHA
    f38979e View commit details
    Browse the repository at this point in the history