Commit c99432c
authored
* improve: 불필요한 Spring 테스트 삭제
* docs: 컨벤션은 별도 패키지로 분리
* docs: ddd 관련 문서를 ddd 패키지 하위로 이동
* docs: 테스트 컨벤션 작성
* refactor: 시간 관련 계약을 고정값으로 하여 테스트 안전성 강화
* refactor: 통합 테스트 네이밍에서 Controller 제거
* refactor: useraccount 도메인의 fixture 정리
* refactor: post 도메인과 source 도메인의 fixture 소유권 정리
* recommendation fixture 소유권 정리
* refactor: activity 도메인 fixture 소유권 정리
* refactor: post 도메인 테스트 fixture 소유권 정리
* refactor: auth 패키지 fixture 소유권 정리
* refactor: useraccount 테스트 fixture 소유권 정리
* refactor: source 테스트 fixture 소유권 정리
* refactor: recommendation 테스트 fixture 소유권 정리
* post 테스트 fixture 소유권 정리 2
* refactor: 테스트 fixture 사용 일괄 정리
* docs: 테스트 실행 계약 정리
* docs: 레거시 코드들은 옮기지 않음을 명시
* docs: 테스트 네이밍 최신화
* docs: 테스트 메서드 네이밍 컨벤션 보강
* test: auth 패키지 구조와 메서드명 정리
* test: useraccount 패키지 구조와 메서드명 정리
* test: activity 패키지 구조와 메서드명 정리
Constraint: 도메인 단위로 Nested 구조와 테스트 메서드명을 함께 정리
Confidence: high
Scope-risk: moderate
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.activity.*'; activity integration tests with temporary MySqlRedisIntegrationTestBase restored afterward
Not-tested: full test suite
* test: post 패키지 구조와 메서드명 정리
Constraint: 도메인 단위로 Nested 구조와 테스트 메서드명을 함께 정리
Confidence: high
Scope-risk: moderate
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.post.*'; post integration tests with temporary MySqlRedisIntegrationTestBase restored afterward
Not-tested: full test suite
* test: personalization 패키지 구조와 메서드명 정리
Constraint: 도메인 단위로 Nested 구조와 테스트 메서드명을 함께 정리
Confidence: high
Scope-risk: moderate
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.personalization.*'; personalization integration test with temporary MySqlRedisIntegrationTestBase restored afterward
Not-tested: full test suite
* test: recommendation 패키지 구조와 메서드명 정리
Constraint: legacy domain/recommendation root 유지
Confidence: medium
Scope-risk: moderate
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.domain.recommendation.*'
Not-tested: RecommendationIntegrationTest blocked by Elasticsearch container startup timeout; MySqlRedisIntegrationTestBase check blocked by schema-validation mismatch
* test: source 패키지 구조와 메서드명 정리
Constraint: legacy domain/source root 유지
Confidence: medium
Scope-risk: moderate
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.domain.source.*'
Not-tested: RssCrawlingJobIntegrationTest blocked by MySqlRedisIntegrationTestBase schema-validation mismatch
* test: search 패키지 구조와 메서드명 정리
Constraint: legacy domain/search root 유지
Confidence: high
Scope-risk: narrow
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.domain.search.*'
Not-tested: full test suite
* test: global 테스트 구조와 메서드명 정리
Constraint: shared global test support files 제외
Confidence: high
Scope-risk: narrow
Tested: ./gradlew compileTestJava; ./gradlew test -PexcludeIntegration --tests 'com.techfork.global.*'
Not-tested: full test suite
* test: 일반 통합 테스트에서 ES 컨테이너 의존 분리
일반 integrationTest 레인은 MySQL/Redis base를 기본으로 맞추고, ES가 필요한 평가/검색 품질 검증은 기존 IntegrationTestBase에 남긴다. 테스트 전용 H2 설정은 @TestConfiguration으로 제한해 SpringBootTest 스캔에 새지 않게 한다.
Constraint: CI에서 Elasticsearch Testcontainer 시작 timeout이 일반 통합 테스트 컨텍스트까지 실패시킴
Rejected: 모든 integrationTest에서 IntegrationTestBase 유지 | ES 비사용 API 테스트가 ES 컨테이너 상태에 종속됨
Confidence: high
Scope-risk: moderate
Directive: ES 실기동이 필요한 테스트만 IntegrationTestBase를 사용하고 일반 API IT는 MySqlRedisIntegrationTestBase를 사용한다
Tested: ./gradlew integrationTest; ./gradlew integrationTest --tests 'com.techfork.auth.integration.AuthIntegrationTest'; ./gradlew integrationTest --tests 'com.techfork.activity.bookmark.integration.BookmarkIntegrationTest'; ./gradlew test --tests 'com.techfork.domain.source.batch.RssFeedReaderTest.enclosureExistsWithoutMedia_ExtractsThumbnail' -PexcludeIntegration
Not-tested: ./gradlew test -PexcludeIntegration 전체 레인은 RssFeedReaderTest enclosure timeout 플래키 1건으로 실패 후 단일 재실행 통과
1 parent b413fb1 commit c99432c
150 files changed
Lines changed: 7248 additions & 6767 deletions
File tree
- docs
- convention
- ddd
- ubiquitous-language
- src/test
- java/com/techfork
- activity
- bookmark
- application
- command
- query
- lookup
- domain
- fixture
- infrastructure
- integration
- readhistory
- application
- command
- query/lookup
- fixture
- infrastructure
- integration
- presentation
- readpost
- application
- command
- query
- lookup
- domain
- fixture
- infrastructure
- integration
- auth
- application/command
- infrastructure/kakao
- integration
- security
- cache
- cookie
- filter
- handler/login
- integration
- jwt
- oauth
- domain
- recommendation
- converter
- entity
- fixture
- integration
- listener
- repository
- scheduler
- service
- search/service
- source
- batch
- config
- entity
- fixture
- listener
- scheduler
- service
- global
- common
- configuration
- util
- personalization
- application
- activity
- event
- generation
- fixture
- integration
- scheduler
- post
- application
- batch
- command
- query
- lookup
- summary
- domain
- fixture
- infrastructure
- batch
- integration
- useraccount
- application
- auth
- command
- query
- lookup
- reader
- domain
- fixture
- infrastructure
- integration
- presentation/request
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
55 | | - | |
56 | | - | |
| 55 | + | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
| 73 | + | |
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
16 | | - | |
17 | 15 | | |
18 | 16 | | |
19 | 17 | | |
20 | | - | |
| 18 | + | |
| 19 | + | |
21 | 20 | | |
22 | 21 | | |
23 | 22 | | |
| |||
File renamed without changes.
File renamed without changes.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
File renamed without changes.
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
88 | | - | |
| 87 | + | |
| 88 | + | |
89 | 89 | | |
90 | 90 | | |
91 | 91 | | |
| |||
127 | 127 | | |
128 | 128 | | |
129 | 129 | | |
130 | | - | |
| 130 | + | |
131 | 131 | | |
132 | 132 | | |
133 | | - | |
| 133 | + | |
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
| |||
File renamed without changes.
File renamed without changes.
Lines changed: 17 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
| 3 | + | |
3 | 4 | | |
4 | | - | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
11 | | - | |
| 12 | + | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
| |||
195 | 196 | | |
196 | 197 | | |
197 | 198 | | |
198 | | - | |
199 | | - | |
| 199 | + | |
| 200 | + | |
200 | 201 | | |
201 | 202 | | |
202 | 203 | | |
| |||
268 | 269 | | |
269 | 270 | | |
270 | 271 | | |
271 | | - | |
272 | | - | |
| 272 | + | |
| 273 | + | |
273 | 274 | | |
274 | 275 | | |
275 | 276 | | |
| |||
333 | 334 | | |
334 | 335 | | |
335 | 336 | | |
336 | | - | |
| 337 | + | |
337 | 338 | | |
338 | 339 | | |
339 | 340 | | |
| |||
413 | 414 | | |
414 | 415 | | |
415 | 416 | | |
416 | | - | |
| 417 | + | |
417 | 418 | | |
418 | 419 | | |
419 | 420 | | |
| |||
427 | 428 | | |
428 | 429 | | |
429 | 430 | | |
430 | | - | |
| 431 | + | |
431 | 432 | | |
432 | 433 | | |
433 | 434 | | |
| |||
446 | 447 | | |
447 | 448 | | |
448 | 449 | | |
449 | | - | |
450 | | - | |
| 450 | + | |
| 451 | + | |
451 | 452 | | |
452 | 453 | | |
453 | 454 | | |
| |||
565 | 566 | | |
566 | 567 | | |
567 | 568 | | |
568 | | - | |
| 569 | + | |
569 | 570 | | |
570 | 571 | | |
571 | 572 | | |
| |||
599 | 600 | | |
600 | 601 | | |
601 | 602 | | |
602 | | - | |
| 603 | + | |
603 | 604 | | |
604 | 605 | | |
605 | 606 | | |
606 | 607 | | |
607 | 608 | | |
608 | | - | |
| 609 | + | |
609 | 610 | | |
610 | 611 | | |
611 | 612 | | |
| |||
630 | 631 | | |
631 | 632 | | |
632 | 633 | | |
633 | | - | |
| 634 | + | |
634 | 635 | | |
635 | 636 | | |
636 | 637 | | |
| |||
721 | 722 | | |
722 | 723 | | |
723 | 724 | | |
724 | | - | |
| 725 | + | |
725 | 726 | | |
726 | 727 | | |
727 | 728 | | |
| |||
0 commit comments