build_apk_test.dart 19.7 KB
Newer Older
Ian Hickson's avatar
Ian Hickson committed
1
// Copyright 2014 The Flutter Authors. All rights reserved.
2 3 4
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

5 6
import 'dart:io';

7 8
import 'package:args/command_runner.dart';
import 'package:flutter_tools/src/android/android_builder.dart';
9
import 'package:flutter_tools/src/android/android_sdk.dart';
10
import 'package:flutter_tools/src/base/context.dart';
11
import 'package:flutter_tools/src/base/file_system.dart';
12

13 14
import 'package:flutter_tools/src/cache.dart';
import 'package:flutter_tools/src/commands/build_apk.dart';
15
import 'package:flutter_tools/src/project.dart';
16
import 'package:flutter_tools/src/reporting/reporting.dart';
17
import 'package:flutter_tools/src/globals.dart' as globals;
18 19
import 'package:mockito/mockito.dart';
import 'package:process/process.dart';
20

21
import '../../src/android_common.dart';
22 23
import '../../src/common.dart';
import '../../src/context.dart';
24
import '../../src/mocks.dart';
25 26 27 28

void main() {
  Cache.disableLocking();

29
  group('Usage', () {
30
    Directory tempDir;
31
    Usage mockUsage;
32 33

    setUp(() {
34
      mockUsage = MockUsage();
35
      tempDir = globals.fs.systemTempDirectory.createTempSync('flutter_tools_packages_test.');
36 37 38 39 40 41 42 43 44
    });

    tearDown(() {
      tryToDelete(tempDir);
    });

    testUsingContext('indicate the default target platforms', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);
45
      final BuildApkCommand command = await runBuildApkCommand(projectPath);
46 47

      expect(await command.usageValues,
48
          containsPair(CustomDimensions.commandBuildApkTargetPlatform, 'android-arm,android-arm64,android-x64'));
49 50 51

    }, overrides: <Type, Generator>{
      AndroidBuilder: () => FakeAndroidBuilder(),
52
    });
53 54 55 56 57

    testUsingContext('split per abi', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

58
      final BuildApkCommand commandWithFlag = await runBuildApkCommand(projectPath,
59 60 61 62
          arguments: <String>['--split-per-abi']);
      expect(await commandWithFlag.usageValues,
          containsPair(CustomDimensions.commandBuildApkSplitPerAbi, 'true'));

63
      final BuildApkCommand commandWithoutFlag = await runBuildApkCommand(projectPath);
64 65 66 67 68
      expect(await commandWithoutFlag.usageValues,
          containsPair(CustomDimensions.commandBuildApkSplitPerAbi, 'false'));

    }, overrides: <Type, Generator>{
      AndroidBuilder: () => FakeAndroidBuilder(),
69
    });
70 71 72 73 74

    testUsingContext('build type', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

75
      final BuildApkCommand commandDefault = await runBuildApkCommand(projectPath);
76 77 78
      expect(await commandDefault.usageValues,
          containsPair(CustomDimensions.commandBuildApkBuildMode, 'release'));

79
      final BuildApkCommand commandInRelease = await runBuildApkCommand(projectPath,
80 81 82 83
          arguments: <String>['--release']);
      expect(await commandInRelease.usageValues,
          containsPair(CustomDimensions.commandBuildApkBuildMode, 'release'));

84
      final BuildApkCommand commandInDebug = await runBuildApkCommand(projectPath,
85 86 87 88
          arguments: <String>['--debug']);
      expect(await commandInDebug.usageValues,
          containsPair(CustomDimensions.commandBuildApkBuildMode, 'debug'));

89
      final BuildApkCommand commandInProfile = await runBuildApkCommand(projectPath,
90 91 92 93 94 95
          arguments: <String>['--profile']);
      expect(await commandInProfile.usageValues,
          containsPair(CustomDimensions.commandBuildApkBuildMode, 'profile'));

    }, overrides: <Type, Generator>{
      AndroidBuilder: () => FakeAndroidBuilder(),
96
    });
97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115

    testUsingContext('logs success', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      await runBuildApkCommand(projectPath);

      verify(mockUsage.sendEvent(
        'tool-command-result',
        'apk',
        label: 'success',
        value: anyNamed('value'),
        parameters: anyNamed('parameters'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidBuilder: () => FakeAndroidBuilder(),
      Usage: () => mockUsage,
    });
116
  });
117 118 119 120 121 122 123 124 125 126 127 128

  group('Gradle', () {
    Directory tempDir;
    ProcessManager mockProcessManager;
    String gradlew;
    AndroidSdk mockAndroidSdk;
    Usage mockUsage;

    setUp(() {
      mockUsage = MockUsage();
      when(mockUsage.isFirstRun).thenReturn(true);

129 130 131
      tempDir = globals.fs.systemTempDirectory.createTempSync('flutter_tools_packages_test.');
      gradlew = globals.fs.path.join(tempDir.path, 'flutter_project', 'android',
          globals.platform.isWindows ? 'gradlew.bat' : 'gradlew');
132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154

      mockProcessManager = MockProcessManager();
      when(mockProcessManager.run(<String>[gradlew, '-v'],
          environment: anyNamed('environment')))
        .thenAnswer((_) => Future<ProcessResult>.value(ProcessResult(0, 0, '', '')));

      when(mockProcessManager.run(<String>[gradlew, 'app:properties'],
          workingDirectory: anyNamed('workingDirectory'),
          environment: anyNamed('environment')))
        .thenAnswer((_) => Future<ProcessResult>.value(ProcessResult(0, 0, 'buildDir: irrelevant', '')));

      when(mockProcessManager.run(<String>[gradlew, 'app:tasks', '--all', '--console=auto'],
          workingDirectory: anyNamed('workingDirectory'),
          environment: anyNamed('environment')))
        .thenAnswer((_) => Future<ProcessResult>.value(ProcessResult(0, 0, 'assembleRelease', '')));
      // Fallback with error.
      final Process process = createMockProcess(exitCode: 1);
      when(mockProcessManager.start(any,
          workingDirectory: anyNamed('workingDirectory'),
          environment: anyNamed('environment')))
        .thenAnswer((_) => Future<Process>.value(process));
      when(mockProcessManager.canRun(any)).thenReturn(false);

155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174
      when(mockProcessManager.runSync(
        argThat(contains(contains('gen_snapshot'))),
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenReturn(ProcessResult(0, 255, '', ''));

      when(mockProcessManager.runSync(
        <String>['/usr/bin/xcode-select', '--print-path'],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenReturn(ProcessResult(0, 0, '', ''));

      when(mockProcessManager.run(
        <String>['which', 'pod'],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenAnswer((_) {
        return Future<ProcessResult>.value(ProcessResult(0, 0, '', ''));
      });

175 176 177 178 179 180 181 182
      mockAndroidSdk = MockAndroidSdk();
      when(mockAndroidSdk.directory).thenReturn('irrelevant');
    });

    tearDown(() {
      tryToDelete(tempDir);
    });

183 184
    group('AndroidSdk', () {
      testUsingContext('validateSdkWellFormed() not called, sdk reinitialized', () async {
185 186
        final String projectPath = await createProject(tempDir,
            arguments: <String>['--no-pub', '--template=app']);
187 188

        await expectLater(
189 190 191 192
          runBuildApkCommand(
            projectPath,
            arguments: <String>['--no-pub'],
          ),
193
          throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'),
194 195 196 197 198 199 200
        );

        verifyNever(mockAndroidSdk.validateSdkWellFormed());
        verify(mockAndroidSdk.reinitialize()).called(1);
      },
      overrides: <Type, Generator>{
        AndroidSdk: () => mockAndroidSdk,
201 202 203 204 205 206 207 208 209 210 211 212 213 214
        FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
        ProcessManager: () => mockProcessManager,
      });

      testUsingContext('throws throwsToolExit if AndroidSdk is null', () async {
        final String projectPath = await createProject(tempDir,
            arguments: <String>['--no-pub', '--template=app']);

        await expectLater(() async {
          await runBuildApkCommand(
            projectPath,
            arguments: <String>['--no-pub'],
          );
        }, throwsToolExit(
215
          message: 'No Android SDK found. Try setting the ANDROID_SDK_ROOT environment variable',
216 217 218 219 220
        ));
      },
      overrides: <Type, Generator>{
        AndroidSdk: () => null,
        FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
221
        ProcessManager: () => mockProcessManager,
222 223 224
      });
    });

Emmanuel Garcia's avatar
Emmanuel Garcia committed
225
    testUsingContext('shrinking is enabled by default on release mode', () async {
226 227 228 229 230 231 232 233 234 235 236
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      await expectLater(() async {
        await runBuildApkCommand(projectPath);
      }, throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'));

      verify(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
237
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
238
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
239
          '-Ptrack-widget-creation=true',
Emmanuel Garcia's avatar
Emmanuel Garcia committed
240
          '-Pshrink=true',
241
          '-Ptree-shake-icons=true',
242 243 244 245 246 247 248 249 250 251
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
252
    });
253

254 255 256 257 258 259 260 261 262 263 264 265
    testUsingContext('--split-debug-info is enabled when an output directory is provided', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      await expectLater(() async {
        await runBuildApkCommand(projectPath, arguments: <String>['--split-debug-info=${tempDir.path}']);
      }, throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'));

      verify(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
266
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
267 268 269 270
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
          '-Ptrack-widget-creation=true',
          '-Pshrink=true',
          '-Psplit-debug-info=${tempDir.path}',
271
          '-Ptree-shake-icons=true',
272 273 274 275 276 277 278 279 280 281 282 283
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
    });

284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303
    testUsingContext('--extra-front-end-options are provided to gradle project', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      await expectLater(() async {
        await runBuildApkCommand(projectPath, arguments: <String>[
          '--extra-front-end-options=foo',
          '--extra-front-end-options=bar',
        ]);
      }, throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'));

      verify(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
          '-Ptrack-widget-creation=true',
          '-Pextra-front-end-options=foo,bar',
          '-Pshrink=true',
304
          '-Ptree-shake-icons=true',
305 306 307 308 309 310 311 312 313 314 315 316
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
    });

Emmanuel Garcia's avatar
Emmanuel Garcia committed
317
    testUsingContext('shrinking is disabled when --no-shrink is passed', () async {
318 319 320 321 322 323
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      await expectLater(() async {
        await runBuildApkCommand(
          projectPath,
Emmanuel Garcia's avatar
Emmanuel Garcia committed
324
          arguments: <String>['--no-shrink'],
325 326 327 328 329 330 331
        );
      }, throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'));

      verify(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
332
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
333
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
334
          '-Ptrack-widget-creation=true',
335
          '-Ptree-shake-icons=true',
336 337 338 339 340 341 342 343 344 345
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
346
    });
347

Emmanuel Garcia's avatar
Emmanuel Garcia committed
348
    testUsingContext('guides the user when the shrinker fails', () async {
349 350 351 352 353 354 355
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      when(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
356
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
357
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
358
          '-Ptrack-widget-creation=true',
Emmanuel Garcia's avatar
Emmanuel Garcia committed
359
          '-Pshrink=true',
360
          '-Ptree-shake-icons=true',
361 362 363 364 365
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenAnswer((_) {
Emmanuel Garcia's avatar
Emmanuel Garcia committed
366
        const String r8StdoutWarning =
367
            "Execution failed for task ':app:transformClassesAndResourcesWithR8ForStageInternal'.\n"
Emmanuel Garcia's avatar
Emmanuel Garcia committed
368
            '> com.android.tools.r8.CompilationFailedException: Compilation failed to complete';
369 370 371
        return Future<Process>.value(
          createMockProcess(
            exitCode: 1,
Emmanuel Garcia's avatar
Emmanuel Garcia committed
372
            stdout: r8StdoutWarning,
373
          ),
374 375 376 377 378 379 380 381 382
        );
      });

      await expectLater(() async {
        await runBuildApkCommand(
          projectPath,
        );
      }, throwsToolExit(message: 'Gradle task assembleRelease failed with exit code 1'));

383 384 385 386 387 388 389 390 391 392 393 394
      expect(
        testLogger.statusText,
        containsIgnoringWhitespace('The shrinker may have failed to optimize the Java bytecode.'),
      );
      expect(
        testLogger.statusText,
        containsIgnoringWhitespace('To disable the shrinker, pass the `--no-shrink` flag to this command.'),
      );
      expect(
        testLogger.statusText,
        containsIgnoringWhitespace('To learn more, see: https://developer.android.com/studio/build/shrink-code'),
      );
395 396

      verify(mockUsage.sendEvent(
397 398
        'build',
        'apk',
399
        label: 'gradle-r8-failure',
400 401 402 403 404 405 406 407
        parameters: anyNamed('parameters'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
      Usage: () => mockUsage,
408
    });
409

410
    testUsingContext("reports when the app isn't using AndroidX", () async {
411
      final String projectPath = await createProject(tempDir,
412 413 414 415 416 417 418
          arguments: <String>['--no-pub', '--template=app']);
      // Simulate a non-androidx project.
      tempDir
        .childDirectory('flutter_project')
        .childDirectory('android')
        .childFile('gradle.properties')
        .writeAsStringSync('android.useAndroidX=false');
419 420 421 422 423

      when(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
424
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
425
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
426
          '-Ptrack-widget-creation=true',
427
          '-Pshrink=true',
428
          '-Ptree-shake-icons=true',
429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445 446 447
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenAnswer((_) {
        return Future<Process>.value(
          createMockProcess(
            exitCode: 0,
            stdout: '',
          ),
        );
      });
      // The command throws a [ToolExit] because it expects an APK in the file system.
      await expectLater(() async {
        await runBuildApkCommand(
          projectPath,
        );
      }, throwsToolExit());

448 449 450 451 452 453 454
      expect(
        testLogger.statusText,
        containsIgnoringWhitespace("Your app isn't using AndroidX"),
      );
      expect(
        testLogger.statusText,
        containsIgnoringWhitespace(
455 456
        'To avoid potential build failures, you can quickly migrate your app by '
        'following the steps on https://goo.gl/CP92wY'
457
        ),
458 459 460 461 462 463 464 465 466 467 468 469 470
      );
      verify(mockUsage.sendEvent(
        'build',
        'apk',
        label: 'app-not-using-android-x',
        parameters: anyNamed('parameters'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
      Usage: () => mockUsage,
471
    });
472 473 474 475 476 477 478 479 480

    testUsingContext('reports when the app is using AndroidX', () async {
      final String projectPath = await createProject(tempDir,
          arguments: <String>['--no-pub', '--template=app']);

      when(mockProcessManager.start(
        <String>[
          gradlew,
          '-q',
481
          '-Ptarget-platform=android-arm,android-arm64,android-x64',
482
          '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
483
          '-Ptrack-widget-creation=true',
484
          '-Pshrink=true',
485
          '-Ptree-shake-icons=true',
486 487 488 489 490 491 492 493 494 495 496 497 498 499 500 501 502 503 504
          'assembleRelease',
        ],
        workingDirectory: anyNamed('workingDirectory'),
        environment: anyNamed('environment'),
      )).thenAnswer((_) {
        return Future<Process>.value(
          createMockProcess(
            exitCode: 0,
            stdout: '',
          ),
        );
      });
      // The command throws a [ToolExit] because it expects an APK in the file system.
      await expectLater(() async {
        await runBuildApkCommand(
          projectPath,
        );
      }, throwsToolExit());

505
      expect(testLogger.statusText.contains("[!] Your app isn't using AndroidX"), isFalse);
506
      expect(
507
        testLogger.statusText.contains(
508 509 510 511 512 513 514 515 516 517 518 519 520 521 522 523 524
          'To avoid potential build failures, you can quickly migrate your app by '
          'following the steps on https://goo.gl/CP92wY'
        ),
        isFalse,
      );
      verify(mockUsage.sendEvent(
        'build',
        'apk',
        label: 'app-using-android-x',
        parameters: anyNamed('parameters'),
      )).called(1);
    },
    overrides: <Type, Generator>{
      AndroidSdk: () => mockAndroidSdk,
      FlutterProjectFactory: () => FakeFlutterProjectFactory(tempDir),
      ProcessManager: () => mockProcessManager,
      Usage: () => mockUsage,
525
    });
526 527 528 529
  });
}

Future<BuildApkCommand> runBuildApkCommand(
530 531 532
  String target, {
  List<String> arguments,
}) async {
533 534 535 536 537
  final BuildApkCommand command = BuildApkCommand();
  final CommandRunner<void> runner = createTestCommandRunner(command);
  await runner.run(<String>[
    'apk',
    ...?arguments,
538
    '--no-pub',
539
    globals.fs.path.join(target, 'lib', 'main.dart'),
540 541
  ]);
  return command;
542
}
543 544 545 546 547

class MockAndroidSdk extends Mock implements AndroidSdk {}
class MockProcessManager extends Mock implements ProcessManager {}
class MockProcess extends Mock implements Process {}
class MockUsage extends Mock implements Usage {}