Unverified Commit df1462fa authored by Zachary Anderson's avatar Zachary Anderson Committed by GitHub

[flutter_tool,fuchsia_tester] Only require a test source dir for coverage (#28168)

parent cb84c157
...@@ -34,7 +34,6 @@ const String _kOptionCoverageDirectory = 'coverage-directory'; ...@@ -34,7 +34,6 @@ const String _kOptionCoverageDirectory = 'coverage-directory';
const List<String> _kRequiredOptions = <String>[ const List<String> _kRequiredOptions = <String>[
_kOptionPackages, _kOptionPackages,
_kOptionShell, _kOptionShell,
_kOptionTestDirectory,
_kOptionSdkRoot, _kOptionSdkRoot,
_kOptionIcudtl, _kOptionIcudtl,
_kOptionTests, _kOptionTests,
...@@ -75,8 +74,6 @@ Future<void> run(List<String> args) async { ...@@ -75,8 +74,6 @@ Future<void> run(List<String> args) async {
fs.systemTempDirectory.createTempSync('flutter_fuchsia_tester.'); fs.systemTempDirectory.createTempSync('flutter_fuchsia_tester.');
try { try {
Cache.flutterRoot = tempDir.path; Cache.flutterRoot = tempDir.path;
final Directory testDirectory =
fs.directory(argResults[_kOptionTestDirectory]);
final String shellPath = argResults[_kOptionShell]; final String shellPath = argResults[_kOptionShell];
if (!fs.isFileSync(shellPath)) { if (!fs.isFileSync(shellPath)) {
...@@ -101,9 +98,9 @@ Future<void> run(List<String> args) async { ...@@ -101,9 +98,9 @@ Future<void> run(List<String> args) async {
final Link testerDestLink = final Link testerDestLink =
fs.link(artifacts.getArtifactPath(Artifact.flutterTester)); fs.link(artifacts.getArtifactPath(Artifact.flutterTester));
testerDestLink.parent.createSync(recursive: true); testerDestLink.parent.createSync(recursive: true);
testerDestLink.createSync(shellPath); testerDestLink.createSync(fs.path.absolute(shellPath));
final Link icudtlLink = testerDestLink.parent.childLink('icudtl.dat'); final Link icudtlLink = testerDestLink.parent.childLink('icudtl.dat');
icudtlLink.createSync(argResults[_kOptionIcudtl]); icudtlLink.createSync(fs.path.absolute(argResults[_kOptionIcudtl]));
final Directory sdkRootDest = final Directory sdkRootDest =
fs.directory(artifacts.getArtifactPath(Artifact.flutterPatchedSdkPath)); fs.directory(artifacts.getArtifactPath(Artifact.flutterPatchedSdkPath));
sdkRootDest.createSync(recursive: true); sdkRootDest.createSync(recursive: true);
...@@ -116,9 +113,14 @@ Future<void> run(List<String> args) async { ...@@ -116,9 +113,14 @@ Future<void> run(List<String> args) async {
PackageMap.globalPackagesPath = PackageMap.globalPackagesPath =
fs.path.normalize(fs.path.absolute(argResults[_kOptionPackages])); fs.path.normalize(fs.path.absolute(argResults[_kOptionPackages]));
Directory testDirectory;
CoverageCollector collector; CoverageCollector collector;
if (argResults['coverage']) { if (argResults['coverage']) {
collector = CoverageCollector(); collector = CoverageCollector();
if (!argResults.options.contains(_kOptionTestDirectory)) {
throwToolExit('Use of --coverage requires setting --test-directory');
}
testDirectory = fs.directory(argResults[_kOptionTestDirectory]);
} }
......
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