Restore handler test coverage for the token-rotation path
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 68/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- aws, github, typescript
Hướng nghiên cứu
Đọc apps/token-rotator/src/main.spec.ts cùng với guard.spec.ts và stringHelpers.spec.ts, sau đó chạy các bài kiểm thử của token-rotator để tái hiện trường hợp bị bỏ qua. Khôi phục độ bao phủ xác định cho đường đi GitHub-auth, registration-token và SSM-put mà không sử dụng thông tin xác thực được commit; hoàn tất khi bài kiểm thử handler vượt qua một cách ổn định và không còn bị bỏ qua.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Context
PR #62 skipped lambdas › token-rotator › completes without error (apps/token-rotator/src/main.spec.ts) because it is a live-API test wired to a dead fixture: the committed .env.test authenticated as GitHub App 217830 in the third-party skedrocket org, which no longer exists — the GitHub API returns "Integration not found" unconditionally, so the test failed 100% of the time on every branch (verified on Renovate branches predating that PR).
That skip leaves the entire GitHub-auth → registration-token → SSM-put path with zero test coverage (the only other specs are guard.spec.ts and stringHelpers.spec.ts; libs/github has none). No working coverage was lost, but this must not become permanent.
Restoration options
- Hermetic (preferred): mock the GitHub API (e.g.
nock) so the handler test runs without live credentials — deterministic, no secrets. - Live: recreate a dedicated test App under the cloudposse org and inject its ID/key via repo secrets (never a committed file).
Hygiene follow-up
.env.test committed the dead App's private key at the repo root; PR #62 replaces it with placeholders, but the key remains in git history — inert (the App is deleted), but worth removing per policy if history-rewrites are ever done.
Refs: #62
- Ngôn ngữ chính
- TypeScript
- Star
- 3
- Fork
- 1
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc 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 cloudposse/lambda-github-action-token-rotator
-
Dependency DashboardĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
Tất cả issue của cloudposse/lambda-github-action-token-rotator
Issue tương tự
-
feat(subscription): add Manage Subscription (Stripe portal) to the Subscription tab plan cardĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Feature] 通知栏合并重复消息并显示次数Đang mởarea:ui enhancement issue-form:feature platform:cross-platform review: high
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
1lck/Lithe-IDEA#1092 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
developmentseed/deck.gl-raster#693 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Marker-Inc-Korea/AutoRAG#1801 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
iii-hq/iii#2278 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày