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 more XSpec based tests #3

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

Add more XSpec based tests #3

wants to merge 9 commits into from

Conversation

dmj
Copy link
Member

@dmj dmj commented Sep 18, 2018

No description provided.

dmj and others added 4 commits September 18, 2018 10:41
This is a compiled Schematron test. Commited by accident.
This is a compiled test Schematron. Committed by accident.
@tgraham-antenna
Copy link
Member

Thanks for those. Sorry, but I haven't looked at them yet. The one that I did run had 3/4 that passed, but I was running it to make sure that XSpec worked on this machine, so I didn't make a note of it.

What I have done is add Travis to 'Schematron/schematron' and 'Schematron/schematron-test'. In theory, if you rebase and push your branch, Travis should run schematron.bats from 'Schematron/schematron-test'. Can you try that?

The second half of the theory is that you should be able to add your new tests to schematron.bats and have Travis run them when you push to your branch. Can you try that?

All this is currently using the Schematron that is baked into XSpec 1.0.0. I haven't yet got it to the point of being able to use the Schematron from HEAD.

dmj and others added 5 commits October 22, 2018 15:34
This is a compiled Schematron test. Commited by accident.
This is a compiled test Schematron. Committed by accident.
@dmj
Copy link
Member Author

dmj commented Oct 25, 2018

The failing tests are all but one due to bugs in the Schematron implementation:

Schematron/schematron#29
Schematron/schematron#69
Schematron/schematron#70
Schematron/schematron#71

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants