remove testUsingContext (#70482)
diff --git a/packages/flutter_tools/lib/src/mdns_discovery.dart b/packages/flutter_tools/lib/src/mdns_discovery.dart
index ac76abd..695ccb8 100644
--- a/packages/flutter_tools/lib/src/mdns_discovery.dart
+++ b/packages/flutter_tools/lib/src/mdns_discovery.dart
@@ -8,24 +8,30 @@
import 'base/common.dart';
import 'base/context.dart';
import 'base/io.dart';
+import 'base/logger.dart';
import 'build_info.dart';
import 'device.dart';
-import 'globals.dart' as globals;
import 'reporting/reporting.dart';
/// A wrapper around [MDnsClient] to find a Dart observatory instance.
class MDnsObservatoryDiscovery {
/// Creates a new [MDnsObservatoryDiscovery] object.
///
- /// The [client] parameter will be defaulted to a new [MDnsClient] if null.
+ /// The [_client] parameter will be defaulted to a new [MDnsClient] if null.
/// The [applicationId] parameter may be null, and can be used to
/// automatically select which application to use if multiple are advertising
/// Dart observatory ports.
- MDnsObservatoryDiscovery({MDnsClient mdnsClient})
- : client = mdnsClient ?? MDnsClient();
+ MDnsObservatoryDiscovery({
+ MDnsClient mdnsClient,
+ @required Logger logger,
+ @required Usage flutterUsage,
+ }): _client = mdnsClient ?? MDnsClient(),
+ _logger = logger,
+ _flutterUsage = flutterUsage;
- /// The [MDnsClient] used to do a lookup.
- final MDnsClient client;
+ final MDnsClient _client;
+ final Logger _logger;
+ final Usage _flutterUsage;
@visibleForTesting
static const String dartObservatoryName = '_dartobservatory._tcp.local';
@@ -53,16 +59,16 @@
// TODO(jonahwilliams): use `deviceVmservicePort` to filter mdns results.
@visibleForTesting
Future<MDnsObservatoryDiscoveryResult> query({String applicationId, int deviceVmservicePort}) async {
- globals.printTrace('Checking for advertised Dart observatories...');
+ _logger.printTrace('Checking for advertised Dart observatories...');
try {
- await client.start();
- final List<PtrResourceRecord> pointerRecords = await client
+ await _client.start();
+ final List<PtrResourceRecord> pointerRecords = await _client
.lookup<PtrResourceRecord>(
ResourceRecordQuery.serverPointer(dartObservatoryName),
)
.toList();
if (pointerRecords.isEmpty) {
- globals.printTrace('No pointer records found.');
+ _logger.printTrace('No pointer records found.');
return null;
}
// We have no guarantee that we won't get multiple hits from the same
@@ -94,9 +100,9 @@
} else {
domainName = pointerRecords[0].domainName;
}
- globals.printTrace('Checking for available port on $domainName');
+ _logger.printTrace('Checking for available port on $domainName');
// Here, if we get more than one, it should just be a duplicate.
- final List<SrvResourceRecord> srv = await client
+ final List<SrvResourceRecord> srv = await _client
.lookup<SrvResourceRecord>(
ResourceRecordQuery.service(domainName),
)
@@ -105,11 +111,11 @@
return null;
}
if (srv.length > 1) {
- globals.printError('Unexpectedly found more than one observatory report for $domainName '
+ _logger.printError('Unexpectedly found more than one observatory report for $domainName '
'- using first one (${srv.first.port}).');
}
- globals.printTrace('Checking for authentication code for $domainName');
- final List<TxtResourceRecord> txt = await client
+ _logger.printTrace('Checking for authentication code for $domainName');
+ final List<TxtResourceRecord> txt = await _client
.lookup<TxtResourceRecord>(
ResourceRecordQuery.text(domainName),
)
@@ -133,7 +139,7 @@
}
return MDnsObservatoryDiscoveryResult(srv.first.port, authCode);
} finally {
- client.stop();
+ _client.stop();
}
}
@@ -166,14 +172,14 @@
// If there's not an ipv4 link local address in `NetworkInterfaces.list`,
// then request user interventions with a `printError()` if possible.
Future<void> _checkForIPv4LinkLocal(Device device) async {
- globals.printTrace(
+ _logger.printTrace(
'mDNS query failed. Checking for an interface with a ipv4 link local address.'
);
final List<NetworkInterface> interfaces = await listNetworkInterfaces(
includeLinkLocal: true,
type: InternetAddressType.IPv4,
);
- if (globals.logger.isVerbose) {
+ if (_logger.isVerbose) {
_logInterfaces(interfaces);
}
final bool hasIPv4LinkLocal = interfaces.any(
@@ -182,14 +188,14 @@
),
);
if (hasIPv4LinkLocal) {
- globals.printTrace('An interface with an ipv4 link local address was found.');
+ _logger.printTrace('An interface with an ipv4 link local address was found.');
return;
}
final TargetPlatform targetPlatform = await device.targetPlatform;
switch (targetPlatform) {
case TargetPlatform.ios:
- UsageEvent('ios-mdns', 'no-ipv4-link-local', flutterUsage: globals.flutterUsage).send();
- globals.printError(
+ UsageEvent('ios-mdns', 'no-ipv4-link-local', flutterUsage: _flutterUsage).send();
+ _logger.printError(
'The mDNS query for an attached iOS device failed. It may '
'be necessary to disable the "Personal Hotspot" on the device, and '
'to ensure that the "Disable unless needed" setting is unchecked '
@@ -198,18 +204,18 @@
);
break;
default:
- globals.printTrace('No interface with an ipv4 link local address was found.');
+ _logger.printTrace('No interface with an ipv4 link local address was found.');
break;
}
}
void _logInterfaces(List<NetworkInterface> interfaces) {
for (final NetworkInterface interface in interfaces) {
- if (globals.logger.isVerbose) {
- globals.printTrace('Found interface "${interface.name}":');
+ if (_logger.isVerbose) {
+ _logger.printTrace('Found interface "${interface.name}":');
for (final InternetAddress address in interface.addresses) {
final String linkLocal = address.isLinkLocal ? 'link local' : '';
- globals.printTrace('\tBound address: "${address.address}" $linkLocal');
+ _logger.printTrace('\tBound address: "${address.address}" $linkLocal');
}
}
}
