google / google/GoogleSignIn-iOS
Refactor and improve `GIDEMMSupport` parameters and tests
- 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
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