測試承諾

Question

我曾經有過下面的代碼中承諾：

function makeCall(userInfo) { 
    api.postUser(userInfo).then(response => { 
    utils.redirect(response.url); 
    }) 

    // other logic 
    return somethingElse; 
} 

，我能夠寫一個測試，是這樣的：

const successPromise = Promise.resolve({ url: 'successUrl' }) 

beforeEach(function() { 
    sinon.stub(api.postUser).returns(successPromise); 
} 

afterEach(function() { 
    api.postUser.restore(); 
} 

it "calls API properly and redirects" do 
    makeCall({}); 
    expect(api.postUser).calledWith(userInfo).toBe(true); 
    successPromise.then(() => { 
    expect(utils.redirect.calledWith('successUrl')).toBe(true); 
    done(); 
    } 
emd 

，一切都是綠色的。現在

，我不得不添加另一個承諾，使另外一個外部調用，這樣做的API postUser調用之前，所以我的代碼如下所示：

function makeCall(names) { 
    fetchUserData(names).then(userData => { 
    return api.postUser(userData).then(response => { 
    utils.redirect(response.url); 
    }) 
    }) 

    // other logic 
    return somethingElse; 
} 

其中fetchUseData是許多承諾，比如像鏈：

function fetchNames(names) { 
    // some name regions 
    return Promise.all(names); 
} 
function fetchUserData(names) { 
    fetchUsersByNames(names).then(users => { 
    // For now we just choose first user 
    { 
     id: users[0].id, 
     name: users[0].name, 
    } 
    }); 
} 

而且我的測試失敗了。我正在嘗試瞭解如何更改我的測試，以確保我仍在測試是否正確執行了最終的API調用，並且重定向也已完成。我想存根fetchUserData(names)，以防止做這個HTTP調用。

Answer 1

您沒有使用正確的承諾。你的代碼沒有單一的return陳述，當它應該有幾個（或者它應該使用箭頭功能，你不需要它們，你不這樣做）。

修復代碼：

function makeCall(names) { 
    // v---- return 
    return fetchUserData(names).then(userData => { 
    // v---- return 
    return api.postUser(userData).then(response => { 
     utils.redirect(response.url); 
    }) 
    }) 
} 


function fetchUserData(names) { 
    // v---- return 
    return fetchUsersByNames(names).then(users => { 
    // For now we just choose first user 
    // v---- return 
    return { 
     id: users[0].id, 
     name: users[0].name, 
    } 
    }); 
} 

一旦你做到了這一點，你可以有你的測試等待所有操作完成。

測試代碼：

makeCall(['name']).then(() => 
    expect(api.postUser).calledWith(userInfo).toBe(true); 
    expect(utils.redirect.calledWith('successUrl')).toBe(true); 
    done(); 
}); 

Answer 2

你應該加一個return語句，否則你是不是無處返回承諾：

function fetchNames(names) { 
    // some name regions 
    return Promise.all(names); 
} 
function fetchUserData(names) { 
    return fetchUsersByNames(names).then(users => { 
    // For now we just choose first user 
    { 
     id: users[0].id, 
     name: users[0].name, 
    } 
    }); 
} 

所以，當你使用Promise.all（），那麼你將有作爲的承諾與所有的數組的結果所有承諾返回的值。 所以後來這個方法看起來應該是這樣叫的時候：

fetchNames(names).then((arrayOfResolvedPromises) => { 
// here you will have all your promised resolved and the array holds all the results 
}); 

所以你的測試，你可以將你的裏面塊，其中所有的承諾將得到解決內部完成。

此外，我強烈建議您使用庫作爲測試承諾的承諾。 它有很多很好的方法來測試你的承諾。

https://github.com/domenic/chai-as-promised