My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 607: Continuing for loop row works differently in RF 2.5 than in earlier versions
  Back to list
Status:  Done
Owner:  janne.t....@gmail.com
Closed:  Aug 2010
Cc:  pekka.klarck

Blocked on:
issue 609


Sign in to add a comment
 
Project Member Reported by pekka.klarck, Aug 4, 2010
***Test Case***
Example
  [Documentation]  This test should fail for syntax error
  :FOR  ${x}  IN  item1
  \  ...  item2  # but this line is ignored and thus the test passes
  \  Log  ${x}


The same occurs also with FOR IN RANGE loop and in dry-run mode.
Aug 11, 2010
Project Member #1 janne.t....@gmail.com
With Robot Framework 2.1.3, the above For loop is executed with both item1 and item2, i.e. it is parsed.
Aug 11, 2010
Project Member #2 pekka.klarck
Because the original examples works with RF 2.1.3 it should not be a syntax error in RF 2.5(.x) either. Changing parsing so that there can be empty cells before `...`, as suggested in  issue 609 , would also fix this issue.
Summary: Continuing for loops works differently in RF 2.5 than in earlier versions
Blockedon: 609
Aug 11, 2010
Project Member #3 pekka.klarck
(No comment was entered for this change.)
Summary: Continuing for loop row works differently in RF 2.5 than in earlier versions
Aug 17, 2010
Project Member #4 pekka.klarck
(No comment was entered for this change.)
Status: Started
Owner: pekka.klarck
Aug 19, 2010
Project Member #5 pekka.klarck
(No comment was entered for this change.)
Owner: janne.t.harkonen
Cc: pekka.klarck
Aug 23, 2010
Project Member #6 janne.t....@gmail.com
(No comment was entered for this change.)
Status: Review
Owner: pekka.klarck
Cc: -pekka.klarck janne.t.harkonen
Aug 23, 2010
Project Member #7 pekka.klarck
Empty cells are now generally allowed before `...` ( issue 609 ) so this bug is fixed.
Status: Done
Owner: janne.t.harkonen
Cc: -janne.t.harkonen pekka.klarck
Aug 27, 2010
Project Member #8 pekka.klarck
(No comment was entered for this change.)
Labels: -Priority-Medium Priority-Low
Sign in to add a comment

Powered by Google Project Hosting