Skip to content

Conversation

@wschidol
Copy link
Contributor

Only the assert messages are parsed out of the schematron output, so
all report messages were effectively muted. The PR converts reports into asserts, negating the test assertion. This should fix #721 .

Note that this will likely unmask further bugs which will need to be fixed subsequently (not part of this PR).

  Only the assert messages are parsed out of the schematron output, so
  all report messages were effectively muted. Fix this.
@Phencys
Copy link
Collaborator

Phencys commented May 27, 2025

@wschidol Both this and related PR #723 seem correct to me, however I am unable to actually reproduce the mentioned issues with the PHP code.

Could you maybe share how to run the schematron separately, as I am unable to find coherent documentation for this anywhere?

@wschidol
Copy link
Contributor Author

create_val_schema.sh contains command lines to convert schematron code into XSLT, a three step process involving XSLT transforms to create an XSLT transform val_schema.xsl out of the input schematron.sch:

java -jar ../saxon9he.jar -versionmsg:off -s:schematron.sch -o:tmp/new_schema1.sch -xsl:schematron/iso_dsdl_include.xsl
java -jar ../saxon9he.jar -versionmsg:off -s:tmp/new_schema1.sch -o:tmp/new_schema2.sch -xsl:schematron/iso_abstract_expand.xsl
java -jar ../saxon9he.jar -versionmsg:off -s:tmp/new_schema2.sch -o:output/val_schema.xsl -xsl:schematron/iso_svrl_for_xslt2.xsl

Then, run_samples.sh shows how to run the XSLT transform on a file:

java -jar ../saxon9he.jar -versionmsg:off -s:manifest-to-test.mpd -o:results.xml -xsl:output/val_schema.xsl

Another option is if you have access to OxygenXML which has schematron support built-in.

@Phencys
Copy link
Collaborator

Phencys commented May 27, 2025

@wschidol Thanks for sharing the instructions. I will also add them to the wiki.

I will get this merged now :)

@Phencys Phencys merged commit 8a8838a into Dash-Industry-Forum:development May 27, 2025
2 checks passed
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.

Failing manifest tests not reported in web frontend

2 participants