Bug: pipes with an argument produce incorrect output

[kybishop]

Here is a test case:

assert_style("c(a() |> b, foo)", "a() |> b() |> c(foo)")

Output (while messing with a local copy):

  1) test big picture unnests multiple steps that have args (Styler.Style.PipesTest)
     test/style/pipes_test.exs:19
     Styling produced unexpected results
     expected:
       a() |> b() |> c(foo)
     styled:
       b(a()) |> c(foo)

     code: assert_style("c(a() |> b, foo)", "a() |> b() |> c(foo)")
     stacktrace:
       test/style/pipes_test.exs:20: (test)

If one changes the result to the current output you get a expected styling to be idempotent, but a second pass resulted in more changes., which does actually produce the correct results, so it seems like there's some missing recursion somewhere.

[novaugust]

hey @kybishop, thanks for the issue and for even writing the test for me ;) you're definitely on point with styler just needing to recurse when it unpipes here.

[novaugust]

sigh, i somehow got the issue # very confused in the commit message, but this was closed by 9404d5f

thanks again!