Skip to content

Commit a27771f

Browse files
committed
Cleanup all the debug stuff
1 parent 4528931 commit a27771f

File tree

3 files changed

+30
-74
lines changed

3 files changed

+30
-74
lines changed

.github/workflows/dart.yml

Lines changed: 13 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dwds/test/fixtures/context.dart

Lines changed: 16 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -154,14 +154,12 @@ class TestContext {
154154

155155
DartUri.currentDirectory = project.absolutePackageDirectory;
156156

157-
void log(String s) {
158-
_logger.info('${DateTime.now()}: $s');
159-
}
160-
161-
log('Serving: ${project.directoryToServe}/${project.filePathToServe}');
162-
log('Project: ${project.absolutePackageDirectory}');
163-
log('Packages: ${project.packageConfigFile}');
164-
log('Entry: ${project.dartEntryFilePath}');
157+
_logger.info(
158+
'Serving: ${project.directoryToServe}/${project.filePathToServe}',
159+
);
160+
_logger.info('Project: ${project.absolutePackageDirectory}');
161+
_logger.info('Packages: ${project.packageConfigFile}');
162+
_logger.info('Entry: ${project.dartEntryFilePath}');
165163

166164
configureLogWriter();
167165

@@ -212,13 +210,13 @@ class TestContext {
212210
line.contains('was started successfully')) {
213211
chromeDriverStartup.complete();
214212
}
215-
log('ChromeDriver stdout: $line');
213+
_logger.finest('ChromeDriver stdout: $line');
216214
});
217-
stdErrLines.listen((line) => log('ChromeDriver stderr: $line'));
215+
stdErrLines.listen(
216+
(line) => _logger.warning('ChromeDriver stderr: $line'),
217+
);
218218

219-
await stdOutLines.first;
220219
await chromeDriverStartup.future;
221-
log('ChromeDriver has now started');
222220
} catch (e) {
223221
throw StateError(
224222
'Could not start ChromeDriver. Is it installed?\nError: $e',
@@ -434,53 +432,15 @@ class TestContext {
434432
],
435433
},
436434
});
437-
log(
438-
'Now creating web driver for chrome driver port; '
439-
'going for port $chromeDriverPort',
440-
);
441-
log(
442-
' -> notice chrome driver identity hash code is '
443-
'${identityHashCode(_chromeDriver)}',
435+
_webDriver = await createDriver(
436+
spec: WebDriverSpec.JsonWire,
437+
desired: capabilities,
438+
uri: Uri.parse(
439+
'http://127.0.0.1:$chromeDriverPort/$chromeDriverUrlBase/',
440+
),
444441
);
445-
try {
446-
final localWebDriver =
447-
_webDriver = await createDriver(
448-
spec: WebDriverSpec.JsonWire,
449-
desired: capabilities,
450-
uri: Uri.parse(
451-
'http://127.0.0.1:$chromeDriverPort/$chromeDriverUrlBase/',
452-
),
453-
);
454-
log(
455-
'After first try: _webDriver = $_webDriver; '
456-
'localWebDriver = $localWebDriver',
457-
);
458-
} on SocketException catch (e) {
459-
log('Got "$e". Will wait a bit and try again.');
460-
await Future.delayed(const Duration(seconds: 2));
461-
log('Back after the wait. Will now try again.');
462-
try {
463-
final localWebDriver =
464-
_webDriver = await createDriver(
465-
spec: WebDriverSpec.JsonWire,
466-
desired: capabilities,
467-
uri: Uri.parse(
468-
'http://127.0.0.1:$chromeDriverPort/$chromeDriverUrlBase/',
469-
),
470-
);
471-
log(
472-
'After second try: _webDriver = $_webDriver; '
473-
'localWebDriver = $localWebDriver',
474-
);
475-
} on SocketException catch (e) {
476-
log('Got exception again: "$e"');
477-
rethrow;
478-
}
479-
}
480442
}
481443

482-
log('Is now after the testSettings.launchChrome stuff.');
483-
484444
// The debugger tab must be enabled and connected before certain
485445
// listeners in DWDS or `main` is run.
486446
final tabConnectionCompleter = Completer();

dwds/test/hot_restart_breakpoints_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import 'fixtures/utilities.dart';
2323

2424
void main() {
2525
// Enable verbose logging for debugging.
26-
const debug = true;
26+
const debug = false;
2727
final provider = TestSdkConfigurationProvider(
2828
verbose: debug,
2929
canaryFeatures: true,

0 commit comments

Comments
 (0)