boakley / robotframework-lint

Linter for robot framework plain text files
Apache License 2.0
126 stars 48 forks source link

KeywordRule: steps table not contain step, if line contains # #41

Closed RiikkaJokinen closed 7 years ago

RiikkaJokinen commented 7 years ago

--configure TooFewKeywordSteps:1

This keyword returns TooFewKeywordSteps warning

My second keyword
    My first keyword    # call My first keyword

When I print out keyword.steps, it returns empy list (= [])

RiikkaJokinen commented 7 years ago

Same issue in TooFewTestSteps rule

This test returns TooFewTestSteps warning

My test
    Step 1    # step 1
    Step 2    # step 2
    Step 3    # step 2
    Step 4    # step 4
boakley commented 7 years ago

Thanks for the report! I've found the bug and will try to post a fix soon.

On Tue, May 30, 2017 at 3:24 AM, RiikkaJokinen notifications@github.com wrote:

Same issue in TooFewTestSteps rule

This test returns TooFewTestSteps warning

My test Step 1 # step 1 Step 2 # step 2 Step 3 # step 2 Step 4 # step 4

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/boakley/robotframework-lint/issues/41#issuecomment-304809015, or mute the thread https://github.com/notifications/unsubscribe-auth/ABEmYubtd2BH92uwRsRXiobSbQyNN_jrks5r-9IlgaJpZM4Np4FN .