firka/flutter
firka/flutter
4
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Update objectToDiagnosticsNode to stop failing. (#129027)

Browse Source 
Prerequisite for https://github.com/flutter/devtools/pull/5918
This commit is contained in:
Polina Cherkasova 2023-06-16 12:39:54 -07:00 committed by GitHub
parent 3f73d2c484
commit 9690394772
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
packages/flutter/lib/src/widgets/widget_inspector.dart
View File

@ -1764,17 +1764,20 @@ mixin WidgetInspectorService {
// TODO(polina-c): start always assuming Diagnosticable, when DevTools stops sending DiagnosticsNode to
// APIs that invoke this method.
// https://github.com/flutter/devtools/issues/3951
final Object? theObject = toObject(diagnosticsOrDiagnosticableId);
if (theObject is DiagnosticsNode) {
return theObject;
final Object? object = toObject(diagnosticsOrDiagnosticableId);
return objectToDiagnosticsNode(object);
}
/// If posiible, returns [DiagnosticsNode] for the object.
@visibleForTesting
static DiagnosticsNode? objectToDiagnosticsNode(Object? object) {
if (object is DiagnosticsNode) {
return object;
}
if (theObject is Diagnosticable) {
return theObject.toDiagnosticsNode();
if (object is Diagnosticable) {
return object.toDiagnosticsNode();
}
if (theObject == null) {
return null;
}
throw StateError('Unexpected object type ${theObject.runtimeType}.');
return null;
}
List<Object> _getChildrenSummaryTree(String? diagnosticsOrDiagnosticableId, String groupName) {

4
packages/flutter/test/widgets/widget_inspector_test.dart
View File

@ -261,6 +261,10 @@ class _TestWidgetInspectorService extends TestWidgetInspectorService {
}
});
test ('objectToDiagnosticsNode returns null for non-diagnosticable', () {
expect(WidgetInspectorService.objectToDiagnosticsNode(Alignment.bottomCenter), isNull);
});
testWidgets('WidgetInspector smoke test', (WidgetTester tester) async {
// This is a smoke test to verify that adding the inspector doesn't crash.
await tester.pumpWidget(