Repository navigation
테스트 가치 기준 적용 및 CI 중복 비용 정리 - #98
Merged
Merged
Conversation
jlc488
marked this pull request as ready for review
October 17, 2026 10:01
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
기능 변경마다 테스트가 누적되고 같은 검사가 반복 실행되는 부담을 줄이기 위한 변경입니다. 중요한 사용자 동작·공개 API·보안·데이터 무결성 검사는 유지합니다.
.github/workflows/ci.yml,.github/workflows/release.yml. 기존 필수 검사 이름과 보안·릴리스 검증은 유지합니다.tests/dds-solid-contracts.test.mjs,tests/dds-table-contracts.test.mjs.정리 근거와 검증
검증:
git diff --check통과, 워크플로 YAML 파싱 통과. 문서/실행 스케줄만 바뀐 저장소는 새로운 테스트를 추가하거나 전체 테스트를 반복 실행하지 않았습니다. 변경된 테스트의 실행 증거는 위 검증 기록 및 PR CI에 구분해 남깁니다. 실제 월 절감액은 병합 후 사용량으로 측정해야 합니다.CI에서 발견한 실행 연결 검사 보완
기존 foundation 검사는 모든 별도 명령 이름이 workflow에 있어야 한다고 요구해, 기본 suite에 포함된 a11y와 동일한 SEO/i18n 명령 통합을 거부했습니다. package/surface/release 인벤토리는 유지하면서 동일 명령 본문 또는 검증된 a11y 포함 관계만 허용하도록 수정했습니다. a11y 파일의 실제 존재, 같은 Vitest config와 include/exclude, 기본 package test 및 workflow 호출을 모두 확인합니다.
node --test --test-name-pattern="every publishable" tests/foundation-contracts.test.mjs1/1 통과.pnpm run verify:source:stage1-223/23 통과.Site-kit 실행 연결 검사
동일한 별도 solid:a11y 명령을 요구하던 site-kit 검사도 foundation의 실제 포함 관계 검사로 통합했습니다. 나머지 hydration/RTL/browser/release 검사는 유지했습니다.
node --test --test-name-pattern="Worker and browser fixtures" tests/site-kit-contracts.test.mjs1/1 통과.취소 그룹 검증
PR 번호가 없는 main/manual 실행은 run_id로 분리해 대기 중인 실행도 교체되지 않게 했습니다. YAML 파싱 및 비교로 concurrency group 외 event/job/step 값이 동일함을 확인했고 독립 리뷰를 통과했습니다.