All committers have signed the CLA.

Pull Request Test Coverage Report for Build 4399142398

• 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

Totals
Change from base Build 4384977037:	0.0%
Covered Lines:	5876
Relevant Lines:	5876

💛 - Coveralls

@jeswr, @rubensworks can I have some feedback.

I think the adjustment is basically fine:

However I wanted my tests to fail before the adjustment (in addition to covering the new code).
But they pass.

In the test, I'm trying to set up a source with 'weird' quads (e.g. with a literal subject) that would not be empty when queried with a malformed QPF selector. This is so I can prove they are empty when malformed selectors are ignored.

Thanks for the PR @langsamu!

As mentioned in #1163 (comment), I would prefer a more general approach, as the same problem could also occur for other source types.
So basically, the same logic could be applied in https://github.com/comunica/comunica/blob/master/packages/actor-query-operation-quadpattern/lib/ActorQueryOperationQuadpattern.ts

To still allow the old mode for people relying on the current behaviour, we'll also have to introduce a generalizedRdf context option (can default to false).

Closing this PR due to staleness.
Feel free to re-open if there are any updates! 🙂