search

fearlessgerbil / equip

0 stars 4 forks source link

complete apiItem test #27

Closed MaZelong closed 8 years ago

MaZelong commented 8 years ago

complete apiItem test created apiMessages tests but could not solve async errors update apiItem at first @dpastoor

dpastoor commented 8 years ago

@MaZelong it looks pretty good! If you can remove the 2 tests that have the isIn:true still in there and re-run to make sure the tests still pass we'll be good

MaZelong commented 8 years ago

@dpastoor do i make a new pull request ? i remove that and it works

MaZelong commented 8 years ago

@dpastoor i removed already and you can see it in the filed changed

dpastoor commented 8 years ago

great thanks