這個單元測試異步JavaScript函數（通過Mocha/Sinon）有什麼問題？

Question

我試圖把一個小例子展示給同事，但不知道這個測試that I've put in a gist有什麼問題。

基本上我想測試做了異步功能，但使用興農的間諜（）功能，以確保它完成：採用摩卡和done（）來解決一個測試，依賴於

function asyncHello(name, delay, cb) { 
    setTimeout(function() { 
    console.log("running after ", delay); 
    cb("hello " + name); 
    }, delay); 
} 



suite('Mega Suite', function(){ 

    suite("testing async hello", function() { 
    test('should call the callback', function(done) { 
     var cb = sinon.spy(); 
     asyncHello("foo", cb); 

     cb.should.have.been.called(); 
     done(); 
    }); 
    }); 
}); 

思想異步函數（setTimeout，在這種情況下）可以工作，但也許有人可以指出我錯在哪裏。謝謝！

Answer 1

你不需要興農此：

function asyncHello(name, delay, cb) { 
    setTimeout(function() { 
    console.log("running after ", delay); 
    cb("hello " + name); 
    }, delay); 
} 

suite('Mega Suite', function(){ 
    suite("testing async hello", function() { 
    test('should call the callback', function(done) { 
     asyncHello("foo", 1000, function() { 
     done(); 
     }); 
    }); 
    }); 
}); 

有兩個問題在此代碼：

1. 你叫asyncHello("foo", cb);這使得它讓你在函數內部delay論據是設置爲cb，函數內部的cb參數未定義。

2. 即使在確定第一項之後cb.should.have.been.called();在傳遞給setTimeout的函數可以執行之前被調用。

   你基本上不需要使用Sinon，因爲如果你只是設置一個回調來呼叫done()那麼你就知道測試是成功的。如果任何地方出現問題，done()將不會被調用，並且測試將失敗。