测试承诺

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