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

manifest validator suppresses firing assertions/successful reports #597

Open
wschidol opened this issue Jul 7, 2022 · 0 comments
Open

Comments

@wschidol
Copy link
Contributor

wschidol commented Jul 7, 2022

Some errors in the manifest file are flagged up by the schematron code, but not surfaced to the UI. This is because only failed statements are displayed, not successful statements.

Attached see an MPD in ex62.zip which does trigger the report "@MimeType shall be common between all Representations in an Adaptation Set" but that report is not shown.

The solution is to either also surface successful reports, or to rewrite all reports into asserts. Even though the former would be the smaller change, I tend to the latter which seems cleaner, stylistically. It could also be confusing to the end user to read "successful report" when in fact something is broken in the manifest.

@wschidol wschidol changed the title manifest validator suppressed firing assertions manifest validator suppresses firing assertions/successful reports Jul 11, 2022
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

No branches or pull requests

1 participant