flutter/packages/flutter_tools/test/integration.shard/xcode_dev_dependencies_test.dart
Loïc Sharma 37e79eb90c
[iOS/macOS] Add Xcode error if dev dependencies are incorrect (#165916)
Recently, the Flutter tool was updated to remove dev dependencies for
release builds and add dev dependencies for debug/profile builds.

However when building from Xcode directly, dev dependencies are not
enabled/disabled. As a result, it was possible for debug builds to not
have dev dependencies (or vice versa). Example:

1. `flutter build ios --release` - Release build using Flutter tool.
Disables dev dependencies
2. `open ios/Runner.xcworkspace` - Open the iOS project in Xcode
3. In Xcode, **Product** > **Build** - Do a debug build

Previously, step 3 would result in debug artifacts that are missing dev
dependencies. This PR now makes this an error:


![image](https://github.com/user-attachments/assets/621c4a8f-1c19-44b9-8866-93bef6b4a384)

Part of https://github.com/flutter/flutter/issues/163874

## Implementation

The Flutter tool now writes a `FLUTTER_DEV_DEPENDENCIES_ENABLED` in the
generated config file. This tracks whether the currently generated
project has dev dependencies.

In the Xcode build:

1. The Xcode backend script passes the
`FLUTTER_DEV_DEPENDENCIES_ENABLED` config to `flutter assemble` using
the `DevDependenciesEnabled` define
6. The new `CheckDevDependencies` target verifies dev dependencies:
1. It checks if the dev dependencies status is correct for the current
build mode
2. It checks whether the app has dev dependencies by reading the
`.flutter-plugins-dependencies` file. If the app has no dev
dependencies, the error is suppressed.

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
2025-04-11 19:34:38 +00:00

543 lines
17 KiB
Dart

// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:file/file.dart';
import 'package:flutter_tools/src/base/io.dart';
import 'package:flutter_tools/src/features.dart';
import '../src/common.dart';
import 'test_utils.dart';
void main() {
late Directory tempDir;
late Directory projectDir;
setUpAll(() async {
// TODO(team-ios): Remove after `explicit-package-dependencies` is enabled by default.
// See https://github.com/flutter/flutter/issues/160257 for details.
if (!explicitPackageDependencies.master.enabledByDefault) {
processManager.runSync(<String>[flutterBin, 'config', '--explicit-package-dependencies']);
}
tempDir = createResolvedTempDirectorySync('xcode_dev_dependencies_test.');
projectDir = tempDir.childDirectory('project')..createSync();
final Directory tempPluginADir = tempDir.childDirectory('plugin_a')..createSync();
// Create a Flutter project.
await processManager.run(<String>[
flutterBin,
'create',
projectDir.path,
'--project-name=testapp',
], workingDirectory: projectDir.path);
// Create a Flutter plugin to add as a dev dependency to the Flutter project.
await processManager.run(<String>[
flutterBin,
'create',
tempPluginADir.path,
'--template=plugin',
'--project-name=plugin_a',
'--platforms=ios,macos',
], workingDirectory: tempPluginADir.path);
// Add a dev dependency on plugin_a
await processManager.run(<String>[
flutterBin,
'pub',
'add',
'dev:plugin_a',
'--path',
tempPluginADir.path,
], workingDirectory: projectDir.path);
});
tearDownAll(() {
// TODO(team-ios): Remove after `explicit-package-dependencies` is enabled by default.
// See https://github.com/flutter/flutter/issues/160257 for details.
if (!explicitPackageDependencies.master.enabledByDefault) {
processManager.runSync(<String>[flutterBin, 'config', '--no-explicit-package-dependencies']);
}
tryToDelete(tempDir);
});
group(
'Xcode build iOS app',
() {
test('succeeds after flutter build ios --config-only', () async {
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(xcodeResult, const ProcessResultMatcher(stdoutPattern: '** BUILD SUCCEEDED **'));
});
test('fails in Release mode if dev dependencies enabled', () async {
// Enable dev dependencies by generating debug configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
// Xcode release build should error that dev dependencies are enabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Release',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'Release builds should not have Dart dev dependencies enabled',
stderrPattern: '** BUILD FAILED **',
),
);
});
test('fails in Debug mode if dev dependencies disabled', () async {
// Disable dev dependencies by generating debug configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--release',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
// Xcode debug build should error that dev dependencies are disabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'Debug builds require Dart dev dependencies',
stderrPattern: '** BUILD FAILED **',
),
);
});
},
skip: !platform.isMacOS, // [intended] iOS builds only work on macos.
);
group(
'Xcode build iOS module',
() {
test('succeeds after flutter build ios --config-only', () async {
final String appDirectory = fileSystem.path.join(
getFlutterRoot(),
'dev',
'integration_tests',
'ios_add2app_life_cycle',
);
final String moduleDirectory = fileSystem.path.join(appDirectory, 'flutterapp');
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: moduleDirectory,
);
expect(flutterResult, const ProcessResultMatcher());
final ProcessResult podResult = await processManager.run(
const <String>['pod', 'install'],
workingDirectory: appDirectory,
environment: const <String, String>{'LANG': 'en_US.UTF-8'},
);
expect(podResult, const ProcessResultMatcher());
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios_add2app.xcworkspace',
'-scheme',
'ios_add2app',
'-sdk',
'iphonesimulator',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: appDirectory,
);
expect(xcodeResult, const ProcessResultMatcher(stdoutPattern: '** BUILD SUCCEEDED **'));
});
test('fails in Release mode if dev dependencies enabled', () async {
final String appDirectory = fileSystem.path.join(
getFlutterRoot(),
'dev',
'integration_tests',
'ios_add2app_life_cycle',
);
final String moduleDirectory = fileSystem.path.join(appDirectory, 'flutterapp');
// Enable dev dependencies by generating debug configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: moduleDirectory,
);
expect(flutterResult, const ProcessResultMatcher());
final ProcessResult podResult = await processManager.run(
const <String>['pod', 'install'],
workingDirectory: appDirectory,
environment: const <String, String>{'LANG': 'en_US.UTF-8'},
);
expect(podResult, const ProcessResultMatcher());
// Xcode release build should error that dev dependencies are enabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios_add2app.xcworkspace',
'-scheme',
'ios_add2app',
'-sdk',
'iphonesimulator',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Release',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: appDirectory,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'Release builds should not have Dart dev dependencies enabled',
stderrPattern: '** BUILD FAILED **',
),
);
});
test('abc fails in Debug mode if dev dependencies disabled', () async {
final String appDirectory = fileSystem.path.join(
getFlutterRoot(),
'dev',
'integration_tests',
'ios_add2app_life_cycle',
);
final String moduleDirectory = fileSystem.path.join(appDirectory, 'flutterapp');
// Disable dev dependencies by generating release configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'ios',
'--config-only',
'--release',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: moduleDirectory,
);
expect(flutterResult, const ProcessResultMatcher());
final ProcessResult podResult = await processManager.run(
const <String>['pod', 'install'],
workingDirectory: appDirectory,
environment: const <String, String>{'LANG': 'en_US.UTF-8'},
);
expect(podResult, const ProcessResultMatcher());
// Xcode debug build should error that dev dependencies are disabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'ios_add2app.xcworkspace',
'-scheme',
'ios_add2app',
'-sdk',
'iphonesimulator',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: appDirectory,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'Debug builds require Dart dev dependencies',
stderrPattern: '** BUILD FAILED **',
),
);
});
},
skip: !platform.isMacOS, // [intended] iOS builds only work on macos.
);
group(
'Xcode build macOS app',
() {
test('succeeds after flutter build macos --config-only', () async {
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'macos',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'macos/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(xcodeResult, const ProcessResultMatcher(stdoutPattern: '** BUILD SUCCEEDED **'));
});
test('fails in Release mode if dev dependencies enabled', () async {
// Enable dev dependencies by generating debug configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'macos',
'--config-only',
'--debug',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
// Xcode release build should error that dev dependencies are enabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'macos/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Release',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'error: Release builds should not have Dart dev dependencies enabled',
stderrPattern: '** BUILD FAILED **',
),
);
});
test('fails in Debug mode if dev dependencies disabled', () async {
// Disable dev dependencies by generating debug configuration files
final List<String> flutterCommand = <String>[
flutterBin,
...getLocalEngineArguments(),
'build',
'macos',
'--config-only',
'--release',
];
final ProcessResult flutterResult = await processManager.run(
flutterCommand,
workingDirectory: projectDir.path,
);
expect(flutterResult, const ProcessResultMatcher());
// Xcode debug build should error that dev dependencies are disabled.
final List<String> xcodeCommand = <String>[
'xcodebuild',
'-workspace',
'macos/Runner.xcworkspace',
'-scheme',
'Runner',
'CODE_SIGNING_ALLOWED=NO',
'CODE_SIGNING_REQUIRED=NO',
'CODE_SIGN_IDENTITY=-',
'EXPANDED_CODE_SIGN_IDENTITY=-',
'COMPILER_INDEX_STORE_ENABLE=NO',
'VERBOSE_SCRIPT_LOGGING=true',
'-configuration',
'Debug',
];
final ProcessResult xcodeResult = await processManager.run(
xcodeCommand,
workingDirectory: projectDir.path,
);
expect(
xcodeResult,
const ProcessResultMatcher(
exitCode: 65,
stdoutPattern: 'error: Debug builds require Dart dev dependencies',
stderrPattern: '** BUILD FAILED **',
),
);
});
},
skip: !platform.isMacOS, // [intended] iOS builds only work on macos.
);
}