google / google/GoogleSignIn-iOS

Refactor and improve `GIDEMMSupport` parameters and tests

Đang mở
#611 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
enhancement triage
Ngôn ngữ chính
Objective-C
Star
750
Fork
282
Merge trung bình
2 ngày 15 giờ
Pull request đã merge (30 ngày)
9

Mô tả

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.

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

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

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.

Đánh giá

Công nghệ
objective-c
Lĩnh vực
authentication, mobile-dev, testing
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
48/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.