fix: quickfix deletes active functionality when removing unused variable

[GuilhermeConstantino]

Description
This was an issue that I opened in a library very_good_analysis, they sugested I opened it here.
Linter will remove function calls when made to a unused variable, even though the function itself might still be needed.

System used:
Windows.

Steps To Reproduce
Create a variable that receives the result of a function but is unused
Accept the quickfix
Expected Behavior
Future _postSomething(var something) {
final result = sendPostRequest(something);
}
Now since I didn't do anything with the result, it will cause a unused_local_variable linter
And when I select the "Remove unused local variable" quickfix it will delete the whole send, not just the variable.

Wanted result:
Future _postSomething(var something) {
sendPostRequest(something);
}

Actual result:
Future _postSomething(var something) {
}

Additional Context
The linter deleted some of my app functionality instead of just the variable.

[FMorschel]

Hey @GuilhermeConstantino, could you please confirm the Dart version you're using?

I just tested on the latest main with the following code, and everything works as you asked, only the code before foo(1); gets removed:

int foo(int p) => 0;
void f(str) {
  final removed = foo(1);
}

We have a similar test case here (but for methods) that was added more than a year ago:

sdk/pkg/analysis_server/test/src/services/correction/fix/remove_unused_local_variable_test.dart

Lines 115 to 128 in ba40d9b

	Future<void> test_assigned_inDeclaration() async {
	await resolveTestCode(r'''
	List<String> l = [];
	void f(str) {
	final removed = l.remove(str);
	}
	''');
	await assertHasFix(r'''
	List<String> l = [];
	void f(str) {
	l.remove(str);
	}
	''');
	}

I still opened https://dart-review.googlesource.com/c/sdk/+/426380 to add a test for functions specifically.

If you can give more information, it would be great! If you want to test on the latest stable Dart and don't have it installed, you can test in https://dartpad.dev/ with no need to install anything else.