-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fewer but bigger acceptance tests #24
Comments
Is this issue already done? I reviwed them and all I could do was create a background for similar test cases for a single feature, the tests themselves look pretty complete. Should I keep going this way? |
No. This issue is about refactoring the existing acceptance tests. Now, prezento has lots of acceptance tests, basically one for each single feature and usage scenario. The idea here is to rewrite the existing tests into bigger ones. Like testing the user creation, reading group creation, configuration creation, project creation, repository creation and processing results into a single and large acceptance test for example. This will save us the time of creation of objects for other test cases and kalibro resets. The example above is just an exaggerated example. Don't take it too seriously :) |
I wonder if the issue relates to what I am doing at the link below: I am reducing the number of scenarios, making scenarios with similar steps =) 2014-12-04 14:14 GMT-02:00 Rafael Reggiani Manzo [email protected]:
Larissa Rodrigues. |
Yes that's the spirit! |
Furthermore we should rethink our acceptance tests in the way that some behaviour can be tested on the unit level. For example, acceptance tests that check just for UI features can be extracted into view specs (https://www.relishapp.com/rspec/rspec-rails/v/3-0/docs/view-specs/view-spec). |
b5b20a6 merged two scenarios into one. |
Now that we have a solid acceptance test coverage and the features are stable, would be nice to minimize the repeated steps.
One way to do that is by writing big scenarios that tests many features.
The text was updated successfully, but these errors were encountered: