pkg/analysis_server/test/src/services/correction/fix/remove_method_declaration_test.dart

// Copyright (c) 2018, 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_reflective_loader/test_reflective_loader.dart';

import 'fix_processor.dart';

void main() {
  defineReflectiveSuite(() {
    defineReflectiveTests(RemoveMethodDeclarationBulkTest);
    defineReflectiveTests(RemoveMethodDeclarationTest);
  });
}

@reflectiveTest
class RemoveMethodDeclarationBulkTest extends BulkFixProcessorTest {
  @override
  String get lintCode => LintNames.unnecessary_overrides;

  Future<void> test_singleFile() async {
    await resolveTestCode('''
class A {
  int foo;
  int bar() => 0;
}

class B extends A {
  @override
  int get foo => super.foo;
  @override
  int bar() => super.bar();
}
''');
    await assertHasFix('''
class A {
  int foo;
  int bar() => 0;
}

class B extends A {
}
''');
  }
}

@reflectiveTest
class RemoveMethodDeclarationTest extends FixProcessorLintTest {
  @override
  FixKind get kind => DartFixKind.REMOVE_METHOD_DECLARATION;

  @override
  String get lintCode => LintNames.unnecessary_overrides;

  Future<void> test_getter() async {
    await resolveTestCode('''
class A {
  int foo = 0;
}

class B extends A {
  @override
  int get foo => super.foo;
}
''');
    await assertHasFix('''
class A {
  int foo = 0;
}

class B extends A {
}
''');
  }

  Future<void> test_method() async {
    await resolveTestCode('''
class A {
  int foo() => 0;
}

class B extends A {
  @override
  int foo() => super.foo();
}
''');
    await assertHasFix('''
class A {
  int foo() => 0;
}

class B extends A {
}
''');
  }

  @FailingTest(issue: 'https://github.com/dart-lang/linter/issues/1997')
  Future<void> test_method_generic() async {
    await resolveTestCode('''
class A<T> {
  T foo() {
    throw 42;
  }
}

class B extends A<int> {
  @override
  int foo() => super.foo();
}
''');
    await assertHasFix('''
class A<T> {
  T foo() {
    throw 42;
  }
}

class B extends A<int> {
}
''');
  }

  @FailingTest(issue: 'https://github.com/dart-lang/linter/issues/1997')
  Future<void> test_method_nullSafety_optIn_fromOptOut() async {
    createAnalysisOptionsFile(lints: [lintCode]);
    newFile('$testPackageLibPath/a.dart', content: r'''
class A {
  int foo() => 0;
}
''');
    await resolveTestCode('''
// @dart = 2.7
import 'a.dart';

class B extends A {
  @override
  int foo() => super.foo();
}
''');
    await assertHasFix('''
// @dart = 2.7
import 'a.dart';

class B extends A {
}
''');
  }

  Future<void> test_method_toString() async {
    await resolveTestCode('''
class A {
  @override
  String toString() => super.toString();
}
''');
    await assertHasFix('''
class A {
}
''');
  }

  @failingTest
  Future<void> test_setter() async {
    // The lint doesn't catch unnecessary setters.
    await resolveTestCode('''
class A {
  int foo;
}

class B extends A {
  @override
  set foo(int value) {
    super.foo = value;
  }
}
''');
    await assertHasFix('''
class A {
  int foo;
}

class B extends A {
}
''');
  }
}