Skip to content

Comments

week3 HW by Ken#3

Open
yoychen wants to merge 4 commits intotrunk-studio:masterfrom
yoychen:master
Open

week3 HW by Ken#3
yoychen wants to merge 4 commits intotrunk-studio:masterfrom
yoychen:master

Conversation

@yoychen
Copy link

@yoychen yoychen commented Aug 2, 2016

麻煩您了

done();
});

it('get friends', async (done) => {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果這邊 it.only 會錯吧?
要在 before 建立一個 friend 比較保險

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

還有我發現你的 測試裡面都沒

try {

} catch(e) {
  done(e)
}

這樣子如果測試有寫錯會卡住

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 this pull request may close these issues.

2 participants