Skip to content

Sqlite3 with tests #11160

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Sqlite3 with tests #11160

wants to merge 4 commits into from

Conversation

groleo
Copy link
Contributor

@groleo groleo commented Sep 27, 2024

I'm adding a way to check the sqlite3 database against the data stored as XML.

Rather than generating XML from sqlite3, I'm using jsonML as a common denominator,
which is easier to handle (see http://www.jsonml.org/)

I've added a (incomplete) jsonML doc-visitor which replaces the xmldocvisitor in sqlite3gen.
The XML generated by runtests.py is then converted to jsonML and used as reference.

Comparing the two jsonML dumps will give the test result.

Edit:
I fixed the warning related error on MSVC but the build still fails: D:\a\doxygen\doxygen\src\sqlite3gen.cpp(1708): fatal error C1021: invalid preprocessor command 'warning'
I guess force-pushed branches are not properly picked up by the CI ?

How to test:

cd doxygen
cmake -B __build .
cmake --build __build -j 3

mkdir -p __test_out
python testing/runtests.py --id 001 \
--xmlxsd \
--keep \
--sqlite3 \
--noredir \
--doxygen __build/bin/doxygen \
--inputdir testing \
--outputdir __test_out

Adrian Negreanu added 4 commits September 27, 2024 20:04
jsonML is used as a common ground when testing.
runtest.py will dump the .db as a jsonML
and also convert the XML to jsonML.

In the end, the two jsonML output must match.
this makes it possible to dump the XML files outside of the source
directory
@albert-github albert-github added the enhancement a request to enhance doxygen, not a bug label Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement a request to enhance doxygen, not a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants