'npm install' invokes tests

[baxang]

사소한 것으로 이슈트래커를 채우는 것 같아 부담스럽지만^^ 설치를 마치고 http://localhost:3000/wikis/note/users 를 열어보니 racoon이라는 유저가 기본적으로 등록되어 있었습니다. 확인해보니 npm install이 postinstall에서 Jakefile을 실행하는데, build task가 test task도 같이 실행하게 되어있었습니다. 이것이 개발팀이 의도하신 동작인지 궁금합니다.

저는 node.js에는 문외한에 가깝습니다만^^ npm install은 packages.json에 적힌 모듈들을 설치하는 명령으로 알고 있습니다. 그런데 이 명령을 실행할 때마다 자동으로 테스트가 수행되면 매번 데이터가 초기화되는데, 의도하지 않게 데이터를 날리는 경우가 생기지 않을까요? 왜냐하면 packages.json에 모듈을 한두개 추가한 다음에도 실행할 수 있는 명령이니까요.

그리고 테스트 실행 후에 유저가 한 명 등록되게 되는데, 저는 제가 등록하지 않은 유저가 있어서 깜짝 놀랐었습니다^^ 그리고 공개된 서버에 설치하는 경우에는 위험할 수도 있다는 생각도 듭니다.

혹시 제가 잘못 생각한 부분이 있다면 가차없이 알려주세요.

[eungjun-yi]

사소하지 않아요. 매우 중요한 이슈입니다. 지속적으로 관심 가져주시는 점 정말 고맙게 생각하고 있습니다.

npm install 시 테스트가 실행되는 것은 의도된 동작이 맞습니다. 개발자노트가 문제없이 설치되었으며 정상 동작할 것임을 사용자에게 알려주기 위한 용도입니다.

제가 기억하기론 테스트시에 생성하는 데이터는 실제 사용자가 사용하는 데이터가 아니라 tests/resources 밑에 테스트 전용으로만 만들어지는 것으로 알고 있는데, 이건 확인을 좀 해봐야겠네요.

유저가 한명 자동으로 등록되는 건 개발과정에서 테스트로 넣은 것인데... 제 생각엔 빼도 될 것 같아요. 아니면 좀 더 예제용 사용자스러운 이름으로 바꾸거나...

[doortts]

커밋 57cbeb3 에서 racoon 유저를 제거하였습니다.

[eungjun-yi]

fa2b208 에서 설치 후에도 테스트를 실행하지 않도록 postinstall을 제거했습니다.

[baxang]

고맙습니다. 테스트가 유저를 만드는걸로 생각했는데, config.json에 기본적으로 저장된 회원 정보였군요^^

한 가지 궁금한게 있는데, 등록하는 회원은 관리자가 되는 것인가요? 노트 생성/편집하고는 관련이 없어 보이네요.

[eungjun-yi]

네, 현재는 권한 개념이 없어서 회원으로 등록되면 모든 권한을 갖습니다.

[baxang]

네, 그렇군요.

그렇다면 회원이나 인증을 어떤 식으로 사용하도록 가정하신 것인지 알려주실 수 있을까요? 만약 로컬에 설치해서 혼자 사용하거나 내부 서버에서만 사용한다면 관리계정이 따로 없어도 될 것 같고, 공개된 서버에 설치한 경우에도 누구나 관리자 등록 페이지에 접근할 수 있으니 의미가 없어 보여서요.

[eungjun-yi]

현재 시점에서 회원 등록의 목적은, 권한을 제어하는 것이 아니라 신원을 식별하는 것입니다. 쉽게 말해서 글을 누가 썼는지 알 수 있게 하는게 목적이죠.

[baxang]

로그인 하고 글을 쓰거나 고쳐도 history에는 guest로 표시되는데 정상적인 작동이 아닌가요?

[eungjun-yi]

히스토리에 글쓴이가 누구인지 남기는 기능이 아직 구현되지 않아서 그렇습니다. 빨리 해야겠네요.

[baxang]

아, 아직 구현되지 않은거였군요^^ 이제 이해가 됐습니다.

On 2012. 8. 26., at 오전 3:06, Yi EungJun notifications@github.com wrote:

히스토리에 글쓴이가 누구인지 남기는 기능이 아직 구현되지 않아서 그렇습니다. 빨리 해야겠네요.

—
Reply to this email directly or view it on GitHub.