Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#58] Reconstruct UI test #60

Merged

Conversation

2wheeh
Copy link
Collaborator

@2wheeh 2wheeh commented Sep 21, 2023

Resolves #58

Changes

각 프로젝트의 test 스크립트가 동작합니다.

  1. mr-c.app: jest with nextjs
  2. common-components: vitest with vite, storybook
    • vite 의 컴파일 파이프라인을 공유할 수 있는 vitest 로 테스트를 실행합니다.
    • storybook 을 위해 작성한 story 를 테스트 작성 시에 사용할 수 있습니다.
    • 참고: vitest-vs-jest, stories-in-unit-tests
  3. common-utils: jest with ts-jest

mrc-ui 루트의 test: 스크립트가 동작합니다.

  1. workspace-tools 플러그인의 foreach 커맨드를 사용하여
    • 변화가 있는 프로젝트와 그 영향을 받는 프로젝트의 테스트를 실행합니다.
    • 모든 프로젝트의 테스트를 실행합니다.
  2. b3ab3db 의 README 참고

UI 테스트 워크플로를 재구성합니다.

  1. Reconstruct UI test system #58 (comment) 에서 논의한 대로 ui-test 를 별도의 파일로 분리합니다.
  2. ui-test 워크플로는 test:affected 스크립트를 실행하며 기대하는 동작은 다음과 같습니다:
    1. PR의 머지 커밋과 base branch의 기존 HEAD 커밋을 비교 한다.
    2. 변화가 있는 프로젝트와 의존하는 프로젝트(dependent) 를 순회하며 test 스크립트를 실행한다.

Additional Info

  1. 추가된 테스트 코드는 동작 확인을 위한 목적입니다. 실제 기능을 위한 코드가 추가되면 삭제 예정입니다.
  2. tsconfig, lint 설정이 아직 작업 중 입니다. 관련된 기능이 정상적으로 동작하지 않을 수 있습니다.

@2wheeh 2wheeh added enhancement New feature or request ui Something about UI labels Sep 21, 2023
@2wheeh 2wheeh self-assigned this Sep 21, 2023
@2wheeh 2wheeh changed the title Feature/issue 58/reconstruct UI test [#58] Reconstruct UI test Sep 21, 2023
@2wheeh 2wheeh force-pushed the feature/issue-58/reconstruct-ui-test branch 3 times, most recently from 7d92413 to 64cc153 Compare September 22, 2023 03:36
@2wheeh 2wheeh force-pushed the feature/issue-58/reconstruct-ui-test branch from 64cc153 to b3ab3db Compare September 22, 2023 03:37
@2wheeh 2wheeh marked this pull request as ready for review September 25, 2023 06:38
@2wheeh 2wheeh added the documentation Improvements or additions to documentation label Sep 26, 2023
@2wheeh 2wheeh requested a review from skgndi12 September 26, 2023 07:52
Copy link
Collaborator

@skgndi12 skgndi12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다! 👍

Copy link
Collaborator

@isutare412 isutare412 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

설명이 상세해서 PR 보기가 편하네요.
고생 많으셨습니다 👍

@2wheeh 2wheeh merged commit af1f117 into MovieReviewComment:develop Sep 28, 2023
@2wheeh 2wheeh deleted the feature/issue-58/reconstruct-ui-test branch September 28, 2023 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request ui Something about UI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Reconstruct UI test system
3 participants