team-ooak / ooak-smartcontract

MIT License
1 stars 0 forks source link

단위 테스트 구축 및 registerNameTag 함수 작성 #2

Closed Woojinger closed 2 years ago

Woojinger commented 2 years ago

ganache를 설치하셨다면 npm run test:ganache -- ./test/ooak/minting_test.js 로 테스트를 돌릴 수 있습니다.

klaytn IDE에서도 테스트 했고, 문제될 것은 없어보입니다 (워낙 간단한 함수다 보니..)

Woojinger commented 2 years ago

truffle에 caver-js 문제가 해결이 안되어 ganache로 실행은 못해봤지만, 코드만 확인했을 때는 괜찮은 것 같습니다! 혹시 민팅 컨트랙트에서 registerNameTag 할 때 lastTokenId가 1000이 넘으면 실행 안되게 하는 코드도 필요할까요?

헉 그 부분을 빼먹었군요;; lastTokenId는 2차나 그 이후 발행할 NFT까지도 포함하기에 1차 프리세일에만 사용할 변수를 하나 만들어서 그 변수가 1000이 넘어가면 revert 시켜야할 것 같습니다. 피드백 감사합니다