사용자 스토리를 기반으로 E2E 테스트 작성하기 #469
Open
+29,829
−3,414
Conversation
This file contains 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
…mage-resize-performence 이미지 리사이즈 성능개선 및 이미지 서비스 리팩토링
…t-url 이미지 host url 수정
…ssets 정적 리소스 assets 폴더로 이동
…ptimization 번들 사이즈 최적화
jeongwusi
requested changes
Sep 21, 2023
client/.DS_Store
Outdated
| @@ -45,6 +45,7 @@ | |||
| "@typescript-eslint/eslint-plugin": "^5.61.0", | |||
| "@typescript-eslint/parser": "^5.61.0", | |||
| "babel-loader": "^9.1.2", | |||
| "babel-plugin-react-remove-properties": "^0.3.0", | |||
There was a problem hiding this comment.
이거 사용하려면 빌드 될 때 없어지게 하려면 따로 웹펙에 환경 설정을 해주어야 하는 거 같아요. 그리고 많이 사용되는 플러그인이 아닌거 같은데... 이게 없으면 테스트 진행이 어렵나용? 그리고 컴포넌트 수정된 것도 PR에 없는 거 같은딩...?!
solo5star
requested changes
Sep 21, 2023
client/package-lock.json
Outdated
client/yarn.lock
Outdated
There was a problem hiding this comment.
main merge한번 하고 yarn install 한번 돌려야겠네용
.DS_Store
Outdated
client/cypress/fixtures/example.json
Outdated
|
고선생님, 솔교수님 리뷰 반영과 코멘트 확인 부탁드립니다.🙋🏻♂️ |
jeongwusi
requested review from
donghae-kim,
nuyh99,
green-kong and
hum02
as code owners
jeongwusi
requested changes
Sep 21, 2023
green-kong
requested changes
Sep 21, 2023
client/cypress/support/component.ts
Outdated
Comment on lines
37
to
39
|
|
||
| // Example use: | ||
| // cy.mount(<MyComponent />) |
Comment on lines
10
to
12
| <div data-cy-root></div> | ||
| </body> | ||
| </html> |
There was a problem hiding this comment.
마지막 줄 개행 안하시나요?
참고 블로그
|
죄송합니다 JS는 안해도 상관없다네요. 까불어서 다시한번 죄송합니다. |
green-kong
approved these changes
Sep 21, 2023
|
폴로가 잘 지적해주신 것 같습니다. 저희가 사용하는 JS Standard Style 에서는 파일 끝에 개행을 넣는 것으로 규정하고 있습니다. prettier를 사용하기 때문에 Ctrl + S 하면 자동으로 포맷팅해줄텐데 아인의 VSCode에서 prettier가 동작하지 않은 것인지 봐야할 것 같네요
|
|
오 바로 태세전환 하겠습니다. 똑바로 하십쇼 아인.🤬 |
2 tasks
geuntaek1013
added
FE
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.
#️⃣ 연관된 이슈
📝 작업 내용
💬 리뷰 요구사항