[iOS] 날짜 정규식, 색상-문자열 변환 단위 테스트 #97
Merged
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
- babel-eslint 추가 - .eslintrc.json 에 parser 옵션 추가 - nodemon 설정 및 package.json 에 dev script 추가
- LabelController 및 LabelService 코드의 가독성을 높이도록 수정 - LabelController 및 LabelService 로직 내 에러 처리 로직 수정 - 선언식 함수를 화살표 함수로 수정 - LabelController.add() 메소드 에러 처리 간 status code 수정 - LabelController 에러 처리 시 응답에 message 추가 Close #58
[server] Label 컨트롤러 수정
- milestones router 추가 - MilestoneController 생성 - MilestoneService 생성
…-init [server] milestones API 초기 세팅
- 마일스톤 수정 API 구현
- 마일스톤 삭제 API 구현 - Controller, Service 함수 구현
- MilestoneController, MilestoneService 에 milestone post API 를 위한 기능 구현 - request의 body 에서 data를 가져와서 date 타입 확인 - date 타입이 유효하지 않으면 400 error - date 타입이 유효하면 db에 create 한 후에 200 응답
…-post [server] Milestone 추가 API
…-update [server] Milestone 수정 API
…-delete [server] Milestone 삭제 API
- milestoneController.getAll() 메소드 구현 - milestoneService.findAll() 메소드 구현 - 정적 쿼리 보관용 query.js 생성 및 findAllMilestones 추가
…-get [server] Milestone 조회 API
- Milestone 모델 title 속성에 unique 제약조건 추가 - milestoneController.add() 메소드 중복 검증 로직 추가
…-fix [server] Milestone 테이블 제약조건 추가
[server] Issue seeder 파일 추가
- 에러 핸들링 작업을 하나의 로직에서 처리할 수 있도록 모듈 구현 - app.js 에 에러 처리용 미들웨어 추가 - common 패키지에 에러 핸들러 모듈 및 에러 메세지 모듈 추가 Close #74
…init [server] 에러 핸들러 모듈 구현
- labels 전체조회 API의 controller에 error handler 적용
- 레이블 추가하는 에러 핸들러 적용
- label 삭제 API 에 error handler 적용 - label 존재하지 않으면 no content 메세지로 error throw
- 마일스톤 수정, 삭제 부분 에러핸들러 적용
- controller 내부 try..catch 코드 삭제 - 전체 error handler 적용
[server] 에러 핸들러 적용 - 레이블 추가, 마일스톤 수정, 삭제
- 라벨 단건 조회 로직 내 에러 핸들링 로직 추가 - 라벨 수정 로직 내 에러 핸들링 로직 추가 - 마일스톤 조회 로직 내 에러 핸들링 로직 추가 Close #75
[server] 에러 핸들러 적용 - 레이블 조회, 삭제, 마일스톤 등록
[server] 에러 핸들러 적용 - 라벨 단건 조회, 수정 및 마일스톤 조회
- date validation check 로직 추가 Close #83
…-fix [server] Milestone 전체 목록 조회 및 수정 API 수정
- 기존: 핸들러 함수 안에서 switch-case 문으로 에러 메시지에 따라 status를 정해주었음. - 문제: switch-case문 길이가 길고 가독성 측면에서도 좋지 않음 - 변경: switch-case 문 제거하고, 함수 외부 객체로 분리해 코드를 간결하게함
- 그룹 인원 부분 수정
…refactor [server] 에러 핸들러 로직 수정
[server] 라벨 API 리팩토링, 마일스톤 API 추가, 에러 핸들링 모듈 추가
…dify [server] 테스트 데이터 수정
- 프로젝트 구조 추가 - 뱃지 추가 - 산출물 링크 변경
[docs] README.md 수정
깃헙 움짤 추가, 프로젝트 설명 추가
- Milestone 전체 조회 API 제공 간 MilestoneService.findAll() 메소드가 ORM 라이브러리를 사용하는 의미를 갖도록 리팩토링 - sequelize.qeury() 대신 Sequelize.Model 의 서브 클래스 Milestone 을 사용하여 기능 구현 Close #92
- 프로젝트 구조 수정 - 기술 스택 추가
- gitHub 에서 표가 깨지는 부분 수정
[common] README.md 수정
[iOS] 레이블, 마일스톤 추가&편집화면 기능 추가, 네트워크 계층 개선
…-api-refactor [server] Milestone 전체 조회 API 리팩토링
[server] 테스트 데이터 수정 및 Milestone 전체 조회 API 리팩토링
- 단위 테스트를 가능케 하기 위해 String Extension 으로 옮김.
chelwoong
approved these changes
Nov 2, 2020
| XCTAssertEqual(UIColor.init(hex: blackHex), UIColor.init(red: 0, green: 0, blue: 0, alpha: 1)) | ||
| } | ||
|
|
||
| func test_hexStringToUIColor_withInvalidString_shouldBeNil() throws { |
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.
날짜 정규식과 색상-문자열 간 변환 단위 테스트 쪼끔 했습니다
#96