-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[local_auth_darwin] MacOS Support #6267
Open
alexrabin-sentracam
wants to merge
119
commits into
flutter:main
Choose a base branch
from
alexrabin-sentracam:macos-support
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,191
−76
Open
Changes from all commits
Commits
Show all changes
119 commits
Select commit
Hold shift + click to select a range
532ed7f
Beginning of macos support
alexrabin-sentracam 0eb64ee
Merge branch 'flutter:main' into macos-support
alexrabin-sentracam 25ca34e
Made macos deployment target 10.15
alexrabin-sentracam c57ac02
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
alexrabin-sentracam 00bd678
Separate ios and macos code
alexrabin-sentracam 509708f
Adjusted format
alexrabin-sentracam fd6c539
Updated change log
alexrabin-sentracam 44dd8c8
Removed duplicated analysis_options.yaml
4f68b52
Removed unnecessary widget_test in darwin
8646867
Added license blocks in macos example files
676b0eb
Changed iOSAuthMessages to DarwinAuthMessages
bcee2e7
Merge branch 'main' into macos-support
alexrabin-sentracam 4aab9d8
Added macos comment
1bded66
Changed macOS capitalization
7335837
Removed unnecessary test, added comment, and changed version number
849c07b
Removed prettier ignore
c810eb4
Reset upgrade version
6867b3a
Revert "Changed iOSAuthMessages to DarwinAuthMessages" and add MacOSA…
995f41d
Changed podspec to use ios deployment_target
8ab5d09
Checks if macos is 10.15 or higher before checking for faceid
507470e
Shows correct window for registrar
05b52ae
Changed wording
8f35101
Merge branch 'main' into macos-support
alexrabin-sentracam 2b0e6d6
Combined @available statements and renamed keywindow
c808c82
Created initWithRegistrar initializer
64bd0b7
Moved runner tests
7dab5b4
Merge branch 'main' into macos-support
alexrabin-sentracam 7ff9871
Fixed format issues
04f6a76
Revert "Moved runner tests"
04c8f12
Fixed tests and platform available check
64366dd
Merge branch 'main' into macos-support
alexrabin-sentracam 7babc9c
Merge branch 'main' into macos-support
alexrabin-sentracam b71f172
Merge branch 'main' into macos-support
alexrabin-sentracam cfabac0
Merge branch 'main' into macos-support
alexrabin-sentracam 5bc080e
Added macos tests
d794eb9
Fixed format
74ae648
Added info.plist key and added license heading
2374375
Added xml language to readme
f053fce
Upgraded project settings
04049a3
Reset example macos project
9be16c3
Fixed macos tests
ff533bc
Removed settings button on macos due to no native support from Apple …
783b0cc
Added @available check
5e9b627
Merge branch 'main' into macos-support
alexrabin-sentracam 1748039
Merge branch 'main' into macos-support
alexrabin-sentracam ebdb5f1
Merge branch 'main' into macos-support
alexrabin-sentracam b73254a
Merge branch 'main' into macos-support
alexrabin-sentracam c4c9f3b
Merge branch 'main' into macos-support
alexrabin-sentracam 6ca1476
Merge branch 'main' into macos-support
alexrabin-sentracam 69db0ec
Merge branch 'main' into macos-support
alexrabin-sentracam a39e82a
Merge branch 'main' into macos-support
alexrabin-sentracam 59ce98a
Merge branch 'main' into macos-support
alexrabin-sentracam e54c7ea
Merge branch 'main' into macos-support
alexrabin-sentracam b8ca53a
Merge branch 'main' into macos-support
alexrabin-sentracam 018fe8f
Merge branch 'main' into macos-support
alexrabin-sentracam ff38a26
Merge branch 'main' into macos-support
alexrabin-sentracam e76f6bc
Addressed additional feedback
72b8212
Fixed plugin tests
113195e
Merge branch 'main' into macos-support
alexrabin-sentracam 7525f01
Merge branch 'main' into macos-support
alexrabin-sentracam 6330a9e
Merge branch 'main' into macos-support
alexrabin-sentracam 79ebae9
Made initWithRegistrar the designated initializer
0c84f2b
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
c84a376
Merge branch 'main' into macos-support
alexrabin-sentracam 5dc7c75
Merge branch 'main' into macos-support
alexrabin-sentracam c33ed13
Merge branch 'main' into macos-support
alexrabin-sentracam 186d89c
Updated messaging
f9e11db
Merge branch 'main' into macos-support
alexrabin-sentracam c45084b
Remove gotosettingsbutton on macos and didn't make it required
7003142
Merge branch 'main' into macos-support
alexrabin-sentracam 18453dc
Merge branch 'main' into macos-support
alexrabin-sentracam fc14107
Merge branch 'main' into macos-support
alexrabin-sentracam 5840a73
Merge branch 'flutter:main' into macos-support
alexrabin-sentracam ce1fa37
Merge branch 'main' into macos-support
alexrabin-sentracam ad83e24
Adjusted format
8beb6dd
Merge branch 'main' into macos-support
alexrabin-sentracam 067a0c0
Added initWithContextFactoryAndRegistrar since tests failing due to m…
e3e89d4
Fixed mock test error
a9167cc
Merge branch 'main' into macos-support
alexrabin-sentracam 0f56ad6
Created base StubAlertFactory
17bd1c7
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
2441440
Added test to check if the alert window is using the registrar window
8af7ec7
Added overridemacosmessages
4fcbc5d
Removes default false for overrideUseMacOSAuthMessages
8bd0531
Merge branch 'main' into macos-support
alexrabin-sentracam 44e5ae3
Merge branch 'main' into macos-support
alexrabin-sentracam 51b3b02
Merge branch 'main' into macos-support
alexrabin-sentracam 6d0684f
Merge branch 'main' into macos-support
alexrabin-sentracam ca5789f
Merge branch 'main' into macos-support
alexrabin-sentracam 6d5a0c6
Merge branch 'main' into macos-support
alexrabin-sentracam 11e6991
Merge branch 'main' into macos-support
alexrabin-sentracam 316bb68
Updated method names
3728490
Fixed comment format
5b72aea
Fixed macos tests
6dd9092
Merge branch 'main' into macos-support
alexrabin-sentracam b5ea028
Merge branch 'main' into macos-support
alexrabin-sentracam dd545e5
Merge branch 'main' of https://github.com/flutter/packages into macos…
85678af
Upgraded version to 1.4.0
679854a
Removed copy property
464ae48
Checks parent window correctly and uses nsa;ert object
05bc529
Merge branch 'main' into macos-support
alexrabin-sentracam d3ce499
Merge branch 'main' into macos-support
alexrabin-sentracam 57c50d3
Added nonnulll keyword
860d198
Updated format
9224412
Added nonnull
9c345f9
Added non null to create alert
71a2824
Merge branch 'main' into macos-support
alexrabin-sentracam 0d420c5
Updated elif check
3415984
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
09236d6
Added target conditionals import
b0de86c
Added declaration comments
288b04b
Merge branch 'main' into macos-support
alexrabin-sentracam 23305d0
Merge remote-tracking branch 'upstream/main' into macos-support
2293ff6
Merge branch 'main' into macos-support
alexrabin-sentracam 46ce252
Merge branch 'main' into macos-support
alexrabin-sentracam 0f7dac8
Merge branch 'main' into macos-support
alexrabin-sentracam a91e4e9
Fixed swift tests
e5c9b97
Merge branch 'main' into macos-support
alexrabin-sentracam f90387f
Merge branch 'main' into macos-support
alexrabin-sentracam File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
22 changes: 22 additions & 0 deletions
22
packages/local_auth/local_auth_darwin/darwin/Tests/OCSwiftMock.h
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
// Copyright 2013 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 <Foundation/Foundation.h> | ||
#import <OCMock/OCMock.h> | ||
|
||
@interface OCSwiftProtocolMock<MockType> : NSObject | ||
|
||
@property(nonatomic, readonly) MockType protocol; | ||
|
||
- (instancetype)init:(Protocol *)mockProtocol; | ||
|
||
@end | ||
|
||
@interface OCSwiftClassMock<MockType> : NSObject | ||
|
||
@property(nonatomic, readonly) MockType object; | ||
|
||
- (instancetype)init:(Class)classObject; | ||
|
||
@end |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Please do not re-introduce OCMock; I removed it intentionally. New mocks/stubs should use the same pattern the other tests now use, which is to introduce a protocol that has the desired methods, and then have tests implement that protocol. See here, here, and here for an example of the pattern. (I would expect the new wrappers you would need to be simpler than that one.)