Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

Commit

Permalink
refactor: organizes coverage defines
Browse files Browse the repository at this point in the history
  • Loading branch information
Ferreira-Jonathan committed Aug 3, 2024
1 parent 5781334 commit 1330dd4
Showing 1 changed file with 17 additions and 11 deletions.
28 changes: 17 additions & 11 deletions packages/patrol_cli/lib/src/commands/build_android.dart
Original file line number Diff line number Diff line change
Expand Up @@ -111,24 +111,30 @@ class BuildAndroidCommand extends PatrolCommand {
packagesRegExps: packagesRegExps,
);

final coveragePackages = await coverageOpts.getCoveragePackages();
final coveragePackagesList = coveragePackages.toList().join(',');
_logger.detail('Received coverage packages: $coveragePackagesList');
final packageConfig = await coverageOpts.getPackageConfigData();
// convert to base64 to avoid issues with special characters
final packageConfigBase64 = base64Encode(utf8.encode(packageConfig));
_logger.detail('Received package config: $packageConfig');
var coverageDefines = <String, String>{};
if (coverage) {
final coveragePackages = await coverageOpts.getCoveragePackages();
final coveragePackagesList = coveragePackages.toList().join(',');
_logger.detail('Received coverage packages: $coveragePackagesList');
final packageConfig = await coverageOpts.getPackageConfigData();
// convert to base64 to avoid issues with special characters
final packageConfigBase64 = base64Encode(utf8.encode(packageConfig));
_logger.detail('Received package config: $packageConfig');
coverageDefines = {
'PATROL_COVERAGE_PACKAGES': coveragePackagesList,
'PATROL_PACKAGE_CONFIG': packageConfigBase64,
};
}

final internalDartDefines = {
'PATROL_WAIT': defaultWait.toString(),
'PATROL_APP_PACKAGE_NAME': packageName,
'PATROL_ANDROID_APP_NAME': config.android.appName,
'PATROL_TEST_LABEL_ENABLED': displayLabel.toString(),
'INTEGRATION_TEST_SHOULD_REPORT_RESULTS_TO_NATIVE': 'false',
'PATROL_COVERAGE': coverageOpts.coverage.toString(),
'PATROL_FUNCTION_COVERAGE': coverageOpts.functionCoverage.toString(),
'PATROL_COVERAGE_PACKAGES': coveragePackagesList,
'PATROL_PACKAGE_CONFIG': packageConfigBase64,
'PATROL_COVERAGE': coverage.toString(),
'PATROL_FUNCTION_COVERAGE': functionCoverage.toString(),
...coverageDefines,
}.withNullsRemoved();

final dartDefines = {...customDartDefines, ...internalDartDefines};
Expand Down

0 comments on commit 1330dd4

Please sign in to comment.