Commit @c2d02b486456e5a5d659de3ad7e8de7d6726cc0f

이호영 07-28

feat: 목록접수 자동 전환에 핀 고정 조건 추가, 공공누리 유형 표기 통일

[18] 목록접수 조건이 자료에 적힌 것과 달랐다.
     자료  : 「목록접수 = Mattermost에 첫 자료 업로드 + 핀 고정 시」
     구현  : 파일이 올라가기만 하면 넘어감 (핀 고정 조건 누락)

공지 고정이 끝난 뒤에 판단하도록 옮겼다. 자료만 올리고 고정하지 않은 경우, 자료 없이
공지만 올린 경우, 고정에 실패한 경우 모두 넘어가지 않는다. 다섯 갈래를 테스트로 잠갔다.

[통일] 공공누리 유형 표기를 「N유형」으로 맞춘다.
     조사표에 「제4유형」으로 적혀 온 자료가 68건 있었는데 나머지는 모두 「0~4유형」
     형태였다. 같은 유형인데 유형별 집계에서 따로 잡히던 문제다.

세 곳을 함께 손봤다.
  · 기존 데이터를 정규식으로 일괄 변환(V18). 뜻이 다른 「유형 확인 필요」는 건드리지 않는다
  · 「제4유형」을 선택지에서 제외
  · 업로드 시점에도 옛 표기를 표준 표기로 바꿔 넣는다 - 이게 없으면 기관이 예전 파일을
    다시 올릴 때 되살아난다

「제4유형」 코드 행은 지우지 않고 비활성 별칭으로 남긴다. 업로드 변환이 이 행의 sameAs를
보고 동작하기 때문이다. 코드표가 별칭까지 관리하는 셈이라 앞으로 다른 표기가 발견돼도
데이터 한 줄로 흡수할 수 있다.

백엔드 288건 통과.

Co-Authored-By: Claude Opus 5 (1M context)

@c2d02b486456e5a5d659de3ad7e8de7d6726cc0f

9e8c4ce

c2d02b4

src/main/java/kr/itn/itnhub/code/CodeService.java

--- src/main/java/kr/itn/itnhub/code/CodeService.java

+++ src/main/java/kr/itn/itnhub/code/CodeService.java

...	...	@@ -108,6 +108,30 @@
108	108	return group(groupId).stream().anyMatch(candidate -> candidate.code().equals(code));
109	109	}
110	110
	111	+ /**
	112	+ * 옛 표기를 표준 표기로 바꿔 준다.
	113	+ *
	114	+ * <p>코드에 {@code sameAs}가 붙어 있으면 그 값으로 옮긴다. 예를 들어 기관 조사표에
	115	+ * 「제4유형」으로 적혀 오는 값은 「4유형」으로 통일한다 - 같은 유형인데 표기가 갈리면
	116	+ * 유형별로 세거나 걸러낼 때 따로 잡히기 때문이다.</p>
	117	+ *
	118	+ * <p>비활성 코드도 본다. 선택지에서는 빠졌어도 옛 표기가 담긴 파일이 다시 올라올 수
	119	+ * 있고, 그때 바꿔 넣어야 하기 때문이다.</p>
	120	+ *
	121	+ * @return 표준 표기. 아는 별칭이 아니면 받은 값을 그대로 돌려준다.
	122	+ */
	123	+ public String canonical(String groupId, String code) {
	124	+ if (code == null \|\| code.isBlank()) {
	125	+ return code;
	126	+ }
	127	+ return find(groupId, code)
	128	+ .map(found -> found.attr(ATTR_SAME_AS))
	129	+ .filter(target -> target != null && !target.isBlank())
	130	+ .orElse(code);
	131	+ }
	132	+
	133	+ private static final String ATTR_SAME_AS = "sameAs";
	134	+
111	135	/** 코드 관리 화면이 쓰는 그룹 메타. 캐시하지 않는다 - 호출 빈도가 낮다. */
112	136	public List<CodeGroup> groups() {
113	137	return mapper.findAllGroups();

9e8c4ce

c2d02b4

src/main/java/kr/itn/itnhub/feed/ChannelFeedService.java

--- src/main/java/kr/itn/itnhub/feed/ChannelFeedService.java

+++ src/main/java/kr/itn/itnhub/feed/ChannelFeedService.java


         String postId = mattermost.createPost(channelId, message, fileIds);
         PostView post = fetchPost(channelId, postId, message);
 
-        // 요구사항 [18]: 채널에 자료가 올라가면 ②목록접수로 저절로 넘어간다.
-        // 파일이 실린 게시글만 계기로 본다 - 그냥 대화만 오간 것은 자료 접수가 아니다.
-        if (!fileIds.isEmpty()) {
-            stageService.autoAdvance(orgId, kr.itn.itnhub.stage.AutoStage.LIST_RECEIVED,
-                    kr.itn.itnhub.stage.AutoStage.REASON_LIST_RECEIVED);
-        }
-
         if (!notice) {
             return new SendResult(post, null);
         }

             }
             mattermost.pinPost(postId);
             mattermost.updateChannelHeader(channelId, noticeHeader(message, files, postId));
+
+            // 요구사항 [18]: 「목록접수 = Mattermost에 첫 자료 업로드 + 핀 고정 시」.
+            // 자료가 실려 있고 공지로 고정까지 된 경우에만 넘긴다 - 그냥 파일만 주고받은 것은
+            // 접수가 아니고, 자료 없는 공지도 접수가 아니다. 핀 고정이 끝난 뒤에 판단해야
+            // "고정까지 됐다"는 조건이 실제로 지켜진다.
+            if (!fileIds.isEmpty()) {
+                stageService.autoAdvance(orgId, kr.itn.itnhub.stage.AutoStage.LIST_RECEIVED,
+                        kr.itn.itnhub.stage.AutoStage.REASON_LIST_RECEIVED);
+            }
             return new SendResult(post, null);
         } catch (MattermostException e) {
+            // 고정에 실패했으면 단계도 넘기지 않는다.
             return new SendResult(post, "공지 등록 처리 중 일부가 실패했습니다: " + e.getMessage());
         }
     }

9e8c4ce

c2d02b4

src/main/java/kr/itn/itnhub/process/ProcessImportService.java

--- src/main/java/kr/itn/itnhub/process/ProcessImportService.java

+++ src/main/java/kr/itn/itnhub/process/ProcessImportService.java


     private final ProcessItemMapper mapper;
     private final OrganizationMapper orgMapper;
     private final kr.itn.itnhub.stage.StageService stageService;
+    private final kr.itn.itnhub.code.CodeService codeService;
     private final TransactionTemplate transactionTemplate;
 
     public ProcessImportService(ProcessParser parser, ProcessItemMapper mapper, OrganizationMapper orgMapper,
                                  kr.itn.itnhub.stage.StageService stageService,
+                                 kr.itn.itnhub.code.CodeService codeService,
                                  PlatformTransactionManager transactionManager) {
         this.parser = parser;
         this.mapper = mapper;
         this.orgMapper = orgMapper;
         this.stageService = stageService;
+        this.codeService = codeService;
         this.transactionTemplate = new TransactionTemplate(transactionManager);
+    }
+
+    /** 공공누리 유형의 옛 표기를 표준 표기로 맞춘다(제4유형 → 4유형). */
+    private String canonicalKogl(String value) {
+        return codeService.canonical(kr.itn.itnhub.code.Codes.KOGL_TYPE, value);
     }
 
     /**

         item.setUrl(row.url());
         item.setDescription(row.description());
         item.setHasAttachment(row.hasAttachment());
-        item.setPriorKoglType(row.priorKoglType());
+        // 옛 표기(제4유형)로 적혀 온 값은 표준 표기(4유형)로 맞춰 넣는다.
+        item.setPriorKoglType(canonicalKogl(row.priorKoglType()));
         item.setContractDocs(row.contractDocs());
         item.setProducedDate(row.producedDate());
         item.setPublishedDate(row.publishedDate());
         item.setReviewMajor(row.reviewMajor());
         item.setReviewMinor(row.reviewMinor());
         item.setReviewResult(row.reviewResult());
-        item.setReviewKoglType(row.reviewKoglType());
+        item.setReviewKoglType(canonicalKogl(row.reviewKoglType()));
         item.setReviewAiType(row.reviewAiType());
         item.setReviewOpinion(row.reviewOpinion());
         item.setReviewNote(row.reviewNote());
         item.setPriorEvidence(row.priorEvidence());
-        item.setJudgedKoglType(row.judgedKoglType());
+        item.setJudgedKoglType(canonicalKogl(row.judgedKoglType()));
         item.setJudgedAiType(row.judgedAiType());
         item.setFinalOpinion(row.finalOpinion());
         item.setJudgmentBasis(row.judgmentBasis());

9e8c4ce

c2d02b4

src/main/java/kr/itn/itnhub/review/ReviewImportService.java

--- src/main/java/kr/itn/itnhub/review/ReviewImportService.java

+++ src/main/java/kr/itn/itnhub/review/ReviewImportService.java


     private final ReviewItemMapper mapper;
     private final OrganizationMapper orgMapper;
     private final kr.itn.itnhub.stage.StageService stageService;
+    private final kr.itn.itnhub.code.CodeService codeService;
     private final TransactionTemplate transactionTemplate;
 
     public ReviewImportService(ReviewParser parser, ReviewItemMapper mapper, OrganizationMapper orgMapper,
                                 kr.itn.itnhub.stage.StageService stageService,
+                                kr.itn.itnhub.code.CodeService codeService,
                                 PlatformTransactionManager transactionManager) {
         this.parser = parser;
         this.mapper = mapper;
         this.orgMapper = orgMapper;
         this.stageService = stageService;
+        this.codeService = codeService;
         this.transactionTemplate = new TransactionTemplate(transactionManager);
+    }
+
+    /** 공공누리 유형의 옛 표기를 표준 표기로 맞춘다(제4유형 → 4유형). */
+    private String canonicalKogl(String value) {
+        return codeService.canonical(kr.itn.itnhub.code.Codes.KOGL_TYPE, value);
     }
 
     /**

         item.setPublishedDate(row.publishedDate());
         item.setHasAttachment(row.hasAttachment());
         item.setKoglAttached(row.koglAttached());
-        item.setKoglType(row.koglType());
+        // 옛 표기(제4유형)로 적혀 온 값은 표준 표기(4유형)로 맞춰 넣는다.
+        item.setKoglType(canonicalKogl(row.koglType()));
         item.setAiType(row.aiType());
         item.setSurveyorNote(row.surveyorNote());
         item.setOpenable(row.openable());
         item.setReviewMajor(row.reviewMajor());
         item.setReviewMinor(row.reviewMinor());
         item.setReviewResult(row.reviewResult());
-        item.setJudgedKoglType(row.judgedKoglType());
+        item.setJudgedKoglType(canonicalKogl(row.judgedKoglType()));
         item.setJudgedAiType(row.judgedAiType());
         item.setOpinion(row.opinion());
         item.setLawyerNote(row.lawyerNote());

9e8c4ce

c2d02b4

src/main/java/kr/itn/itnhub/stage/AutoStage.java

--- src/main/java/kr/itn/itnhub/stage/AutoStage.java

+++ src/main/java/kr/itn/itnhub/stage/AutoStage.java


     private AutoStage() {
     }
 
-    /** ②목록접수 — 기관 채널에 첫 자료가 올라갔을 때. */
+    /** ②목록접수 — 기관 채널에 자료가 올라가고 공지로 고정까지 됐을 때. */
     public static final int LIST_RECEIVED = 2;
-    public static final String REASON_LIST_RECEIVED = "채널에 자료 업로드";
+    public static final String REASON_LIST_RECEIVED = "채널에 자료 업로드 + 공지 고정";
 
     /** ⑤법률검토(권리확인) — 권리확인 자료가 올라갔을 때. */
     public static final int REVIEW_STARTED = 5;

c2d02b4

src/main/resources/db/migration/V18__normalize_kogl_type.sql (added)

+++ src/main/resources/db/migration/V18__normalize_kogl_type.sql

...	...	@@ -0,0 +1,35 @@
	1	+-- 공공누리 유형 표기를 「N유형」으로 통일한다.
	2	+--
	3	+-- 기관 조사표에 「제4유형」으로 적혀 온 자료가 68건 있었다. 나머지 자료와 코드표는 전부
	4	+-- 「0유형 ~ 4유형」 형태라 같은 유형인데도 다른 값으로 잡혔다. 유형별로 세거나 걸러낼 때
	5	+-- 둘이 따로 집계되므로 표기를 맞춘다.
	6	+--
	7	+-- '제숫자유형' 형태만 바꾼다. 「유형 확인 필요」처럼 뜻이 다른 값은 건드리지 않는다.
	8	+
	9	+update review_item
	10	+ set kogl_type = regexp_replace(kogl_type, '^제([0-9]+)유형$', '\1유형')
	11	+ where kogl_type ~ '^제[0-9]+유형$';
	12	+
	13	+update review_item
	14	+ set judged_kogl_type = regexp_replace(judged_kogl_type, '^제([0-9]+)유형$', '\1유형')
	15	+ where judged_kogl_type ~ '^제[0-9]+유형$';
	16	+
	17	+update process_item
	18	+ set prior_kogl_type = regexp_replace(prior_kogl_type, '^제([0-9]+)유형$', '\1유형')
	19	+ where prior_kogl_type ~ '^제[0-9]+유형$';
	20	+
	21	+update process_item
	22	+ set review_kogl_type = regexp_replace(review_kogl_type, '^제([0-9]+)유형$', '\1유형')
	23	+ where review_kogl_type ~ '^제[0-9]+유형$';
	24	+
	25	+update process_item
	26	+ set judged_kogl_type = regexp_replace(judged_kogl_type, '^제([0-9]+)유형$', '\1유형')
	27	+ where judged_kogl_type ~ '^제[0-9]+유형$';
	28	+
	29	+-- 「제4유형」은 이제 선택지에서 뺀다. 다만 행을 지우지는 않는다 -
	30	+-- 기관이 예전 표기 그대로 엑셀을 다시 올릴 수 있고, 그때 sameAs를 보고 「4유형」으로
	31	+-- 바꿔 넣어야 하기 때문이다(별칭 역할로 남긴다).
	32	+update code
	33	+ set active = false,
	34	+ label = '제4유형 (옛 표기 → 4유형)'
	35	+ where group_id = 'KOGL_TYPE' and code = '제4유형';

9e8c4ce

c2d02b4

src/test/java/kr/itn/itnhub/code/CodeControllerTest.java

--- src/test/java/kr/itn/itnhub/code/CodeControllerTest.java

+++ src/test/java/kr/itn/itnhub/code/CodeControllerTest.java


                 .andExpect(jsonPath("$.STAGE[12].code").value("13"))
                 .andExpect(jsonPath("$.STAGE[12].label").value("보고서 제출 완료"))
                 .andExpect(jsonPath("$.STAGE[12].attrs.progress").value("done"))
-                // 0~4유형 + 개방불가 + 보류 + 조사표에서 온 2개(유형 확인 필요, 제4유형)
-                .andExpect(jsonPath("$.KOGL_TYPE.length()").value(9))
+                // 0~4유형 + 개방불가 + 보류 + 조사표에서 온 '유형 확인 필요'.
+                // '제4유형'은 '4유형'으로 통일해 선택지에서 빠졌다(V18).
+                .andExpect(jsonPath("$.KOGL_TYPE.length()").value(8))
                 .andExpect(jsonPath("$.CONTACT_CATEGORY[0].code").value("APPLICANT"))
                 .andExpect(jsonPath("$.CONTACT_CATEGORY[0].label").value("신청기관"));
     }

9e8c4ce

c2d02b4

src/test/java/kr/itn/itnhub/code/CodeSeedTest.java

--- src/test/java/kr/itn/itnhub/code/CodeSeedTest.java

+++ src/test/java/kr/itn/itnhub/code/CodeSeedTest.java

...	...	@@ -74,16 +74,32 @@
74	74	@Test
75	75	void 공공누리_유형은_사용처별로_다르게_노출된다() {
76	76	// 세 화면의 목록이 7개/5개/6개로 달랐던 이유는 쓰이는 자리가 달라서다.
77		- assertThat(koglScope("review")).hasSize(7).doesNotContain("유형 확인 필요");
	77	+ assertThat(koglScope("review")).hasSize(7)
	78	+ .doesNotContain("유형 확인 필요")
	79	+ .doesNotContain("제4유형");
78	80	assertThat(koglScope("process")).hasSize(6).doesNotContain("보류");
79	81	}
80	82
81	83	@Test
82	84	void 기존_부착유형에는_조사표에서_온_값도_들어간다() {
83		- // 운영 DB 실측(2026-07-28): kogl_type에 '유형 확인 필요' 1,858건, '제4유형' 68건이
84		- // 저장돼 있었다. 코드로 등록해 두지 않으면 상세 화면에서 선택 안 됨으로 보인다.
	85	+ // 운영 DB 실측(2026-07-28): kogl_type에 '유형 확인 필요' 1,858건이 저장돼 있었다.
	86	+ // 코드로 등록해 두지 않으면 상세 화면에서 선택 안 됨으로 보인다.
	87	+ // '제4유형'은 '4유형'으로 통일했으므로 선택지에서는 빠진다(V18).
85	88	assertThat(koglScope("prior")).containsExactly(
86		- "0유형", "1유형", "2유형", "3유형", "4유형", "유형 확인 필요", "제4유형");
	89	+ "0유형", "1유형", "2유형", "3유형", "4유형", "유형 확인 필요");
	90	+ }
	91	+
	92	+ /**
	93	+ * 「제4유형」은 지웠으면 안 된다. 기관이 옛 표기 그대로 엑셀을 다시 올릴 수 있고,
	94	+ * 그때 sameAs를 보고 「4유형」으로 바꿔 넣어야 한다.
	95	+ */
	96	+ @Test
	97	+ void 통일된_옛_표기는_별칭으로_남아_있다() {
	98	+ String active = jdbc.queryForObject(
	99	+ "select active::text from code where group_id = 'KOGL_TYPE' and code = '제4유형'",
	100	+ String.class);
	101	+ assertThat(active).isEqualTo("false");
	102	+ assertThat(attr("KOGL_TYPE", "제4유형", "sameAs")).isEqualTo("4유형");
87	103	}
88	104
89	105	@Test
...	...	@@ -99,9 +115,10 @@
99	115	assertThat(leaked).isZero();
100	116	}
101	117
	118	+ /** 화면 선택지에 실제로 나가는 것만 본다. 서버는 활성 코드만 내려준다. */
102	119	private List<String> koglScope(String scope) {
103	120	return jdbc.queryForList(
104		- "select code from code c where c.group_id = 'KOGL_TYPE' "
	121	+ "select code from code c where c.group_id = 'KOGL_TYPE' and c.active "
105	122	+ " and exists (select 1 from jsonb_array_elements_text(c.attrs->'scopes') s(v) "
106	123	+ " where s.v = ?) "
107	124	+ "order by sort_order",

9e8c4ce

c2d02b4

src/test/java/kr/itn/itnhub/code/CodeServiceTest.java

--- src/test/java/kr/itn/itnhub/code/CodeServiceTest.java

+++ src/test/java/kr/itn/itnhub/code/CodeServiceTest.java

...	...	@@ -58,12 +58,24 @@
58	58	assertThat(service.scoped(Codes.KOGL_TYPE, Codes.SCOPE_PROCESS))
59	59	.hasSize(6)
60	60	.noneMatch(c -> c.code().equals("보류"));
61		- // prior에는 기관 조사표에서 온 값('유형 확인 필요', '제4유형')까지 들어간다.
	61	+ // prior에는 기관 조사표에서 온 값('유형 확인 필요')까지 들어간다.
62	62	// 운영 DB에 실제로 저장돼 있는 값이라 등록해 두지 않으면 화면에서 선택 안 됨으로 보인다.
63	63	assertThat(service.scoped(Codes.KOGL_TYPE, Codes.SCOPE_PRIOR))
64	64	.extracting(Code::code)
65		- .containsExactly("0유형", "1유형", "2유형", "3유형", "4유형",
66		- "유형 확인 필요", "제4유형");
	65	+ .containsExactly("0유형", "1유형", "2유형", "3유형", "4유형", "유형 확인 필요");
	66	+ }
	67	+
	68	+ @Test
	69	+ void 옛_표기를_표준_표기로_바꿔_준다() {
	70	+ // 「제4유형」으로 적혀 온 값은 「4유형」으로 통일한다.
	71	+ assertThat(service.canonical(Codes.KOGL_TYPE, "제4유형")).isEqualTo("4유형");
	72	+ // 표준 표기는 그대로 둔다.
	73	+ assertThat(service.canonical(Codes.KOGL_TYPE, "4유형")).isEqualTo("4유형");
	74	+ // 뜻이 다른 값은 건드리지 않는다.
	75	+ assertThat(service.canonical(Codes.KOGL_TYPE, "유형 확인 필요")).isEqualTo("유형 확인 필요");
	76	+ // 모르는 값도 그대로 돌려준다.
	77	+ assertThat(service.canonical(Codes.KOGL_TYPE, "이상한값")).isEqualTo("이상한값");
	78	+ assertThat(service.canonical(Codes.KOGL_TYPE, null)).isNull();
67	79	}
68	80
69	81	@Test

c2d02b4

src/test/java/kr/itn/itnhub/feed/ChannelUploadAutoStageTest.java (added)

+++ src/test/java/kr/itn/itnhub/feed/ChannelUploadAutoStageTest.java

...	...	@@ -0,0 +1,124 @@
	1	+package kr.itn.itnhub.feed;
	2	+
	3	+import kr.itn.itnhub.AbstractDbTest;
	4	+import kr.itn.itnhub.mattermost.MattermostClient;
	5	+import kr.itn.itnhub.mattermost.MattermostException;
	6	+import kr.itn.itnhub.org.Organization;
	7	+import kr.itn.itnhub.org.OrganizationMapper;
	8	+import kr.itn.itnhub.stage.AutoStage;
	9	+import kr.itn.itnhub.stage.StageHistoryMapper;
	10	+import org.junit.jupiter.api.BeforeEach;
	11	+import org.junit.jupiter.api.Test;
	12	+import org.springframework.beans.factory.annotation.Autowired;
	13	+import org.springframework.boot.test.mock.mockito.MockBean;
	14	+import org.springframework.jdbc.core.JdbcTemplate;
	15	+
	16	+import java.util.List;
	17	+
	18	+import static org.assertj.core.api.Assertions.assertThat;
	19	+import static org.mockito.ArgumentMatchers.any;
	20	+import static org.mockito.ArgumentMatchers.anyString;
	21	+import static org.mockito.Mockito.doThrow;
	22	+import static org.mockito.Mockito.when;
	23	+
	24	+/**
	25	+ * 「목록접수 = Mattermost에 첫 자료 업로드 + 핀 고정 시」(기능확인 PPT 14쪽, 요구사항 [18]).
	26	+ *
	27	+ * <p>조건이 둘이라는 점이 핵심이다. 자료만 올리고 공지로 고정하지 않았으면 접수가 아니고,
	28	+ * 자료 없이 공지만 올린 것도 접수가 아니다.</p>
	29	+ */
	30	+class ChannelUploadAutoStageTest extends AbstractDbTest {
	31	+
	32	+ @Autowired
	33	+ ChannelFeedService feedService;
	34	+
	35	+ @Autowired
	36	+ OrganizationMapper orgMapper;
	37	+
	38	+ @Autowired
	39	+ StageHistoryMapper historyMapper;
	40	+
	41	+ @Autowired
	42	+ JdbcTemplate jdbc;
	43	+
	44	+ @MockBean
	45	+ MattermostClient mattermost;
	46	+
	47	+ private Long orgId;
	48	+
	49	+ @BeforeEach
	50	+ void setUp() {
	51	+ jdbc.update("delete from stage_history");
	52	+ jdbc.update("delete from organization where org_no = '940'");
	53	+
	54	+ Organization org = new Organization();
	55	+ org.setOrgNo("940");
	56	+ org.setOrgName("업로드전환테스트기관");
	57	+ org.setChannelSlug("940");
	58	+ orgMapper.upsertBySeed(org);
	59	+ orgId = orgMapper.findByOrgNoAndOrgName("940", "업로드전환테스트기관").getId();
	60	+ orgMapper.updateChannelIdMj(orgId, "chan-mj");
	61	+ orgMapper.updateChannelIdLaw(orgId, "chan-law");
	62	+
	63	+ // 채널 생성으로 ①신청까지 와 있는 상태를 만든다.
	64	+ orgMapper.updateStage(orgId, 1);
	65	+ historyMapper.insert(orgId, 1, true, "채널 생성");
	66	+
	67	+ when(mattermost.uploadFile(anyString(), anyString(), any(), anyString()))
	68	+ .thenReturn("file-1");
	69	+ when(mattermost.createPost(anyString(), anyString(), any())).thenReturn("post-1");
	70	+ when(mattermost.getRecentPosts(anyString(), any(Integer.class))).thenReturn(List.of());
	71	+ when(mattermost.getPinnedPosts(anyString())).thenReturn(List.of());
	72	+ }
	73	+
	74	+ private UploadedFile file() {
	75	+ return new UploadedFile("목록.xlsx", new byte[]{1, 2, 3},
	76	+ "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet");
	77	+ }
	78	+
	79	+ private Integer stage() {
	80	+ return orgMapper.findById(orgId).getStage();
	81	+ }
	82	+
	83	+ @Test
	84	+ void 자료를_공지로_고정해서_올리면_목록접수로_넘어간다() {
	85	+ feedService.sendMessage(orgId, "mj", "목록 보냅니다", List.of(file()), true);
	86	+
	87	+ assertThat(stage()).isEqualTo(AutoStage.LIST_RECEIVED);
	88	+ assertThat(historyMapper.findByOrg(orgId).get(0).isAuto()).isTrue();
	89	+ assertThat(historyMapper.findByOrg(orgId).get(0).getReason())
	90	+ .isEqualTo(AutoStage.REASON_LIST_RECEIVED);
	91	+ }
	92	+
	93	+ @Test
	94	+ void 자료만_올리고_고정하지_않으면_넘어가지_않는다() {
	95	+ feedService.sendMessage(orgId, "mj", "그냥 파일 하나 드립니다", List.of(file()), false);
	96	+
	97	+ assertThat(stage()).isEqualTo(1);
	98	+ }
	99	+
	100	+ @Test
	101	+ void 자료_없이_공지만_올리면_넘어가지_않는다() {
	102	+ feedService.sendMessage(orgId, "mj", "안내드립니다", List.of(), true);
	103	+
	104	+ assertThat(stage()).isEqualTo(1);
	105	+ }
	106	+
	107	+ @Test
	108	+ void 고정에_실패하면_단계도_넘기지_않는다() {
	109	+ doThrow(new MattermostException("핀 고정 실패")).when(mattermost).pinPost(anyString());
	110	+
	111	+ feedService.sendMessage(orgId, "mj", "목록 보냅니다", List.of(file()), true);
	112	+
	113	+ assertThat(stage()).isEqualTo(1);
	114	+ }
	115	+
	116	+ @Test
	117	+ void 이미_더_앞선_단계면_뒤로_밀지_않는다() {
	118	+ orgMapper.updateStage(orgId, 7);
	119	+
	120	+ feedService.sendMessage(orgId, "mj", "목록 보냅니다", List.of(file()), true);
	121	+
	122	+ assertThat(stage()).isEqualTo(7);
	123	+ }
	124	+}

Add a comment

Open 0
Closed 0

List

...	...	@@ -72,13 +72,6 @@
72	72	String postId = mattermost.createPost(channelId, message, fileIds);
73	73	PostView post = fetchPost(channelId, postId, message);
74	74
75		- // 요구사항 [18]: 채널에 자료가 올라가면 ②목록접수로 저절로 넘어간다.
76		- // 파일이 실린 게시글만 계기로 본다 - 그냥 대화만 오간 것은 자료 접수가 아니다.
77		- if (!fileIds.isEmpty()) {
78		- stageService.autoAdvance(orgId, kr.itn.itnhub.stage.AutoStage.LIST_RECEIVED,
79		- kr.itn.itnhub.stage.AutoStage.REASON_LIST_RECEIVED);
80		- }
81		-
82	75	if (!notice) {
83	76	return new SendResult(post, null);
84	77	}
...	...	@@ -89,8 +82,18 @@
89	82	}
90	83	mattermost.pinPost(postId);
91	84	mattermost.updateChannelHeader(channelId, noticeHeader(message, files, postId));
	85	+
	86	+ // 요구사항 [18]: 「목록접수 = Mattermost에 첫 자료 업로드 + 핀 고정 시」.
	87	+ // 자료가 실려 있고 공지로 고정까지 된 경우에만 넘긴다 - 그냥 파일만 주고받은 것은
	88	+ // 접수가 아니고, 자료 없는 공지도 접수가 아니다. 핀 고정이 끝난 뒤에 판단해야
	89	+ // "고정까지 됐다"는 조건이 실제로 지켜진다.
	90	+ if (!fileIds.isEmpty()) {
	91	+ stageService.autoAdvance(orgId, kr.itn.itnhub.stage.AutoStage.LIST_RECEIVED,
	92	+ kr.itn.itnhub.stage.AutoStage.REASON_LIST_RECEIVED);
	93	+ }
92	94	return new SendResult(post, null);
93	95	} catch (MattermostException e) {
	96	+ // 고정에 실패했으면 단계도 넘기지 않는다.
94	97	return new SendResult(post, "공지 등록 처리 중 일부가 실패했습니다: " + e.getMessage());
95	98	}
96	99	}

...	...	@@ -18,16 +18,24 @@
18	18	private final ProcessItemMapper mapper;
19	19	private final OrganizationMapper orgMapper;
20	20	private final kr.itn.itnhub.stage.StageService stageService;
	21	+ private final kr.itn.itnhub.code.CodeService codeService;
21	22	private final TransactionTemplate transactionTemplate;
22	23
23	24	public ProcessImportService(ProcessParser parser, ProcessItemMapper mapper, OrganizationMapper orgMapper,
24	25	kr.itn.itnhub.stage.StageService stageService,
	26	+ kr.itn.itnhub.code.CodeService codeService,
25	27	PlatformTransactionManager transactionManager) {
26	28	this.parser = parser;
27	29	this.mapper = mapper;
28	30	this.orgMapper = orgMapper;
29	31	this.stageService = stageService;
	32	+ this.codeService = codeService;
30	33	this.transactionTemplate = new TransactionTemplate(transactionManager);
	34	+ }
	35	+
	36	+ /** 공공누리 유형의 옛 표기를 표준 표기로 맞춘다(제4유형 → 4유형). */
	37	+ private String canonicalKogl(String value) {
	38	+ return codeService.canonical(kr.itn.itnhub.code.Codes.KOGL_TYPE, value);
31	39	}
32	40
33	41	/**
...	...	@@ -78,19 +86,20 @@
78	86	item.setUrl(row.url());
79	87	item.setDescription(row.description());
80	88	item.setHasAttachment(row.hasAttachment());
81		- item.setPriorKoglType(row.priorKoglType());
	89	+ // 옛 표기(제4유형)로 적혀 온 값은 표준 표기(4유형)로 맞춰 넣는다.
	90	+ item.setPriorKoglType(canonicalKogl(row.priorKoglType()));
82	91	item.setContractDocs(row.contractDocs());
83	92	item.setProducedDate(row.producedDate());
84	93	item.setPublishedDate(row.publishedDate());
85	94	item.setReviewMajor(row.reviewMajor());
86	95	item.setReviewMinor(row.reviewMinor());
87	96	item.setReviewResult(row.reviewResult());
88		- item.setReviewKoglType(row.reviewKoglType());
	97	+ item.setReviewKoglType(canonicalKogl(row.reviewKoglType()));
89	98	item.setReviewAiType(row.reviewAiType());
90	99	item.setReviewOpinion(row.reviewOpinion());
91	100	item.setReviewNote(row.reviewNote());
92	101	item.setPriorEvidence(row.priorEvidence());
93		- item.setJudgedKoglType(row.judgedKoglType());
	102	+ item.setJudgedKoglType(canonicalKogl(row.judgedKoglType()));
94	103	item.setJudgedAiType(row.judgedAiType());
95	104	item.setFinalOpinion(row.finalOpinion());
96	105	item.setJudgmentBasis(row.judgmentBasis());

...	...	@@ -18,16 +18,24 @@
18	18	private final ReviewItemMapper mapper;
19	19	private final OrganizationMapper orgMapper;
20	20	private final kr.itn.itnhub.stage.StageService stageService;
	21	+ private final kr.itn.itnhub.code.CodeService codeService;
21	22	private final TransactionTemplate transactionTemplate;
22	23
23	24	public ReviewImportService(ReviewParser parser, ReviewItemMapper mapper, OrganizationMapper orgMapper,
24	25	kr.itn.itnhub.stage.StageService stageService,
	26	+ kr.itn.itnhub.code.CodeService codeService,
25	27	PlatformTransactionManager transactionManager) {
26	28	this.parser = parser;
27	29	this.mapper = mapper;
28	30	this.orgMapper = orgMapper;
29	31	this.stageService = stageService;
	32	+ this.codeService = codeService;
30	33	this.transactionTemplate = new TransactionTemplate(transactionManager);
	34	+ }
	35	+
	36	+ /** 공공누리 유형의 옛 표기를 표준 표기로 맞춘다(제4유형 → 4유형). */
	37	+ private String canonicalKogl(String value) {
	38	+ return codeService.canonical(kr.itn.itnhub.code.Codes.KOGL_TYPE, value);
31	39	}
32	40
33	41	/**
...	...	@@ -82,14 +90,15 @@
82	90	item.setPublishedDate(row.publishedDate());
83	91	item.setHasAttachment(row.hasAttachment());
84	92	item.setKoglAttached(row.koglAttached());
85		- item.setKoglType(row.koglType());
	93	+ // 옛 표기(제4유형)로 적혀 온 값은 표준 표기(4유형)로 맞춰 넣는다.
	94	+ item.setKoglType(canonicalKogl(row.koglType()));
86	95	item.setAiType(row.aiType());
87	96	item.setSurveyorNote(row.surveyorNote());
88	97	item.setOpenable(row.openable());
89	98	item.setReviewMajor(row.reviewMajor());
90	99	item.setReviewMinor(row.reviewMinor());
91	100	item.setReviewResult(row.reviewResult());
92		- item.setJudgedKoglType(row.judgedKoglType());
	101	+ item.setJudgedKoglType(canonicalKogl(row.judgedKoglType()));
93	102	item.setJudgedAiType(row.judgedAiType());
94	103	item.setOpinion(row.opinion());
95	104	item.setLawyerNote(row.lawyerNote());

...	...	@@ -11,9 +11,9 @@
11	11	private AutoStage() {
12	12	}
13	13
14		- /** ②목록접수 — 기관 채널에 첫 자료가 올라갔을 때. */
	14	+ /** ②목록접수 — 기관 채널에 자료가 올라가고 공지로 고정까지 됐을 때. */
15	15	public static final int LIST_RECEIVED = 2;
16		- public static final String REASON_LIST_RECEIVED = "채널에 자료 업로드";
	16	+ public static final String REASON_LIST_RECEIVED = "채널에 자료 업로드 + 공지 고정";
17	17
18	18	/** ⑤법률검토(권리확인) — 권리확인 자료가 올라갔을 때. */
19	19	public static final int REVIEW_STARTED = 5;

...	...	@@ -31,8 +31,9 @@
31	31	.andExpect(jsonPath("$.STAGE[12].code").value("13"))
32	32	.andExpect(jsonPath("$.STAGE[12].label").value("보고서 제출 완료"))
33	33	.andExpect(jsonPath("$.STAGE[12].attrs.progress").value("done"))
34		- // 0~4유형 + 개방불가 + 보류 + 조사표에서 온 2개(유형 확인 필요, 제4유형)
35		- .andExpect(jsonPath("$.KOGL_TYPE.length()").value(9))
	34	+ // 0~4유형 + 개방불가 + 보류 + 조사표에서 온 '유형 확인 필요'.
	35	+ // '제4유형'은 '4유형'으로 통일해 선택지에서 빠졌다(V18).
	36	+ .andExpect(jsonPath("$.KOGL_TYPE.length()").value(8))
36	37	.andExpect(jsonPath("$.CONTACT_CATEGORY[0].code").value("APPLICANT"))
37	38	.andExpect(jsonPath("$.CONTACT_CATEGORY[0].label").value("신청기관"));
38	39	}

Delete comment