-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Ensure Messaging instance is usable after FIRApp's delete
#3579
Conversation
This fixes #3411. Test fails before the code change, and succeeds afterwards.
@@ -52,7 +52,7 @@ device, and it is completely free. | |||
s.dependency 'Protobuf', '~> 3.1' | |||
|
|||
s.test_spec 'unit' do |unit_tests| | |||
unit_tests.source_files = 'Example/Messaging/Tests/*.[mh]' | |||
unit_tests.source_files = 'Example/Messaging/Tests/*.{m,h,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.
Awesome! swift unit tests!
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.
LGTM
import FirebaseMessaging | ||
import XCTest | ||
|
||
class FIRMessagingInstanceTest: XCTestCase { |
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.
Nice, tests in Swift!
Add to release notes and bug to milestone if this fix is going in M54 (6.7.0) cc: @chliangGoogle |
@chliangGoogle does this make sense that |
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.
Thanks for the quick fix!
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.
Can you also update the CHANGELOG if this goes into M54 release?
Done! |
Firebase/Messaging/CHANGELOG.md
Outdated
@@ -1,6 +1,7 @@ | |||
# 2019-08-20 -- v4.1.3 | |||
- [changed] Cleaned up the documents, unused macros, and folders. (#3490, #3537, #3556, #3498) | |||
- [changed] Updated the header path to pod repo relative. (#3527) | |||
- [fixed] Fix singleton functionality after a FirebaseApp is deleted and recreated. (#3579) |
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.
Use bug number - #3411
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.
Good catch!
Building the swift file fails with the --use-libraries test in the cron job. https://travis-ci.org/firebase/firebase-ios-sdk/jobs/572152503 |
This fixes #3411. Test fails before the code change, and succeeds
afterwards.