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 test suite #123

Closed
dandv opened this issue Aug 16, 2012 · 6 comments
Closed

Add test suite #123

dandv opened this issue Aug 16, 2012 · 6 comments

Comments

@dandv
Copy link
Contributor

dandv commented Aug 16, 2012

Would a framework like Selenium be useful?

@hakimel
Copy link
Owner

hakimel commented Aug 17, 2012

Suppose it's probably time to introduce unit tests, been thinking about adding a build script too. Will give this some more thought. Main hesitation is quite simply that I want to keep the repo light and easy to navigate/understand for beginners.

@dandv
Copy link
Contributor Author

dandv commented Oct 15, 2012

Well, beginners just won't look at tests... been there, done that ;)

@owenversteeg
Copy link
Contributor

Agree with @dandv that beginners won't look at tests - just make them as separate as possible from the normal code and mention that the tests might not be good for beginners and you're good.

@hakimel
Copy link
Owner

hakimel commented Nov 24, 2012

Renamed from question to be actionable. Will add a test suite when there's time and integrate with the grunt.js build.

@dandv
Copy link
Contributor Author

dandv commented Jul 21, 2013

I plan to upgrade form a rather old RevealJS to the latest master. When I do so, I'll be creating a suite of manual tests, mostly regression. Maybe that will help seed the automated test suite?

@hakimel
Copy link
Owner

hakimel commented Aug 23, 2013

This has been added in to the dev branch, the tests run as part of the grunt build. More tests will be added over time.

@hakimel hakimel closed this as completed Aug 23, 2013
enovajuan pushed a commit to enovajuan/reveal.js that referenced this issue Dec 2, 2014
enovajuan pushed a commit to enovajuan/reveal.js that referenced this issue Dec 2, 2014
enovajuan pushed a commit to enovajuan/reveal.js that referenced this issue Dec 2, 2014
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

3 participants