-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Start of adding XSuite tests to Maven for XQSuite #291
base: master
Are you sure you want to change the base?
Conversation
There is also a bug in eXist-db's |
phew i know that @dizzzz has already done work in that area, so he might have some ideas. Looking at the mavenization of the docs in general i have a two feature wishes though.
This leads me to think that we could,
We could also adopt a similar approach to what I did with yeoman. If my local mocha can access the apps inside a running exist instance it executes the XQSuite tests, if it gets a The copy a an exist-db plugin seems more suited, and future-proof overall. |
see #333 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
point of caution exist-testkit
is not available since 4.7.0
Is this PR being worked on in the future or should we rather close it? |
85a2251
to
d5c9715
Compare
176b0e3
to
ca2ad67
Compare
@line-o Yes, it just requires a newer version of eXist-db first, and then some refactoring of the tests. |
DO NOT MERGE YET.
I made a start on having the XQSuite tests execute via XSuite with Maven.
There is a problem where XSuite is run outside of eXist-db, and an XQuery module imports another module, the resolution of the module fails. Unfortunately there is not a quick solution to this. XSuite assumes that all dependencies are available in eXist-db, but eXist-db does not know where to find the dependencies.
There are a number of options I can think of here:
Either way, it will take some time to write some Java code and test it.
My preference would likely be (1). This could be done either by:
conf.xml
which specify Xars which should be autodeployed at startup if they are not already available.