데모 전 운영 서버 최종 반영

[EpicFn]

📢 기능 설명

필요시 실행결과 스크린샷 첨부

🩷 Approve 하기 전 확인해주세요!

• 리뷰어가 확인해줬으면 하는 사항 적어주세요.
• [ ]

✅ 체크리스트

• PR 제목 규칙 잘 지켰는가?
• 추가/수정사항을 설명하였는가?
• 이슈넘버를 적었는가?
• Approve 하기 전 확인 사항 체크했는가?

[Copilot]

Pull Request Overview

This PR adds a comprehensive test suite and shared test utilities to validate core domains (member, space, archive, datasource, dashboard, news) and controller endpoints, including integration with security and external clients.

• Introduces a reusable ControllerTestSupport for consistent MockMvc usage and common assertions.
• Adds extensive unit/integration tests across services and controllers (space, membership, archive/folder, datasource, dashboard, news), including messaging and signature verification.
• Mocks/stubs external dependencies (Liveblocks, AI/Chat, tag repo) for deterministic tests.

Reviewed Changes

Copilot reviewed 258 out of 265 changed files in this pull request and generated 11 comments.

Show a summary per file

File	Description
src/test/java/org/tuna/zoopzoop/backend/testSupport/ControllerTestSupport.java	Provides shared MockMvc/ObjectMapper helpers and common response assertions for controller tests.
src/test/java/org/tuna/zoopzoop/backend/domain/space/space/service/SpaceServiceTest.java	Service tests for create/update/delete space behavior and duplicate name handling.
src/test/java/org/tuna/zoopzoop/backend/domain/space/space/controller/ApiV1SpaceControllerTest.java	Controller tests for space CRUD and list endpoints with authority checks.
src/test/java/org/tuna/zoopzoop/backend/domain/space/membership/service/NotificationServiceTest.java	Verifies SSE notification dispatch integration.
src/test/java/org/tuna/zoopzoop/backend/domain/space/membership/service/MembershipServiceTest.java	Membership add/invite/accept/reject/authority change tests.
src/test/java/org/tuna/zoopzoop/backend/domain/space/membership/controller/ApiV1MembershipControllerTest.java	Controller tests for membership listing/invite/authority/expel/leave.
src/test/java/org/tuna/zoopzoop/backend/domain/space/membership/controller/ApiV1InviteControllerTest.java	Invite accept/reject and listing tests.
src/test/java/org/tuna/zoopzoop/backend/domain/space/archive/service/SpaceArchiveFolderServiceTest.java	Service tests for shared archive folder operations with authority checks.
src/test/java/org/tuna/zoopzoop/backend/domain/space/archive/service/SpaceArchiveDataSourceServiceTest.java	Service tests for shared archive datasource operations (import/move/update/delete).
src/test/java/org/tuna/zoopzoop/backend/domain/space/archive/controller/SpaceArchiveFolderControllerTest.java	Controller tests for shared archive folder endpoints.
src/test/java/org/tuna/zoopzoop/backend/domain/space/archive/controller/SpaceArchiveDataSourceControllerTest.java	Controller tests for shared archive datasource endpoints.
src/test/java/org/tuna/zoopzoop/backend/domain/news/service/NewsServiceTest.java	Tests for tag frequency extraction.
src/test/java/org/tuna/zoopzoop/backend/domain/news/service/NewsAPIServiceTest.java	Integration-like test for news search service.
src/test/java/org/tuna/zoopzoop/backend/domain/member/service/MemberServiceTest.java	Member service create/find/update/delete tests.
src/test/java/org/tuna/zoopzoop/backend/domain/member/repository/MemberRepositoryTest.java	Repository test asserting cascade creation of personal archive and default folder.
src/test/java/org/tuna/zoopzoop/backend/domain/member/controller/MemberSearchControllerTest.java	Member search endpoint tests.
src/test/java/org/tuna/zoopzoop/backend/domain/member/controller/MemberControllerTest.java	Member info/edit/delete controller tests.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/service/PersonalArchiveDataSourceServiceTest.java	Service tests for personal datasource operations.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/service/DataSourceServiceTest.java	Domain datasource service tests (create/update/move/delete/search).
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/service/DataProcessorServiceTest.java	Tests for URL processing (network fetch) and error handling.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/service/CrawlerManagerServiceTest.java	Tests for crawler manager with specific/generic site DTOs.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/service/AiServiceTest.java	AI analysis service test with mocked ChatClient.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/repository/DataSourceQRepositoryImplTest.java	Querydsl repository tests for search/sort/paging/active flags.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/crawler/service/VelogCrawlerTest.java	Velog crawler tests.
src/test/java/org/tuna/zoopzoop/backend/domain/datasource/controller/DataSourceControllerTest.java	Personal archive datasource controller tests.
src/test/java/org/tuna/zoopzoop/backend/domain/dashboard/service/GraphServiceTest.java	Graph/nodes/edges persistence test.
src/test/java/org/tuna/zoopzoop/backend/domain/dashboard/extraComponent/GraphUpdateConsumerTest.java	Message consumer integration test with Rabbit + graph update.
src/test/java/org/tuna/zoopzoop/backend/domain/dashboard/controller/DashboardControllerTest.java	Dashboard graph get/update (with webhook signature) tests.
src/test/java/org/tuna/zoopzoop/backend/domain/archive/folder/service/PersonalArchiveFolderServiceTest.java	Personal folder service tests.
src/test/java/org/tuna/zoopzoop/backend/domain/archive/folder/service/FolderServiceTest.java	Folder domain service tests.
src/test/java/org/tuna/zoopzoop/backend/domain/archive/folder/controller/FolderControllerTest.java	Personal folder controller tests.

Files not reviewed (5)

• .idea/.gitignore: Language not supported
• .idea/.name: Language not supported
• .idea/compiler.xml: Language not supported
• .idea/misc.xml: Language not supported
• .idea/vcs.xml: Language not supported

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

These tests perform real network calls (e.g., example.com) which can be flaky and slow in CI. Replace external HTTP calls by stubbing/mocking the fetch (e.g., use WireMock or inject a Jsoup Connection factory), or provide in-memory HTML to unit test parsing logic.

[Copilot]

This test hits a live Velog URL, making the suite dependent on external network and page structure. For reliability, switch to a static HTML fixture (Jsoup.parse) or stub the connection to return deterministic content.

Suggested change

	@Test
	@DisplayName("Velog 크롤러 작동 테스트")
	void testExtract() throws IOException {
	Document doc = Jsoup.connect("https://velog.io/@imcute0703123/%EC%BA%A0%ED%8D%BC%EC%8A%A4%EC%8B%9C%EA%B7%B8%EB%84%90-2025-%EB%B0%B1%EC%84%9D%EC%97%B0%ED%99%94-1%EB%93%B1-%EB%B6%80%EC%8A%A4-%EB%9F%AC%EB%B8%8C%EB%9D%BC%EC%9D%B8-%EB%A7%A4%EC%B9%AD-%EC%84%9C%EB%B9%84%EC%8A%A4-%ED%9A%8C%EA%B3%A0%EB%A1%9D").get();
	// Static HTML fixture representing a Velog post (simplified for test)
	private static final String VELOG_HTML_FIXTURE = "<html><head><title>Test Velog Post</title></head><body>"
	+ "<div class='post-title'>캠퍼스시그널 2025 백석연화 1등 부스 러브라인 매칭 서비스 회고록</div>"
	+ "<div class='post-content'>이것은 테스트용 Velog 포스트 내용입니다.</div>"
	+ "<img class='post-image' src='https://example.com/test-image.jpg' />"
	+ "<div class='post-source'>Velog</div>"
	+ "<div class='post-date'>2024-06-01</div>"
	+ "</body></html>";
	@Test
	@DisplayName("Velog 크롤러 작동 테스트")
	void testExtract() throws IOException {
	Document doc = Jsoup.parse(VELOG_HTML_FIXTURE);

[Copilot]

This mock is never used and not registered in the Spring context (should be @MockBean if overriding a bean). Remove it or convert to @MockBean and use it explicitly to avoid confusion.

Suggested change

@Mock PersonalDataSourceService personalApp;

[Copilot]

Using @qualifier("dataProcessorService") here prevents the @primary stub from being injected and will bind the real bean by name. Since this field is not used in the test, remove the qualifier (or the field) to avoid conflicting wiring.

Suggested change

	@Qualifier("dataProcessorService")
	@Autowired
	private DataProcessorService dataProcessorService;

[Copilot]

If you intend to target a named qualifier, consider naming this bean 'dataProcessorService' (e.g., @bean(name = "dataProcessorService")) or adding @qualifier on injection sites; otherwise, rely solely on @primary without qualifiers to ensure the stub is used.

Suggested change

	@Bean @Primary
	@Bean(name = "dataProcessorService") @Primary

[Copilot]

Directly assigning 'sa.archive = a;' bypasses encapsulation and likely won't compile if the field is private. Use the setter consistently: 'sa.setArchive(a);'.

Suggested change

	sa.archive = a;
	sa.setArchive(a);

[Copilot]

[nitpick] Most tests here use Spring's @transactional (org.springframework.transaction.annotation.Transactional); standardize imports/annotations to the same variant for consistency and clearer rollback semantics in the Spring test context.

Suggested change

	import jakarta.transaction.Transactional;
	import org.springframework.transaction.annotation.Transactional;

[Copilot]

[nitpick] This test doesn't use MockMvc/ObjectMapper from ControllerTestSupport; consider removing the inheritance to avoid pulling in extra test configuration unnecessarily.

Suggested change

	class GraphUpdateConsumerTest extends ControllerTestSupport {
	class GraphUpdateConsumerTest {

[Copilot]

Catching and ignoring all exceptions can hide real setup failures. Prefer checking existence first or catch specific duplicate-key exceptions so genuine errors fail fast.

Suggested change

	try { memberService.createMember("spaceOwner", "http://img/owner.png", OWNER_PK, Provider.KAKAO); } catch (Exception ignored) {}
	try { memberService.createMember("spaceReader", "http://img/reader.png", READER_PK, Provider.KAKAO); } catch (Exception ignored) {}
	if (memberRepository.findByProviderAndProviderKey(Provider.KAKAO, OWNER_PK).isEmpty()) {
	memberService.createMember("spaceOwner", "http://img/owner.png", OWNER_PK, Provider.KAKAO);
	}
	if (memberRepository.findByProviderAndProviderKey(Provider.KAKAO, READER_PK).isEmpty()) {
	memberService.createMember("spaceReader", "http://img/reader.png", READER_PK, Provider.KAKAO);
	}

[Copilot]

Avoid swallowing exceptions in test setup; check for existing member or catch a specific duplicate exception to prevent masking actual setup issues.

Suggested change

	try {
	memberService.createMember("testUser_sc1111", "http://img", TEST_PROVIDER_KEY, Provider.KAKAO);
	} catch (Exception ignored) {}
	if (memberRepository.findByProviderAndProviderKey(Provider.KAKAO, TEST_PROVIDER_KEY).isEmpty()) {
	memberService.createMember("testUser_sc1111", "http://img", TEST_PROVIDER_KEY, Provider.KAKAO);
	}