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

Test dumpDictionary not being called with an input HDT file #254

Closed
donpellegrino opened this issue Mar 13, 2022 · 0 comments · Fixed by #255
Closed

Test dumpDictionary not being called with an input HDT file #254

donpellegrino opened this issue Mar 13, 2022 · 0 comments · Fixed by #255

Comments

@donpellegrino
Copy link
Contributor

The make check test runner seems to call the dumpDictionary test case incorrectly. A required inputHDTFile CLI parameter is not specified:

FAIL: dumpDictionary
====================

ERROR: You must supply an input HDT File

$ dumpDictionary <inputHDTFile>
Process the input HDT file and generate <termURI>;EstimatedTriples;<rdf:label>

        -o                      Dump also objects

        -u                      Dump only URIs

        -p <exportPredicateFile>                        exportPredicateFile (outPred.txt by default)

        -t <exportTermsFile>            ^[xportTermsFile (outTerms.txt by default)

        -h                      This help

FAIL dumpDictionary (exit status: 1)

Manually running the command and passing in the .hdt from the ../data directory gives a reasonable result:

u133615@s001-n054:~/src/hdt-cpp/libhdt/tests$ ./dumpDictionary ../data/literals.hdt
Predicate Bitmap in 25 us
Count predicates in 5 us
Count Objects in 3 us Max was: 1
Bitmap in 6 us
Bitmap bits: 9 Ones: 9
Object references in 37 us
Sort lists in 7 us
Index generated in 159 us
p;9;
s;9;
donpellegrino pushed a commit to donpellegrino/hdt-cpp that referenced this issue Mar 13, 2022
… from the data directory under revision control instead of user-supplied input to support automated testing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant