Skip to content

Several improvements to fuchsia_ctl. - #134

Merged
godofredoc merged 4 commits into
flutter:masterfrom
godofredoc:timeouts
Apr 11, 2020
Merged

Several improvements to fuchsia_ctl.#134
godofredoc merged 4 commits into
flutter:masterfrom
godofredoc:timeouts

Conversation

@godofredoc

Copy link
Copy Markdown
Contributor
  • Adds timeouts for paving process.
  • Adds timeouts to ssh commands and tests.
  • Support to generate authorized_keys with existing pub key.
  • Add flags to pass public key to pave subcommand.

flutter/flutter#54416

* Adds timeouts for paving process.
* Adds timeouts to ssh commands and tests.
* Support to generate authorized_keys with existing pub key.
* Add flags to pass public key to pave subcommand.
flutter/flutter#54416
@godofredoc
godofredoc requested a review from dnfieldApril 11, 2020 01:33
@godofredoc

Copy link
Copy Markdown
ContributorAuthor

Fixes: flutter/flutter#54256

Comment threadpackages/fuchsia_ctl/bin/main.dart Outdated
defaultsTo: '.ssh/pkey', help: 'The key to use when SSHing.');
defaultsTo: '.ssh/pkey', help: 'The key to use when SSHing.')
..addOption('timeout-seconds',
defaultsTo: '120', help: 'Ssh command timeout in secs.');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: in seconds.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

Comment threadpackages/fuchsia_ctl/bin/main.dart Outdated
.addCommand('publishRepo')
.addMultiOption('far', abbr: 'f', help: 'The .far files to publish.');
pmSubCommand.addCommand('publishRepo')
..addMultiOption('far', abbr: 'f', help: 'The .far files to publish.');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit - why change this?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a second option for help command that I removed. Getting it back to "."

Comment threadpackages/fuchsia_ctl/bin/main.dart Outdated
final OperationResult result = await sshClient.runCommand(targetIp,
identityFilePath: identityFile,
command: args['command'].split(' '),
timeoutMs: int.parse(args['timeout-seconds']) * 1000);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This parameter should just be a Duration.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment threadpackages/fuchsia_ctl/bin/main.dart Outdated
..addOption('timeout-seconds',
defaultsTo: '120', help: 'Ssh command timeout in secs.');
parser.addCommand('pave')
..addOption('pubkey',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: we're not using abbreviations anywhere else for the full command name, just call this public-key.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

import 'tar.dart';

/// The default timeout in milliseconds for the pave command.
const int defaultPaveTimeoutMs = 5 * 60 * 1000;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just make this a duration.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

bool verbose = true,
}) async {
String imageTgzPath, String deviceName, String pubKeyPath,
{bool verbose = true, int timeoutMs = defaultPaveTimeoutMs}) async {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a comma after defaultPaveTimeoutMs, better formatting. And this should be a Duration rather than an int.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

'an unexpected device.');
}
final SshKeyManager sshKeyManager = sshKeyManagerProvider(
processManager: processManager, pubKeyPath: pubKeyPath, fs: fs);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma after fs.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

import 'operation_result.dart';

/// The default timeout in milliseconds for the ssh command.
const int defaultSshTimeoutMs = 5 * 60 * 1000;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe make this a static const member of SshClient and then it can just be dropped from image_paver.dart.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.


/// Function signature for a [SshKeyManager] provider.
typedef SshKeyManagerProvider = SshKeyManager Function(
{ProcessManager processManager, FileSystem fs, String pubKeyPath});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma after pubKeyPath. Also, avoid abbreviations - publicKeyPath.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

bool force = false,
});
Future<OperationResult> createKeys(
{String destinationPath = '.ssh', bool force = false});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma after last param

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

return SystemSshKeyManager(
processManager: processManager ?? const LocalProcessManager(),
fs: fs,
pkeyPubPath: pubKeyPath);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma after pubKeyPath. Also, make the parameter match - publicKeyPath.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

sshKeyManagerProvider: (
{ProcessManager processManager,
FileSystem fs,
String pubKeyPath}) =>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: comma after last parameter to make formatting better. Also, avoid using => for things that do not fit on a single line per Flutter style.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

final OperationResult result = await paver.pave(
'generic-x64.tgz',
deviceName,
null,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be a named parameter if it's sometimes not used.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

sshKeyManagerProvider: (
{ProcessManager processManager,
FileSystem fs,
String pubKeyPath}) =>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed


expect(
paver.pave('generic-x64.tgz', deviceName, null,
verbose: false, timeoutMs: 1),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma after last param for formatting.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

sshKeyManagerProvider: (
{ProcessManager processManager,
FileSystem fs,
String pubKeyPath}) =>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@dnfielddnfield left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly just nits, core of this change lgtm

@godofredoc
godofredoc merged commit 5859d77 into flutter:masterApr 11, 2020
stuartmorgan-g pushed a commit to stuartmorgan-g/packages that referenced this pull request Apr 30, 2021
stuartmorgan-g pushed a commit that referenced this pull request Oct 31, 2024
* Fix fill type for applyOp
* format
* fix incorrect tests
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
* Several improvements to fuchsia_ctl.
* Adds timeouts for paving process.
* Adds timeouts to ssh commands and tests.
* Support to generate authorized_keys with existing pub key.
* Add flags to pass public key to pave subcommand.
flutter/flutter#54416
* Link CLI flags to command implementations.
* Enables command help.
* Address code review comments.
chunhtai pushed a commit to chunhtai/packages that referenced this pull request Jun 12, 2026
…extMenuBuilder (#128114)
Close #128113 Fun fact: This is caught by monkey testing I have written (will soon be open sourced as well) that runs on my app!
Without the fix, the test fails as expected:
<details>
```
(base) � flutter git:(feat/text-field-npe) /Volumes/MyExternal/ExternalRefCode/flutter/bin/flutter test test/material/text_field_test.dart --name 'changes from default'
00:06 +0: context menu contextMenuBuilder changes from default to null ��� EXCEPTION CAUGHT BY WIDGETS LIBRARY ������������������������������������������������������������
The following _TypeError was thrown building
_OverlayEntryWidget-[LabeledGlobalKey<_OverlayEntryWidgetState>#e3717](state:
_OverlayEntryWidgetState#7666a):
Null check operator used on a null value
When the exception was thrown, this was the stack:
#0 EditableTextState._createSelectionOverlay.<anonymous closure> (package:flutter/src/widgets/editable_text.dart:3331:43)
flutter#1 SelectionOverlay.showToolbar.<anonymous closure> (package:flutter/src/widgets/text_selection.dart:1357:36)
flutter#2 ContextMenuController.show.<anonymous closure> (package:flutter/src/widgets/context_menu_controller.dart:65:54)
flutter#3 _OverlayEntryWidgetState.build (package:flutter/src/widgets/overlay.dart:351:36)
flutter#4 StatefulElement.build (package:flutter/src/widgets/framework.dart:5198:27)
flutter#5 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5086:15)
flutter#6 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#7 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#8 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#9 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#10 RenderObjectElement.updateChildren (package:flutter/src/widgets/framework.dart:6093:32)
flutter#11 MultiChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6595:17)
flutter#12 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#13 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#14 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#15 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#16 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#17 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#18 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#19 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#20 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#21 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#22 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#23 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#24 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#25 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#26 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#27 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#28 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#29 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#30 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#31 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#32 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#33 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#34 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#35 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#36 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#37 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#38 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#39 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#40 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#41 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#42 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#43 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#44 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#45 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#46 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#47 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#48 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#49 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#50 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#51 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#52 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#53 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#54 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#55 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#56 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#57 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#58 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#59 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#60 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#61 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#62 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#63 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#64 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#65 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#66 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#67 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#68 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#69 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#70 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#71 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#72 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#73 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#74 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#75 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#76 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#77 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#78 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#79 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#80 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#81 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#82 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#83 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#84 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#85 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#86 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#87 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#88 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#89 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#90 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#91 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#92 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#93 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#94 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#95 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#96 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#97 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#98 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#99 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#100 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#101 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#102 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#103 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#104 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#105 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#106 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#107 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#108 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#109 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#110 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#111 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#112 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#113 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#114 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#115 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#116 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#117 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#118 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#119 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#120 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#121 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#122 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#123 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#124 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#125 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#126 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#127 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#128 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#129 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#130 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#131 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#132 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#133 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#134 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#135 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#136 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#137 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#138 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#139 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#140 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#141 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#142 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#143 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#144 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#145 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#146 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#147 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#148 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#149 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#150 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#151 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#152 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#153 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#154 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#155 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#156 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#157 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#158 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#159 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#160 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#161 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#162 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#163 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#164 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#165 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#166 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#167 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#168 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#169 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#170 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#171 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#172 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#173 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#174 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#175 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#176 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#177 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#178 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#179 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#180 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#181 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#182 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#183 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#184 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#185 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#186 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#187 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#188 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#189 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#190 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#191 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#192 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#193 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#194 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#195 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#196 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#197 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#198 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#199 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#200 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#201 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#202 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#203 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#204 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#205 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#206 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#207 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#208 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#209 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#210 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#211 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#212 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#213 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#214 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#215 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#216 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#217 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#218 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#219 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#220 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#221 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#222 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#223 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#224 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#225 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#226 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#227 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#228 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#229 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#230 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#231 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#232 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#233 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#234 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#235 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#236 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#237 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#238 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#239 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#240 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#241 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#242 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#243 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#244 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#245 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#246 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#247 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#248 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#249 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#250 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#251 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#252 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#253 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#254 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#255 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#256 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#257 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#258 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#259 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#260 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#261 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#262 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#263 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#264 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#265 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#266 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#267 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#268 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#269 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#270 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#271 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#272 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#273 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#274 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#275 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#276 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#277 SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6442:14)
flutter#278 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#279 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#280 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#281 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#282 _InheritedNotifierElement.update (package:flutter/src/widgets/inherited_notifier.dart:107:11)
flutter#283 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#284 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#285 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#286 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#287 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#288 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#289 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#290 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#291 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#292 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#293 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#294 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#295 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#296 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#297 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#298 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#299 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#300 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#301 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#302 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#303 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#304 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#305 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#306 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#307 StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:5251:11)
flutter#308 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#309 StatefulElement.update (package:flutter/src/widgets/framework.dart:5274:5)
flutter#310 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#311 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#312 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#313 ProxyElement.update (package:flutter/src/widgets/framework.dart:5417:5)
flutter#314 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#315 ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5111:16)
flutter#316 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#317 StatelessElement.update (package:flutter/src/widgets/framework.dart:5162:5)
flutter#318 Element.updateChild (package:flutter/src/widgets/framework.dart:3686:15)
flutter#319 RenderObjectToWidgetElement._rebuild (package:flutter/src/widgets/binding.dart:1253:16)
flutter#320 RenderObjectToWidgetElement.update (package:flutter/src/widgets/binding.dart:1230:5)
flutter#321 RenderObjectToWidgetElement.performRebuild (package:flutter/src/widgets/binding.dart:1244:7)
flutter#322 Element.rebuild (package:flutter/src/widgets/framework.dart:4805:7)
flutter#323 BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2780:19)
flutter#324 AutomatedTestWidgetsFlutterBinding.drawFrame (package:flutter_test/src/binding.dart:1396:19)
flutter#325 RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:358:5)
flutter#326 SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1297:15)
flutter#327 SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1227:9)
flutter#328 AutomatedTestWidgetsFlutterBinding.pump.<anonymous closure> (package:flutter_test/src/binding.dart:1246:9)
flutter#331 TestAsyncUtils.guard (package:flutter_test/src/test_async_utils.dart:68:41)
flutter#332 AutomatedTestWidgetsFlutterBinding.pump (package:flutter_test/src/binding.dart:1232:27)
flutter#333 WidgetTester._pumpWidget (package:flutter_test/src/widget_tester.dart:587:20)
flutter#334 WidgetTester.pumpWidget.<anonymous closure> (package:flutter_test/src/widget_tester.dart:572:14)
flutter#337 TestAsyncUtils.guard (package:flutter_test/src/test_async_utils.dart:68:41)
flutter#338 WidgetTester.pumpWidget (package:flutter_test/src/widget_tester.dart:571:27)
flutter#339 main.<anonymous closure>.<anonymous closure> (file:///Volumes/MyExternal/ExternalRefCode/flutter/packages/flutter/test/material/text_field_test.dart:15687:20)
<asynchronous suspension>
<asynchronous suspension>
(elided 5 frames from dart:async and package:stack_trace)
����������������������������������������������������������������������������������������������������
00:06 +0 -1: context menu contextMenuBuilder changes from default to null [E] Test failed. See exception logs above.
The test description was: contextMenuBuilder changes from default to null
To run this test again: /Volumes/MyExternal/ExternalRefCode/flutter/bin/cache/dart-sdk/bin/dart test /Volumes/MyExternal/ExternalRefCode/flutter/packages/flutter/test/material/text_field_test.dart -p vm --plain-name 'context menu contextMenuBuilder changes from default to null'
00:06 +0 -1: Some tests failed. (base) � flutter git:(feat/text-field-npe) ```
</details>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@godofredoc@dnfield