-
-
Notifications
You must be signed in to change notification settings - Fork 529
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
feat: RunnerTests for iOS/MacOS #812
feat: RunnerTests for iOS/MacOS #812
Conversation
end | ||
|
||
post_install do |installer| | ||
installer.pods_project.targets.each do |target| | ||
flutter_additional_ios_build_settings(target) | ||
target.build_configurations.each do |config| | ||
config.build_settings['ENABLE_BITCODE'] = 'NO' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bitcode hasn't been supported anymore for a while now.
@@ -8,14 +8,26 @@ | |||
|
|||
/* Begin PBXBuildFile section */ | |||
1498D2341E8E89220040F4C2 /* GeneratedPluginRegistrant.m in Sources */ = {isa = PBXBuildFile; fileRef = 1498D2331E8E89220040F4C2 /* GeneratedPluginRegistrant.m */; }; | |||
331C808B294A63AB00263BE5 /* RunnerTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 331C807B294A618700263BE5 /* RunnerTests.swift */; }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of these changes are reorganising for the addition of the RunnerTests. This was done by the flutter create
template.
@@ -21,15 +21,24 @@ | |||
/* End PBXAggregateTarget section */ | |||
|
|||
/* Begin PBXBuildFile section */ | |||
331C80D8294CF71000263BE5 /* RunnerTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 331C80D7294CF71000263BE5 /* RunnerTests.swift */; }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto, but for MacOS.
…pp_ios_macos_cleanup feat: RunnerTests for iOS/MacOS
This PR adds a RunnerTests setup to the iOS/MacOS example app.
It adds the RunnerTests stub that is added by the template. (with the original function commented out, since it will be replaced in the end)
With the large changes to the iOS/MacOS workspace files, I've tested iOS/MacOS and they still run correctly.
The new RunnerTests also show up in the project navigator for iOS/MacOS, so that works as well.