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

Add Integration tests #343

Merged
merged 10 commits into from
May 21, 2019
Merged

Add Integration tests #343

merged 10 commits into from
May 21, 2019

Conversation

duncdrum
Copy link
Contributor

@duncdrum duncdrum commented May 17, 2019

This brings integrations tests to the docs, it also adds XQsuite execution to travis. Better integration with mvn TBD.

Fleshed out the readme with respect to testing.

closes #281
see eXist-db/generator-exist#253
see #291

@duncdrum duncdrum requested a review from a team May 17, 2019 19:55
@duncdrum duncdrum added this to the 4.1.3 milestone May 17, 2019
@@ -11,14 +11,16 @@ declare namespace test = "http://exist-db.org/xquery/xqsuite";

import module namespace docbook = "http://docbook.org/ns/docbook" at "docbook.xql";
import module namespace config = "http://exist-db.org/xquery/apps/config" at "config.xqm";
import module namespace diag = "http://exist-db.org/xquery/diagnostics" at "diagnostics.xql";
import module namespace rvds = "http://exist-db.org/xquery/rvd-support" at "rvd-support.xql";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rvds module appears to be unused?

@@ -0,0 +1,5 @@
{
"name": "Using fixtures to represent data",
"email": "[email protected]",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this email correct?

@adamretter adamretter merged commit 0e3b2b9 into eXist-db:master May 21, 2019
@duncdrum duncdrum deleted the int-test branch May 23, 2019 09:59
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 this pull request may close these issues.

3 participants