-
Notifications
You must be signed in to change notification settings - Fork 31
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
check for @graph keyword in manifest #25
Comments
Interesting... would it be possible to add your manifest as a test fixture with accompanying test? https://github.com/viewdir/manifesto/tree/master/test/fixtures Maybe just testing that it loads correctly. This would also ensure it doesn't break any other tests. |
@edsilv thank you for the reply. I added a test that checks if it loads correctly, and that resources can be read (I reused getManifestType() and getSequenceByIndex(0) ). See lines 117-124 in this file. All other tests still pass with the change. |
See UniversalViewer/universalviewer#385
The text was updated successfully, but these errors were encountered: