[CHORE] Swagger + Rest Docs 로 API 문서 생성 #44

leegwichan · 2025-01-07T04:01:58Z

🚩 연관 이슈

closed #41

🗣️ 리뷰 요구사항 (선택)

Rest Docs를 잘 몰라도 사용할 수 있도록 코드를 만들었다.
- 가독성 개선, ... 등 코드 수정해달라고 요청주세요
콜리 : Swagger 관련 설정 추가할 것 있으면 알려주세요~ 혹은 참고 자료도 괜찮습니다.

기타

드디어 끝났다으아으아으아~~ 이젠 쉬고 싶어...
다음 회의에 Rest Docs & Swagger 설명회 있을 예정입니다.
기회 되면 문서화도 진행하겠습니다.

# Conflicts: # src/test/java/com/debatetimer/controller/parliamentary/ParliamentaryControllerTest.java

github-actions · 2025-01-07T04:03:52Z

Test Results

68 tests 68 ✅ 7s ⏱️
34 suites 0 💤
34 files 0 ❌

Results for commit 98ca7eb.

♻️ This comment has been updated with latest results.

github-actions · 2025-01-07T04:03:54Z

📝 Test Coverage Report

Overall Project	84.67%	🍏
Files changed	100%	🍏

File	Coverage
Member.java	100%	🍏
MemberController.java	100%	🍏
ErrorResponse.java	100%	🍏
ParliamentaryController.java	100%	🍏

leegwichan · 2025-01-07T05:23:11Z

Swagger 확인하는 방법

Gradle task 중 documentation 하위 generateSwaggerUI 테스트 실행
build/resources/main/static/docs/openapi3.yaml 에서 확인

예시

unifolio0 · 2025-01-07T06:07:34Z

/noti
@debate-timer/backend
만약 저희 태스크가 많은 게 아니면 목욜에 커찬의 설명 듣고서 리뷰 하는건 어떤가요?

coli-geonwoo · 2025-01-07T06:53:54Z

좋아요!

coli-geonwoo

/noti

커찬! 정말 정말 고생많았습니다. 사실 사용법이나 설정 관련해서는 너무 이해가 잘갔어요. 다만 사소한 부분들에서 제 의견 남겼으니 천천히 의견 남겨주면 좋을 것 같아요!

논의했던 대로 화요일까지는 approve&merge하겠습니다 😄

coli-geonwoo · 2025-01-11T08:40:53Z

build.gradle

    implementation 'org.springdoc:springdoc-openapi-starter-webmvc-ui:2.6.0'
+    testImplementation 'io.rest-assured:rest-assured:5.5.0'
+    testImplementation 'org.springframework.restdocs:spring-restdocs-restassured'
+    testImplementation 'com.epages:restdocs-api-spec-mockmvc:0.18.2'


[질문 ✋ ]

mockMvc가 webMvc를 사용하지 않음에도 불구하고 필요한 설정인것이죠?
(제 기억에는 RestAssured 사용하더라도 추가해야 했던 기억이 있긴 한데...)

asciidoc generate 태스크는 그날 회의 이후로 커찬이 제거한 것인가요?

제가 에러 핸들링한 자료 공유해드려요~ 링크

아마도 해당 부분은 라이브러리 쪽 버그인 것 같아요. 어쩔 수 없이 의존성 추가했습니다.

감사합니다~

coli-geonwoo · 2025-01-11T08:47:26Z

src/test/java/com/debatetimer/controller/RestDocumentationRequest.java

+
+    public RestDocumentationRequest description(String description) {
+        resourceBuilder.description(description)
+                .summary(description);


[사소한 제안]

summary는 간단한 요약 이라는 설정이고 desription은 파라미터에 대한 상세한 설명을 기술하는 것이라 생각되는데요. 메서드명은 description인데 하나의 파라미터로 둘다 초기화해주는 이유가 있나요?

아니라면 분리하거나 혹은 description만 설정해주는 건 어떤가요?

Swagger UI 부분에서도 중복되는 부분이라 제거하는 게 좋을 것 같네요
반영하도록 하겠습니다~

해당 부분을 요약해서 설명한다는 느낌에서 summary로 변경하고 summary만 설정해주도록 했습니다~

coli-geonwoo · 2025-01-11T08:51:48Z

src/test/java/com/debatetimer/controller/RestDocumentationRequest.java

+        return this;
+    }
+
+    public RestDocumentationRequest pathParameter(ParameterDescriptor... descriptors) {


[사소한 제안]

가변 인자를 좋아하진 않지만 빌더같은 유틸성 클래스를 만들때는 좋다고 생각해요. 다만 가변인자의 경우 배열이기에 null 값도 허용하여 인수가 0인 값이 런타임에 잡히게 되는 것으로 알고 있어요.

하나 이상의 null이 아닌 파라미터를 무조건 받는다를 시그니처에 녹여도 좋을 것 같아요!

Suggested change

public RestDocumentationRequest pathParameter(ParameterDescriptor... descriptors) {

public RestDocumentationRequest pathParameter(ParameterDescriptor descriptor, ParameterDescriptor... descriptors) {

reference)
https://velog.io/@injoon2019/%EC%9D%B4%ED%8E%99%ED%8B%B0%EB%B8%8C-%EC%9E%90%EB%B0%94-%EC%95%84%EC%9D%B4%ED%85%9C-53.-%EA%B0%80%EB%B3%80%EC%9D%B8%EC%88%98%EB%8A%94-%EC%8B%A0%EC%A4%91%ED%9E%88-%EC%82%AC%EC%9A%A9%ED%95%98%EB%9D%BC

매우 좋은 의견이라고 생각해요! 하지만 아래 부분은 더 생각해봐야 할 것 같아요.

그러면 매번 descriptor, descriptors를 추가한 새로운 List or Array를 만들어야 하는데, 해당 부분은 정말 많은 부분에서 사용하고 있어요. 그렇다면 정말로 새로운 객체를 만들어야 할 정도로 중요하게 컴파일 타임에 신경써야 할 부분일까요?

해당 부분을 사용하는 테스트 코드는 특정 부분이 누락되면 실패하게 되어있어요. 내가 특정 파라미터를 추가하려고 해당 메서드를 사용했지만, 누락했을 경우에는 쉽게 알 수 있게 되요. 이러한 환경에서도 하나 이상의 null이 아닌 파라미터를 무조건 받는다는 점을 강조해야 할까요?

인정합니다. 현재 구조로도 괜찮다고 생각해요!

coli-geonwoo · 2025-01-11T08:52:49Z

src/test/java/com/debatetimer/controller/RestDocumentationRequest.java

+    }
+
+    public List<Snippet> getSnippets() {
+        return List.copyOf(snippets);


방어적 복사 좋네요 ㄷㄷ 👍

개인적으로는 어짜피 RestDocumentFilterBuilder에만 사용할 껀데 방어적 복사를 해야되나 고민 했음...
그런데 애초에 내부적으로 가변 객체를 사용하니, 이건 방어적 복사를 추가해야 된다고 생각했음

src/main/java/com/debatetimer/controller/auth/AuthMember.java

coli-geonwoo · 2025-01-11T09:23:36Z

src/test/java/com/debatetimer/controller/BaseDocumentTest.java

+    }
+
+    protected RestDocumentationFilterBuilder document(String identifierPrefix, int status) {
+        return new RestDocumentationFilterBuilder(identifierPrefix, Integer.toString(status));


RestDocumentationFilterBuilder가 있는 라이브러린줄 알았는데 만든거였군요? 😮
대단한 사람...

Rest Docs의 static import가 너무 귀찮아서 참을 수 없었다...
Builder 만들어서 실제 DocumentTest에서는 어떤 static import가 필요한지 모르도록 최선을 다했다...

coli-geonwoo · 2025-01-11T09:28:13Z

src/test/java/com/debatetimer/controller/member/MemberControllerTest.java

                    .contentType(ContentType.JSON)
                    .body(request)
                    .when().post("/api/member")
-                    .then().log().all()


[질문 ✋ ]

일전에 설명해주엇던 것 같긴한데 로깅을 뺀 이유는 필요하지 않아서가 맞을 까요?

이번에 BaseControllerTest에, RequestLoggingFilter, ResponseLoggingFilter를 추가해 주었는데요.
따로 log().all()을 명시해주지 않더라고 볼 수 있게끔 해주었습니다~

이해했습니다 👍

coli-geonwoo · 2025-01-11T09:35:24Z

src/test/java/com/debatetimer/controller/member/MemberDocumentTest.java

+        }
+
+        @EnumSource(
+                value = ClientErrorCode.class,


[제안]
이거 문자열로 관리하기가 힘들 것 같은데 MethodSource 사용해보는 건 어떨까요?
관리 용이성과 타입안전성에 좋을 것 같아요.

Suggested change

value = ClientErrorCode.class,

@MethodSource("memberCreateError")

@ParameterizedTest

void 회원_생성_실패(ClientErrorCode errorCode) {

MemberCreateRequest request = new MemberCreateRequest("커찬");

when(memberService.createMember(request)).thenThrow(new DTClientErrorException(errorCode));

var document = document("member/create", errorCode)

.request(requestDocument)

.response(ERROR_RESPONSE)

.build();

given(document)

.contentType(ContentType.JSON)

.body(request)

.when().post("/api/member")

.then().statusCode(errorCode.getStatus().value());

}

private static Stream<ClientErrorCode> memberCreateError() {

return Stream.of(

ClientErrorCode.INVALID_MEMBER_NICKNAME_FORM,

ClientErrorCode.INVALID_MEMBER_NICKNAME_LENGTH

);

}

개인적으로 비선호하긴 합니다. 저는 어떤 ERROR CODE를 사용하는지 최대한 테스트와 붙여 놓은 것을 선호하긴 합니다.
물론 컴파일 환경에서 관리할 수 있다는 것이 큰 장점이긴 하지만, 저 에러 코드가 바뀌는 케이스를 거이 못봐서 @EnumSource를 이용했습니다.

인정합니다. 개인적인 취향으로 제가 MethodSource를 선호하는 이유는 문자열 관리를 싫어하고 + 오타가 많아서 그런가봐요 😓 지금으로도 괜찮다 생각해요!

coli-geonwoo · 2025-01-11T09:40:43Z

src/test/java/com/debatetimer/controller/BaseDocumentTest.java

+    protected static long EXIST_MEMBER_ID = 123L;
+    protected static Member EXIST_MEMBER = new Member(EXIST_MEMBER_ID, "존재하는 멤버");
+
+    protected static RestDocumentationResponse ERROR_RESPONSE = new RestDocumentationResponse()


[제안]
이거 description이 "에러 메시지" 로 통일되기 보다 실제 반환되는 에러 메시지를 넘겨주도록 protected method로 선언하는 건 어떨까요?

ex)

Suggested change

protected static RestDocumentationResponse ERROR_RESPONSE = new RestDocumentationResponse()

protected RestDocumentationResponse errorResponse(ErrorCode errorCode){

return new RestDocumentationResponse()

.responseBodyField(

fieldWithPath("message").type(STRING).description(errorCode.getMessage())

);

}

PR에서 표기하실 때 '변경하고 싶은 부분'을 '앞 뒤 맥락 포함해서' 표시해주세요. 내가 코드를 어떻게 썼는지 한 눈에 안들어오고, 콜리의 의견과 비교하기가 어려워요.

저 부분의 description이 Swagger UI에 안나타나는 부분이네요. 해당 description을 지우는 건 어떨까요?

넵. 문제 개선안을 혼자 고민하다보니 당연히 커찬도 해당 문제를 인식하고 있을 줄 알았는데 넘겨짚은 감이 있는 것 같습니다. 유의할게요 😅
이 코멘트에서 변경하고 싶었던 부분은 api docs에서 ErrorResponse를 "에러 메시지" 통일하는 것보다 각 에러 상황에 맞도록 에러 객체를 지정하기 위해 method를 만들어주는 건 어떨까? 하는 부분이었어요.

description이 Swagger UI 에 나타나지 않음은 RestAssured의 실행 결과를 기반으로 adoc 파일이 생성되기 때문이군요! 그럼 지워도 괜찮을 것 같아요.

coli-geonwoo · 2025-01-11T09:54:18Z

src/test/java/com/debatetimer/controller/parliamentary/ParliamentaryDocumentTest.java

+
+    @Nested
+    class Save {
+


공통 필드는 변수로 빼주는 건 어떨까요? 다만 request, response builder 시그니처 수정이 조금 필요할 것 같아요.

private final FieldDescriptor[] tableFields = { fieldWithPath("info").type(OBJECT).description("토론 테이블 정보"), fieldWithPath("info.name").type(STRING).description("테이블 이름"), fieldWithPath("info.agenda").type(STRING).description("토론 주제"), fieldWithPath("table").type(ARRAY).description("토론 테이블 구성"), fieldWithPath("table[].stance").type(STRING).description("입장"), fieldWithPath("table[].type").type(STRING).description("발언 유형"), fieldWithPath("table[].time").type(NUMBER).description("발언 시간(초)"), fieldWithPath("table[].speakerNumber").type(NUMBER).description("발언자 번호").optional() }; private final RestDocumentationRequest requestDocument = request() .tag(Tag.PARLIAMENTARY_API) .description("새로운 의회식 토론 시간표 생성") .queryParameter( parameterWithName("memberId").description("멤버 ID") ) .requestBodyField(tableFields); private final RestDocumentationResponse responseDocument = response() .responseBodyField( fieldWithPath("id").type(NUMBER).description("테이블 ID"), tableFields );

공통 필드는 변수로 빼주는 건 어떨까요?

싫어요.
생각을 안해본 건 아닙니다. 하지만, Request Response가 어떻게 생겼는지 한 눈에 잘 안들어 올 것 같아요.

이해했습니다. 커찬이 한번 생각하고 내린 결정이라면 그 방식이 더 합리적인 것 같아요. DocumentTest가 요청-응답을 기록하기 위한 테스트이니 목적에 더 알맞는 명세라는 생각이 들기도 하네요!

unifolio0

/noti
이미 콜리가 상세히 리뷰해줬고 직접 써봐야 될 것 같아 approve합니다

leegwichan · 2025-01-11T12:39:25Z

/noti @coli-geonwoo
생각보다 작업이 빨리 끝났네요~ 제 코멘트 확인 부탁드릴께요~
(c.f. 생각보다 의견 많음)

coli-geonwoo

/noti

커찬! 답변 잘 확인하여 approve 합니다. 대부분은 설득되었으나 한두가지에서 의견 더 남겼으니 확인해주세요. 모든 의견은 반영보다는 커찬이 한번쯤 생각해주었으만 하는 의견들일 뿐입니다 ㅎㅎ

의견에 대한 반영은 커찬의 자유에 맡기고자 merge는 커찬의 몫으로 남겨두겠습니다.
고생많으셨어요! (<- 코드에서 시간 쏟은게 느껴짐! )

coli-geonwoo · 2025-01-12T07:06:01Z

build.gradle

    implementation 'org.springdoc:springdoc-openapi-starter-webmvc-ui:2.6.0'
+    testImplementation 'io.rest-assured:rest-assured:5.5.0'
+    testImplementation 'org.springframework.restdocs:spring-restdocs-restassured'
+    testImplementation 'com.epages:restdocs-api-spec-mockmvc:0.18.2'


감사합니다~

coli-geonwoo · 2025-01-12T07:07:10Z

src/main/java/com/debatetimer/domain/member/Member.java

+        this.id = id;
+        this.nickname = nickname;
+    }
+
    public Member(String nickname) {


아... 그러네요 이해했습니다! 별도의 변경 없이 지금도 괜찮은 것 같아요. 1/2번이 합리적으로 생성자 체이닝을 못하는 이유라 생각합니다.

coli-geonwoo · 2025-01-12T07:10:25Z

src/test/java/com/debatetimer/controller/BaseDocumentTest.java

+    protected static long EXIST_MEMBER_ID = 123L;
+    protected static Member EXIST_MEMBER = new Member(EXIST_MEMBER_ID, "존재하는 멤버");
+
+    protected static RestDocumentationResponse ERROR_RESPONSE = new RestDocumentationResponse()


넵. 문제 개선안을 혼자 고민하다보니 당연히 커찬도 해당 문제를 인식하고 있을 줄 알았는데 넘겨짚은 감이 있는 것 같습니다. 유의할게요 😅
이 코멘트에서 변경하고 싶었던 부분은 api docs에서 ErrorResponse를 "에러 메시지" 통일하는 것보다 각 에러 상황에 맞도록 에러 객체를 지정하기 위해 method를 만들어주는 건 어떨까? 하는 부분이었어요.

description이 Swagger UI 에 나타나지 않음은 RestAssured의 실행 결과를 기반으로 adoc 파일이 생성되기 때문이군요! 그럼 지워도 괜찮을 것 같아요.

coli-geonwoo · 2025-01-12T07:23:07Z

src/test/java/com/debatetimer/controller/BaseDocumentTest.java

+    private MemberRepository memberRepository;
+
+    @MockitoBean
+    protected MemberService memberService;


[의견]

커찬의 입장을 이해했습니다. RestAssured에서 뜨는 전반적인 빈들을 다른 documentTest에서도Mocking하는 작업이 필요하다는 거군요!

반영을 바라는 건 아니지만 의견을 말하자면 해당 memberService와 소통하는 다른 controller 계층이 있을지에 대해선 조금 의문이 남는 것 같아요. 서비스가 다계층으로 복잡해진다면 전반적으로 모든 모킹을 BaseXXXtest에 넣는 것이 맞겠지만 그럼 DocumentTest가 추가될때마다 BaseTest가 두꺼워질 것 같아서요. 개인적으로 인증 작업은 MemberRepository 모킹으로 충분하다 생각하고 각각의 DocumentTest에서 필요한 모킹 설정을 해주는 것이 응집성에 좋다 생각합니다. 그런데 현재 서비스는 크게 복잡하지 않으니 지금 구조로도 괜찮은 것 같아요.

coli-geonwoo · 2025-01-12T07:24:15Z

src/test/java/com/debatetimer/controller/RestDocumentationRequest.java

+        return this;
+    }
+
+    public RestDocumentationRequest pathParameter(ParameterDescriptor... descriptors) {


인정합니다. 현재 구조로도 괜찮다고 생각해요!

coli-geonwoo · 2025-01-12T07:24:37Z

src/test/java/com/debatetimer/controller/member/MemberControllerTest.java

                    .contentType(ContentType.JSON)
                    .body(request)
                    .when().post("/api/member")
-                    .then().log().all()


이해했습니다 👍

coli-geonwoo · 2025-01-12T07:25:32Z

src/test/java/com/debatetimer/controller/member/MemberDocumentTest.java

+        }
+
+        @EnumSource(
+                value = ClientErrorCode.class,


인정합니다. 개인적인 취향으로 제가 MethodSource를 선호하는 이유는 문자열 관리를 싫어하고 + 오타가 많아서 그런가봐요 😓 지금으로도 괜찮다 생각해요!

coli-geonwoo · 2025-01-12T07:26:54Z

src/test/java/com/debatetimer/controller/parliamentary/ParliamentaryDocumentTest.java

+
+    @Nested
+    class Save {
+


이해했습니다. 커찬이 한번 생각하고 내린 결정이라면 그 방식이 더 합리적인 것 같아요. DocumentTest가 요청-응답을 기록하기 위한 테스트이니 목적에 더 알맞는 명세라는 생각이 들기도 하네요!

leegwichan · 2025-01-13T06:26:08Z

/noti
누락된 코멘트에 답장하고 이만 머지하도록 하겠습니다!

leegwichan added 7 commits January 4, 2025 16:43

refactor: 기존 Swagger 의존성 제거

527f256

feat: Rest Docs + Swagger UI 적용

65963b5

feat: RestDocumentationFilterBuilder 에 description 추가

6a68e38

feat: RestDocumentationFilterBuilder 에 tag 추가

1dc8a1c

Merge branch 'develop' into chore/#41

277344c

# Conflicts: # src/test/java/com/debatetimer/controller/parliamentary/ParliamentaryControllerTest.java

test: 에러 케이스 테스트 추가

8666560

test: 문서화 테스트 추기 및 리팩터링

8d744a4

unifolio0 added the chore 기타 label Jan 7, 2025

unifolio0 requested review from unifolio0 and coli-geonwoo January 7, 2025 04:02

unifolio0 assigned leegwichan Jan 7, 2025

refactor: 사용하지 않는 메서드 제거

5968f16

leegwichan added 2 commits January 7, 2025 14:26

test: MemberDocumentTest identifier 수정

9ea50ee

test: ParliamentaryDocumentTest identifier 수정

06fd843

style: 필요없는 공백 제거

be78043

coli-geonwoo requested changes Jan 11, 2025

View reviewed changes

unifolio0 approved these changes Jan 11, 2025

View reviewed changes

leegwichan added 2 commits January 11, 2025 21:30

fix: RestDocumentationFilterBuilder 의 description 을 summary 로 변경

6f403a5

chore: Swagger UI 설정 추가

98ca7eb

coli-geonwoo approved these changes Jan 12, 2025

View reviewed changes

leegwichan merged commit eff690e into develop Jan 13, 2025
5 checks passed

leegwichan deleted the chore/#41 branch January 13, 2025 06:26

	public RestDocumentationRequest pathParameter(ParameterDescriptor... descriptors) {
	public RestDocumentationRequest pathParameter(ParameterDescriptor descriptor, ParameterDescriptor... descriptors) {

-                value = ClientErrorCode.class,
+        @MethodSource("memberCreateError")
+        @ParameterizedTest
+        void 회원_생성_실패(ClientErrorCode errorCode) {
+            MemberCreateRequest request = new MemberCreateRequest("커찬");
+            when(memberService.createMember(request)).thenThrow(new DTClientErrorException(errorCode));
+            var document = document("member/create", errorCode)
+                    .request(requestDocument)
+                    .response(ERROR_RESPONSE)
+                    .build();
+            given(document)
+                    .contentType(ContentType.JSON)
+                    .body(request)
+                    .when().post("/api/member")
+                    .then().statusCode(errorCode.getStatus().value());
+        }
+        private static Stream<ClientErrorCode> memberCreateError() {
+            return Stream.of(
+                    ClientErrorCode.INVALID_MEMBER_NICKNAME_FORM,
+                    ClientErrorCode.INVALID_MEMBER_NICKNAME_LENGTH
+            );
+        }

-    protected static RestDocumentationResponse ERROR_RESPONSE = new RestDocumentationResponse()
+    protected  RestDocumentationResponse errorResponse(ErrorCode errorCode){
+        return new RestDocumentationResponse()
+                .responseBodyField(
+                        fieldWithPath("message").type(STRING).description(errorCode.getMessage())
+                );
+    }

[CHORE] Swagger + Rest Docs 로 API 문서 생성 #44

[CHORE] Swagger + Rest Docs 로 API 문서 생성 #44

Conversation

leegwichan commented Jan 7, 2025 • edited Loading

🚩 연관 이슈

🗣️ 리뷰 요구사항 (선택)

기타

github-actions bot commented Jan 7, 2025 • edited Loading

Test Results

github-actions bot commented Jan 7, 2025 • edited Loading

📝 Test Coverage Report

leegwichan commented Jan 7, 2025 • edited Loading

Swagger 확인하는 방법

예시

unifolio0 commented Jan 7, 2025

coli-geonwoo commented Jan 7, 2025

coli-geonwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

unifolio0 left a comment

Choose a reason for hiding this comment

leegwichan commented Jan 11, 2025

coli-geonwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leegwichan commented Jan 13, 2025

leegwichan commented Jan 7, 2025 •

edited

Loading

github-actions bot commented Jan 7, 2025 •

edited

Loading

github-actions bot commented Jan 7, 2025 •

edited

Loading

leegwichan commented Jan 7, 2025 •

edited

Loading