-
Notifications
You must be signed in to change notification settings - Fork 28.6k
integration_test embedded in shipping iOS apps when added to pubspec #74274
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
For the iOS side at least: Avoid embedding in app
Then the podhelper can exclude those plugins: flutter/packages/flutter_tools/bin/podhelper.rb Lines 252 to 253 in f63055a
Something like return [] if dependencies_hash['plugins'].has_key?('dev_dependency')
Embed in testTo add target 'RunnerTests' do
pod 'integration_test'
end Or we can figure out how to do this as a Swift Package #72482. If there's plans to add the test target to the Xcode project template, this embedding can be done automatically. |
I'd really like to see it fixed. It makes it harder than necessary to develop a Flutter plugin that is intended to use only in integration tests. |
@cbracken Just checking in - this functionality is currently working e2e in Android, but not iOS. I'd love to get this into the next (not this) stable release so we can announce at the same time that both Android and iOS support it, but it's not critical. |
This is ready to get worked on whenever prioritized; see #163874 (can dedupe) |
Uh oh!
There was an error while loading. Please reload this page.
@matanlurey update as of 2024-11-20:
As of HEAD today, with the following feature flag opt-in:
The generated
.flutter-plugins-dependencies
file now includesdev_dependency: true|false
per plugin. See #158009 for implementation details. We should be able to use the explicit appearance ofdev_dependency: true
in order to strip out plugins (such asintegration_test
) that are dev dependencies in production builds.Original post:
integration_test
was added to the pubspec template in #70240. This caused all newlyflutter create
d iOS apps to ship with an embeddedintegration_test
framework, and it requires all developers to install CocoaPods, even if they had not previously used any plugins. It also embedded the app in add-to-app host apps.#70240 was reverted in #74068 due to NNBD issues. Before it is relanded, there should be a plan for how to prevent the above.
See also #68818
The text was updated successfully, but these errors were encountered: