-
Notifications
You must be signed in to change notification settings - Fork 27.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate more tool unit tests to null safety #80002
Conversation
ee012fb
to
6b7bc59
Compare
@@ -118,7 +118,7 @@ class AppContext { | |||
if (value == null && _parent != null) { | |||
value = _parent!.get<T>(); | |||
} | |||
return _unboxNull(value ?? _generateIfNecessary(T, _fallbacks)) as T; | |||
return _unboxNull(value ?? _generateIfNecessary(T, _fallbacks)) as T?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also in #79985, let's see which one merges first.
@@ -48,7 +48,7 @@ bool _inUnitTest() { | |||
/// | |||
/// The specification for the language version tag is defined at: | |||
/// https://github.com/dart-lang/language/blob/master/accepted/future-releases/language-versioning/feature-specification.md#individual-library-language-version-override | |||
LanguageVersion determineLanguageVersion(File file, Package package, String flutterRoot) { | |||
LanguageVersion determineLanguageVersion(File file, Package? package, String flutterRoot) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only actual non-test change.
@@ -9,7 +9,7 @@ import 'package:flutter_tools/src/doctor_validator.dart'; | |||
import 'package:flutter_tools/src/linux/linux_doctor.dart'; | |||
|
|||
import '../../src/common.dart'; | |||
import '../../src/context.dart'; | |||
import '../../src/fake_process_manager.dart'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Getting ready to migrate fake_process_manager
.
6b7bc59
to
e1d3944
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Part of #71511