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

Finishing the test coverage for everything that can reasonably be tested #7

Merged
merged 22 commits into from
Sep 12, 2014

Conversation

mboynes
Copy link
Contributor

@mboynes mboynes commented Sep 5, 2014

No description provided.

ob_start();
SP_Admin()->sync();
ob_end_clean();
} catch ( WPDieException $e ) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can assert specific exceptions in PHPDoc now!

Also, this whole test is commented-out. Assume you know that.

@netaustin
Copy link

Is that fake cron mine? Big fan. 🐴

@mboynes
Copy link
Contributor Author

mboynes commented Sep 6, 2014

Yep, that's your fake cron! The only working WordPress cron I've seen in a unit test.

mboynes added a commit that referenced this pull request Sep 12, 2014
Finishing the test coverage for everything that can reasonably be tested
@mboynes mboynes merged commit 7228e88 into master Sep 12, 2014
@mboynes mboynes deleted the more-tests branch September 12, 2014 03: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