-
Notifications
You must be signed in to change notification settings - Fork 48
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
add rule for too many steps in test case #9
Conversation
I'd like to include this rule, but I want it to be configurable. (see Configurable Rules on the wiki). I'll be glad to make it configurable if you want, or you can do it yourself. All you need to do is add a configure method that takes a string as a parameter and sets the maximum number allowed. You can see the otherRules.py:LineTooLong rule for an example. Sorry it took so long for me to respond. |
No problem. I'll make the necessary changes. |
@boakley have you pushed the support for configurable rules to github yet? I can't find it in the project source, other than the reference you linked on the wiki. |
My bad. I didn't realize I had a bunch of stuff I hadn't pushed. It should On Wed, Jan 14, 2015 at 4:19 PM, Guy Kisel [email protected] wrote:
|
Cool, thanks! |
Conflicts: tests/acceptance/self-test.robot
@boakley I've implemented configurableness. If it looks to you like I've done it correctly, I'll do the same for my other rule PR. |
Cool. I'll try to take a look at it this weekend. On Fri, Jan 16, 2015 at 3:25 PM, Guy Kisel [email protected] wrote:
|
add configurable rule for too many steps in test case
Based on https://code.google.com/p/robotframework/wiki/HowToWriteGoodTestCases#Workflow_tests
"Max 10 steps, preferably less"
Example output: