blob: 4f2edff90a38d3814e51babcabfde258c1931e4a [file] [log] [blame]
// Copyright (c) 2023, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
import 'package:analysis_server/src/services/correction/fix.dart';
import 'package:analysis_server/src/services/linter/lint_names.dart';
import 'package:analyzer_plugin/utilities/fixes/fixes.dart';
import 'package:test/expect.dart';
import 'package:test_reflective_loader/test_reflective_loader.dart';
import 'fix_processor.dart';
void main() {
defineReflectiveSuite(() {
defineReflectiveTests(MakeRequiredNamedParametersFirstBulkTest);
defineReflectiveTests(MakeRequiredNamedParametersFirstTest);
defineReflectiveTests(MakeRequiredNamedParametersInFileTest);
});
}
@reflectiveTest
class MakeRequiredNamedParametersFirstBulkTest extends BulkFixProcessorTest {
@override
String get lintCode => LintNames.always_put_required_named_parameters_first;
Future<void> test_singleFile() async {
await resolveTestCode('''
void f({int? b, required int a}) {}
void g({int? b, required int a,}) {}
''');
await assertHasFix('''
void f({required int a, int? b}) {}
void g({required int a, int? b,}) {}
''');
}
}
@reflectiveTest
class MakeRequiredNamedParametersFirstTest extends FixProcessorLintTest {
@override
FixKind get kind => DartFixKind.MAKE_REQUIRED_NAMED_PARAMETERS_FIRST;
@override
String get lintCode => LintNames.always_put_required_named_parameters_first;
Future<void> test_comma() async {
await resolveTestCode('''
void f({int? b, required int a,}) {}
''');
await assertHasFix('''
void f({required int a, int? b,}) {}
''');
}
Future<void> test_comment() async {
await resolveTestCode('''
void f({int? b, /* aa */ required int a}) {}
''');
await assertHasFix('''
void f({/* aa */ required int a, int? b}) {}
''');
}
Future<void> test_comment_both() async {
await resolveTestCode('''
void f({/* bb */ int? b, /* aa */ required int a}) {}
''');
await assertHasFix('''
void f({/* aa */ required int a, /* bb */ int? b}) {}
''');
}
Future<void> test_comment_both_multiple() async {
await resolveTestCode('''
void f({/* b1 */ /* b2 */ int? b, /* a1 */ /* a2 */ required int a}) {}
''');
await assertHasFix('''
void f({/* a1 */ /* a2 */ required int a, /* b1 */ /* b2 */ int? b}) {}
''');
}
Future<void> test_comment_first() async {
await resolveTestCode('''
void f({/* bb */ int? b, required int a}) {}
''');
await assertHasFix('''
void f({required int a, /* bb */ int? b}) {}
''');
}
Future<void> test_comments() async {
await resolveTestCode('''
void f({int? a /* a */, required int b /* b1 */ /* b2 */}) {}
''');
await assertHasFix('''
void f({required int b /* b1 */ /* b2 */, int? a /* a */}) {}
''');
}
Future<void> test_single() async {
await resolveTestCode('''
void f({int? b, required int a}) {}
''');
await assertHasFix('''
void f({required int a, int? b}) {}
''');
}
}
@reflectiveTest
class MakeRequiredNamedParametersInFileTest extends FixInFileProcessorTest {
Future<void> test_file() async {
createAnalysisOptionsFile(
lints: [LintNames.always_put_required_named_parameters_first]);
await resolveTestCode(r'''
void f({int? c, required int a, required int b}) {}
''');
var fixes = await getFixesForFirstError();
expect(fixes, hasLength(1));
assertProduces(fixes.first, r'''
void f({required int a, required int b, int? c}) {}
''');
}
Future<void> test_multiple() async {
createAnalysisOptionsFile(
lints: [LintNames.always_put_required_named_parameters_first]);
await resolveTestCode(r'''
void f({required int a, int? d, required num b, required String c,}) {}
''');
var fixes = await getFixesForFirstError();
expect(fixes, hasLength(1));
assertProduces(fixes.first, r'''
void f({required int a, required num b, required String c, int? d,}) {}
''');
}
}