PyCQA / redbaron

Bottom-up approach to refactoring in python
http://redbaron.pycqa.org/
694 stars 74 forks source link

ComprehensionLoopNode target/iterator attributes transposed #196

Open graingert opened 4 years ago

graingert commented 4 years ago

ComprehensionLoopNode target/iterator attributes transposed

actual

In [112]: RedBaron("[x for y in z]")[0].generators[0].help(deep=True)
ComprehensionLoopNode()
  # identifiers: comprehension_loop, comprehension_loop_, comprehensionloop, comprehensionloopnode
  iterator ->
    NameNode()
      # identifiers: name, name_, namenode
      value='y'
  target ->
    NameNode()
      # identifiers: name, name_, namenode
      value='z'
  ifs ->

expected

In [112]: RedBaron("[x for y in z]")[0].generators[0].help(deep=True)
ComprehensionLoopNode()
  # identifiers: comprehension_loop, comprehension_loop_, comprehensionloop, comprehensionloopnode
  target ->
    NameNode()
      # identifiers: name, name_, namenode
      value='y'
  iterator ->
    NameNode()
      # identifiers: name, name_, namenode
      value='z'
  ifs ->