Skip to content

Inject the device and LAContext into GIDEMMSupport and GIDMDMPasscodeCache - #659

Merged
w-goog merged 4 commits into
mainfrom
codegen/ocmock-02-device-lacontext-seams
Oct 28, 2026
Merged

w-goog merged 4 commits into
mainfrom
codegen/ocmock-02-device-lacontext-seams

Conversation

@w-goog

@w-goog w-goog commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Second of the OCMock removal stack, on top of #658. This one removes the UIDevice.systemName and LAContext swizzles (and the MDM test's iOS 9 guards).

  • GIDEMMSupport gets the device dep it needs. The class methods stay and delegate.
  • GIDMDMPasscodeCache gets LAContext through the initializer (yay for DI).
  • UIDevice and LAContext each get a protocol + a fake for the tests to use.

@w-goog
w-goog requested a review from mingyokim October 21, 2026 00:57
@w-goog
w-goog force-pushed the codegen/ocmock-02-device-lacontext-seams branch from eac9534 to 39a9801 Compare October 21, 2026 22:03
Base automatically changed from codegen/ocmock-01-bundle-seam to main October 21, 2026 22:37
@w-goog
w-goog force-pushed the codegen/ocmock-02-device-lacontext-seams branch from 39a9801 to c7207b1 Compare October 23, 2026 23:21
Comment thread GoogleSignIn/Sources/GIDEMMSupport.h Resolved
@w-goog
w-goog merged commit 059416c into main Oct 28, 2026
32 of 33 checks passed
@w-goog
w-goog deleted the codegen/ocmock-02-device-lacontext-seams branch October 28, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants