-
Notifications
You must be signed in to change notification settings - Fork 28
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
Refactor: UNVALIDATED 에서 GUEST 로 권한을 변경하고 가입시 기존 권한을 CREW 에서 GUEST 로 변경한다. #1564
Merged
The head ref may contain hidden characters: "refactor/guest-\uAD8C\uD55C"
Merged
Changes from 9 commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
2292d67
Merge pull request #1536 from woowacourse/advenced-develop
D0Dam 66280be
refactor: UNVALIDATED 에서 GUEST 로 네이밍 변경
splitCoding 13b3869
refactor: Github 로그인시 부여되는 권한 GUEST 로 변경
splitCoding 5ae3f98
feat: Coach 권한이상의 회원이 id에 해당하는 멤버의 권한을 변경하는 기능 추가
splitCoding 46145f4
feat: Role 에 중요도 필드 추가, test 격리를 위해 application 속성 추가
splitCoding c2ff0af
feat: 아티클 기능에 권한 검증 로직 추가, 테스트 작성
splitCoding c2e275e
test: 인수 테스트 작성 및 수정
splitCoding d937d05
Merge branch 'develop' into refactor/guest-권한
splitCoding d242e22
refactor: 권한 검증 로직 importance를 사용하는 로직으로 변경
splitCoding 778fc15
refactor: QA를 위해 dev, local 환경에서 manger 권한 guest 로 설정
splitCoding File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
40 changes: 20 additions & 20 deletions
40
backend/src/acceptanceTest/resources/wooteco/prolog/article.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,28 +1,28 @@ | ||
@api | ||
Feature: 아티클 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "브라운"이 로그인을 하고 | ||
Background: 사전 작업 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
|
||
Scenario: 아티클을 작성하기 | ||
When 아티클을 작성하면 | ||
Then 아티클이 작성된다 | ||
Scenario: 아티클을 작성하기 | ||
When 아티클을 작성하면 | ||
Then 아티클이 작성된다 | ||
|
||
Scenario: 아티클을 전체 조회하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 아티클을 전체 조회 하면 | ||
Then 아티클이 전체 조회 된다 | ||
Scenario: 아티클을 전체 조회하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 아티클을 전체 조회 하면 | ||
Then 아티클이 전체 조회 된다 | ||
|
||
Scenario: 아티클을 수정하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 1번 아티클을 수정하면 | ||
Then 아티클이 수정된다 | ||
Scenario: 아티클을 수정하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 1번 아티클을 수정하면 | ||
Then 아티클이 수정된다 | ||
|
||
Scenario: 아티클을 삭제하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 1번 아티클을 삭제하면 | ||
Then 아티클이 삭제된다 | ||
Scenario: 아티클을 삭제하기 | ||
Given 아티클을 여러개 작성하고 | ||
When 1번 아티클을 삭제하면 | ||
Then 아티클이 삭제된다 | ||
|
||
Scenario: Url og태그 파싱하기 | ||
When Url을 입력하면 | ||
Then og태그를 파싱해서 반환한다. | ||
Scenario: Url og태그 파싱하기 | ||
When Url을 입력하면 | ||
Then og태그를 파싱해서 반환한다. |
22 changes: 11 additions & 11 deletions
22
backend/src/acceptanceTest/resources/wooteco/prolog/badges.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,18 @@ | ||
@api | ||
Feature: 배지 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given 세션 여러개를 생성하고 | ||
And 미션 여러개를 생성하고 | ||
Background: 사전 작업 | ||
Given 세션 여러개를 생성하고 | ||
And 미션 여러개를 생성하고 | ||
|
||
Scenario: 존재하지 않는 멤버로 배지 조회하기 | ||
When 존재하지 않는 멤버의 배지를 조회하면 | ||
Then 존재하지 않는 멤버 관련 예외가 발생한다 | ||
Scenario: 존재하지 않는 멤버로 배지 조회하기 | ||
When 존재하지 않는 멤버의 배지를 조회하면 | ||
Then 존재하지 않는 멤버 관련 예외가 발생한다 | ||
|
||
Scenario: 배지목록 조회하기 | ||
Given "브라운"이 로그인을 하고 | ||
And 여러개의 스터디로그를 작성하고 | ||
When 배지를 조회하면 | ||
Then 열정왕 배지를 부여한다. | ||
Scenario: 배지목록 조회하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
And 여러개의 스터디로그를 작성하고 | ||
When 배지를 조회하면 | ||
Then 열정왕 배지를 부여한다. | ||
|
||
|
44 changes: 22 additions & 22 deletions
44
backend/src/acceptanceTest/resources/wooteco/prolog/comment.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,29 +1,29 @@ | ||
@api | ||
Feature: 댓글 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "브라운"이 로그인을 하고 | ||
And 세션 여러개를 생성하고 | ||
And 미션 여러개를 생성하고 | ||
And 스터디로그를 작성하면 | ||
Background: 사전 작업 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
And 세션 여러개를 생성하고 | ||
And 미션 여러개를 생성하고 | ||
And 스터디로그를 작성하면 | ||
|
||
Scenario: 댓글 작성하기 | ||
When 1번 스터디로그에 대한 댓글을 작성하면 | ||
Then 댓글이 작성된다 | ||
Scenario: 댓글 작성하기 | ||
When 1번 스터디로그에 대한 댓글을 작성하면 | ||
Then 댓글이 작성된다 | ||
|
||
Scenario: 스터디로그의 댓글 목록 조회하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
And "웨지"가 로그인을 하고 | ||
And 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그의 댓글을 조회하면 | ||
Then 해당 스터디로그의 댓글 목록을 조회한다 | ||
Scenario: 스터디로그의 댓글 목록 조회하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
And "웨지"가 크루역할로 로그인을 하고 | ||
And 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그의 댓글을 조회하면 | ||
Then 해당 스터디로그의 댓글 목록을 조회한다 | ||
|
||
Scenario: 댓글 수정하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그에 대한 1번 댓글을 수정하면 | ||
Then 댓글이 수정된다 | ||
Scenario: 댓글 수정하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그에 대한 1번 댓글을 수정하면 | ||
Then 댓글이 수정된다 | ||
|
||
Scenario: 댓글 삭제하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그에 대한 1번 댓글을 삭제하면 | ||
Then 댓글이 삭제된다 | ||
Scenario: 댓글 삭제하기 | ||
Given 1번 스터디로그에 대한 댓글을 작성하고 | ||
When 1번 스터디로그에 대한 1번 댓글을 삭제하면 | ||
Then 댓글이 삭제된다 |
34 changes: 17 additions & 17 deletions
34
backend/src/acceptanceTest/resources/wooteco/prolog/curriculum.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,25 @@ | ||
@api | ||
Feature: 커리큘럼 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "수달"이 로그인을 하고 | ||
Background: 사전 작업 | ||
Given "수달"이 크루역할로 로그인을 하고 | ||
|
||
Scenario: 커리큘럼 생성하기 | ||
When 커리큘럼을 생성하면 | ||
Then 커리큘럼이 생성된다 | ||
Scenario: 커리큘럼 생성하기 | ||
When 커리큘럼을 생성하면 | ||
Then 커리큘럼이 생성된다 | ||
|
||
Scenario: 커리큘럼 목록 조회하기 | ||
When 커리큘럼을 생성하고 | ||
And 커리쿨럼을 조회하면 | ||
Then 커리큘럼이 조회된다 | ||
Scenario: 커리큘럼 목록 조회하기 | ||
When 커리큘럼을 생성하고 | ||
And 커리쿨럼을 조회하면 | ||
Then 커리큘럼이 조회된다 | ||
|
||
Scenario: 커리큘럼 수정하기 | ||
When 커리큘럼을 생성하고 | ||
And 1번 커리쿨럼을 수정하면 | ||
Then 커리큘럼이 수정된다 | ||
Scenario: 커리큘럼 수정하기 | ||
When 커리큘럼을 생성하고 | ||
And 1번 커리쿨럼을 수정하면 | ||
Then 커리큘럼이 수정된다 | ||
|
||
Scenario: 커리큘럼 삭제하기 | ||
When 커리큘럼을 생성하고 | ||
And 1번 커리쿨럼을 삭제하면 | ||
Then 커리큘럼이 삭제된다 | ||
Scenario: 커리큘럼 삭제하기 | ||
When 커리큘럼을 생성하고 | ||
And 1번 커리쿨럼을 삭제하면 | ||
Then 커리큘럼이 삭제된다 | ||
|
48 changes: 24 additions & 24 deletions
48
backend/src/acceptanceTest/resources/wooteco/prolog/essayanswer.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,32 +1,32 @@ | ||
@api | ||
Feature: 로드맵 답변 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "2022 백엔드 레벨1" 세션을 생성하고 - 1번 세션 | ||
And 1번 세션에 "자바"라는 키워드를 순서 1, 중요도 1로 작성하고 | ||
And 1번 세션, 1번 키워드에 퀴즈를 작성하고 | ||
And "브라운"이 로그인을 하고 | ||
Background: 사전 작업 | ||
Given "2022 백엔드 레벨1" 세션을 생성하고 - 1번 세션 | ||
And 1번 세션에 "자바"라는 키워드를 순서 1, 중요도 1로 작성하고 | ||
And 1번 세션, 1번 키워드에 퀴즈를 작성하고 | ||
And "브라운"이 크루역할로 로그인을 하고 | ||
|
||
Scenario: 답변 생성하기 | ||
When 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하면 | ||
Then 답변이 생성된다 | ||
Scenario: 답변 생성하기 | ||
When 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하면 | ||
Then 답변이 생성된다 | ||
|
||
Scenario: 답변 조회하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 조회하면 | ||
Then 답변이 조회된다 | ||
Scenario: 답변 조회하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 조회하면 | ||
Then 답변이 조회된다 | ||
|
||
Scenario: 답변 수정하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 "Integer은 wrapper 클래스"로 수정하면 | ||
Then 답변이 수정된다 | ||
Scenario: 답변 수정하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 "Integer은 wrapper 클래스"로 수정하면 | ||
Then 답변이 수정된다 | ||
|
||
Scenario: 답변 삭제하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 삭제하면 | ||
Then 답변이 삭제된다 | ||
Scenario: 답변 삭제하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 답변을 삭제하면 | ||
Then 답변이 삭제된다 | ||
|
||
Scenario: 퀴즈에 대한 모든 답변 조회하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 퀴즈에 대한 답변들을 조회하면 | ||
Then 답변들이 조회된다 | ||
Scenario: 퀴즈에 대한 모든 답변 조회하기 | ||
Given 1번 퀴즈에 "varargs는 가변 인자"라는 답변을 생성하고 | ||
When 1번 퀴즈에 대한 답변들을 조회하면 | ||
Then 답변들이 조회된다 |
14 changes: 7 additions & 7 deletions
14
backend/src/acceptanceTest/resources/wooteco/prolog/filter.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,11 @@ | ||
@api | ||
Feature: 필터 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "현구막"이 로그인을 하고 | ||
And "브라운"이 로그인을 하고 | ||
And "서니"가 로그인을 하고 | ||
Background: 사전 작업 | ||
Given "현구막"이 크루역할로 로그인을 하고 | ||
And "브라운"이 크루역할로 로그인을 하고 | ||
And "서니"가 크루역할로 로그인을 하고 | ||
|
||
Scenario: 필터 목록 조회하기 | ||
When 필터요청이 들어오면 | ||
Then nickname을 기준으로 멤버데이터들을 오름차순 정렬하여 반환한다 | ||
Scenario: 필터 목록 조회하기 | ||
When 필터요청이 들어오면 | ||
Then nickname을 기준으로 멤버데이터들을 오름차순 정렬하여 반환한다 |
32 changes: 16 additions & 16 deletions
32
backend/src/acceptanceTest/resources/wooteco/prolog/levellogs.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,25 @@ | ||
@api | ||
Feature: 레벨 로그 관련 기능 | ||
|
||
Background: 사전 작업 | ||
Given "브라운"이 로그인을 하고 | ||
Background: 사전 작업 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
|
||
Scenario: 레벨 로그 작성하기 | ||
When 레벨로그를 작성하면 | ||
Then 레벨로그가 조회된다 | ||
Scenario: 레벨 로그 작성하기 | ||
When 레벨로그를 작성하면 | ||
Then 레벨로그가 조회된다 | ||
|
||
Scenario: 레벨 로그 삭제하기 | ||
When 레벨로그를 작성하고 | ||
And 레벨로그를 삭제하면 | ||
Then 레벨로그가 삭제된다 | ||
Scenario: 레벨 로그 삭제하기 | ||
When 레벨로그를 작성하고 | ||
And 레벨로그를 삭제하면 | ||
Then 레벨로그가 삭제된다 | ||
|
||
Scenario: 레벨 로그 수정하기 | ||
When 레벨로그를 작성하고 | ||
And 레벨로그를 수정하면 | ||
Then 레벨로그가 수정된다 | ||
Scenario: 레벨 로그 수정하기 | ||
When 레벨로그를 작성하고 | ||
And 레벨로그를 수정하면 | ||
Then 레벨로그가 수정된다 | ||
|
||
|
||
Scenario: 레벨 로그 목록 조회하기 | ||
When 레벨로그를 여러개 작성하면 | ||
Then 레벨로그가 여러개 조회된다 | ||
Scenario: 레벨 로그 목록 조회하기 | ||
When 레벨로그를 여러개 작성하면 | ||
Then 레벨로그가 여러개 조회된다 | ||
|
16 changes: 8 additions & 8 deletions
16
backend/src/acceptanceTest/resources/wooteco/prolog/member.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,12 @@ | ||
@api | ||
Feature: 멤버 관련 기능 | ||
|
||
Scenario: 자신의 멤버정보 조회하기 | ||
Given "브라운"이 로그인을 하고 | ||
When 자신의 멤버 정보를 조회하면 | ||
Then 멤버 정보가 조회된다 | ||
Scenario: 자신의 멤버정보 조회하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
When 자신의 멤버 정보를 조회하면 | ||
Then 멤버 정보가 조회된다 | ||
|
||
Scenario: 자신의 정보를 수정하기 | ||
Given "브라운"이 로그인을 하고 | ||
When 자신의 닉네임을 "brown"으로 수정하면 | ||
Then "브라운"의 닉네임이 "brown"으로 수정 | ||
Scenario: 자신의 정보를 수정하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
When 자신의 닉네임을 "brown"으로 수정하면 | ||
Then "브라운"의 닉네임이 "brown"으로 수정 |
32 changes: 16 additions & 16 deletions
32
backend/src/acceptanceTest/resources/wooteco/prolog/mission.feature
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,22 +1,22 @@ | ||
@api | ||
Feature: 로그인 기능 | ||
|
||
Background: 사전 작업 | ||
Given 세션 여러개를 생성하고 | ||
Background: 사전 작업 | ||
Given 세션 여러개를 생성하고 | ||
|
||
Scenario: 미션 등록하기 | ||
Given "브라운"이 로그인을 하고 | ||
When "이런저런" 미션 등록을 하면 | ||
Then 미션이 등록된다 | ||
Scenario: 미션 등록하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
When "이런저런" 미션 등록을 하면 | ||
Then 미션이 등록된다 | ||
|
||
Scenario: 미션 조회하기 | ||
Given "브라운"이 로그인을 하고 | ||
And "이런저런" 미션 등록을 하고 | ||
When 미션 목록을 조회하면 | ||
Then 미션 목록을 받는다 | ||
Scenario: 미션 조회하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
And "이런저런" 미션 등록을 하고 | ||
When 미션 목록을 조회하면 | ||
Then 미션 목록을 받는다 | ||
|
||
Scenario: 중복된 이름으로 미션 등록하기 | ||
Given "브라운"이 로그인을 하고 | ||
And "이런저런" 미션 등록을 하고 | ||
When "이런저런" 미션 등록을 하면 | ||
Then 미션을 실패한다 | ||
Scenario: 중복된 이름으로 미션 등록하기 | ||
Given "브라운"이 크루역할로 로그인을 하고 | ||
And "이런저런" 미션 등록을 하고 | ||
When "이런저런" 미션 등록을 하면 | ||
Then 미션을 실패한다 |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
원래 띄워쓰기에 _(언더스코어) 사용하지 않나용?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
새로 작성된 인수테스트를 제외한 기존 인수테스트의 조건에 맵핑될 메서드명을 다 해당 방식으로 작성했더라구요 컨벤션이였던 것으로 파악되서 통일되게 작성했습니다!! ( 저도 언더 스코어가 좋아요ㅎㅎ )