migrate `avoid_returning_null` nnbd experiment test
Fixes: https://github.com/dart-lang/linter/issues/4726
Change-Id: Ifbb056c046be24d1f153662ea6bbf66334ab2ca9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/323222
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Phil Quitslund <pquitslund@google.com>
diff --git a/pkg/linter/test/rules/all.dart b/pkg/linter/test/rules/all.dart
index 5c490b6..5e0a13c 100644
--- a/pkg/linter/test/rules/all.dart
+++ b/pkg/linter/test/rules/all.dart
@@ -25,6 +25,7 @@
as avoid_renaming_method_parameters;
import 'avoid_return_types_on_setters_test.dart'
as avoid_return_types_on_setters;
+import 'avoid_returning_null_test.dart' as avoid_returning_null;
import 'avoid_returning_this_test.dart' as avoid_returning_this;
import 'avoid_setters_without_getters_test.dart'
as avoid_setters_without_getters;
@@ -242,6 +243,7 @@
avoid_relative_lib_imports.main();
avoid_renaming_method_parameters.main();
avoid_return_types_on_setters.main();
+ avoid_returning_null.main();
avoid_returning_this.main();
avoid_setters_without_getters.main();
avoid_shadowing_type_parameters.main();
diff --git a/pkg/linter/test/rules/avoid_returning_null_test.dart b/pkg/linter/test/rules/avoid_returning_null_test.dart
new file mode 100644
index 0000000..0e79d54
--- /dev/null
+++ b/pkg/linter/test/rules/avoid_returning_null_test.dart
@@ -0,0 +1,26 @@
+// 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:test_reflective_loader/test_reflective_loader.dart';
+
+import '../rule_test_support.dart';
+
+main() {
+ defineReflectiveSuite(() {
+ defineReflectiveTests(AvoidReturningNullTest);
+ });
+}
+
+@reflectiveTest
+class AvoidReturningNullTest extends LintRuleTest {
+ @override
+ String get lintRule => 'avoid_returning_null';
+
+ /// https://github.com/dart-lang/linter/issues/2636
+ test_nullableValue() async {
+ await assertNoDiagnostics(r'''
+ int? getFoo() => null;
+''');
+ }
+}
diff --git a/pkg/linter/test_data/rules/experiments/nnbd/rules/avoid_returning_null.dart b/pkg/linter/test_data/rules/experiments/nnbd/rules/avoid_returning_null.dart
deleted file mode 100644
index 4f14c70..0000000
--- a/pkg/linter/test_data/rules/experiments/nnbd/rules/avoid_returning_null.dart
+++ /dev/null
@@ -1,9 +0,0 @@
-// Copyright (c) 2021, 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.
-
-// test w/ `dart test -N avoid_returning_null`
-
-/// See: https://github.com/dart-lang/linter/issues/2636
-
-int? getFoo() => null; // OK
