fix: 416 python hanging comma

[Alex-ley-scrub]

/claim #416
fixes #416
follows on from PR #419 and uses those test cases but makes the multi line example also work

verified fix with:

cargo test --package marzano-core --lib -- --exact --show-output
cargo test --package marzano-core --lib -- test::trailing_comma_after_argument_removal --exact --show-output
cargo test --package marzano-core --lib -- test::trailing_comma_after_argument_removal_one_line --exact --show-output

Greptile Summary

This is an auto-generated summary

Hi! Looks like you've reached your API usage limit. You can increase it from your account settings page here: app.greptile.com/settings/usage

[greptile-apps]

LGTM

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[morgante]

Do we need the identifier restriction?

[Alex-ley-scrub]

good point, actually we don't: dcfcf4d

that also passes all tests locally

I had gone down a few different rabbit holes / routes trying to make this work whilst ensuring all other tests passed as expected - one test that broke with one of my approaches was test_notebooks::test_insertion so I was trying to make it pretty strict but actually asserting that the previous node is empty is pretty strict and makes sense for this issue 👌

[morgante]

Perfect, looks like this passes everything. I agree that empty is probably the best check.