Unverified Commit 0a66d8a7 authored by Alexandre Ardhuin's avatar Alexandre Ardhuin Committed by GitHub

implicit-casts:false in fuchsia_remote_debug_protocol (#45239)

parent e2cf2f0f
# Override the parent analysis_options until all the repo has implicit-casts: false
include: ../analysis_options.yaml
analyzer:
strong-mode:
implicit-casts: false
implicit-dynamic: false
linter:
rules:
avoid_as: false
...@@ -171,7 +171,7 @@ class DartVm { ...@@ -171,7 +171,7 @@ class DartVm {
await invokeRpc('getVM', timeout: timeout); await invokeRpc('getVM', timeout: timeout);
final List<IsolateRef> result = <IsolateRef>[]; final List<IsolateRef> result = <IsolateRef>[];
for (Map<String, dynamic> jsonIsolate in jsonVmRef['isolates']) { for (Map<String, dynamic> jsonIsolate in jsonVmRef['isolates']) {
final String name = jsonIsolate['name']; final String name = jsonIsolate['name'] as String;
if (pattern.matchAsPrefix(name) != null) { if (pattern.matchAsPrefix(name) != null) {
_log.fine('Found Isolate matching "$pattern": "$name"'); _log.fine('Found Isolate matching "$pattern": "$name"');
result.add(IsolateRef._fromJson(jsonIsolate, this)); result.add(IsolateRef._fromJson(jsonIsolate, this));
...@@ -197,7 +197,7 @@ class DartVm { ...@@ -197,7 +197,7 @@ class DartVm {
'Peer connection timed out during RPC call', 'Peer connection timed out during RPC call',
timeout, timeout,
); );
}); }) as Map<String, dynamic>;
return result; return result;
} }
...@@ -243,11 +243,11 @@ class FlutterView { ...@@ -243,11 +243,11 @@ class FlutterView {
/// All other cases return a [FlutterView] instance. The name of the /// All other cases return a [FlutterView] instance. The name of the
/// view may be null, but the id will always be set. /// view may be null, but the id will always be set.
factory FlutterView._fromJson(Map<String, dynamic> json) { factory FlutterView._fromJson(Map<String, dynamic> json) {
final Map<String, dynamic> isolate = json['isolate']; final Map<String, dynamic> isolate = json['isolate'] as Map<String, dynamic>;
final String id = json['id']; final String id = json['id'] as String;
String name; String name;
if (isolate != null) { if (isolate != null) {
name = isolate['name']; name = isolate['name'] as String;
if (name == null) { if (name == null) {
throw RpcFormatError('Unable to find name for isolate "$isolate"'); throw RpcFormatError('Unable to find name for isolate "$isolate"');
} }
...@@ -286,9 +286,9 @@ class IsolateRef { ...@@ -286,9 +286,9 @@ class IsolateRef {
IsolateRef._(this.name, this.number, this.dartVm); IsolateRef._(this.name, this.number, this.dartVm);
factory IsolateRef._fromJson(Map<String, dynamic> json, DartVm dartVm) { factory IsolateRef._fromJson(Map<String, dynamic> json, DartVm dartVm) {
final String number = json['number']; final String number = json['number'] as String;
final String name = json['name']; final String name = json['name'] as String;
final String type = json['type']; final String type = json['type'] as String;
if (type == null) { if (type == null) {
throw RpcFormatError('Unable to find type within JSON "$json"'); throw RpcFormatError('Unable to find type within JSON "$json"');
} }
......
...@@ -99,6 +99,6 @@ class SshCommandRunner { ...@@ -99,6 +99,6 @@ class SshCommandRunner {
'Command failed: $command\nstdout: ${result.stdout}\nstderr: ${result.stderr}'); 'Command failed: $command\nstdout: ${result.stdout}\nstderr: ${result.stderr}');
} }
_log.fine('SSH command stdout in brackets:[${result.stdout}]'); _log.fine('SSH command stdout in brackets:[${result.stdout}]');
return result.stdout.split('\n'); return (result.stdout as String).split('\n');
} }
} }
...@@ -53,11 +53,11 @@ void main() { ...@@ -53,11 +53,11 @@ void main() {
interface: interface, interface: interface,
sshConfigPath: '/whatever', sshConfigPath: '/whatever',
); );
when<String>(mockProcessResult.stdout).thenReturn('somestuff'); when<dynamic>(mockProcessResult.stdout).thenReturn('somestuff');
when(mockProcessResult.exitCode).thenReturn(0); when(mockProcessResult.exitCode).thenReturn(0);
await runner.run('ls /whatever'); await runner.run('ls /whatever');
final List<String> passedCommand = final List<String> passedCommand =
verify(mockProcessManager.run(captureAny)).captured.single; verify(mockProcessManager.run(captureAny)).captured.single as List<String>;
expect(passedCommand, contains('$ipV6Addr%$interface')); expect(passedCommand, contains('$ipV6Addr%$interface'));
}); });
...@@ -67,11 +67,11 @@ void main() { ...@@ -67,11 +67,11 @@ void main() {
mockProcessManager, mockProcessManager,
address: ipV6Addr, address: ipV6Addr,
); );
when<String>(mockProcessResult.stdout).thenReturn('somestuff'); when<dynamic>(mockProcessResult.stdout).thenReturn('somestuff');
when(mockProcessResult.exitCode).thenReturn(0); when(mockProcessResult.exitCode).thenReturn(0);
await runner.run('ls /whatever'); await runner.run('ls /whatever');
final List<String> passedCommand = final List<String> passedCommand =
verify(mockProcessManager.run(captureAny)).captured.single; verify(mockProcessManager.run(captureAny)).captured.single as List<String>;
expect(passedCommand, contains(ipV6Addr)); expect(passedCommand, contains(ipV6Addr));
}); });
...@@ -79,7 +79,7 @@ void main() { ...@@ -79,7 +79,7 @@ void main() {
const String addr = '192.168.1.1'; const String addr = '192.168.1.1';
runner = SshCommandRunner.withProcessManager(mockProcessManager, runner = SshCommandRunner.withProcessManager(mockProcessManager,
address: addr); address: addr);
when<String>(mockProcessResult.stdout).thenReturn('''this when<dynamic>(mockProcessResult.stdout).thenReturn('''this
has has
four four
lines'''); lines''');
...@@ -92,7 +92,7 @@ void main() { ...@@ -92,7 +92,7 @@ void main() {
const String addr = '192.168.1.1'; const String addr = '192.168.1.1';
runner = SshCommandRunner.withProcessManager(mockProcessManager, runner = SshCommandRunner.withProcessManager(mockProcessManager,
address: addr); address: addr);
when<String>(mockProcessResult.stdout).thenReturn('whatever'); when<dynamic>(mockProcessResult.stdout).thenReturn('whatever');
when(mockProcessResult.exitCode).thenReturn(1); when(mockProcessResult.exitCode).thenReturn(1);
Future<void> failingFunction() async { Future<void> failingFunction() async {
await runner.run('oihaw'); await runner.run('oihaw');
...@@ -109,11 +109,11 @@ void main() { ...@@ -109,11 +109,11 @@ void main() {
address: addr, address: addr,
sshConfigPath: config, sshConfigPath: config,
); );
when<String>(mockProcessResult.stdout).thenReturn('somestuff'); when<dynamic>(mockProcessResult.stdout).thenReturn('somestuff');
when(mockProcessResult.exitCode).thenReturn(0); when(mockProcessResult.exitCode).thenReturn(0);
await runner.run('ls /whatever'); await runner.run('ls /whatever');
final List<String> passedCommand = final List<String> passedCommand =
verify(mockProcessManager.run(captureAny)).captured.single; verify(mockProcessManager.run(captureAny)).captured.single as List<String>;
expect(passedCommand, contains('-F')); expect(passedCommand, contains('-F'));
final int indexOfFlag = passedCommand.indexOf('-F'); final int indexOfFlag = passedCommand.indexOf('-F');
final String passedConfig = passedCommand[indexOfFlag + 1]; final String passedConfig = passedCommand[indexOfFlag + 1];
...@@ -126,11 +126,11 @@ void main() { ...@@ -126,11 +126,11 @@ void main() {
mockProcessManager, mockProcessManager,
address: addr, address: addr,
); );
when<String>(mockProcessResult.stdout).thenReturn('somestuff'); when<dynamic>(mockProcessResult.stdout).thenReturn('somestuff');
when(mockProcessResult.exitCode).thenReturn(0); when(mockProcessResult.exitCode).thenReturn(0);
await runner.run('ls /whatever'); await runner.run('ls /whatever');
final List<String> passedCommand = final List<String> passedCommand =
verify(mockProcessManager.run(captureAny)).captured.single; verify(mockProcessManager.run(captureAny)).captured.single as List<String>;
final int indexOfFlag = passedCommand.indexOf('-F'); final int indexOfFlag = passedCommand.indexOf('-F');
expect(indexOfFlag, equals(-1)); expect(indexOfFlag, equals(-1));
}); });
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment