Unverified Commit ce2c708a authored by Jonah Williams's avatar Jonah Williams Committed by GitHub

Add more information to cannot find Chrome message (#41695)

parent 5e1f600f
...@@ -22,7 +22,8 @@ class WebValidator extends DoctorValidator { ...@@ -22,7 +22,8 @@ class WebValidator extends DoctorValidator {
ValidationMessage('$kChromeEnvironment = $chrome') ValidationMessage('$kChromeEnvironment = $chrome')
else else
if (!canRunChrome) if (!canRunChrome)
ValidationMessage.hint('$kChromeEnvironment not set') ValidationMessage.hint('Cannot find Chrome. Try setting '
'$kChromeEnvironment to a Chrome executable.')
else else
ValidationMessage('Chrome at $chrome'), ValidationMessage('Chrome at $chrome'),
]; ];
......
...@@ -50,7 +50,7 @@ void main() { ...@@ -50,7 +50,7 @@ void main() {
when(mockProcessManager.canRun(kMacOSExecutable)).thenReturn(false); when(mockProcessManager.canRun(kMacOSExecutable)).thenReturn(false);
final ValidationResult result = await webValidator.validate(); final ValidationResult result = await webValidator.validate();
expect(result.messages, <ValidationMessage>[ expect(result.messages, <ValidationMessage>[
ValidationMessage.hint('CHROME_EXECUTABLE not set'), ValidationMessage.hint('Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.'),
]); ]);
expect(result.type, ValidationType.missing); expect(result.type, ValidationType.missing);
})); }));
......
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