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

Unit test failures with Fuseki 3.10 snapshot #828

Closed
osma opened this issue Dec 14, 2018 · 3 comments
Closed

Unit test failures with Fuseki 3.10 snapshot #828

osma opened this issue Dec 14, 2018 · 3 comments
Milestone

Comments

@osma
Copy link
Member

osma commented Dec 14, 2018

The Travis builds using Fuseki snapshots have all failed recently. In particular, these tests are failing:

1) VocabularyTest::testSearchConceptsAlphabetical
Failed asserting that actual size 0 matches expected size 2.

/var/www/html/Skosmos/tests/VocabularyTest.php:240

2) VocabularyTest::testSearchConceptsAlphabeticalIncludingDeprecated
Failed asserting that actual size 0 matches expected size 1.

/var/www/html/Skosmos/tests/VocabularyTest.php:260

An example failed build job

This should be investigated, it looks like a regression in Jena (most likely jena-text) but could be something else too.

@osma osma added this to the 2.2 milestone Dec 14, 2018
@osma
Copy link
Member Author

osma commented Dec 14, 2018

It has something to do with the Analyzer in jena-text. This part of the query gives no results when run on the groups test vocabulary which has concepts with the prefLabels Guppy and Grouped fish:

(?s ?score ?match) text:query (skos:prefLabel 'G*' 100000 'lang:en*')

but this one works:

(?s ?score ?match) text:query (skos:prefLabel 'g*' 100000 'lang:en*')

Note the difference in case - G vs g. The assembler configuration specifies the use of LowerCaseKeywordAnalyzer which should fold query strings into lower case - at least it used to in Jena 3.9.0 and earlier.

Next step would be creating a minimal test case and filing a Jena issue.

@osma
Copy link
Member Author

osma commented Dec 14, 2018

Reported as JENA-1652

@osma
Copy link
Member Author

osma commented Dec 17, 2018

JENA-1652 was fixed by @xristy and now the tests pass again: https://travis-ci.org/NatLibFi/Skosmos/jobs/466985759

@osma osma closed this as completed Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant