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

TestUtils simulate actions #44

Merged
merged 3 commits into from
Feb 25, 2015

Conversation

tappleby
Copy link
Contributor

This PR implements the TestUtils simulate api discussed in #35.

The API is implemented but the docs still need to be finished.

@tappleby tappleby mentioned this pull request Feb 25, 2015
@acdlite
Copy link
Owner

acdlite commented Feb 25, 2015

Looks great! Do you mind changing the indents to two spaces to match the rest of the repo?

@acdlite
Copy link
Owner

acdlite commented Feb 25, 2015

I think the file should be test-utils.js instead of testutils.js.

@tappleby
Copy link
Contributor Author

two spaces + test-utils.js have been pushed.

@acdlite acdlite merged commit bcbee58 into acdlite:master Feb 25, 2015
@acdlite
Copy link
Owner

acdlite commented Feb 25, 2015

Awesome! Merged and released in 2.11

@tappleby
Copy link
Contributor Author

The docs for the TestUtils are still just a shell, I wont be able to write anything for these until tomorrow night.

@acdlite
Copy link
Owner

acdlite commented Feb 25, 2015

No problem. Thanks.

@tappleby tappleby deleted the test-utils-simulate-actions branch February 26, 2015 01:19
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