-
Notifications
You must be signed in to change notification settings - Fork 0
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
Integration of XProcSpec testing #7
Comments
All the linked xprocspec issues should be fixed now:
There's no issue for the "custom comparator could return more detailed result". I'm not sure exactly what that entails so please create an issue for it which describes it in more detail. I suppose the issues "html-report" (daisy/xproc-maven-plugin#3) and "more logging" (daisy/xproc-maven-plugin#5) are non-essential for v1.8. But we need the ability to provide a custom XML catalog (daisy/xproc-maven-plugin#2) before enabling xprocspec tests in all modules by default, because otherwise we can't perform unit tests. I've created a separate issue for integration tests, but that's probably a post-1.8 feature (daisy/xproc-maven-plugin#7). What remains after all this is ready? Do we add xproc-maven-plugin to the modules-parent pom? Does this depend on #8 being finished first? |
I'm on daisy/xproc-maven-plugin#2, daisy/xproc-maven-plugin#3 and daisy/xproc-maven-plugin#5. I'm starting with that ASAP. I don't think we depend on #8. |
@bertfrees: great! :) |
The memory issue is still not solved by the way. The issue is there both when using Pax-Exam and when using xproc-maven-plugin with the catalog.xml feature. |
XProcSpec testing s/b enabled by default in a module's build, via the
xproc-maven-plugin
Notes from the Paris f2f
this issue can be fixed in fileset-utils)
(see c:errors does not contain a c:error in p:catch ndw/xmlcalabash1#97)
The text was updated successfully, but these errors were encountered: