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

Adds support for Cedar-style +beforeEach and +afterEach class methods #21

Closed
wants to merge 7 commits into from

Conversation

nerdyc
Copy link
Member

@nerdyc nerdyc commented Jun 12, 2013

I've found Cedar's global beforeEach and afterEach methods really useful. They allow me to encapsulate global test setup related to a particular class inside a category. I've missed this when using specta, so I implemented it in this pull request.

Here's the relevant Cedar info:

https://github.com/pivotal/cedar#global-beforeeach-and-aftereach

@nerdyc
Copy link
Member Author

nerdyc commented Jun 12, 2013

Oops! I also developed some Xcode templates, which I should have put on a branch and submitted separately. LMK if you want me to break this up, or if you're good with both commits.

@petejkim
Copy link
Member

I will take a look at the commits as soon as I get some free time, thanks!

I mistakenly tried to map a test class to a file on disk. This was inherently flawed since test files will only exist on the desktop.
…of it() or describe(). When specs are focused, all unfocused specs are skipped.
@nerdyc
Copy link
Member Author

nerdyc commented Jun 18, 2013

I've continued to work on new features, in particular:

  • New output format that's easier to read, while maintaining the stuff Xcode needs for its integration
  • Report pending specs in the output, and support Cedar's PENDING macro.
  • Focused specs, allowing a subset of specs to be run.

I'd like to keep this on my master branch, so if you want to integrate any of this into Specta main, let me know and I can split this into more appropriate feature branches.

@petejkim
Copy link
Member

merged

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.

None yet

2 participants