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

User - user, storeManager Sign in #29

Closed
reasonSun opened this issue Nov 1, 2022 · 0 comments · Fixed by #30
Closed

User - user, storeManager Sign in #29

reasonSun opened this issue Nov 1, 2022 · 0 comments · Fixed by #30
Assignees

Comments

@reasonSun
Copy link
Contributor

reasonSun commented Nov 1, 2022

user, storeManager 로그인 구현 (스프링시큐리티 적용 전)

@reasonSun reasonSun self-assigned this Nov 1, 2022
reasonSun added a commit that referenced this issue Nov 12, 2022
@chanwooo chanwooo linked a pull request Nov 14, 2022 that will close this issue
reasonSun added a commit that referenced this issue Nov 14, 2022
reasonSun added a commit that referenced this issue Nov 24, 2022
* #17-불필요한 파일 삭제

* #17-Entity파일 수정(toResponse(), of()추가 및 컬러명 변경)

* #17-Repository 파일 수정

* #17-DTO 파일 줄바꿈 및 @builder 어노테이션 추가

* #17-store exception파일 추가

* #17-store 서비스 구현(update 제외하고 구현)

* #17-store 서비스 테스트코드 작성(스토어 저장)

* #17-Repository id로 조회하도록 변경, StoreRepository findByStoreId 삭제

* #17-update 구현 및 Menu,Store Entity storeId 제거

* #17-메뉴저장,수정 및 스토어 저장 테스트코드 작성

* #17-StoreService 메소드 이름 변경

* #17-update 메소드에 save()추가

* #17-BaseEntity package 이동

* #29-User, StoreManager 로그인 구현

* #29-암호화 파일 추가

* #29-로그인 exception 파일 추가, 불필요한 exception 파일 삭제

* #29-Store로그인 테스트케이스 추가

* #29-생성자 어노테이션 추가

* #29-URI 수정, store 로그인 exception 처리

* #29-URI 수정, user 로그인 수정 및 updateUserName 메서드 추가

* #29-리뷰반영

* #29-sha512메서드 Exception 수정, 암호화 파일 Util 클래스로 변경

* #29-CustomPasswordEncoder util 클래스로 변경으로 인한 테스트코드 수정

* #29-build.gralde에 mockito-inline dependencies 추가

* #29-PasswordEncoder.java 파일 미사용으로 삭제

* #29-signUp_StoreManager_Success 테스트 메소드 static mocking 제거

* #29-UserServiceImplTest static 메소드 모킹 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant