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

The test suite should no longer be a bottleneck #104

Closed
gonzalo-bulnes opened this issue Sep 21, 2014 · 0 comments
Closed

The test suite should no longer be a bottleneck #104

gonzalo-bulnes opened this issue Sep 21, 2014 · 0 comments

Comments

@gonzalo-bulnes
Copy link
Owner

As a developer
In order to ensure Simple Token Authentication is stable
I want its features to be properly tested

But, while the current test suite performs very well in preventing regressions, it runs (very) slowly, and writing new scenarios is not always trivial. Therefore, testing properly new features uses to take more time than available. One way to work around that time bottleneck is to make the tests suite faster, another is to make contributions easier. Both solutions require in fact pretty much the same preparation.

In order to make the tests suite faster
I want the context of each test to be easier to set, that's to say:
I want unit tests to replace integration tests where possible

In order to make unit testing possible
I want to improve the Simple Token Authentication modules design through refactoring
And I need their interfaces to be properly tested and documented

In order to make contributions easier
And to ensure contributors build on the same interfaces as I do
I want the module both public and private interfaces to be documented
And I want their tests suites to be easy to use and extend

See also: the issues needs testing tag.

gonzalo-bulnes added a commit that referenced this issue Oct 20, 2014
gonzalo-bulnes added a commit that referenced this issue Oct 20, 2014
I also forgot them in #104.
gonzalo-bulnes added a commit that referenced this issue Oct 24, 2014
Fix circular dependency (!)
Minor fix harmless variables shadowing in dynamically defined methods

Remove forgotten hack related to the deprecated Cucumber scenarios
See #104
This was referenced Oct 24, 2014
gonzalo-bulnes added a commit that referenced this issue Nov 9, 2014
I forgot to remove it when I removed a deprecated Cucumber hack,
see 21ee5ed and #104
gonzalo-bulnes added a commit that referenced this issue Nov 27, 2014
I forgot to remove it when I removed a deprecated Cucumber hack,
see 21ee5ed and #104
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant