Refactor and improve `GIDEMMSupport` parameters and tests

Đang mở

#611 mở ngày 31/7/2026 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó: 4/5
Thời gian dự kiến: 3-5 ngày
Mức phù hợp với người mới: 48/100

Loại issue: Tái cấu trúc
Độ rõ ràng: Khá rõ ràng
Mức độ hoạt động: Ít trao đổi
Công nghệ: objective-c
Lĩnh vực: authentication, mobile-dev, testing

Hướng nghiên cứu

Bắt đầu bằng việc đọc phần triển khai của GIDEMMSupport và các unit test hiện có, đặc biệt là parametersWithParameters và các test thực hiện swizzle UIDevice. Kiểm tra cách API công khai cung cấp hành vi này và xác định những test nào phụ thuộc vào trạng thái thiết bị thực. Hoàn thành khi các tham số có key là mảng lồng nhau được loại bỏ an toàn, việc mocking UIDevice nhất quán và hermetic, đồng thời coverage kiểm thử API công khai khi phù hợp.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement triage

This issue tracks follow-up improvements identified during the review of PR #608. We need to address the following technical debt and testing improvements:

Handle nested data structures: Ensure that the parametersWithParameters function safely drops key parameters that contain values like a nested array.
Standardize UIDevice mocking: Fix the tests that are swizzling UIDevice to "Not Do That," or ensure all tests are doing it the same way. Relying on the real UIDevice will make tests non-hermetic.
Uplevel tests to the public API: Consider upleveling the new GIDEMMSupport unit tests to test against the public API instead of testing internal classes in isolation.

Ngôn ngữ chính: Objective-C
Star: 751
Fork: 282
Merge trung bình: 4 ngày 20 giờ
Pull request đã merge (30 ngày): 11

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
Fork repository và làm thay đổi trên một nhánh.
Mở pull request có tham chiếu số hiệu của issue.

Issue khác của google/GoogleSignIn-iOS

Revoke request uses GET instead of POST Đang mở

enhancement triage

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100

google/GoogleSignIn-iOS#621 · Mở ngày 7/8/2026
Add test coverage for the userinfo profile-fetch fallback Đang mở

enhancement triage

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100

google/GoogleSignIn-iOS#620 · Mở ngày 7/8/2026
Allow only requesting the oauth scopes that an app needs Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100

google/GoogleSignIn-iOS#440 · 4 bình luận · 2 reaction · Mở ngày 11/6/2024
Remove Cocoapods Support Đang mở

Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100

google/GoogleSignIn-iOS#607 · Mở ngày 6/7/2026
Add App Check support for macOS Đang mở

enhancement triage

Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100

google/GoogleSignIn-iOS#602 · Mở ngày 6/6/2026

Tất cả issue của google/GoogleSignIn-iOS

Issue tương tự

Suzhou subway logo Đang mở

Icons Metro Public Transport

Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100

organicmaps/organicmaps#13617 · 1 bình luận · Mở ngày 23/9/2026
Make "Details & Options" clickable Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100

objective-see/LuLu#925 · Mở ngày 23/9/2026
macOS下查找brew::ncurse失败 Đang mở

bug

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100

xmake-io/xmake#7786 · 3 bình luận · Mở ngày 20/9/2026
The function key icon is missing [bug] Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100

noah-nuebling/mac-mouse-fix#2054 · Mở ngày 20/9/2026
macOS: every obs-websocket GetStats leaks ~200 autoreleased objects (os_get_free_space has no autorelease pool on the websocket thread) — ~2 MB/min at one poll per second Đang mở

Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100

obsproject/obs-studio#13918 · Mở ngày 19/9/2026

Thêm issue về Objective-C