PTH201 fix does not consider trailing comma

[InSyncWithFoo]

from pathlib import Path

# Before:
_ = Path(".",)
#        ^^^ PTH201

# After:
_ = Path(,)

This happens because the fix is a simple range deletion:

let [Expr::StringLiteral(ast::ExprStringLiteral { value, range })] = &*arguments.args else {
    return;
};

if matches!(value.to_str(), "" | ".") {
    let mut diagnostic = Diagnostic::new(PathConstructorCurrentDirectory, *range);
    diagnostic.set_fix(Fix::safe_edit(Edit::range_deletion(*range)));
    checker.diagnostics.push(diagnostic);
}

Perhaps it should instead replace arguments.range with ()?

[MichaReiser]

We could also just search for a trailing comma using the checker.tokens or the SimpleLexer. It might also be worth to see if there's an existing helper method for removing an argument

[dylwil3]

It might also be worth to see if there's an existing helper method for removing an argument

There is! It's here:

https://github.com/astral-sh/ruff/blob/main/crates/ruff_linter/src/fix/edits.rs#L207