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 support for testing examples. #866

Merged
merged 2 commits into from
Jul 13, 2018
Merged

Add more support for testing examples. #866

merged 2 commits into from
Jul 13, 2018

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Jul 12, 2018

Prior to this, all example testing was done via image comparison, which is relatively heavy to add. This mimics the testing support in tutorials, where examples can test if their dom or internal state matches expected values. These are different tests than image comparison, but should facilitate adding basic tests to most examples.

Also, this now loads an example's javascript at the end of the body rather than at the end of the head. This allows skipping the onready callback.

Prior to this, all example testing was done via image comparison, which
is relatively heavy to add.  This mimics the testing support in
tutorials, where examples can test if their dom or internal state
matches expected values.  These are different tests than image
comparison, but should facilitate adding basic tests to most examples.

Also, this now loads an example's javascript at the end of the body
rather than at the end of the head.  This allows skipping the onready
callback.
@manthey
Copy link
Contributor Author

manthey commented Jul 12, 2018

This closes #329, as the infrastructure is fully in place.

@manthey manthey merged commit d24f8b3 into master Jul 13, 2018
@manthey manthey deleted the example-tests branch July 13, 2018 12:15
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.

2 participants