Use the same shape to rewrite import when reordering them or not

[ytmimi]

Fixes #4794

The code paths for formatting imports diverge based on the value of reorder_imports. Each code path used a slightly different shape when rewriting the imports which lead to issues when reorder_imports = false and imports_indent = Visual.

Now the code path that rewrites imports without reordering uses the same shape as the path that also reorders imports.

[calebcartwright]

I think this is good, but could you add some test cases that have imports nested a few levels (e.g. imports within some in-line mod or other item)

[ytmimi]

could you add some test cases that have imports nested a few levels (e.g. imports within some in-line mod or other item)

@calebcartwright I added the test cases you were looking for. CI is still running right now, but assuming it passes I think we're good to go here!